Re: [PATCH v10 6/6] selftests/mm: add hwpoison-panic destructive test
Miaohe Lin <[email protected]> Tue, 28 Jul 2026 16:57:13 +0800
| Newsgroups | org.kernel.vger.linux-trace-kernel,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel,org.kernel.vger.linux-kselftest,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
On 2026/7/28 12:24, Andrew Morton wrote: > On Thu, 9 Jul 2026 11:34:05 +0800 Miaohe Lin <[email protected]> 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 <[email protected]> >> >> With Mike's comment addressed: >> >> Acked-by: Miaohe Lin <[email protected]> > > 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? Sure. Should I make this change or should we wait until Breno has time to do it? Thanks. .