[dpdk-dev] [PATCH v3 20/20] eal: set log level from command line

2014-09-18 Thread David Marchand
Hello Neil, On Wed, Sep 17, 2014 at 4:45 PM, Neil Horman wrote: > > This is a nit, but since you're working in this code anyway, would you mind > fixing the long options parsing please? Instead of having a single case > statement that just does a never ending if..else..if series of strcmps tha

[dpdk-dev] [PATCH v3 20/20] eal: set log level from command line

2014-09-18 Thread Neil Horman
On Thu, Sep 18, 2014 at 09:46:54AM +0200, David Marchand wrote: > Hello Neil, > > > On Wed, Sep 17, 2014 at 4:45 PM, Neil Horman wrote: > > > > > This is a nit, but since you're working in this code anyway, would you mind > > fixing the long options parsing please? Instead of having a single c

[dpdk-dev] [PATCH v3 20/20] eal: set log level from command line

2014-09-17 Thread David Marchand
Add a --log-level option to set the default eal log level. Signed-off-by: David Marchand Acked-by: Bruce Richardson --- lib/librte_eal/bsdapp/eal/eal.c| 42 +++ .../bsdapp/eal/include/eal_internal_cfg.h |1 + lib/librte_eal/linuxapp/eal/eal.c

[dpdk-dev] [PATCH v3 20/20] eal: set log level from command line

2014-09-17 Thread Neil Horman
On Wed, Sep 17, 2014 at 03:46:52PM +0200, David Marchand wrote: > Add a --log-level option to set the default eal log level. > > Signed-off-by: David Marchand > Acked-by: Bruce Richardson > --- > #else > @@ -652,6 +679,18 @@ eal_parse_args(int argc, char **argv) >