Re: [PATCH blktests 0/2] add nvme test for creating sleep while atomic kernel BUG

Chaitanya Kulkarni
2024-12-03T04:20:34Z

On 11/30/24 01:10, Shinichiro Kawasaki wrote:
> On Nov 29, 2024 / 13:31, Nilay Shroff wrote:
>> Hi,
>>
>> There're two patches in this series. The first patch is a preparation patch
>> for reusing a common function nvmf_wait_for_ns from multiple nvme test scripts.
>> The second patch adds a new nvme regression[1] test for commit 505363957fad
>> ("nvmet: fix nvme status code when namespace is disabled").
> Hi Nilay, thank your very much for the fix actions. Much appreciated.
>
> I tried these blktests patches with the kernel just before the commit
> 505363957fad, at the git hash 6825bdde4434. I expected the new test case fails,
> but it passes. I increased the number of iterations from 10 to 100, but it still
> passes. Do you observe the test case failure on your test systems?

testcase seems really useful to me, but we need to tune it in a such way
it will
have consistent result yet finding the bug in absence of the fix.

-ck