From 74050eacd76a4b47526adca45e29add63eb82a3d Mon Sep 17 00:00:00 2001 From: WANGHAILIANG Date: Mon, 8 Jun 2020 15:24:50 +0800 Subject: [PATCH] script/perf: move "make clean" before "./configure" I suggest putting "make clean" before "./configure". Like Migration-tc3a.sh (spdk/test/vhost/migration): ssh_remote $MGMT_INITIATOR_IP "cd $spdk_repo_share_dir/spdk; make clean; ./configure --with-rdma --enable-debug; make -j40" Otherwise, once the configuration is changed, some outdated compiled files may remain after "make clean". Change-Id: I7300749986129cbfaef44f8ff72fad0449f4e081 Signed-off-by: WANGHAILIANG Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/2805 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins Reviewed-by: Reviewed-by: Karol Latecki Reviewed-by: Liang Yan Reviewed-by: Ben Walker Reviewed-by: Tomasz Zawadzki --- scripts/perf/nvmf/run_nvmf.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/perf/nvmf/run_nvmf.py b/scripts/perf/nvmf/run_nvmf.py index 7cf0e78cb..ae7b53e6c 100755 --- a/scripts/perf/nvmf/run_nvmf.py +++ b/scripts/perf/nvmf/run_nvmf.py @@ -735,8 +735,8 @@ class SPDKInitiator(Initiator): self.log_print("Sources unpacked") self.log_print("Using fio binary %s" % self.fio_bin) - self.remote_call("cd %s; git submodule update --init; ./configure --with-rdma --with-fio=%s;" - "make clean; make -j$(($(nproc)*2))" % (self.spdk_dir, os.path.dirname(self.fio_bin))) + self.remote_call("cd %s; git submodule update --init; make clean; ./configure --with-rdma --with-fio=%s;" + "make -j$(($(nproc)*2))" % (self.spdk_dir, os.path.dirname(self.fio_bin))) self.log_print("SPDK built") self.remote_call("sudo %s/scripts/setup.sh" % self.spdk_dir)