[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