Commit 07be871a authored by Changpeng Liu's avatar Changpeng Liu Committed by Jim Harris
Browse files

vhost: remove unnecessary vhost lock 



_stop_session() is called while holding the global vhost lock,
and in the caller we do release the vhost lock, so even for the
error return from device backend, we don't need to release it
in _stop_session().

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


Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarDong Yi <dongx.yi@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent 245394ca

lib/vhost/vhost.c

+0 −1
Original line number Diff line number Diff line
@@ -1213,7 +1213,6 @@ _stop_session(struct spdk_vhost_session *vsession) 
	rc = vdev->backend->stop_session(vsession);

	if (rc != 0) {

		SPDK_ERRLOG("Couldn't stop device with vid %d.\n", vsession->vid);

		pthread_mutex_unlock(&g_vhost_mutex);

		return rc;

	}