Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756145AbdIHIlZ (ORCPT ); Fri, 8 Sep 2017 04:41:25 -0400 Received: from mail-wr0-f174.google.com ([209.85.128.174]:32924 "EHLO mail-wr0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754171AbdIHIlX (ORCPT ); Fri, 8 Sep 2017 04:41:23 -0400 X-Google-Smtp-Source: ADKCNb5lD3HXrNjjZQ55A9pLRgKcBYGnolvvlXpa9srKjU4wGxnv+Igk7+/7ICO1iM2aGTdy35qXhcrIXxVcBDfyrNQ= MIME-Version: 1.0 In-Reply-To: <20170629093953.5790-1-fathi.boudra@linaro.org> References: <20170629093953.5790-1-fathi.boudra@linaro.org> From: Fathi Boudra Date: Fri, 8 Sep 2017 11:41:21 +0300 Message-ID: Subject: Re: [PATCH] selftests: breakpoints: re-order TEST_GEN_PROGS targets To: Shuah Khan Cc: linux-kselftest@vger.kernel.org, "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1749 Lines: 50 Shuah, should I resend? 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