Re:
From: Shuah Khan
Date: Mon Nov 02 2026 - 16:24:40 EST
On 10/31/26 12:20, Regen Yang wrote:
run_kselftest.sh sources runner.sh under /bin/sh, but run_in_netns()
uses Bash's &> redirection for running a test and deleting its namespace.
With dash, this backgrounds the command and performs a separate
redirection-only command. Consequently, rc captures the successful
redirection instead of the test status, and namespace cleanup can race
with test execution.
A failing test run with run_kselftest.sh -n can therefore be counted as a
pass and return success. The totals can also appear before the test
result, and skipped and expected-failure tests are counted as passes.
Use portable stdout and stderr redirections so the runner waits for the
test and namespace cleanup, preserves the test status, and reads the log
after the test has finished.
Please send a proper patch with patch summary? Add output before and
after to the commit log.
thanks,
-- Shuah