[dpdk-dev] [PATCH v6 0/5] net/softnic: sw fall-back pmd for traffic mgmt and others

[Singh, Jasvinder]

> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Friday, October 6, 2017 7:57 PM
> To: Singh, Jasvinder <jasvinder.singh at intel.com>; dev at dpdk.org
> Cc: Dumitrescu, Cristian <cristian.dumitrescu at intel.com>;
> thomas at monjalon.net; Lu, Wenzhuo <wenzhuo.lu at intel.com>
> Subject: Re: [PATCH v6 0/5] net/softnic: sw fall-back pmd for traffic mgmt
> and others
> 
> On 10/6/2017 5:59 PM, Jasvinder Singh wrote:
> > The SoftNIC PMD is intended to provide SW fall-back options for
> > specific ethdev APIs in a generic way to the NICs not supporting those
> features.
> >
> > Currently, the only implemented ethdev API is Traffic Management (TM),
> > but other ethdev APIs such as rte_flow, traffic metering & policing,
> > etc can be easily implemented.
> 
> <...>
> 
> > Cristian Dumitrescu (4):
> > Jasvinder Singh (4):
> >   net/softnic: add softnic PMD
> >   net/softnic: add traffic management support
> >   net/softnic: add TM capabilities ops
> >   net/softnic: add TM hierarchy related ops
> >
> > Jasvinder Singh (1):
> >   app/testpmd: add traffic management forwarding mode
> 
> Hi Jasvinder,
> 
> There is a patchset has been merged into main and next-net repo to convert
> port_id from 8bits to 16bits.
> This PMD also should reflect this conversion, can you please rebase PMD on
> top of latest next-net?
> 
> Please keep existing Acks in next version.

Ok, will do in the next version.

> Also there are lots of UNNECESSARY_PARENTHESES checkpatch warnings, can
> you please fix them?

Ok, will fix them. 
> >
> >  MAINTAINERS                                        |    5 +
> 
> Checkpatch also gives warning about MAINTAINERS file updates, not sure
> why, can you please check this?
> 
The latest checkpatch script issues warning on the use of "space" instead of "Tab" for defining the type. The existing pattern in the MAINTEINERS
files doesn't follow this.  "Space" is used for each Type everywhere instead of "Tab", that's why I ignored this warning.