Re: [PATCH v10 6/6] selftests/mm: add hwpoison-panic destructive test

[David Hildenbrand (Arm)]

On 7/28/26 06:24, Andrew Morton wrote:
> On Thu, 9 Jul 2026 11:34:05 +0800 Miaohe Lin <linmiaohe@xxxxxxxxxx> wrote:
> 
>>> The script is intentionally NOT wired into run_vmtests.sh: every
>>> successful run panics the kernel, which is incompatible with the
>>> sequential "run each category in the same VM" model that
>>> run_vmtests.sh assumes.  It is also not registered as a TEST_PROGS /
>>> ksft_* wrapper so a default kselftest run does not opt itself into
>>> a panic.  The script is meant to be executed manually inside a
>>> disposable VM (e.g. virtme-ng), one variant per VM boot, and
>>> requires RUN_DESTRUCTIVE=1 in the environment as a safety net.
>>>
>>> Signed-off-by: Breno Leitao <leitao@xxxxxxxxxx>
>>
>> With Mike's comment addressed:
>>
>> Acked-by: Miaohe Lin <linmiaohe@xxxxxxxxxx>
> 
> As I understand it, Breno would prefer not to rewrite the selftest in C
> at this time.  IOW, we stick with the bash version for now and add
> "convert this to C" to the todo list, OK?
> 

No, seriously, this is code we have to maintain. Better to not have this test
than have it.

-- 
Cheers,

David