diff mbox series

[2/2] ltp: Try re-enabling problematic test

Message ID 20240206102246.2569704-2-richard.purdie@linuxfoundation.org
State Accepted, archived
Commit 8832ff65ba91587fbe739173bb2d33119a1c6f1b
Headers show
Series [1/2] ltp: Enable extra test groups | expand

Commit Message

Richard Purdie Feb. 6, 2024, 10:22 a.m. UTC 
The proc01 test used to hang when reading /proc/kmsg. This issue could
have been a number of issues which were fixed in the meantime. Try
re-enabling the test to see if the issue is still present.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 .../ltp/ltp/disable_hanging_tests.patch       | 27 -------------------
 meta/recipes-extended/ltp/ltp_20240129.bb     |  1 -
 2 files changed, 28 deletions(-)
 delete mode 100644 meta/recipes-extended/ltp/ltp/disable_hanging_tests.patch

Comments

Petr Vorel Feb. 6, 2024, 5:48 p.m. UTC | #1 
Hi Richard,

> The proc01 test used to hang when reading /proc/kmsg. This issue could
> have been a number of issues which were fixed in the meantime. Try
> re-enabling the test to see if the issue is still present.

Reviewed-by: Petr Vorel <petr.vorel@gmail.com>

Crossed fingers to get it either fixed or logs which shows what's wrong.

Thanks!

Kind regards,
Petr
diff mbox series

Patch

diff --git a/meta/recipes-extended/ltp/ltp/disable_hanging_tests.patch b/meta/recipes-extended/ltp/ltp/disable_hanging_tests.patch
deleted file mode 100644
index f25a692dc0f..00000000000
--- a/meta/recipes-extended/ltp/ltp/disable_hanging_tests.patch
+++ /dev/null
@@ -1,27 +0,0 @@ 
-This patch disables tests which we've found "hang" on our infrastructure.
-
-proc01:
-https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/1748 (arm)
-https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/1781 (arm)
-https://autobuilder.yoctoproject.org/typhoon/#/builders/96/builds/2782 (arm)
-(was trying to read /proc/kmsg, looks like a horrible test anyway)
-
-Upstream-Status: Inappropriate [OE Configuration]
-Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-
-Index: git/runtest/fs
-===================================================================
---- git.orig/runtest/fs
-+++ git/runtest/fs
-@@ -64,11 +64,6 @@ writetest01	writetest
- #Also run the fs_di (Data Integrity tests)
- fs_di fs_di -d $TMPDIR
- 
--# Read every file in /proc. Not likely to crash, but does enough
--# to disturb the kernel. A good kernel latency killer too.
--# Was not sure why it should reside in runtest/crashme and won't get tested ever
--proc01 proc01 -m 128
--
- read_all_dev read_all -d /dev -p -q -r 3
- read_all_proc read_all -d /proc -q -r 3
- read_all_sys read_all -d /sys -q -r 3
diff --git a/meta/recipes-extended/ltp/ltp_20240129.bb b/meta/recipes-extended/ltp/ltp_20240129.bb
index 5cd33bb6a38..acd857f4a90 100644
--- a/meta/recipes-extended/ltp/ltp_20240129.bb
+++ b/meta/recipes-extended/ltp/ltp_20240129.bb
@@ -28,7 +28,6 @@  SRCREV = "68737d20556d37364c95776044b1119c0912a36a"
 
 SRC_URI = "git://github.com/linux-test-project/ltp.git;branch=master;protocol=https \
            file://0001-Remove-OOM-tests-from-runtest-mm.patch \
-           file://disable_hanging_tests.patch \
            "
 
 S = "${WORKDIR}/git"