Hi Jan,
> On 19 Oct 2021, at 13:29, Jan Beulich <jbeul...@suse.com> wrote:
>
> On 19.10.2021 12:40, Bertrand Marquis wrote:
>> --- a/xen/drivers/passthrough/pci.c
>> +++ b/xen/drivers/passthrough/pci.c
>> @@ -756,11 +756,6 @@ int pci_add_device(u16 seg, u8 bus, u8 devfn,
>> if ( !pdev->domain )
>> {
>> pdev->domain = hardware_domain;
>> -#ifdef CONFIG_ARM
>> - /*
>> - * On ARM PCI devices discovery will be done by Dom0. Add vpci
>> handler
>> - * when Dom0 inform XEN to add the PCI devices in XEN.
>> - */
>> ret = vpci_add_handlers(pdev);
>
> I think it wouldn't hurt to retain a (re-worded) comment here, maybe
> along the lines of "For devices not discovered by Xen during boot,
> add vPCI handlers when Dom0 first informs Xen about such devices."
Ok, I will add that in v2.
Cheers
Bertrand
>
> Jan
>
