Hi, Xuemig This patch looks like a fix - could you, please add the "Fixes:" tag and modify the headline?
With best regards, Slava > -----Original Message----- > From: Xueming Li <xuemi...@nvidia.com> > Sent: Thursday, April 22, 2021 10:25 > Cc: dev@dpdk.org; Xueming(Steven) Li <xuemi...@nvidia.com>; Matan > Azrad <ma...@nvidia.com>; Shahaf Shuler <shah...@nvidia.com>; Slava > Ovsiienko <viachesl...@nvidia.com> > Subject: [PATCH] net/mlx5: probe LAG representor with PF1 PCI address > > In case of bonding, orchestrator wants to use same devargs for LAG and non- > LAG scenario, to probe representor on PF1 using PF1 PCI address like > "<DBDF_PF1>,representor=pf1vf[0-3]". > > This patch changes PCI address check policy to allow PF1 PCI address for > representors on PF1. > > Note: detaching PF0 device can't remove representors on PF1. It's > recommended to use primary(PF0) PCI address to probe representors on > both PFs. > > Signed-off-by: Xueming Li <xuemi...@nvidia.com> > --- > drivers/net/mlx5/linux/mlx5_os.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/mlx5/linux/mlx5_os.c > b/drivers/net/mlx5/linux/mlx5_os.c > index 76c72d0e38..22271e289a 100644 > --- a/drivers/net/mlx5/linux/mlx5_os.c > +++ b/drivers/net/mlx5/linux/mlx5_os.c > @@ -1875,11 +1875,14 @@ mlx5_device_bond_pci_match(const struct > ibv_device *ibv_dev, > tmp_str); > break; > } > - /* Match PCI address. */ > + /* Match PCI address, allows BDF0+pfx or BDFx+pfx. */ > if (pci_dev->domain == pci_addr.domain && > pci_dev->bus == pci_addr.bus && > pci_dev->devid == pci_addr.devid && > - pci_dev->function + owner == pci_addr.function) > + ((pci_dev->function == 0 && > + pci_dev->function + owner == pci_addr.function) || > + (pci_dev->function == owner && > + pci_addr.function == owner))) > pf = info.port_name; > /* Get ifindex. */ > snprintf(tmp_str, sizeof(tmp_str), > -- > 2.25.1
