From 835c593c70b9374055e76fe8c045ded2b69d510b Mon Sep 17 00:00:00 2001 From: Shuhei Matsumoto Date: Tue, 10 Sep 2019 11:34:42 +0900 Subject: [PATCH] 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: Shuhei Matsumoto Change-Id: Ic097e37c2d74aa3860bb06931f320c9f18c45640 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/467896 Tested-by: SPDK CI Jenkins Reviewed-by: Changpeng Liu Reviewed-by: Jim Harris --- test/bdev/bdevperf/bdevperf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/bdev/bdevperf/bdevperf.c b/test/bdev/bdevperf/bdevperf.c index 6b122555d..9b818e74f 100644 --- a/test/bdev/bdevperf/bdevperf.c +++ b/test/bdev/bdevperf/bdevperf.c @@ -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;