Re: [Xen-devel] [PATCH v3 03/26] ARM: GICv3 ITS: allocate device and collection table

[Julien Grall <julien.grall@xxxxxxx>]

Hi Andre,

On 03/04/17 16:38, Julien Grall wrote:
> On 31/03/17 19:05, Andre Przywara wrote:
> > Each ITS maps a pair of a DeviceID (for instance derived from a PCI
> > b/d/f triplet) and an EventID (the MSI payload or interrupt ID) to a
> > pair of LPI number and collection ID, which points to the target CPU.
> > This mapping is stored in the device and collection tables, which
> > software
> > has to provide for the ITS to use.
> > Allocate the required memory and hand it to the ITS.
> > The maximum number of devices is limited to a compile-time constant
> > exposed in Kconfig.
> >
> > Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx>
> Reviewed-by: Julien Grall <julien.grall@xxxxxxx>
Actually I will withdraw my reviewed-by. I didn't spot you keep the 
command line around which I clearly say no and gave some reasons why. 
Sorry for the mess.
To explain it again, no-one can possible know how the DeviceID will be 
spread on the platform without having the platform data sheet in hand. 
If the platform provide more DeviceID and is not able to cope with that. 
Then it is a platform specific quirk.
When we spoke f2f you agree on this. So please drop this command line.

Cheers,

--
Julien Grall

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