From: Alexey Dobriyan <adobriyan@gmail.com>
Date: Mon, 22 Feb 2021 13:21:22 +0000 (+0300)
Subject: zbd: simplify zoneskip= validness check
X-Git-Tag: fio-3.26~6
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=4316f6409721bc42e878df1057de0e4be7df3d44;p=fio.git

zbd: simplify zoneskip= validness check

Simply check the remainder:

	(zoneskip % zone_size) > 0

It will do the right thing for all zoneskip= values, and
zone size being positive is checked earlier.

Reviewed-by: Dmitry Fomichev <dmitry.fomichev@wdc.com>
Signed-off-by: Alexey Dobriyan (SK hynix) <adobriyan@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
---

diff --git a/zbd.c b/zbd.c
index 6a26fe10..a93a977e 100644
--- a/zbd.c
+++ b/zbd.c
@@ -285,9 +285,7 @@ static bool zbd_verify_sizes(void)
 				return false;
 			}
 
-			if (td->o.zone_skip &&
-			    (td->o.zone_skip < td->o.zone_size ||
-			     td->o.zone_skip % td->o.zone_size)) {
+			if (td->o.zone_skip % td->o.zone_size) {
 				log_err("%s: zoneskip %llu is not a multiple of the device zone size %llu.\n",
 					f->file_name, (unsigned long long) td->o.zone_skip,
 					(unsigned long long) td->o.zone_size);