Hi Stefano, On 11/20/2014 03:54 PM, 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.stabell...@eu.citrix.com> >>>> Reported-and-Tested-by: Andrii Tseglytskyi >>>> <andrii.tseglyts...@globallogic.com> >>>> CC: konrad.w...@oracle.com >>>> --- >>>> >>>> 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
Ok. I will give a quick try on the model today or tomorrow. Aside from that, and after reading the spec. This patch looks good to me: Reviewed-by: Julien Grall <julien.gr...@linaro.org> Regards, -- Julien Grall _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
