Commit ec2ad00c authored by Jim Harris's avatar Jim Harris
Browse files

test/unit/raid: fix set-but-not-used error 



verify_io() keeps track of a buf pointer, but the
buf pointer never actually gets used.  So remove
this buf pointer.

Found by clang-13.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: I79dfeac7f004b56f7d4404f41b2ff18b96968a20
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10056


Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
parent ab19b5de

test/unit/lib/bdev/raid/bdev_raid.c/bdev_raid_ut.c

+0 −2
Original line number Diff line number Diff line
@@ -630,7 +630,6 @@ verify_io(struct spdk_bdev_io *bdev_io, uint8_t num_base_drives, 
	uint64_t pd_lba;

	uint64_t pd_blocks;

	uint32_t index = 0;

	uint8_t *buf = bdev_io->u.bdev.iovs->iov_base;

	struct io_output *output;



	if (io_status == INVALID_IO_SUBMIT) {
@@ -666,7 +665,6 @@ verify_io(struct spdk_bdev_io *bdev_io, uint8_t num_base_drives, 
		CU_ASSERT(ch_ctx->base_channel[pd_idx] == output->ch);

		CU_ASSERT(raid_bdev->base_bdev_info[pd_idx].desc == output->desc);

		CU_ASSERT(bdev_io->type == output->iotype);

		buf += (pd_blocks << spdk_u32log2(g_block_len));

	}

	CU_ASSERT(g_io_comp_status == io_status);

}