All of lore.kernel.org
 help / color / mirror / Atom feed
* [OE-core][PATCH v1] procps: ptest: skip flaky pgrep full process name match test
@ 2026-08-18  8:34 Pratik Farkase
  2026-08-20 10:28 ` Alexander Kanavin
  0 siblings, 1 reply; 3+ messages in thread
From: Pratik Farkase @ 2026-08-18  8:34 UTC (permalink / raw)
  To: openembedded-core; +Cc: pratik.farkase, Pratik Farkase

The 'pgrep match against full process name' test uses pgrep -f with a
regex pattern close to ARG_MAX in size (~100KB). This intermittently
fails on qemuriscv64 where the process cmdline may not be fully visible
in /proc/<pid>/cmdline at the time pgrep reads it, or the large regex
match times out under TCG emulation.

Skip this single test case while keeping all other 30+ pgrep tests
which are reliable.

[YOCTO #16290]

Signed-off-by: Pratik Farkase <pratik.farkase@est.tech>
---
 ...p-pgrep-full-process-name-match-test.patch | 39 +++++++++++++++++++
 meta/recipes-extended/procps/procps_4.0.6.bb  |  1 +
 2 files changed, 40 insertions(+)
 create mode 100644 meta/recipes-extended/procps/procps/0001-testsuite-skip-pgrep-full-process-name-match-test.patch

diff --git a/meta/recipes-extended/procps/procps/0001-testsuite-skip-pgrep-full-process-name-match-test.patch b/meta/recipes-extended/procps/procps/0001-testsuite-skip-pgrep-full-process-name-match-test.patch
new file mode 100644
index 0000000000..43c930ce08
--- /dev/null
+++ b/meta/recipes-extended/procps/procps/0001-testsuite-skip-pgrep-full-process-name-match-test.patch
@@ -0,0 +1,39 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Pratik Farkase <pratik.farkase@est.tech>
+Date: Mon, 17 Aug 2026 10:00:00 +0000
+Subject: [PATCH] testsuite: skip pgrep full process name match test
+
+The "pgrep match against full process name" test uses pgrep -f with a
+regex pattern close to ARG_MAX in size (~100KB). This intermittently
+fails on qemuriscv64 where the process cmdline may not be fully visible
+in /proc/<pid>/cmdline at the time pgrep reads it, or the large regex
+match times out under TCG emulation.
+
+Skip this single test case while keeping all other pgrep tests which
+are reliable.
+
+https://bugzilla.yoctoproject.org/show_bug.cgi?id=16290
+
+Upstream-Status: Inappropriate [OE-ptest specific: fails only under QEMU TCG emulation]
+Signed-off-by: Pratik Farkase <pratik.farkase@est.tech>
+---
+ testsuite/pgrep.test/pgrep.exp | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/testsuite/pgrep.test/pgrep.exp b/testsuite/pgrep.test/pgrep.exp
+index 54e75df..66edbaa 100644
+--- a/testsuite/pgrep.test/pgrep.exp
++++ b/testsuite/pgrep.test/pgrep.exp
+@@ -35,9 +35,9 @@ set test "pgrep with : delimiter"
+ spawn $pgrep -d : $testproc_comm
+ expect_pass "$test" "^${testproc1_pid}:${testproc2_pid}\\s*$"
+ 
+-set test "pgrep match against full process name"
+-spawn $pgrep -f "$testproc_path\\s+$testproc_arg_str"
+-expect_pass "$test" "^$testproc1_pid\\s*$"
++# Skip: intermittent failure on riscv64 - pgrep -f with large ARG_MAX
++# patterns can fail under QEMU emulation (timing/cmdline visibility race)
++untested "pgrep match against full process name"
+ 
+ set test "pgrep with matching gid"
+ spawn $pgrep -G $gid $testproc_comm
diff --git a/meta/recipes-extended/procps/procps_4.0.6.bb b/meta/recipes-extended/procps/procps_4.0.6.bb
index 5cb5a2831d..1f0203d111 100644
--- a/meta/recipes-extended/procps/procps_4.0.6.bb
+++ b/meta/recipes-extended/procps/procps_4.0.6.bb
@@ -16,6 +16,7 @@ SRC_URI = "git://gitlab.com/procps-ng/procps.git;protocol=https;branch=master;ta
            file://sysctl.conf \
            file://0001-tests-Disable-twice-total-pmap-X-tests.patch \
            file://0001-testsuite-skip-signal-based-kill-tests.patch \
+           file://0001-testsuite-skip-pgrep-full-process-name-match-test.patch \
            file://run-ptest \
            "
 SRCREV = "4dafddf4c3f4646caa517f039a2307e92657ec93"
-- 
2.43.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [OE-core][PATCH v1] procps: ptest: skip flaky pgrep full process name match test
  2026-08-18  8:34 [OE-core][PATCH v1] procps: ptest: skip flaky pgrep full process name match test Pratik Farkase
@ 2026-08-20 10:28 ` Alexander Kanavin
  2026-08-20 11:07   ` Pratik Farkase
  0 siblings, 1 reply; 3+ messages in thread
From: Alexander Kanavin @ 2026-08-20 10:28 UTC (permalink / raw)
  To: pratik.farkase; +Cc: openembedded-core, pratik.farkase

On Tue, 18 Aug 2026 at 10:34, Pratik Farkase via
lists.openembedded.org
<pratik.farkase=est.tech@lists.openembedded.org> wrote:
> +Subject: [PATCH] testsuite: skip pgrep full process name match test
> +
> +The "pgrep match against full process name" test uses pgrep -f with a
> +regex pattern close to ARG_MAX in size (~100KB). This intermittently
> +fails on qemuriscv64 where the process cmdline may not be fully visible
> +in /proc/<pid>/cmdline at the time pgrep reads it, or the large regex
> +match times out under TCG emulation.
> +
> +Skip this single test case while keeping all other pgrep tests which
> +are reliable.
> +
> +https://bugzilla.yoctoproject.org/show_bug.cgi?id=16290
> +
> +Upstream-Status: Inappropriate [OE-ptest specific: fails only under QEMU TCG emulation]

Hello Pratik,

even though this shows up only under slow software emulation, it's
still a race condition in the test, and should be reported upstream,
so they can figure out how to sync the test properly. Can you please
open a ticket, and update the patch with the link?

Alex


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [OE-core][PATCH v1] procps: ptest: skip flaky pgrep full process name match test
  2026-08-20 10:28 ` Alexander Kanavin
@ 2026-08-20 11:07   ` Pratik Farkase
  0 siblings, 0 replies; 3+ messages in thread
From: Pratik Farkase @ 2026-08-20 11:07 UTC (permalink / raw)
  To: Alexander Kanavin
  Cc: openembedded-core@lists.openembedded.org,
	pratik.farkase@ericsson.com

Hello Alex,

Thanks for the review!

It abolutely makes sense i have filed the ticket here : https://gitlab.com/procps-ng/procps/-/work_items/435

Updated the patch with link too here : https://lists.openembedded.org/g/openembedded-core/message/243853

Best Regards,
Pratik

________________________________________
From: Alexander Kanavin <alex.kanavin@gmail.com>
Sent: Thursday, August 20, 2026 12:28 PM
To: Pratik Farkase
Cc: openembedded-core@lists.openembedded.org; pratik.farkase@ericsson.com
Subject: Re: [OE-core][PATCH v1] procps: ptest: skip flaky pgrep full process name match test

On Tue, 18 Aug 2026 at 10:34, Pratik Farkase via
lists.openembedded.org
<pratik.farkase=est.tech@lists.openembedded.org> wrote:
> +Subject: [PATCH] testsuite: skip pgrep full process name match test
> +
> +The "pgrep match against full process name" test uses pgrep -f with a
> +regex pattern close to ARG_MAX in size (~100KB). This intermittently
> +fails on qemuriscv64 where the process cmdline may not be fully visible
> +in /proc/<pid>/cmdline at the time pgrep reads it, or the large regex
> +match times out under TCG emulation.
> +
> +Skip this single test case while keeping all other pgrep tests which
> +are reliable.
> +
> +https://bugzilla.yoctoproject.org/show_bug.cgi?id=16290
> +
> +Upstream-Status: Inappropriate [OE-ptest specific: fails only under QEMU TCG emulation]

Hello Pratik,

even though this shows up only under slow software emulation, it's
still a race condition in the test, and should be reported upstream,
so they can figure out how to sync the test properly. Can you please
open a ticket, and update the patch with the link?

Alex


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-08-20 11:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-18  8:34 [OE-core][PATCH v1] procps: ptest: skip flaky pgrep full process name match test Pratik Farkase
2026-08-20 10:28 ` Alexander Kanavin
2026-08-20 11:07   ` Pratik Farkase

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.