Commit 186c834f authored by Shuhei Matsumoto's avatar Shuhei Matsumoto Committed by Tomasz Zawadzki
Browse files

bdev_nvme/opal: Use spdk_bdev_part_base_construct_ext() to pass bdev_name 



This is a drop-in replacement of spdk_bdev_part_base_construct() by
spdk_bdev_part_base_construct_ext().

Signed-off-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Change-Id: I0c719683b5a9cd6dffce057552982d8e44871ab1
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4580


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: default avatarPaul Luse <paul.e.luse@intel.com>
parent 5bb0d8f2

module/bdev/nvme/vbdev_opal.c

+11 −7
Original line number Diff line number Diff line
@@ -378,15 +378,19 @@ vbdev_opal_create(const char *nvme_ctrlr_name, uint32_t nsid, uint8_t locking_ra 
		}

		TAILQ_INIT(&opal_part_base->part_tailq);



		opal_part_base->part_base = spdk_bdev_part_base_construct(spdk_bdev_get_by_name(base_bdev_name),

		rc = spdk_bdev_part_base_construct_ext(base_bdev_name,

						       vbdev_opal_base_bdev_hotremove_cb, &opal_if,

					    &opal_vbdev_fn_table, &opal_part_base->part_tailq, vbdev_opal_base_free,

					    opal_part_base, sizeof(struct vbdev_opal_channel), NULL, NULL);

		if (opal_part_base->part_base == NULL) {

						       &opal_vbdev_fn_table, &opal_part_base->part_tailq,

						       vbdev_opal_base_free, opal_part_base,

						       sizeof(struct vbdev_opal_channel), NULL, NULL,

						       &opal_part_base->part_base);

		if (rc != 0) {

			if (rc != -ENODEV) {

				SPDK_ERRLOG("Could not allocate part_base\n");

			}

			free(opal_bdev);

			free(opal_part_base);

			return -ENOMEM;

			return rc;

		}

		opal_part_base->nvme_ctrlr_name = strdup(nvme_ctrlr_name);

		if (opal_part_base->nvme_ctrlr_name == NULL) {