[dpdk-dev] [PATCH v4 3/3] app/testpmd: set packet dump based on verbosity level
Iremonger, Bernard
bernard.iremonger at intel.com
Thu Oct 11 17:24:35 CEST 2018
Hi Feruh,
> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Thursday, October 11, 2018 4:00 PM
> To: Raslan Darawsheh <rasland at mellanox.com>; Wu, Jingjing
> <jingjing.wu at intel.com>
> Cc: Thomas Monjalon <thomas at monjalon.net>; dev at dpdk.org; Shahaf Shuler
> <shahafs at mellanox.com>; Xueming(Steven) Li <xuemingl at mellanox.com>; Ori
> Kam <orika at mellanox.com>; jerin.jacob at caviumnetworks.com;
> david.marchand at 6wind.com; Iremonger, Bernard
> <bernard.iremonger at intel.com>
> Subject: Re: [dpdk-dev] [PATCH v4 3/3] app/testpmd: set packet dump based on
> verbosity level
>
> On 10/7/2018 8:38 AM, Raslan Darawsheh wrote:
> > when changing verbosity level it will configure rx/tx callbacks to
> > dump packets based on the verbosity value as following:
> > 1- dump only received packets:
> > testpmd> set verbose 1
> > 2- dump only sent packets:
> > testpmd> set verbose 2
> > 3- dump sent and received packets:
> > testpmd> set verbose (any number > 2)
> > 4- disable dump
> > testpmd> set verbose 0
>
> It is good to able to enable Rx/Tx separately but you are overloading "verbose"
> meaning here. "verbose" is kind of log_level internal to testpmd and can be used
> by many testpmd debug log.
>
> Why not create a separate setting for it, perhaps like pkt_verbose?
The original code in rxonly.c used the verbose level, I don't think it is good idea to add a new pkt_verbose.
Users will be expecting it to work as before.
Regards,
Bernard.
More information about the dev
mailing list