On Sun, Aug 20, 2023 at 05:58:34PM +0300, Sagi Grimberg wrote: > > diff --git a/tests/nvme/003 b/tests/nvme/003 > > index 71b82ce758a3..eed1f549866a 100755 > > --- a/tests/nvme/003 > > +++ b/tests/nvme/003 > > @@ -22,15 +22,8 @@ test() { > > _setup_nvmet > > - local loop_dev > > - local port > > - port="$(_create_nvmet_port "${nvme_trtype}")" > > - > > - loop_dev="$(losetup -f)" > > - > > - _create_nvmet_subsystem "${def_subsysnqn}" "${loop_dev}" > > - _add_nvmet_subsys_to_port "${port}" "${def_subsysnqn}" > > + _nvmet_target_setup --blkdev=device > > --blkdev=device by default no? Yes. I thought it is better to be explicit in the tests. I don't mind dropping --blkdev=device if you think we should use the defaults.