lkml.org 
[lkml]   [2017]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets
Shuah, should I resend?

On 29 June 2017 at 12:39, Fathi Boudra <fathi.boudra@linaro.org> wrote:
> breakpoint_test can fail on arm64 with older/unpatched glibc:
>
> breakpoint_test_arm64.c: In function 'run_test':
> breakpoint_test_arm64.c:170:25: error: 'TRAP_HWBKPT' undeclared (first use
> in this function)
>
> due to glibc missing several of the TRAP_* constants in the userspace
> definitions. Specifically TRAP_BRANCH and TRAP_HWBKPT.
> See https://sourceware.org/bugzilla/show_bug.cgi?id=21286
>
> It prevents to build step_after_suspend_test afterward, since make won't
> continue.
>
> We still want to be able to build and run the test, independently of
> breakpoint_test_arm64 build failure. Re-order TEST_GEN_PROGS to be able to
> build step_after_suspend_test first.
>
> Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>
> ---
> tools/testing/selftests/breakpoints/Makefile | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/tools/testing/selftests/breakpoints/Makefile b/tools/testing/selftests/breakpoints/Makefile
> index 6b214b7b10fb..247b0a1899d7 100644
> --- a/tools/testing/selftests/breakpoints/Makefile
> +++ b/tools/testing/selftests/breakpoints/Makefile
> @@ -2,14 +2,14 @@
> uname_M := $(shell uname -m 2>/dev/null || echo not)
> ARCH ?= $(shell echo $(uname_M) | sed -e s/i.86/x86/ -e s/x86_64/x86/)
>
> +TEST_GEN_PROGS := step_after_suspend_test
> +
> ifeq ($(ARCH),x86)
> -TEST_GEN_PROGS := breakpoint_test
> +TEST_GEN_PROGS += breakpoint_test
> endif
> ifneq (,$(filter $(ARCH),aarch64 arm64))
> -TEST_GEN_PROGS := breakpoint_test_arm64
> +TEST_GEN_PROGS += breakpoint_test_arm64
> endif
>
> -TEST_GEN_PROGS += step_after_suspend_test
> -
> include ../lib.mk
>
> --
> 2.13.0

\
 
 \ /
  Last update: 2017-09-08 10:41    [W:0.055 / U:0.748 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site