Hi Jasvinder,
> -----Original Message----- > From: dev [mailto:[email protected]] On Behalf Of Jasvinder Singh > Sent: Monday, September 18, 2017 5:10 PM > To: [email protected] > Cc: Dumitrescu, Cristian <[email protected]>; Yigit, Ferruh > <[email protected]>; [email protected] > Subject: [dpdk-dev] [PATCH v4 3/4] net/softnic: add TM capabilities ops > > Implement ethdev TM capability APIs in SoftNIC PMD. > > Signed-off-by: Cristian Dumitrescu <[email protected]> > Signed-off-by: Jasvinder Singh <[email protected]> > --- > drivers/net/softnic/rte_eth_softnic.c | 12 +- > drivers/net/softnic/rte_eth_softnic_internals.h | 32 ++ > drivers/net/softnic/rte_eth_softnic_tm.c | 500 > ++++++++++++++++++++++++ > 3 files changed, 543 insertions(+), 1 deletion(-) The same concern of the naming as patch 2.The function and structure names are too common. Better add the prefix 'softnic_' too. Except that, the patch looks good to me.
