diff options
| -rw-r--r-- | meta/recipes-devtools/strace/strace/0001-landlock-update-expected-string.patch | 67 | ||||
| -rw-r--r-- | meta/recipes-devtools/strace/strace/skip-load.patch | 40 | ||||
| -rw-r--r-- | meta/recipes-devtools/strace/strace_5.18.bb (renamed from meta/recipes-devtools/strace/strace_5.17.bb) | 3 |
3 files changed, 18 insertions, 92 deletions
diff --git a/meta/recipes-devtools/strace/strace/0001-landlock-update-expected-string.patch b/meta/recipes-devtools/strace/strace/0001-landlock-update-expected-string.patch deleted file mode 100644 index 9d67d68331..0000000000 --- a/meta/recipes-devtools/strace/strace/0001-landlock-update-expected-string.patch +++ /dev/null | |||
| @@ -1,67 +0,0 @@ | |||
| 1 | From d0dae2fb30b907bc9bf70382f37c9e00207ae036 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Bruce Ashfield <bruce.ashfield@gmail.com> | ||
| 3 | Date: Sat, 30 Apr 2022 01:09:42 -0400 | ||
| 4 | Subject: [PATCH] landlock: update expected string | ||
| 5 | MIME-Version: 1.0 | ||
| 6 | Content-Type: text/plain; charset=UTF-8 | ||
| 7 | Content-Transfer-Encoding: 8bit | ||
| 8 | |||
| 9 | Kernel commit: | ||
| 10 | |||
| 11 | commit 3d4b396a616d0d67bf95d6823ad1197f6247292e | ||
| 12 | Author: Christian Brauner <christian.brauner@ubuntu.com> | ||
| 13 | Date: Mon Oct 11 15:37:04 2021 +0200 | ||
| 14 | |||
| 15 | landlock: Use square brackets around "landlock-ruleset" | ||
| 16 | |||
| 17 | commit aea0b9f2486da8497f35c7114b764bf55e17c7ea upstream. | ||
| 18 | |||
| 19 | Make the name of the anon inode fd "[landlock-ruleset]" instead of | ||
| 20 | "landlock-ruleset". This is minor but most anon inode fds already | ||
| 21 | carry square brackets around their name: | ||
| 22 | |||
| 23 | [eventfd] | ||
| 24 | [eventpoll] | ||
| 25 | [fanotify] | ||
| 26 | [fscontext] | ||
| 27 | [io_uring] | ||
| 28 | [pidfd] | ||
| 29 | [signalfd] | ||
| 30 | [timerfd] | ||
| 31 | [userfaultfd] | ||
| 32 | |||
| 33 | For the sake of consistency lets do the same for the landlock-ruleset anon | ||
| 34 | inode fd that comes with landlock. We did the same in | ||
| 35 | 1cdc415f1083 ("uapi, fsopen: use square brackets around "fscontext" [ver #2]") | ||
| 36 | for the new mount api. | ||
| 37 | |||
| 38 | Cc: linux-security-module@vger.kernel.org | ||
| 39 | Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com> | ||
| 40 | Link: https://lore.kernel.org/r/20211011133704.1704369-1-brauner@kernel.org | ||
| 41 | Cc: stable@vger.kernel.org | ||
| 42 | Signed-off-by: Mickaël Salaün <mic@linux.microsoft.com> | ||
| 43 | Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> | ||
| 44 | |||
| 45 | Changed the format of the landlock tracing. We need to update the strace | ||
| 46 | expected string to match. | ||
| 47 | |||
| 48 | Upstream-Status: Submitted [https://lists.strace.io/pipermail/strace-devel/2022-April/011064.html] | ||
| 49 | |||
| 50 | Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com> | ||
| 51 | --- | ||
| 52 | tests/landlock_create_ruleset-y.c | 2 +- | ||
| 53 | 1 file changed, 1 insertion(+), 1 deletion(-) | ||
| 54 | |||
| 55 | diff --git a/tests/landlock_create_ruleset-y.c b/tests/landlock_create_ruleset-y.c | ||
| 56 | index a30966b..50e19c2 100644 | ||
| 57 | --- a/tests/landlock_create_ruleset-y.c | ||
| 58 | +++ b/tests/landlock_create_ruleset-y.c | ||
| 59 | @@ -1,4 +1,4 @@ | ||
| 60 | -#define FD_PATH "<anon_inode:landlock-ruleset>" | ||
| 61 | +#define FD_PATH "<anon_inode:[landlock-ruleset]>" | ||
| 62 | #define SKIP_IF_PROC_IS_UNAVAILABLE skip_if_unavailable("/proc/self/fd/") | ||
| 63 | |||
| 64 | #include "landlock_create_ruleset.c" | ||
| 65 | -- | ||
| 66 | 2.19.1 | ||
| 67 | |||
diff --git a/meta/recipes-devtools/strace/strace/skip-load.patch b/meta/recipes-devtools/strace/strace/skip-load.patch index fa3ddb8ddc..b1acfda5d8 100644 --- a/meta/recipes-devtools/strace/strace/skip-load.patch +++ b/meta/recipes-devtools/strace/strace/skip-load.patch | |||
| @@ -4,6 +4,12 @@ care about timing. | |||
| 4 | Upstream-Status: Inappropriate | 4 | Upstream-Status: Inappropriate |
| 5 | Signed-off-by: Ross Burton <ross.burton@arm.com> | 5 | Signed-off-by: Ross Burton <ross.burton@arm.com> |
| 6 | 6 | ||
| 7 | --- | ||
| 8 | tests/clock_nanosleep.gen.test | 1 + | ||
| 9 | tests/delay.test | 1 + | ||
| 10 | tests/strace-r.test | 1 + | ||
| 11 | 3 files changed, 3 insertions(+) | ||
| 12 | |||
| 7 | diff --git a/tests/clock_nanosleep.gen.test b/tests/clock_nanosleep.gen.test | 13 | diff --git a/tests/clock_nanosleep.gen.test b/tests/clock_nanosleep.gen.test |
| 8 | index 7a6025b..f0e6dbb 100755 | 14 | index 7a6025b..f0e6dbb 100755 |
| 9 | --- a/tests/clock_nanosleep.gen.test | 15 | --- a/tests/clock_nanosleep.gen.test |
| @@ -15,41 +21,29 @@ index 7a6025b..f0e6dbb 100755 | |||
| 15 | +skip_ "Test not reliable under load" | 21 | +skip_ "Test not reliable under load" |
| 16 | run_strace_match_diff -e trace=clock_nanosleep,clock_gettime | 22 | run_strace_match_diff -e trace=clock_nanosleep,clock_gettime |
| 17 | diff --git a/tests/delay.test b/tests/delay.test | 23 | diff --git a/tests/delay.test b/tests/delay.test |
| 18 | index f74e27f..328087e 100755 | 24 | index f74e27f..6172c04 100755 |
| 19 | --- a/tests/delay.test | 25 | --- a/tests/delay.test |
| 20 | +++ b/tests/delay.test | 26 | +++ b/tests/delay.test |
| 21 | @@ -9,6 +9,8 @@ | 27 | @@ -8,6 +8,7 @@ |
| 28 | # SPDX-License-Identifier: GPL-2.0-or-later | ||
| 22 | 29 | ||
| 23 | . "${srcdir=.}/init.sh" | 30 | . "${srcdir=.}/init.sh" |
| 24 | |||
| 25 | +skip_ "Test not reliable under load" | 31 | +skip_ "Test not reliable under load" |
| 26 | + | 32 | |
| 27 | while read -r denter dexit denter_us dexit_us; do | 33 | while read -r denter dexit denter_us dexit_us; do |
| 28 | [ -n "$denter" ] || continue | 34 | [ -n "$denter" ] || continue |
| 29 | |||
| 30 | diff --git a/tests/strace-T.test b/tests/strace-T.test | ||
| 31 | index adca3e8..d4acfe9 100755 | ||
| 32 | --- a/tests/strace-T.test | ||
| 33 | +++ b/tests/strace-T.test | ||
| 34 | @@ -9,6 +9,8 @@ | ||
| 35 | |||
| 36 | . "${srcdir=.}/init.sh" | ||
| 37 | |||
| 38 | +skip_ "Test not reliable under load" | ||
| 39 | + | ||
| 40 | T_opt="${1:--T}" | ||
| 41 | |||
| 42 | run_prog ../sleep 0 | ||
| 43 | diff --git a/tests/strace-r.test b/tests/strace-r.test | 35 | diff --git a/tests/strace-r.test b/tests/strace-r.test |
| 44 | index 8299737..c360344 100755 | 36 | index 8299737..d89c7df 100755 |
| 45 | --- a/tests/strace-r.test | 37 | --- a/tests/strace-r.test |
| 46 | +++ b/tests/strace-r.test | 38 | +++ b/tests/strace-r.test |
| 47 | @@ -9,6 +9,8 @@ | 39 | @@ -8,6 +8,7 @@ |
| 40 | # SPDX-License-Identifier: GPL-2.0-or-later | ||
| 48 | 41 | ||
| 49 | . "${srcdir=.}/init.sh" | 42 | . "${srcdir=.}/init.sh" |
| 50 | |||
| 51 | +skip_ "Test not reliable under load" | 43 | +skip_ "Test not reliable under load" |
| 52 | + | 44 | |
| 53 | r_opt="${1:--r}" | 45 | r_opt="${1:--r}" |
| 54 | 46 | ||
| 55 | run_prog ../sleep 0 | 47 | -- |
| 48 | 2.25.1 | ||
| 49 | |||
diff --git a/meta/recipes-devtools/strace/strace_5.17.bb b/meta/recipes-devtools/strace/strace_5.18.bb index c7fd9edf40..3164ebca51 100644 --- a/meta/recipes-devtools/strace/strace_5.17.bb +++ b/meta/recipes-devtools/strace/strace_5.18.bb | |||
| @@ -13,9 +13,8 @@ SRC_URI = "https://strace.io/files/${PV}/strace-${PV}.tar.xz \ | |||
| 13 | file://ptest-spacesave.patch \ | 13 | file://ptest-spacesave.patch \ |
| 14 | file://0001-strace-fix-reproducibilty-issues.patch \ | 14 | file://0001-strace-fix-reproducibilty-issues.patch \ |
| 15 | file://skip-load.patch \ | 15 | file://skip-load.patch \ |
| 16 | file://0001-landlock-update-expected-string.patch \ | ||
| 17 | " | 16 | " |
| 18 | SRC_URI[sha256sum] = "5fb298dbd1331fd1e1bc94c5c32395860d376101b87c6cd3d1ba9f9aa15c161f" | 17 | SRC_URI[sha256sum] = "60293ea79ac9253d600cdc9be077ad2988ca22284a439c9e66be5150db3d1187" |
| 19 | 18 | ||
| 20 | inherit autotools ptest | 19 | inherit autotools ptest |
| 21 | 20 | ||
