Re: [PATCH v3 0/4] nvmet-fcloop: unblock module removal

From: Daniel Wagner
Date: Wed Apr 19 2023 - 05:03:42 EST


On Tue, Apr 18, 2023 at 04:26:27PM +0200, Daniel Wagner wrote:
> On Tue, Apr 18, 2023 at 03:43:22PM +0200, Daniel Wagner wrote:
> > On Tue, Apr 18, 2023 at 03:01:55PM +0200, Daniel Wagner wrote:
> > > nvme/041 (Create authenticated connections) [failed]
> > > nvme/042 (Test dhchap key types for authenticated connections) [failed]
> > > nvme/043 (Test hash and DH group variations for authenticated connections) [passed]
> > > nvme/044 (Test bi-directional authentication) [failed]
> > > nvme/045 (Test re-authentication) [passed]
> >
> > I suppose these should be disabled for fc as all this is tcp specific.
>
> After a fresh reboot the deleter tport, lport and rport trouble is back...
>
> nvme/003 (test if we're sending keep-alives to a discovery controller) [passed]
> runtime 10.265s ... 10.365s
> tests/nvme/rc: line 198: /sys/class/fcloop/ctl/del_target_port: No such file or directory
> tests/nvme/rc: line 190: /sys/class/fcloop/ctl/del_local_port: No such file or directory
> tests/nvme/rc: line 182: /sys/class/fcloop/ctl/del_remote_port: No such file or directory

Eventually, I figured the out the root problem. The modules got unloaded before
the resource were freed. This explains a lot of the nasty problems I saw.
Anyway, I posted an updated blktests fixes but I think we should still consider
these patches here.

https://lore.kernel.org/linux-nvme/20230419084757.24846-1-dwagner@xxxxxxx/

BTW, the authentication tests fail for fc, but not for the rest. And after
reading up on it, it supposed to work on fc as well. So here we go first real
bugs found.