blktests icon indicating copy to clipboard operation
blktests copied to clipboard

common/nvme: remove allowed_hosts before remove subsys in nvmet configfs

Open yizhanglinux opened this issue 1 month ago • 3 comments

$ nvme_trtype=tcp ./check nvme/063 nvme/063 (tr=tcp) (Create authenticated TCP connections with secure concatenation) runtime 2.220s ... WARNING: Test did not clean up port: 0 nvme/063 (tr=tcp) (Create authenticated TCP connections with secure concatenation) [failed] runtime 2.220s ... 2.217sost: nqn.2014-08.org.nvmexpress:uuid:0f01fb42-9f7f-4856-b0b3-51e60b8de349 --- tests/nvme/063.out 2025-11-02 02:47:38.826227836 -0500 +++ /root/blktests/results/nodev_tr_tcp/nvme/063.out.bad 2025-11-02 08:43:09.743150576 -0500 @@ -1,7 +1,3 @@ Running nvme/063 Test secure concatenation with SHA256 -Reset controller -disconnected 1 controller(s) -Test secure concatenation with SHA384 -disconnected 1 controller(s) -Test complete ... (Run 'diff -u tests/nvme/063.out /root/blktests/results/nodev_tr_tcp/nvme/063.out.bad' to see the entire diff)

Link: https://github.com/linux-blktests/blktests/issues/207

yizhanglinux avatar Nov 13 '25 14:11 yizhanglinux

@yizhanglinux I guess the failure of nvme/063 indicates a bug in kernel side, and it should be fixed with this patch. Could you try it out?

kawasaki avatar Nov 16 '25 11:11 kawasaki

@yizhanglinux I guess the failure of nvme/063 indicates a bug in kernel side, and it should be fixed with this patch. Could you try it out?

Yes, just confirmed the failure issue was fixed by the patch.

yizhanglinux avatar Nov 17 '25 06:11 yizhanglinux

I think adding the cleanup code would still make sense, as it makes the test suite more robust, no?

igaw avatar Nov 25 '25 09:11 igaw