log: fix default level
authorOlivier Matz <olivier.matz@6wind.com>
Tue, 18 Apr 2017 14:22:20 +0000 (16:22 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 19 Apr 2017 23:28:37 +0000 (01:28 +0200)
The initialization of the default log level (from configuration) was
removed by mistake in a previous commit. The global log level was
wrongly set to debug when no --log-level argument was passed. Restore
this initialization.

Before:
  $ ./build/app/test
  RTE>>dump_log_types
  global log level is debug
  ...

After:
  $ ./build/app/test
  RTE>>dump_log_types
  global log level is info
  ...

Fixes: 845afe51e428 ("eal: change specific log levels at startup")

Reported-by: Ferruh Yigit <ferruh.yigit@intel.com>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
lib/librte_eal/bsdapp/eal/eal.c
lib/librte_eal/linuxapp/eal/eal.c

index 3b66da6..db154db 100644 (file)
@@ -324,6 +324,7 @@ eal_log_level_parse(int argc, char **argv)
        optreset = 1;
 
        eal_reset_internal_config(&internal_config);
+       rte_log_set_global_level(internal_config.log_level);
 
        while ((opt = getopt_long(argc, argvopt, eal_short_options,
                                  eal_long_options, &option_index)) != EOF) {
index f3fbeb9..b275879 100644 (file)
@@ -493,6 +493,7 @@ eal_log_level_parse(int argc, char **argv)
        optind = 1;
 
        eal_reset_internal_config(&internal_config);
+       rte_log_set_global_level(internal_config.log_level);
 
        while ((opt = getopt_long(argc, argvopt, eal_short_options,
                                  eal_long_options, &option_index)) != EOF) {