Re: [ovs-dev] [PATCH 2/2] Adding configuration option to whitelist DPDK physical ports.

2017-12-08 Thread O Mahony, Billy
r 7, 2017 5:53 PM > To: Chandran, Sugesh <sugesh.chand...@intel.com>; O Mahony, Billy > <billy.o.mah...@intel.com>; d...@openvswitch.org; b...@ovn.org > Cc: Mooney, Sean K <sean.k.moo...@intel.com> > Subject: RE: [ovs-dev] [PATCH 2/2] Adding configuration op

Re: [ovs-dev] [PATCH 2/2] Adding configuration option to whitelist DPDK physical ports.

2017-12-07 Thread Mooney, Sean K
ndran, Sugesh <sugesh.chand...@intel.com>; > d...@openvswitch.org; > > b...@ovn.org > > Subject: RE: [ovs-dev] [PATCH 2/2] Adding configuration option to > > whitelist DPDK physical ports. > > > > Hi Sugesh, > > > > > -Original Message- &g

Re: [ovs-dev] [PATCH 2/2] Adding configuration option to whitelist DPDK physical ports.

2017-12-07 Thread Chandran, Sugesh
essage- > > > From: O Mahony, Billy > > > Sent: Wednesday, December 6, 2017 5:31 PM > > > To: Chandran, Sugesh <sugesh.chand...@intel.com>; > > > d...@openvswitch.org; b...@ovn.org > > > Subject: RE: [ovs-dev] [PATCH 2/2] Adding configuration opti

Re: [ovs-dev] [PATCH 2/2] Adding configuration option to whitelist DPDK physical ports.

2017-12-07 Thread O Mahony, Billy
Hi Sugesh, > -Original Message- > From: Chandran, Sugesh > Sent: Wednesday, December 6, 2017 6:23 PM > To: O Mahony, Billy <billy.o.mah...@intel.com>; d...@openvswitch.org; > b...@ovn.org > Subject: RE: [ovs-dev] [PATCH 2/2] Adding configuration option to whiteli

Re: [ovs-dev] [PATCH 2/2] Adding configuration option to whitelist DPDK physical ports.

2017-12-06 Thread Chandran, Sugesh
ct: RE: [ovs-dev] [PATCH 2/2] Adding configuration option to whitelist > DPDK physical ports. > > Hi Sugesh, > > This is definitely a very useful feature. I'm looking forward to running trex > on the > same DUT as my ovs-dpdk. > > However I'd suggest adding an ssca

Re: [ovs-dev] [PATCH 2/2] Adding configuration option to whitelist DPDK physical ports.

2017-12-06 Thread O Mahony, Billy
Hi Sugesh, This is definitely a very useful feature. I'm looking forward to running trex on the same DUT as my ovs-dpdk. However I'd suggest adding an sscanf or some such to verify that the domain is also specified for each whitelist member. And either add the default of '' or complain