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

Groups > linux.kernel > #1728696 > unrolled thread

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

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  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

#1728696 — Re: [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

[toc] | [next] | [standalone]

#1728752

On Fri, Sep 08, 2017 at 11:41:21AM +0300, Fathi Boudra wrote:
> Shuah, should I resend?

Hi,

I also did send some patches to his kernel.org address after lazy searching
for "Shuah" in my inbox. But maybe his other email address is the correct one.

with kind regards
thomas

[toc] | [prev] | [next] | [standalone]

#1728904

On 09/08/2017 04:32 AM, Thomas Meyer wrote:
> On Fri, Sep 08, 2017 at 11:41:21AM +0300, Fathi Boudra wrote:
>> Shuah, should I resend?
> 
> Hi,
> 
> I also did send some patches to his kernel.org address after lazy searching
> for "Shuah" in my inbox. But maybe his other email address is the correct one.
> 
> with kind regards
> thomas
> 


You have the right address for me which is shuah@kernel.org. I see your patches
in my Inbox. Please continue to use the address get_maintainers.

I will review the patches once I get done with the merge window.

thanks,
-- Shuah

[toc] | [prev] | [next] | [standalone]

#1728949

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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web