Re: [Xen-devel] [PATCH v3] xen/arm: introduce PLATFORM_QUIRK_GUEST_PIRQ_NEED_EOI

[Julien Grall <julien.grall@xxxxxxxxxx>]

On 07/09/2014 03:40 PM, Ian Campbell wrote:
>> I think we should check GIC Dist address passed in DTS by
>> u-boot to Xen. If GIC Dist address is secured address then
>> enable the quirk otherwise disable the qurik.
> 
> Isn't IGROUPR readable from NS? In which case we should be able to spot
> the difference, I think?
> 
> Stefano, you might want to arrange in your patch to cache the value of
> the quirk in the gicv2 struct -- otherwise all the calls to
> platform_has_quirk are going to add up.

Can't we cache the value in the platform code? I don't think it's too
expensive to call the platform_has_quirk in this case.

Regards,

-- 
Julien Grall

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