From 1a8d510a69ea95b8bdc4c05f49f257b7cafddd02 Mon Sep 17 00:00:00 2001 From: Dmitry Fomichev Date: Wed, 27 Jan 2021 13:19:28 +0900 Subject: [PATCH] zbd: avoid failing assertion in zbd_convert_to_open_zone() The test run against null_blk with the following command line - t/zbd/run-tests-against-nullb -l -q -s 12 -t 51 -n 100 stops with a failure and the message below can be seen in the test log: fio: zbd.c:1110: zbd_convert_to_open_zone: Assertion `open_zone_idx < f->zbd_info->num_open_zones' failed. This assertion fails because pick_random_zone_idx() function returns index 0 if no zones are currently open. In this case, open_zone_idx and f->zbd_info->num_open_zones are both zero. Since this situation is normal, simply modify the assert statement to avoid failing. Signed-off-by: Dmitry Fomichev Reviewed-by: Shin'ichiro Kawasaki Signed-off-by: Jens Axboe --- zbd.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/zbd.c b/zbd.c index 1ee238e8..8583f2ac 100644 --- a/zbd.c +++ b/zbd.c @@ -1107,7 +1107,8 @@ static struct fio_zone_info *zbd_convert_to_open_zone(struct thread_data *td, * Ignore zones which don't belong to thread's offset/size area. */ open_zone_idx = pick_random_zone_idx(f, io_u); - assert(open_zone_idx < f->zbd_info->num_open_zones); + assert(!open_zone_idx || + open_zone_idx < f->zbd_info->num_open_zones); tmp_idx = open_zone_idx; for (i = 0; i < f->zbd_info->num_open_zones; i++) { uint32_t tmpz; -- 2.25.1