Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1728949

Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets

From Shuah Khan <shuah@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets
Date 2017-09-08 16:00 +0200
Message-ID <unrZ9-7KT-49@gated-at.bofh.it> (permalink)
References <unn97-4mN-19@gated-at.bofh.it> <unn97-4mN-17@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 09/08/2017 02:41 AM, Fathi Boudra wrote:
> Shuah, should I resend?

No need to resend. Sorry about this.I just missed it. I will get
this into 4.14-rc2 or so. Thanks for the ping.

-- Shuah
> 
> 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
> 
> 

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets Fathi Boudra <fathi.boudra@linaro.org> - 2017-09-08 10:50 +0200
  Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets Thomas Meyer <thomas@m3y3r.de> - 2017-09-08 12:40 +0200
    Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets Shuah Khan <shuahkh@osg.samsung.com> - 2017-09-08 15:50 +0200
  Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets Shuah Khan <shuah@kernel.org> - 2017-09-08 16:00 +0200

csiph-web