diff options
| author | Alexey Dobriyan <adobriyan@gmail.com> | 2021-02-22 16:21:22 +0300 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2021-02-22 07:35:34 -0700 |
| commit | 4316f6409721bc42e878df1057de0e4be7df3d44 (patch) | |
| tree | 0b4d37404a77752f75acb5e14a3f8382de9941b0 /zbd.c | |
| parent | b75c0fae66129883dc554fecb6eaa4b35938c14e (diff) | |
| download | fio-4316f6409721bc42e878df1057de0e4be7df3d44.tar.gz fio-4316f6409721bc42e878df1057de0e4be7df3d44.tar.bz2 | |
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>
Diffstat (limited to 'zbd.c')
| -rw-r--r-- | zbd.c | 4 |
1 files changed, 1 insertions, 3 deletions
@@ -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); |