> 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.
Ok, Thanks. Will clarity and add comments as well.
