Commit d2677013 authored by Paul Luse's avatar Paul Luse Committed by Jim Harris
Browse files

bdev/crypto: respect return value of vbdev_crypto_claim() 



Found via inspection while invetigating a CI failure. In
vbdev_crypto_examine() we were not looking at the rc from
vbdev_crypto_claim()

Change-Id: I8be09b5844e18e35b95f19e378fe280323d183fa
Signed-off-by: default avatarPaul Luse <paul.e.luse@intel.com>
Reviewed-on: https://review.gerrithub.io/432930


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 197998ed

lib/bdev/crypto/vbdev_crypto.c

+6 −2
Original line number Diff line number Diff line
@@ -1495,7 +1495,7 @@ vbdev_crypto_claim(struct spdk_bdev *bdev) 
			goto error_claim;

		}



		SPDK_NOTICELOG("registered crypto_bdev for: %s\n", name->vbdev_name);

		SPDK_NOTICELOG("registered io_device for: %s\n", name->vbdev_name);

	}



	return rc;
@@ -1562,7 +1562,11 @@ vbdev_crypto_examine(struct spdk_bdev *bdev) 
	struct vbdev_crypto *crypto_bdev, *tmp;

	int rc;



	vbdev_crypto_claim(bdev);

	rc = vbdev_crypto_claim(bdev);

	if (rc) {

		spdk_bdev_module_examine_done(&crypto_if);

		return;

	}



	TAILQ_FOREACH_SAFE(crypto_bdev, &g_vbdev_crypto, link, tmp) {

		if (strcmp(crypto_bdev->base_bdev->name, bdev->name) == 0) {