From f594537a485b2b77e8762a6b2f45bf731f2eb5c9 Mon Sep 17 00:00:00 2001 From: Shuhei Matsumoto Date: Thu, 9 Jan 2020 20:40:25 -0500 Subject: [PATCH] bdevperf: Reset g_run_rc to 0 after sending RPC reply When RPC kicks the test and the test results in error, RPC response should return error but bdevperf application should not return error. RPC can kick the test multiple times, and if RPC kicks the test again and it completes normally, RPC response should return success. So reset g_run_rc to 0 after sending RPC response. Signed-off-by: Shuhei Matsumoto Change-Id: I59a7c06a293ba7f7b5caa8171bcfb54d736524da Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/479914 Tested-by: SPDK CI Jenkins Community-CI: SPDK CI Jenkins Reviewed-by: Changpeng Liu Reviewed-by: Tomasz Zawadzki Reviewed-by: Jim Harris --- test/bdev/bdevperf/bdevperf.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/bdev/bdevperf/bdevperf.c b/test/bdev/bdevperf/bdevperf.c index 6d170a5fc..b91cd32f3 100644 --- a/test/bdev/bdevperf/bdevperf.c +++ b/test/bdev/bdevperf/bdevperf.c @@ -1460,6 +1460,9 @@ rpc_perform_tests_cb(void) spdk_jsonrpc_send_error_response_fmt(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR, "bdevperf failed with error %s", spdk_strerror(-g_run_rc)); } + + /* Reset g_run_rc to 0 for the next test run. */ + g_run_rc = 0; } static void