On Tue, May 02, 2023 at 05:45:46PM +0200, Daniel Wagner wrote: > On Fri, Apr 28, 2023 at 04:00:54AM +0000, Shinichiro Kawasaki wrote: > > On Apr 21, 2023 / 08:05, Daniel Wagner wrote: > > > Introduce two new function to calculate the IO size for fio jobs. > > > > > > _nvme_calc_io_size() returns the jobs size for _run_fio_verify_io() > > > function. Reduce the max size of the job by one megabyte to make the > > > test more robust not to run out of space by accident. Note these fio > > > calls run with just one jobs. > > > > It is not clear for me what kind of issue happens without the 1MB decrement. > > Could you share failure symptoms you observed? > > As I said, this is just to make the test more robust as this the size limits > are not the main objective of these tests. I don't care about this too > much, I'll just drop it then. BTW, this is how it would look like if the disk is too small: nvme/035 => nvme0n1 (run mkfs and data verification fio job on an NVMeOF passthru controller) [failed] runtime 2.383s ... 51.954s --- tests/nvme/035.out 2023-04-18 17:43:18.163745956 +0200 +++ /home/wagi/work/blktests/results/nvme0n1/nvme/035.out.bad 2023-05-02 18:21:09.442382196 +0200 @@ -1,3 +1,20 @@ Running nvme/035 +fio: io_u error on file /mnt/blktests//verify.0.0: No space left on device: write offset=925274112, buflen=4096 +fio: io_u error on file /mnt/blktests//verify.0.0: No space left on device: write offset=406040576, buflen=4096 +fio: io_u error on file /mnt/blktests//verify.0.0: No space left on device: write offset=498868224, buflen=4096 +fio: io_u error on file /mnt/blktests//verify.0.0: No space left on device: write offset=217063424, buflen=4096 +fio: io_u error on file /mnt/blktests//verify.0.0: No space left on device: write offset=1049411584, buflen=4096 +fio: io_u error on file /mnt/blktests//verify.0.0: No space left on device: write offset=348282880, buflen=4096 ... (Run 'diff -u tests/nvme/035.out /home/wagi/work/blktests/results/nvme0n1/nvme/035.out.bad' to see the entire diff)