Hi David Thanks for review Please find responses inline
> -----Original Message----- > From: David Marchand <david.march...@redhat.com> > Sent: Friday, January 12, 2024 6:12 PM > To: Harman Kalra <hka...@marvell.com> > Cc: Ajit Khaparde <ajit.khapa...@broadcom.com>; Somnath Kotur > <somnath.ko...@broadcom.com>; John Daley <johnd...@cisco.com>; > Hyong Youb Kim <hyon...@cisco.com>; Yuying Zhang > <yuying.zh...@intel.com>; Beilei Xing <beilei.x...@intel.com>; Qiming Yang > <qiming.y...@intel.com>; Qi Zhang <qi.z.zh...@intel.com>; Wenjun Wu > <wenjun1...@intel.com>; Dariusz Sosnowski <dsosnow...@nvidia.com>; > Viacheslav Ovsiienko <viachesl...@nvidia.com>; Ori Kam > <or...@nvidia.com>; Suanming Mou <suanmi...@nvidia.com>; Matan > Azrad <ma...@nvidia.com>; Chaoyong He <chaoyong...@corigine.com>; > Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>; Thomas Monjalon > <tho...@monjalon.net>; Ferruh Yigit <ferruh.yi...@amd.com>; > dev@dpdk.org > Subject: [EXT] Re: [PATCH 1/2] ethdev: parsing multiple representor devargs > string > > External Email > > ---------------------------------------------------------------------- > On Thu, Jan 11, 2024 at 7:45 AM Harman Kalra <hka...@marvell.com> > wrote: > > > > Adding support for parsing multiple representor devargs strings passed > > to a PCI BDF. There may be scenario where port representors for > > various PFs or VFs under PFs are required and all these are > > representor ports shall be backed by single pci device. In such case > > port representors can be created using devargs string: > > <PCI > > BDF>,representor=pf[0-1],representor=pf2vf[1,2-3],representor=[4-5] > > Is it possible to make the representor devargs value a list? > Like: <PCI BDF>,representor=[pf[0-1],pf2vf[1,2-3],[4-5]] Thanks for suggestion, even I was not so happy with repeated representor keywork. Please review the changes: https://patches.dpdk.org/project/dpdk/patch/20240115155715.111154-2-hka...@marvell.com/ Thanks Harman > > > -- > David Marchand
