From ec2ad00c9260159e3cf2125b8997c61689ae3b33 Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Thu, 28 Oct 2021 21:25:31 +0000 Subject: [PATCH] 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: Jim Harris Change-Id: I79dfeac7f004b56f7d4404f41b2ff18b96968a20 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10056 Community-CI: Broadcom CI Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Changpeng Liu Reviewed-by: Aleksey Marchuk --- test/unit/lib/bdev/raid/bdev_raid.c/bdev_raid_ut.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/test/unit/lib/bdev/raid/bdev_raid.c/bdev_raid_ut.c b/test/unit/lib/bdev/raid/bdev_raid.c/bdev_raid_ut.c index 574f8b09d..b30c23f9d 100644 --- a/test/unit/lib/bdev/raid/bdev_raid.c/bdev_raid_ut.c +++ b/test/unit/lib/bdev/raid/bdev_raid.c/bdev_raid_ut.c @@ -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); }