From 9eae89f10c32c7a55e4048a9d46d0a06120835fe Mon Sep 17 00:00:00 2001
From: wuzhouhui <wuzhouhui@kingsoft.com>
Date: Tue, 28 Aug 2018 15:23:36 +0800
Subject: [PATCH] app: initializing log after DPDK

DPDK initializes log as well (in rte_eal_init()), so we should
re-initialize log after DPDK did that.

Change-Id: I36843a6d09d1e2d84e76284899348b84364e18b7
Signed-off-by: wuzhouhui <wuzhouhui@kingsoft.com>
Reviewed-on: https://review.gerrithub.io/423630
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
---
 lib/event/app.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/event/app.c b/lib/event/app.c
index 9ae9fbc67..8e3910bc7 100644
--- a/lib/event/app.c
+++ b/lib/event/app.c
@@ -495,7 +495,7 @@ spdk_app_setup_env(struct spdk_app_opts *opts)
 	free(env_opts.pci_whitelist);
 
 	if (rc < 0) {
-		SPDK_ERRLOG("Unable to initialize SPDK env\n");
+		fprintf(stderr, "Unable to initialize SPDK env\n");
 	}
 
 	return rc;
@@ -591,12 +591,12 @@ spdk_app_start(struct spdk_app_opts *opts, spdk_event_fn start_fn,
 	}
 
 	spdk_log_set_level(SPDK_APP_DEFAULT_LOG_LEVEL);
-	spdk_log_open();
 
 	if (spdk_app_setup_env(opts) < 0) {
-		goto app_start_log_close_err;
+		goto app_start_setup_conf_err;
 	}
 
+	spdk_log_open();
 	SPDK_NOTICELOG("Total cores available: %d\n", spdk_env_get_core_count());
 
 	spdk_thread_lib_init();