[dpdk-dev] [PATCH v2 1/3] app/testpmd: add profiling flags set command

Thomas Monjalon thomas at monjalon.net
Mon Apr 13 14:23:02 CEST 2020


13/04/2020 09:56, Slava Ovsiienko:
> From: Ferruh Yigit <ferruh.yigit at intel.com>
> > > +set fwdprof
> > > +~~~~~~~~~~~
> > > +
> > > +Set the flags controlling the datapath profiling statistics::
> > > +
> > > +   testpmd> set fwdprof (flags)
> > 
> > What do you think a little longer command 'fwdprofile", which is more clear I
> > think.
> Yes, it is clearer, but longer to type 😊
> If you insist on - I will extend, please, let me know your final opinion.

I agree with Ferruh in general and especially about "fwdprof" not nice to read.
What about fwd-profiling?
I would be also OK with fwd-prof.

Is it common to use hyphen sign in testpmd commands?




More information about the dev mailing list