Re: [PATCH blktests 2/2] block/011: Perform PCI reset while doing IO

Jens Axboe
2017-06-23T15:36:14Z

On 06/23/2017 08:29 AM, Johannes Thumshirn wrote:
> From: Omar Sandoval <osandov@fb.com>
>
> This test-case performs I/O with fio while doing PCI disable/enable
> cycles.
>
> In the results we don't care for I/O errors but for hiccups in dmesg only.

Let's get this in, that would be a very useful test. A few comments -
not necessarily on this patch in particular, but for future cleanups
and improvements.

> + if _test_dev_is_rotational; then
> + size="32m"
> + else
> + size="1g"
> + fi

I introduced this idea in one of my previous patches. I wonder if we
should turn that into a helper. Pass in the dev, get returned a
suitable fio size, instead of hard coding this in each job that
needs it.

> + # start fio job
> + _run_fio --bs=4k --rw=randread --norandommap \
> + --name=reads --filename="$TEST_DEV" --size="$size" \
> + --numjobs=8 --direct=1 2>/dev/null &

I don't believe we check for fio errors right now, but we probably
should in the future. So I think you'd want to add something ala:

--ignore_error=EIO,ENXIO,ENODEV

to your options to make it explicit that you don't care about IO
errors for this test.

--
Jens Axboe