On 15 March 2017 at 17:56, Shuah Khan <shuah@xxxxxxxxxx> 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 <fathi.boudra@xxxxxxxxxx> >> --- >> 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 \ >> > -- To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html