From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Shinichiro Kawasaki Subject: Re: [PATCH v3 00/38] ZBD fixes and improvements Date: Fri, 22 Jan 2021 09:24:57 +0000 Message-ID: <20210122092457.kya3nis4g3bs2j6i@shindev.dhcp.fujisawa.hgst.com> References: <20210106215739.264524-1-dmitry.fomichev@wdc.com> In-Reply-To: <20210106215739.264524-1-dmitry.fomichev@wdc.com> Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-ID: <5424CEEB5D31984FBDE1BFA5D32A5C7E@namprd04.prod.outlook.com> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 To: Dmitry Fomichev Cc: Jens Axboe , "fio@vger.kernel.org" , Aravind Ramesh , Bart Van Assche , Naohiro Aota , Niklas Cassel , Damien Le Moal List-ID: On Jan 07, 2021 / 06:57, Dmitry Fomichev wrote: > v2 -> v3: >=20 > - fix two libzbc-related bugs in test scripts (found by Shinichiro). > - modify libzbc ioengine code to allow fio to operate successfully > against null_blk devices while using this ioengine. > - add -l option to run-tests-against-nullb script to allow running > the tests while using libzbc ioengine. > - add -n option to run-tests-against-nullb to allow running the > specified tests multiple times. This option is the most useful > in combination with the existing -s and -t options. > - fix sporadic failed assertion in test #51 that is easily triggered > by running run-tests-against-nullb script with -l and -n options. > - make minor coding style improvements in run-tests-against-nullb. >=20 > v1 -> v2: >=20 > - replace both run-tests-against-conventional-nullb and=20 > run-tests-against-conventional-nullb with a single script, > run-tests-against-nullb, that runs test-zbd-support unit tests > over a variety on different zoned configurations. > - add five new test cases to test-zbd-support to cover the code > changes made in zbd.c as a part of this series. > - mark all test cases in test-zbd-support script that are not > applicable for the device configuration as SKIP instead of > reporting them as passed. > - properly handle writes to conventional zones that cross over > to sequential zones. > - make additional improvements in zone locking parts of zbd.c. > - implement miscellaneous test script enhancements. >=20 > This patch series contains bug fixes and refactoring changes > related to support for Zoned Block Devices (ZBD) in fio. > The highlights: >=20 > - fix several errors related to running workloads that span > a mix of conventional zones and write pointer zones. > - improve counting of sectors with data (SWD). > - remove dependencies on particular zone types in the code. > - add code to gracefully handle offline zones. >=20 > Aravind Ramesh (1): > zbd: initialize sectors with data at start time >=20 > Dmitry Fomichev (25): > zbd: return ENOMEM if zone buffer allocation fails > zbd: use zbd_zone_nr() more actively in the code > zbd: add get_zone() helper function > zbd: introduce zone_unlock() > zbd: engines/libzbc: don't fail on assert for offline zones > zbd: remove dependency on zone type during i/o > zbd: skip offline zones in zbd_convert_to_open_zone() > zbd: avoid zone buffer overrun > zbd: don't unlock zone mutex after verify replay > zbd: use zone_lock() in zbd_process_swd() > zbd: don't log "zone nnnn is not open" message > zbd: handle conventional start zone in zbd_convert_to_open_zone() > zbd: improve replay range validation > engines/libzbc: enable block backend > zbd: avoid failing assertion in zbd_convert_to_open_zone() > zbd: set thread errors in zbd_adjust_block() > t/zbd: check for error in test #2 > t/zbd: add run-tests-against-nullb script Dmitry, thank you for posting this series. I reviewed all of your patches i= n it. Overall, they are all good and I believe they worth up-streaming. One thing= , I think the patch above needs update. Please find my comments on it in other = mail. > t/zbd: add an option to bail on a failed test > t/zbd: prevent test #31 from looping > t/zbd: add checks for offline zone condition > t/zbd: add test #54 to exercise ZBD verification > t/zbd: show elapsed time in test-zbd-support > t/zbd: increase timeout in test #48 > t/zbd: avoid looping on invalid command line options >=20 > Shin'ichiro Kawasaki (12): > zbd: do not lock conventional zones on I/O adjustment > zbd: do not set zbd handlers for conventional zones > zbd: count sectors with data for write pointer zones Regarding my patches, you kindly reviewed them and added your SOB tags, except this patch above. Could you check it again? Thanks! --=20 Best Regards, Shin'ichiro Kawasaki=