Looks good. Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@xxxxxxx> On 09/11/2019 01:54 AM, Yi Zhang wrote: > Add one test to cover NVMe SSD rescan/reset/remove operation during > IO, the steps found several issues during my previous testing, check > them here: > http://lists.infradead.org/pipermail/linux-nvme/2017-February/008358.html > http://lists.infradead.org/pipermail/linux-nvme/2017-May/010259.html > > Signed-off-by: Yi Zhang<yi.zhang@xxxxxxxxxx> > > --- > > changes from v2: > - add check_sysfs function for rescan/reset/remove operation > - declare all local variables at the start > - alignment fix > - add udevadm settle > - change to QUICK=1 > changes from v1: > - add variable for "/sys/bus/pci/devices/${pdev}" > - add kill $!; wait; for background fio > - add rescan/reset/remove sysfs node check > - add loop checking for nvme reinitialized > > ---