Commit 13481a59 authored by Krzysztof Karas's avatar Krzysztof Karas Committed by Tomasz Zawadzki
Browse files

test: ensure unique address for FIO plugin 



FIO plugin (fio/spdk_bdev) always starts RPC server on
SPDK_DEFAULT_RPC_ADDR address. This is unnecessary, because in most
cases there will be no need for RPC interface to be active for
communication. We can change plugin's implementation similarly to what
we have in bdevperf: RPC server is going to be started only when
'--spdk_rpc_listen_addr' is specified.


Change-Id: I99227626c71ae5615cbdd5090fcdd4c42f193eaa
Signed-off-by: default avatarKrzysztof Karas <krzysztof.karas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/22682


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Reviewed-by: default avatarBoris Glimcher <Boris.Glimcher@emc.com>
parent a5fcb030

app/fio/bdev/fio_plugin.c

+5 −3
Original line number Diff line number Diff line
@@ -108,7 +108,7 @@ struct spdk_fio_oat_ctx { 


static bool g_spdk_env_initialized = false;

static const char *g_json_config_file = NULL;

static const char *g_rpc_listen_addr = SPDK_DEFAULT_RPC_ADDR;

static const char *g_rpc_listen_addr = NULL;



static int spdk_fio_init(struct thread_data *td);

static void spdk_fio_cleanup(struct thread_data *td);
@@ -265,10 +265,12 @@ spdk_fio_bdev_init_done(int rc, void *cb_arg) 
{

	*(bool *)cb_arg = true;



	if (g_rpc_listen_addr != NULL) {

		if (spdk_rpc_initialize(g_rpc_listen_addr, NULL) == 0) {

			spdk_rpc_set_state(SPDK_RPC_RUNTIME);

		}

	}

}



static void

spdk_fio_bdev_init_start(void *arg)