Re: [PATCH v10 15/17] xen/arm: account IO handlers for emulated PCI MSI-X

[Julien Grall <julien@xxxxxxx>]

Hi,

On 13/10/2023 14:06, Volodymyr Babchuk wrote:
> Hi Julien,
>
> Julien Grall <julien@xxxxxxx> writes:
>
> > Hi Volodymyr,
> >
> > On 12/10/2023 23:09, Volodymyr Babchuk wrote:
> > > From: Oleksandr Andrushchenko <oleksandr_andrushchenko@xxxxxxxx>
> > > At the moment, we always allocate an extra 16 slots for IO handlers
> > > (see MAX_IO_HANDLER). So while adding IO trap handlers for the emulated
> > > MSI-X registers we need to explicitly tell that we have additional IO
> > > handlers, so those are accounted.
> > > Signed-off-by: Oleksandr Andrushchenko
> > > <oleksandr_andrushchenko@xxxxxxxx>
> > Some process remark. All the patches you send (even if they are
> > unmodified) should also contain your signed-off-by. This is to comply
> > with point (b) in the DCO certificate:
> Oh, sorry. I assumed that it is enough to have signed-off-by tag of the
> original author. I'll add my tags in the next version.
Thanks!

> > https://urldefense.com/v3/__https://cert-manager.io/docs/contributing/sign-off/__;!!GF_29dbcQIUBPA!0mzdEfHOZMm2OmzFc6TZukGgMYRHxDWLdEQvbhUlDmOg3tZNeDbWb8vHz38zLzcYv8GUZeHLn-5sWTYCkvkb$
> >  [cert-manager[.]io]
> >
> > Please check the other patches in this series.
> >
> > > Acked-by: Julien Grall <jgrall@xxxxxxxxxx>
> > Is this patch depends on the rest of the series? If not we can merge
> > it in the for-4.19 branch Stefano created. This will reduce the number
> > of patches you need to resend.
> It uses VPCI_MAX_VIRT_DEV constant which was introduced in ("vpci: add
> initial support for virtual PCI bus topology").
Ok. I will wait before committing. Please let me know if there are any 
Arm patches that can be already committed (or could potentially be 
reviewed independently).
Cheers,

--
Julien Grall