Re: [PATCH] selftests/damon/sysfs.py: stop obsolete_target shells hanging the test

[SJ Park <sj@kernel.org>]

On Sun, 18 Oct 2026 04:04:15 +0200 Jaeyeon Lee <jaeyeon.lee.dev@gmail.com> wrote:

> The obsolete_target test spawns three 'sh' processes as monitoring
> targets.  The processes inherit stdin from the test.  When the test is
> run via 'make run_tests' from an interactive terminal, the kselftest
> runner executes it under 'timeout', which puts it in a background
> process group.  The 'sh' processes then try to read the terminal, so
> the kernel sends SIGTTIN to the whole process group.  This stops the
> 'sh' processes, and also the child the test has just forked to spawn
> the next 'sh', before it can run 'sh'.  The test then waits on that
> child forever, until the kselftest runner kills it after 45 seconds.
> The test passes when run directly.
> 
> Fix this by passing stdin=subprocess.PIPE when spawning the processes.
> The processes then read from a pipe that never receives input, so they
> stay alive and idle as the test expects.

Thank you for finding and fixing the bug!

> 
> Assisted-by: LLM
> Signed-off-by: Jaeyeon Lee <jaeyeon.lee.dev@gmail.com>

Reviewed-by: SJ Park <sj@kernel.org>

I applied  this patch to damon/next [1] tree.  We are now quite close to next
merge window.  It may better to focus on making mm.git more stabilized and
therefore ready for the next merge window, rather than adding more changes that
are not really urgent.  I understand this series is not really urgent.  Hence,
Andrew might not add this patch until next -rc1 release.  In the case, I will
request adding this to mm.git after next -rc1 release.  So, no action from your
side is needed for now.  Let me know if you think this is really  urgent.

[1] https://origin.kernel.org/doc/html/latest/mm/damon/maintainer-profile.html#scm-trees


Thanks,
SJ

[...]