Hi Xueming,
> -----Original Message----- > From: Xueming(Steven) Li [mailto:[email protected]] > Sent: Wednesday, January 10, 2018 4:52 PM > To: Lu, Wenzhuo <[email protected]> > Cc: Wu, Jingjing <[email protected]>; [email protected]; Olivier MATZ > <[email protected]>; Burakov, Anatoly <[email protected]> > Subject: RE: [dpdk-dev] [PATCH v1] app/testpmd: support command echo in > CLI batch loading > > > > -----Original Message----- > > From: Lu, Wenzhuo [mailto:[email protected]] > > Sent: Wednesday, January 10, 2018 4:36 PM > > To: Xueming(Steven) Li <[email protected]> > > Cc: Wu, Jingjing <[email protected]>; [email protected]; Olivier MATZ > > <[email protected]>; Burakov, Anatoly <[email protected]> > > Subject: RE: [dpdk-dev] [PATCH v1] app/testpmd: support command echo > > in CLI batch loading > > > > Hi Xueming, > > > > > -----Original Message----- > > > From: dev [mailto:[email protected]] On Behalf Of Xueming Li > > > Sent: Tuesday, December 26, 2017 10:26 PM > > > Cc: Xueming Li <[email protected]>; Wu, Jingjing > > > <[email protected]>; [email protected]; Olivier MATZ > > > <[email protected]>; Burakov, Anatoly > > > <[email protected]> > > > Subject: [dpdk-dev] [PATCH v1] app/testpmd: support command echo in > > > CLI batch loading > > > > > > Use first bit of verbose_level to enable CLI echo of batch loading. > > After this patch, the first bit of verbose_level is ambiguous. It can > > still enable/disable the log print. > > Is it by design? > You are correct, there are some code in testpmd simply testing verbose>0. > How about changing all the test to: verbose & 1? I have another patchset > using other bits of verbose... I don't object to give more meaning to verbose. Agree with "verbose & 1". As you said, now only 0 and 1 has meaning. Don't forget to comment this variable carefully. And also update the doc 'testpmd_funcs.rst' to let the users know how to use the new functions of this CLI 'set verbose (level)'.
