[OE-core][PATCH v1] procps: ptest: skip kill test due to race condition

[OE-core][PATCH v1] procps: ptest: skip kill test due to race condition

[Pratik Farkase]

The kill dejagnu test has a race condition where the test_process helper
can exit or become unavailable before the expect script reads its output,
resulting in 'spawn id not open' errors. This causes intermittent
'kill with SIGUSR1' failures on the autobuilder, particularly on
qemuarm64.

This is tracked upstream at https://gitlab.com/procps-ng/procps/-/issues/423

[YOCTO #16263]

Signed-off-by: Pratik Farkase <pratik.farkase@est.tech>
---
 meta/recipes-extended/procps/procps/run-ptest | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/meta/recipes-extended/procps/procps/run-ptest b/meta/recipes-extended/procps/procps/run-ptest
index 7e132653b0..40dbd35265 100644
--- a/meta/recipes-extended/procps/procps/run-ptest
+++ b/meta/recipes-extended/procps/procps/run-ptest
@@ -3,6 +3,13 @@
 # Run testsuite which use dejagnu
 cd ./testsuite
 for tool in @DEJATOOL@; do
+    # Skip the kill test as it has a race condition where test_process
+    # can exit before the expect script reads from it, causing
+    # "spawn id not open" errors (upstream: https://gitlab.com/procps-ng/procps/-/issues/423)
+    if [ "$tool" = "kill" ]; then
+        echo "SKIP: kill (flaky, upstream issue #423)"
+        continue
+    fi
     runtest -a --tool $tool --outdir ../log
 done
 cd -
-- 
2.43.0

[OE-core][PATCH v2] procps: ptest: skip signal-based kill tests due to race condition

[Pratik Farkase]

The kill dejagnu tests that send signals to a pipeproc helper (SIGUSR1,
SIGUSR2, and queued signal) have a race condition where test_process can
exit or become unavailable before the expect script reads its output,
resulting in 'spawn id not open' errors. This causes intermittent
'kill with SIGUSR1' failures on the autobuilder, particularly on
qemuarm64.

Skip only the three signal-based test cases while keeping the nine
reliable argument-parsing tests (signal name/number conversion, etc.).

This is tracked upstream at https://gitlab.com/procps-ng/procps/-/issues/423

[YOCTO #16263]

Signed-off-by: Pratik Farkase <pratik.farkase@est.tech>
---
Changes in v2:
- Skip only the 3 flaky signal-based test cases via a source patch
  instead of skipping the entire kill dejagnu tool in run-ptest
- Keeps the 9 reliable argument-parsing tests running

 ...stsuite-skip-signal-based-kill-tests.patch | 54 +++++++++++++++++++
 meta/recipes-extended/procps/procps_4.0.6.bb  |  1 +
 2 files changed, 55 insertions(+)
 create mode 100644 meta/recipes-extended/procps/procps/0001-testsuite-skip-signal-based-kill-tests.patch

diff --git a/meta/recipes-extended/procps/procps/0001-testsuite-skip-signal-based-kill-tests.patch b/meta/recipes-extended/procps/procps/0001-testsuite-skip-signal-based-kill-tests.patch
new file mode 100644
index 0000000000..6d28f5c36a
--- /dev/null
+++ b/meta/recipes-extended/procps/procps/0001-testsuite-skip-signal-based-kill-tests.patch
@@ -0,0 +1,54 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Pratik Farkase <pratik.farkase@est.tech>
+Date: Mon, 17 Aug 2026 08:00:00 +0000
+Subject: [PATCH] testsuite: skip signal-based kill tests
+
+The kill tests that send signals to a pipeproc helper (SIGUSR1, SIGUSR2,
+and queued signal) have a race condition where test_process can exit or
+become unavailable before the expect script reads its output, resulting
+in "spawn id not open" errors. This causes intermittent failures on
+loaded systems, particularly under QEMU emulation.
+
+Skip these tests with 'untested' while keeping the non-signal kill tests
+(argument parsing, signal name/number conversion) which are reliable.
+
+https://bugzilla.yoctoproject.org/show_bug.cgi?id=16263
+
+Upstream-Status: Inappropriate [upstream ticket: https://gitlab.com/procps-ng/procps/-/issues/423]
+Signed-off-by: Pratik Farkase <pratik.farkase@est.tech>
+---
+ testsuite/kill.test/kill.exp | 21 +++++++--------------
+ 1 file changed, 7 insertions(+), 14 deletions(-)
+
+diff --git a/testsuite/kill.test/kill.exp b/testsuite/kill.test/kill.exp
+index 49bac96..30a602f 100644
+--- a/testsuite/kill.test/kill.exp
++++ b/testsuite/kill.test/kill.exp
+@@ -45,21 +45,9 @@ set test "kill convert signal number to name with space"
+ spawn  $kill -l 1
+ expect_pass "$test" "^HUP\\s*"
+ 
+-make_pipeproc
+-
+-set test "kill with SIGUSR1"
+-spawn $kill -USR1 $pipeproc_pid
+-expect_blank_continue "$test"
+-expect_pipeproc_pass "$test" "SIG SIGUSR1"
+-
+-set test "kill with long SIGUSR2"
+-spawn $kill -s SIGUSR2 $pipeproc_pid
+-expect_blank_continue "$test"
+-expect_pipeproc_pass "$test" "SIG SIGUSR2"
+-
+-set test "kill with queued int"
+-spawn $kill -USR1 -q 42 $pipeproc_pid
+-expect_blank_continue "$test"
+-expect_pipeproc_pass "$test" "SIG SIGUSR1 value=42"
+-
+-kill_pipeproc
++# Skip signal-based tests: they have a race condition where test_process
++# can become unavailable before expect reads its output, causing
++# "spawn id not open" errors under load (upstream issue #423)
++untested "kill with SIGUSR1"
++untested "kill with long SIGUSR2"
++untested "kill with queued int"
diff --git a/meta/recipes-extended/procps/procps_4.0.6.bb b/meta/recipes-extended/procps/procps_4.0.6.bb
index 2437be1c3a..5cb5a2831d 100644
--- a/meta/recipes-extended/procps/procps_4.0.6.bb
+++ b/meta/recipes-extended/procps/procps_4.0.6.bb
@@ -15,6 +15,7 @@ inherit autotools gettext pkgconfig update-alternatives ptest
 SRC_URI = "git://gitlab.com/procps-ng/procps.git;protocol=https;branch=master;tag=v${PV} \
            file://sysctl.conf \
            file://0001-tests-Disable-twice-total-pmap-X-tests.patch \
+           file://0001-testsuite-skip-signal-based-kill-tests.patch \
            file://run-ptest \
            "
 SRCREV = "4dafddf4c3f4646caa517f039a2307e92657ec93"
-- 
2.43.0