Shuah, should I resend?
On 29 June 2017 at 12:39, Fathi Boudra <[email protected]> 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 <[email protected]>
> ---
> 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
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
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 [email protected]. 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
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 <[email protected]> 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 <[email protected]>
>> ---
>> 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
>
>
On 09/08/2017 07:52 AM, Shuah Khan wrote:
> 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.
>
Applied to linux-kselftest fixes for 4.14-rc2
thanks,
-- Shuah