Commit 835c593c authored by Shuhei Matsumoto's avatar Shuhei Matsumoto Committed by Jim Harris
Browse files

bdevperf: Fix unclear return code check in bdevperf_complete() 



Two return code checks are added to bdevperf_complete().
Checking if return code is not false will be enough.

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


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent 62615117

test/bdev/bdevperf/bdevperf.c

+1 −1
Original line number Diff line number Diff line
@@ -461,7 +461,7 @@ bdevperf_complete(struct spdk_bdev_io *bdev_io, bool success, void *cb_arg) 
				 spdk_bdev_get_block_size(target->bdev),

				 task->md_buf, spdk_bdev_io_get_md_buf(bdev_io),

				 spdk_bdev_get_md_size(target->bdev),

				 target->io_size_blocks, md_check) != 0) {

				 target->io_size_blocks, md_check)) {

			printf("Buffer mismatch! Disk Offset: %lu\n", task->offset_blocks);

			target->is_draining = true;

			g_run_failed = true;