Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751239AbdCQM4T (ORCPT ); Fri, 17 Mar 2017 08:56:19 -0400 Received: from mail-wr0-f181.google.com ([209.85.128.181]:35674 "EHLO mail-wr0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751073AbdCQM4S (ORCPT ); Fri, 17 Mar 2017 08:56:18 -0400 MIME-Version: 1.0 In-Reply-To: References: <20170315131535.11438-1-fathi.boudra@linaro.org> From: Fathi Boudra Date: Fri, 17 Mar 2017 14:48:23 +0200 Message-ID: Subject: Re: [PATCH 1/2] selftests: keep consistency for uname -m usage To: Shuah Khan Cc: "linux-kernel@vger.kernel.org" , linux-kselftest@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: 3758 Lines: 93 On 15 March 2017 at 17:56, Shuah Khan wrote: > Hi Fathi, > > On 03/15/2017 07:15 AM, Fathi Boudra wrote: >> powerpc selftests allow to override ARCH for cross-compilation by making >> the first ARCH assignment weak. >> Use the same approach in breakpoints, ipc and prctl tests to: >> - keep uname usage consistent across selftests >> - make it easier to cross-compile > > How does it make it easier to cross-compile? Could you please > elaborate on what this patch is fixing? Nevermind, ARCH was already a weak assignment. My patches changed the behavior. I've been confused by the different approach used by powerpc vs breakpoints/ipc/prctl tests. > Also I would like to see 3 separate patches one for each > of the tests? Will you accept a patch to get rid on uname_M variable, keep ARCH usage consistent across selftests? In other words, align with the way it's done in powerpc test. It won't fix any cross-compilation issues but improve consistency in the code. > thanks, > -- Shuah > >> >> Signed-off-by: Fathi Boudra >> --- >> tools/testing/selftests/breakpoints/Makefile | 4 ++-- >> tools/testing/selftests/ipc/Makefile | 4 ++-- >> tools/testing/selftests/powerpc/Makefile | 2 +- >> tools/testing/selftests/prctl/Makefile | 4 ++-- >> 4 files changed, 7 insertions(+), 7 deletions(-) >> >> diff --git a/tools/testing/selftests/breakpoints/Makefile b/tools/testing/selftests/breakpoints/Makefile >> index 72aa103e4141..9d5a1af4b5c9 100644 >> --- a/tools/testing/selftests/breakpoints/Makefile >> +++ b/tools/testing/selftests/breakpoints/Makefile >> @@ -1,6 +1,6 @@ >> # Taken from perf makefile >> -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/) >> +ARCH ?= $(shell uname -m 2>/dev/null || echo not) >> +ARCH := $(shell echo $(ARCH) | sed -e s/i.86/x86/ -e s/x86_64/x86/) > > > >> >> ifeq ($(ARCH),x86) >> TEST_GEN_PROGS := breakpoint_test >> diff --git a/tools/testing/selftests/ipc/Makefile b/tools/testing/selftests/ipc/Makefile >> index 30ef4c7f53ea..7e5a459842bb 100644 >> --- a/tools/testing/selftests/ipc/Makefile >> +++ b/tools/testing/selftests/ipc/Makefile >> @@ -1,5 +1,5 @@ >> -uname_M := $(shell uname -m 2>/dev/null || echo not) >> -ARCH ?= $(shell echo $(uname_M) | sed -e s/i.86/i386/) >> +ARCH ?= $(shell uname -m 2>/dev/null || echo not) >> +ARCH := $(shell echo $(ARCH) | sed -e s/i.86/i386/) >> ifeq ($(ARCH),i386) >> ARCH := x86 >> CFLAGS := -DCONFIG_X86_32 -D__i386__ >> diff --git a/tools/testing/selftests/powerpc/Makefile b/tools/testing/selftests/powerpc/Makefile >> index 1c5d0575802e..731e015d170d 100644 >> --- a/tools/testing/selftests/powerpc/Makefile >> +++ b/tools/testing/selftests/powerpc/Makefile >> @@ -1,7 +1,7 @@ >> # Makefile for powerpc selftests >> >> # ARCH can be overridden by the user for cross compiling >> -ARCH ?= $(shell uname -m) >> +ARCH ?= $(shell uname -m 2>/dev/null || echo not) >> ARCH := $(shell echo $(ARCH) | sed -e s/ppc.*/powerpc/) >> >> ifeq ($(ARCH),powerpc) >> diff --git a/tools/testing/selftests/prctl/Makefile b/tools/testing/selftests/prctl/Makefile >> index 35aa1c8f2df2..08aeb8721ce6 100644 >> --- a/tools/testing/selftests/prctl/Makefile >> +++ b/tools/testing/selftests/prctl/Makefile >> @@ -1,6 +1,6 @@ >> ifndef CROSS_COMPILE >> -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/) >> +ARCH ?= $(shell uname -m 2>/dev/null || echo not) >> +ARCH := $(shell echo $(ARCH) | sed -e s/i.86/x86/ -e s/x86_64/x86/) >> >> ifeq ($(ARCH),x86) >> TEST_PROGS := disable-tsc-ctxt-sw-stress-test disable-tsc-on-off-stress-test \ >> >