Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754884AbdIHNwh (ORCPT ); Fri, 8 Sep 2017 09:52:37 -0400 Received: from mailout.easymail.ca ([64.68.200.34]:49096 "EHLO mailout.easymail.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754550AbdIHNwa (ORCPT ); Fri, 8 Sep 2017 09:52:30 -0400 Reply-To: shuah@kernel.org Subject: Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets To: Fathi Boudra Cc: linux-kselftest@vger.kernel.org, "linux-kernel@vger.kernel.org" , Shuah Khan , Shuah Khan References: <20170629093953.5790-1-fathi.boudra@linaro.org> From: Shuah Khan Message-ID: <4c9eac14-4234-63f9-f335-e19df5d15f67@kernel.org> Date: Fri, 8 Sep 2017 07:52:21 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1975 Lines: 58 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 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 >> --- >> 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 > >