[PATCH v2 6/7] common/multipath-over-rdma: don't retry module unload
Sagi Grimberg
sagi at grimberg.me
Fri Aug 7 13:23:24 EDT 2020
>> There is no need to retry module unload for rdma_rxe
>> and siw. This also creates a dependency on
>> tests/nvmeof/rc which prevents it from using in
>> other test subsystems.
>
> If it wouldn't be necessary to retry module unload I wouldn't have
> introduced a loop.
I thought it was to work-around a driver issue as these drivers
traditionally had plenty of stability issues.
To be honest this retry loop to me indicated that either the
driver has a bug or the test. But maybe there is a need I
am not seeing.
> How about moving the unload_module() function definitions from tests/srp/rc
> and tests/nvmeof-mp/rc into common/rc instead?
Don't have an issue with that.
More information about the Linux-nvme
mailing list