[Skiboot] [PATCH v2] xive: disable store EOI support
Benjamin Herrenschmidt
benh at kernel.crashing.org
Tue Mar 27 08:50:54 AEDT 2018
On Mon, 2018-03-26 at 17:08 +0200, Cédric Le Goater wrote:
> @@ -5027,7 +5031,7 @@ static void phb4_create(struct dt_node *np)
>
> /* Register all interrupt sources with XIVE */
> irq_flags = XIVE_SRC_SHIFT_BUG | XIVE_SRC_TRIGGER_PAGE;
> - if (p->rev >= PHB4_REV_NIMBUS_DD20)
> + if (PHB4_CAN_STORE_EOI(p))
> irq_flags |= XIVE_SRC_STORE_EOI;
> xive_register_hw_source(p->base_msi, p->num_irqs - 8, 16,
> p->int_mmio, irq_flags, NULL, NULL);
Looking at this, it looks like the old code was a bit broken already...
If STORE_EOI is enabled, there is no trigger page.
Also the shift bug only happens on DD1.0 afaik.
Cheers,
Ben.
More information about the Skiboot
mailing list