diff mbox series

elfutils: skip run-stackprof-system.sh ptest under qemu

Message ID 20261030133350.20515-1-jaipaul.cheernam@est.tech
State New
Headers show
Series elfutils: skip run-stackprof-system.sh ptest under qemu | expand

Commit Message

Jaipaul Cheernam Oct. 30, 2026, 1:33 p.m. UTC
Non-deterministic under qemu as needs userspace perf samples in a fixed window
Issue already reported upstream and updated ticket with details

Reported upstream: https://sourceware.org/bugzilla/show_bug.cgi?id=34544

Skip test case via ELFUTILS_PTEST_SKIP

ptest results:
Before:
glibc: 263 PASSED, 45 SKIPPED, 0 FAILED
musl: 255 PASSED, 44 SKIPPED, 0 FAILED
clang: 263 PASSED, 45 SKIPPED, 0 FAILED

with change:
glibc: 262 PASSED, 46 SKIPPED, 0 FAILED
musl: 254 PASSED, 45 SKIPPED, 0 FAILED,
clang: 262 PASSED, 46 SKIPPED, 0 FAILED

[YOCTO #16473]

Signed-off-by: Jaipaul Cheernam <jaipaul.cheernam@est.tech>
---
 meta/recipes-devtools/elfutils/elfutils_0.197.bb | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Joao Marcos Costa Oct. 30, 2026, 2:52 p.m. UTC | #1
Hello, Jaipaul

On 10/30/26 2:33 PM, Jaipaul Cheernam wrote:
> Non-deterministic under qemu as needs userspace perf samples in a fixed window
> Issue already reported upstream and updated ticket with details
> 
> Reported upstream: https://sourceware.org/bugzilla/show_bug.cgi?id=34544
> 
> Skip test case via ELFUTILS_PTEST_SKIP
> 
> ptest results:
> Before:
> glibc: 263 PASSED, 45 SKIPPED, 0 FAILED
> musl: 255 PASSED, 44 SKIPPED, 0 FAILED
> clang: 263 PASSED, 45 SKIPPED, 0 FAILED
> 
> with change:
> glibc: 262 PASSED, 46 SKIPPED, 0 FAILED
> musl: 254 PASSED, 45 SKIPPED, 0 FAILED,
> clang: 262 PASSED, 46 SKIPPED, 0 FAILED
> 
> [YOCTO #16473]
> 
> Signed-off-by: Jaipaul Cheernam <jaipaul.cheernam@est.tech>
> ---
>   meta/recipes-devtools/elfutils/elfutils_0.197.bb | 10 ++++++++++
>   1 file changed, 10 insertions(+)
(...)

Thanks for the patch.

Out of curiosity, why using an :append (unconditional) for 
do_install_ptest() instead of simply adding the chunk of code into the 
task itself?

Reviewed-by: João Marcos Costa <joaomarcos.costa@bootlin.com>
diff mbox series

Patch

diff --git a/meta/recipes-devtools/elfutils/elfutils_0.197.bb b/meta/recipes-devtools/elfutils/elfutils_0.197.bb
index 6e58afb2745..f309a3643fe 100644
--- a/meta/recipes-devtools/elfutils/elfutils_0.197.bb
+++ b/meta/recipes-devtools/elfutils/elfutils_0.197.bb
@@ -65,6 +65,9 @@  PTEST_XFAILS ?= ""
 PTEST_XFAILS:append = "${@bb.utils.contains('DISTRO_FEATURES', 'ld-is-lld', ' run-strip-strmerge.sh run-elflint-self.sh run-backtrace-data.sh run-reverse-sections-self.sh', '', d)}"
 PTEST_XFAILS:append:libc-musl = " run-large-elf-file.sh run-backtrace-data.sh run-backtrace-native.sh run-backtrace-native-core.sh run-stack-d-test.sh run-stack-i-test.sh run-stack-demangled-test.sh run-deleted.sh run-compress-test.sh"
 
+ELFUTILS_PTEST_SKIP ?= ""
+ELFUTILS_PTEST_SKIP:append:qemuall = " run-stackprof-system.sh"
+
 do_install_ptest() {
 	# copy the files which needed by the cases
 	TEST_FILES="strip strip.o addr2line elfcmp objdump readelf size.o nm.o nm elflint elfcompress elfclassify stack unstrip srcfiles stackprof"
@@ -106,6 +109,13 @@  do_install_ptest() {
 	sed -i -e 's|@XFAIL_TESTS@|${PTEST_XFAILS}|' ${D}${PTEST_PATH}/run-ptest
 }
 
+do_install_ptest:append() {
+	for t in ${ELFUTILS_PTEST_SKIP}; do
+		sed -i -e '\|stackprof-subr.sh|a echo "unreliable under qemu, skipping (PR34544)"; exit 77' \
+			${D}${PTEST_PATH}/tests/$t
+	done
+}
+
 EXTRA_OEMAKE:class-native = ""
 EXTRA_OEMAKE:class-nativesdk = ""