Commit 835490b1 authored by Changpeng Liu's avatar Changpeng Liu Committed by Tomasz Zawadzki
Browse files

lib/vhost: check memory table earlier 



Before starting a device, the memory table is already
there, so we can check it earlier.

Change-Id: I4996705501577cfa78c89621f7081eb0c3d4dd78
Signed-off-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/14923


Community-CI: Mellanox Build Bot
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent d941d138

lib/vhost/rte_vhost_user.c

+5 −5
Original line number Diff line number Diff line
@@ -1036,6 +1036,11 @@ start_device(int vid) 
		goto out;

	}



	if (!vsession->mem) {

		SPDK_ERRLOG("Session %s doesn't set memory table yet\n", vsession->name);

		goto out;

	}



	if (vhost_get_negotiated_features(vid, &vsession->negotiated_features) != 0) {

		SPDK_ERRLOG("vhost device %d: Failed to get negotiated driver features\n", vid);

		goto out;
@@ -1120,11 +1125,6 @@ start_device(int vid) 
		vsession->max_queues = i + 1;

	}



	if (!vsession->mem) {

		SPDK_ERRLOG("Session %s doesn't set memory table yet\n", vsession->name);

		goto out;

	}



	vhost_user_session_set_coalescing(vdev, vsession, NULL);

	vsession->initialized = true;

	rc = vhost_user_session_start(vdev, vsession);