!54 [sync] PR-53: Fix run-ptrace-mapper and other failed test case

From: @openeuler-sync-bot 
Reviewed-by: @hubin95 
Signed-off-by: @hubin95
This commit is contained in:
openeuler-ci-bot 2023-07-12 02:05:21 +00:00 committed by Gitee
commit 868094c7c7
No known key found for this signature in database
GPG Key ID: 173E9B9CA92EEF8F
3 changed files with 83 additions and 1 deletions

View File

@ -0,0 +1,46 @@
From ffb69e2384538a7d4b634bdc491d7da9b7220f57 Mon Sep 17 00:00:00 2001
From: chenziyang <chenziyang4@huawei.com>
Date: Tue, 11 Jul 2023 10:19:22 +0800
Subject: [PATCH] avoid calling printf because OE glibc-2.34 used
-mno-outline-atomics buildflag, it will cause printf to be non-atomic
operations on ARMv8.1 platform. This will cause printf to stuck into dead
loop because ptrace sing-step execution cause race condition during printf
instructions.
---
tests/mapper.c | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/tests/mapper.c b/tests/mapper.c
index b47ae78..fcfb080 100644
--- a/tests/mapper.c
+++ b/tests/mapper.c
@@ -43,6 +43,15 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. */
# define MAP_NORESERVE 0
#endif
+void __attribute__((noinline)) push_some_stacks(int n)
+{
+ if (n >= 1)
+ {
+ push_some_stacks(n - 1);
+ push_some_stacks(n - 1);
+ }
+}
+
int
main (void)
{
@@ -71,7 +80,7 @@ main (void)
printf ("Turning on single-stepping...\n");
kill (getpid (), SIGUSR1); /* tell test-ptrace to start single-stepping */
- printf ("Va bene?\n");
+ push_some_stacks (4);
kill (getpid (), SIGUSR2); /* tell test-ptrace to stop single-stepping */
printf ("Turned single-stepping off...\n");
return 0;
--
2.33.0

View File

@ -0,0 +1,28 @@
From 0224ac01f620ff226bdf877e2b278d20c0fa1b56 Mon Sep 17 00:00:00 2001
From: chenziyang <chenziyang4@huawei.com>
Date: Mon, 3 Jul 2023 14:27:47 +0800
Subject: [PATCH] Fix run-ptrace-mapper test case failed by allowing
unw_get_proc_info() return unsuccessful result. Because it is not an error
unw_get_proc_info() does not return the auxiliary information, so we edit
test case and don't treat it as error.
---
tests/test-ptrace.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/test-ptrace.c b/tests/test-ptrace.c
index 846bcd8..32f42e8 100644
--- a/tests/test-ptrace.c
+++ b/tests/test-ptrace.c
@@ -111,7 +111,7 @@ do_backtrace (void)
printf ("%016lx %-32s (sp=%016lx)\n", (long) ip, buf, (long) sp);
}
- if ((ret = unw_get_proc_info (&c, &pi)) < 0)
+ if ((ret = unw_get_proc_info (&c, &pi)) < 0 && ret != -UNW_ENOINFO) /* It's possible unw_get_proc_info don't return infomation */
panic ("unw_get_proc_info(ip=0x%lx) failed: ret=%d\n", (long) ip, ret);
else if (verbose)
printf ("\tproc=%016lx-%016lx\n\thandler=%lx lsda=%lx",
--
2.33.0

View File

@ -1,7 +1,7 @@
Name: libunwind
Epoch: 2
Version: 1.3.1
Release: 5
Release: 6
Summary: Libunwind provides a C ABI to determine the call-chain of a program
License: BSD
URL: http://savannah.nongnu.org/projects/libunwind
@ -19,6 +19,8 @@ Patch6008: dwarf-Push-correct-CFA-onto-stack-for-dwarf-expressi.patch
Patch6009: backport-check-namespace.sh-adjust-aarch64-symbols.patch
Patch6010: backport-Ltest-mem-validate-Disable-inlining-for-consume_and_.patch
Patch6011: backport-tests-run-coredump-unwind-Skip-test-if-no-coredump-h.patch
Patch6012: backport-avoid-calling-printf-because-OE-glibc-2.34-used-mno-.patch
Patch6013: backport-fix-run-ptrace-mapper-test-case-failed.patch
ExclusiveArch: aarch64 %{ix86} x86_64
@ -96,6 +98,12 @@ make check || true
%{_mandir}/*/*
%changelog
* Tue Jul 11 2023 chenziyang <chenziyang4@huawei.com> - 2:1.3.1-6
- Type:bugfix
- ID:NA
- SUG:NA
- DESC:backport upstream patch to fix failed run-ptrace-mapper testcase
* Tue Feb 21 2021 shixuantong <shixuantong1@huawei.com> - 2:1.3.1-5
- fix some test fail