Re: [Xen-devel] [PATCH v2 for-4.5] xen/arm: clear UIE on hypervisor entry

[Julien Grall <julien.grall@xxxxxxxxxx>]

Hi Stefano,

On 20/11/2014 15:54, Stefano Stabellini wrote:
> On Thu, 20 Nov 2014, Julien Grall wrote:
> > On 11/20/2014 11:02 AM, Julien Grall wrote:
> > > Hi Stefano,
> > >
> > > On 11/20/2014 10:53 AM, Stefano Stabellini wrote:
> > > > UIE being set can cause maintenance interrupts to occur when Xen writes
> > > > to one or more LR registers. The effect is a busy loop around the
> > > > interrupt handler in Xen
> > > > (http://marc.info/?l=xen-devel&m=141597517132682): everything gets stuck.
> > > >
> > > > Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> > > > Reported-and-Tested-by: Andrii Tseglytskyi <andrii.tseglytskyi@xxxxxxxxxxxxxxx>
> > > > CC: konrad.wilk@xxxxxxxxxx
> > > > ---
> > > >
> > > > Konrad, this fixes an actual bug, at least on OMAP5. It should have no
> > > > bad side effects on any other platforms as far as I can tell. It should
> > > > go in 4.5.
> > >  From Andrii's mail, there is a bad side effect. We can receive spurious
> > > interrupt.
> > >
> > > On V1, you said that you tried this patch on midway. I would prefer if
> > > you give a try on  platform that would really be used (such as xgene or
> > > seattle). As we know Midway won't be used in prod.
> > And maybe give a try to gicv3 too as it's common code.
> I don't have a gicv3 test environment ready but it works on xgene too
I gave a try on the Foundation Model and didn't see any specific regression:

Tested-by: Julien Grall <julien.grall@xxxxxxxxxx>

Regards,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel