> -----Original Message----- > From: Yuanhan Liu [mailto:[email protected]] > Sent: Friday, January 19, 2018 2:48 PM > To: Stokes, Ian <[email protected]> > Cc: [email protected]; Shahaf Shuler <[email protected]>; Loftus, > Ciara <[email protected]>; Thomas Monjalon <[email protected]>; > Kevin Traynor <[email protected]>; Ilya Maximets > ([email protected]) <[email protected]>; Olga Shern > <[email protected]> > Subject: Re: [PATCH v3] netdev-dpdk: fix port addition for ports sharing > same PCI id > > On Fri, Jan 19, 2018 at 11:03:52AM +0000, Stokes, Ian wrote: > > > -----Original Message----- > > > From: Yuanhan Liu [mailto:[email protected]] > > > Sent: Wednesday, January 10, 2018 3:05 AM > > > To: [email protected] > > > Cc: Stokes, Ian <[email protected]>; Shahaf Shuler > > > <[email protected]>; Yuanhan Liu <[email protected]>; Loftus, > > > Ciara <[email protected]>; Thomas Monjalon > > > <[email protected]>; Kevin Traynor <[email protected]> > > > Subject: [PATCH v3] netdev-dpdk: fix port addition for ports sharing > > > same PCI id > > > > > > Some NICs have only one PCI address associated with multiple ports. > > > This patch extends the dpdk-devargs option's format to cater for such > devices. > > > > > > To achieve that, this patch uses a new syntax that will be adapted > > > and implemented in future DPDK release (likely, v18.05): > > > http://dpdk.org/ml/archives/dev/2017-December/084234.html > > > > > > And since it's the DPDK duty to parse the (complete and full) syntax > > > and this patch is more likely to serve as an intermediate > > > workaround, here I take a simpler and shorter syntax from it (note > > > it's allowed to have only one category being provided): > > > class=eth,mac=00:11:22:33:44:55:66 > > > > > > Also, old compatibility is kept. Users can still go on with using > > > the PCI id to add a port (if that's enough for them). Meaning, this > > > patch will not break anything. > > > > > > This patch is basically based on the one from Ciara: > > > https://mail.openvswitch.org/pipermail/ovs-dev/2017- > > > October/339496.html > > > > > > Cc: Loftus Ciara <[email protected]> > > > Cc: Thomas Monjalon <[email protected]> > > > Cc: Kevin Traynor <[email protected]> > > > Signed-off-by: Yuanhan Liu <[email protected]> > > > > Thanks for the work on this Yuanhan, it's a valuable feature/bug fix. > > > > At this point I think this aligns to the DPDK model? > > Yes, I think so. > > > > > If so, and if there are no other comments on this patchset then I plan > to merge this to DPDK merge over the weekend. > > > > Thank you for taking care of it.
Thanks for all the work on this, I've pushed this to DPDK_MERGE and will also push it to DPDK_MERGE_2_9 for it to be part of the 2.9 release. Ian > > --yliu _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
