Re: [PATCH v2 1/2] xen-pciback: redo VF placement in the virtual topology

2021-05-20 Thread Boris Ostrovsky
On 5/20/21 10:46 AM, Jan Beulich wrote: > On 20.05.2021 16:44, Jan Beulich wrote: >> On 20.05.2021 16:38, Boris Ostrovsky wrote: >>> On 5/20/21 3:43 AM, Jan Beulich wrote: On 20.05.2021 02:36, Boris Ostrovsky wrote: > On 5/18/21 12:13 PM, Jan Beulich wrote: >> >> @@ -95,22 +95,

Re: [PATCH v2 1/2] xen-pciback: redo VF placement in the virtual topology

2021-05-20 Thread Jan Beulich
On 20.05.2021 16:44, Jan Beulich wrote: > On 20.05.2021 16:38, Boris Ostrovsky wrote: >> >> On 5/20/21 3:43 AM, Jan Beulich wrote: >>> On 20.05.2021 02:36, Boris Ostrovsky wrote: On 5/18/21 12:13 PM, Jan Beulich wrote: > > @@ -95,22 +95,25 @@ static int __xen_pcibk_add_pci_dev(struc

Re: [PATCH v2 1/2] xen-pciback: redo VF placement in the virtual topology

2021-05-20 Thread Jan Beulich
On 20.05.2021 16:38, Boris Ostrovsky wrote: > > On 5/20/21 3:43 AM, Jan Beulich wrote: >> On 20.05.2021 02:36, Boris Ostrovsky wrote: >>> On 5/18/21 12:13 PM, Jan Beulich wrote: @@ -95,22 +95,25 @@ static int __xen_pcibk_add_pci_dev(struc /* * Keep multi-functio

Re: [PATCH v2 1/2] xen-pciback: redo VF placement in the virtual topology

2021-05-20 Thread Boris Ostrovsky
On 5/20/21 3:43 AM, Jan Beulich wrote: > On 20.05.2021 02:36, Boris Ostrovsky wrote: >> On 5/18/21 12:13 PM, Jan Beulich wrote: >>> >>> @@ -95,22 +95,25 @@ static int __xen_pcibk_add_pci_dev(struc >>> >>> /* >>> * Keep multi-function devices together on the virtual PCI bus, except >>

Re: [PATCH v2 1/2] xen-pciback: redo VF placement in the virtual topology

2021-05-20 Thread Jan Beulich
On 20.05.2021 02:36, Boris Ostrovsky wrote: > > On 5/18/21 12:13 PM, Jan Beulich wrote: >> >> @@ -95,22 +95,25 @@ static int __xen_pcibk_add_pci_dev(struc >> >> /* >> * Keep multi-function devices together on the virtual PCI bus, except >> - * virtual functions. >> + * that

Re: [PATCH v2 1/2] xen-pciback: redo VF placement in the virtual topology

2021-05-19 Thread Boris Ostrovsky
On 5/18/21 12:13 PM, Jan Beulich wrote: > > @@ -95,22 +95,25 @@ static int __xen_pcibk_add_pci_dev(struc > > /* >* Keep multi-function devices together on the virtual PCI bus, except > - * virtual functions. > + * that we want to keep virtual functions at func 0 on the

[PATCH v2 1/2] xen-pciback: redo VF placement in the virtual topology

2021-05-18 Thread Jan Beulich
The commit referenced below was incomplete: It merely affected what would get written to the vdev- xenstore node. The guest would still find the function at the original function number as long as __xen_pcibk_get_pci_dev() wouldn't be in sync. The same goes for AER wrt __xen_pcibk_get_pcifront_dev