Commit f217aa46 authored Dec 22, 2017 by Dariusz Stojaczyk Committed by Ben Walker Dec 27, 2017

bdev/virtio: don't allow registering the same target twice



Change-Id: I0c0b6cfded7fa2450c652c477ab8c1d88fc6209f
Signed-off-by: Dariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/392775


Reviewed-by: Daniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Tested-by: SPDK Automated Test System <sys_sgsw@intel.com>

parent c3cee7ef

lib/bdev/virtio/bdev_virtio.c

+7 −0

Original line number	Diff line number	Diff line
		@@ -1187,6 +1187,13 @@ virtio_scsi_dev_add_tgt(struct virtio_scsi_dev *svdev, struct virtio_scsi_scan_i
		struct spdk_bdev *bdev;
		int rc;

		TAILQ_FOREACH(disk, &svdev->luns, link) {
		if (disk->info.target == info->target) {
		/* Target is already attached and param change is not supported */
		return 0;
		}
		}

		disk = calloc(1, sizeof(*disk));
		if (disk == NULL) {
		SPDK_ERRLOG("could not allocate disk\n");