* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' [not found] <tip-5a8a2d13b1526e306ff2a9fe12dc9d5878d355f9@git.kernel.org> @ 2009-09-04 14:48 ` Ingo Molnar 2009-09-04 15:33 ` David Woodhouse 0 siblings, 1 reply; 15+ messages in thread From: Ingo Molnar @ 2009-09-04 14:48 UTC (permalink / raw) To: linux-kernel, mingo, hpa, dwmw2, sam, tglx, David.Woodhouse Cc: linux-tip-commits * tip-bot for David Woodhouse <dwmw2@infradead.org> wrote: > Commit-ID: 5a8a2d13b1526e306ff2a9fe12dc9d5878d355f9 > Gitweb: http://git.kernel.org/tip/5a8a2d13b1526e306ff2a9fe12dc9d5878d355f9 > Author: David Woodhouse <dwmw2@infradead.org> > AuthorDate: Tue, 30 Jun 2009 12:10:21 +0100 > Committer: H. Peter Anvin <hpa@zytor.com> > CommitDate: Thu, 3 Sep 2009 10:57:20 -0700 > > x86: Don't silently override CONFIG_64BIT in 'make oldconfig' -tip testing found that this commit broke the UML build: /home/mingo/tip/arch/um/Makefile:52: /home/mingo/tip/arch/um/Makefile-x86: No such file or directory make[1]: *** No rule to make target `/home/mingo/tip/arch/um/Makefile-x86'. Stop. make: *** [sub-make] Error 2 Ingo ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 14:48 ` [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' Ingo Molnar @ 2009-09-04 15:33 ` David Woodhouse 2009-09-04 16:27 ` H. Peter Anvin 0 siblings, 1 reply; 15+ messages in thread From: David Woodhouse @ 2009-09-04 15:33 UTC (permalink / raw) To: Ingo Molnar Cc: linux-kernel, mingo, hpa, sam, tglx, David.Woodhouse, linux-tip-commits On Fri, 4 Sep 2009, Ingo Molnar wrote: >> x86: Don't silently override CONFIG_64BIT in 'make oldconfig' > > -tip testing found that this commit broke the UML build: > > /home/mingo/tip/arch/um/Makefile:52: > /home/mingo/tip/arch/um/Makefile-x86: No such file or directory > make[1]: *** No rule to make target `/home/mingo/tip/arch/um/Makefile-x86'. Stop. > make: *** [sub-make] Error 2 Hm, doesn't that mean that UML has always been broken for ARCH=x86? -- dwmw2 ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 15:33 ` David Woodhouse @ 2009-09-04 16:27 ` H. Peter Anvin 2009-09-04 16:38 ` Randy Dunlap 2009-09-04 18:31 ` Ingo Molnar 0 siblings, 2 replies; 15+ messages in thread From: H. Peter Anvin @ 2009-09-04 16:27 UTC (permalink / raw) To: David Woodhouse Cc: Ingo Molnar, linux-kernel, mingo, sam, tglx, David.Woodhouse, linux-tip-commits On 09/04/2009 08:33 AM, David Woodhouse wrote: > On Fri, 4 Sep 2009, Ingo Molnar wrote: > > >>> x86: Don't silently override CONFIG_64BIT in 'make oldconfig' >> >> -tip testing found that this commit broke the UML build: >> >> /home/mingo/tip/arch/um/Makefile:52: >> /home/mingo/tip/arch/um/Makefile-x86: No such file or directory >> make[1]: *** No rule to make target >> `/home/mingo/tip/arch/um/Makefile-x86'. Stop. >> make: *** [sub-make] Error 2 > > Hm, doesn't that mean that UML has always been broken for ARCH=x86? > Quite possible. ARCH=x86 hasn't exactly been widely used. -hpa -- H. Peter Anvin, Intel Open Source Technology Center I work for Intel. I don't speak on their behalf. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 16:27 ` H. Peter Anvin @ 2009-09-04 16:38 ` Randy Dunlap 2009-09-04 16:53 ` H. Peter Anvin 2009-09-04 21:40 ` David Woodhouse 2009-09-04 18:31 ` Ingo Molnar 1 sibling, 2 replies; 15+ messages in thread From: Randy Dunlap @ 2009-09-04 16:38 UTC (permalink / raw) To: H. Peter Anvin Cc: David Woodhouse, Ingo Molnar, linux-kernel, mingo, sam, tglx, David.Woodhouse, linux-tip-commits On Fri, 04 Sep 2009 09:27:18 -0700 H. Peter Anvin wrote: > On 09/04/2009 08:33 AM, David Woodhouse wrote: > > On Fri, 4 Sep 2009, Ingo Molnar wrote: > > > > > >>> x86: Don't silently override CONFIG_64BIT in 'make oldconfig' > >> > >> -tip testing found that this commit broke the UML build: > >> > >> /home/mingo/tip/arch/um/Makefile:52: > >> /home/mingo/tip/arch/um/Makefile-x86: No such file or directory > >> make[1]: *** No rule to make target > >> `/home/mingo/tip/arch/um/Makefile-x86'. Stop. > >> make: *** [sub-make] Error 2 > > > > Hm, doesn't that mean that UML has always been broken for ARCH=x86? > > > > Quite possible. ARCH=x86 hasn't exactly been widely used. what does ARCH=x86 mean? --- ~Randy LPC 2009, Sept. 23-25, Portland, Oregon http://linuxplumbersconf.org/2009/ ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 16:38 ` Randy Dunlap @ 2009-09-04 16:53 ` H. Peter Anvin 2009-09-04 21:40 ` David Woodhouse 1 sibling, 0 replies; 15+ messages in thread From: H. Peter Anvin @ 2009-09-04 16:53 UTC (permalink / raw) To: Randy Dunlap Cc: David Woodhouse, Ingo Molnar, linux-kernel, mingo, sam, tglx, David.Woodhouse, linux-tip-commits On 09/04/2009 09:38 AM, Randy Dunlap wrote: >>> >> Quite possible. ARCH=x86 hasn't exactly been widely used. > > what does ARCH=x86 mean? > Treat 32 vs 64 bit as a normal Kconfig variable. Similar as for other 32/64-bit architectures. -hpa -- H. Peter Anvin, Intel Open Source Technology Center I work for Intel. I don't speak on their behalf. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 16:38 ` Randy Dunlap 2009-09-04 16:53 ` H. Peter Anvin @ 2009-09-04 21:40 ` David Woodhouse 2009-09-04 22:54 ` H. Peter Anvin 1 sibling, 1 reply; 15+ messages in thread From: David Woodhouse @ 2009-09-04 21:40 UTC (permalink / raw) To: Randy Dunlap Cc: H. Peter Anvin, Ingo Molnar, linux-kernel, mingo, sam, tglx, linux-tip-commits On Fri, 2009-09-04 at 09:38 -0700, Randy Dunlap wrote: > > what does ARCH=x86 mean? A bit like ARCH=powerpc. It's the new combined architecture where 64-bit vs. 32-bit is just a config choice. ARCH=i386 and ARCH=x86_64 are just historical baggage, and should be dropped like ARCH=ppc and ARCH=ppc64 were. -- dwmw2 ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 21:40 ` David Woodhouse @ 2009-09-04 22:54 ` H. Peter Anvin 0 siblings, 0 replies; 15+ messages in thread From: H. Peter Anvin @ 2009-09-04 22:54 UTC (permalink / raw) To: David Woodhouse Cc: Randy Dunlap, Ingo Molnar, linux-kernel, mingo, sam, tglx, linux-tip-commits On 09/04/2009 02:40 PM, David Woodhouse wrote: > On Fri, 2009-09-04 at 09:38 -0700, Randy Dunlap wrote: >> >> what does ARCH=x86 mean? > > A bit like ARCH=powerpc. It's the new combined architecture where 64-bit > vs. 32-bit is just a config choice. ARCH=i386 and ARCH=x86_64 are just > historical baggage, and should be dropped like ARCH=ppc and ARCH=ppc64 > were. > >From a cursory look it looks like ARCH=powerpc won't work for UM, either. -hpa ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 16:27 ` H. Peter Anvin 2009-09-04 16:38 ` Randy Dunlap @ 2009-09-04 18:31 ` Ingo Molnar 2009-09-04 18:57 ` H. Peter Anvin 1 sibling, 1 reply; 15+ messages in thread From: Ingo Molnar @ 2009-09-04 18:31 UTC (permalink / raw) To: H. Peter Anvin Cc: David Woodhouse, linux-kernel, mingo, sam, tglx, David.Woodhouse, linux-tip-commits * H. Peter Anvin <hpa@zytor.com> wrote: > On 09/04/2009 08:33 AM, David Woodhouse wrote: > > On Fri, 4 Sep 2009, Ingo Molnar wrote: > > > > > >>> x86: Don't silently override CONFIG_64BIT in 'make oldconfig' > >> > >> -tip testing found that this commit broke the UML build: > >> > >> /home/mingo/tip/arch/um/Makefile:52: > >> /home/mingo/tip/arch/um/Makefile-x86: No such file or directory > >> make[1]: *** No rule to make target > >> `/home/mingo/tip/arch/um/Makefile-x86'. Stop. > >> make: *** [sub-make] Error 2 > > > > Hm, doesn't that mean that UML has always been broken for ARCH=x86? > > Quite possible. ARCH=x86 hasn't exactly been widely used. Note, i used 'make ARCH=um' so this commit cannot be pushed upwards until this problem is fixed. It could very well be some missing changes on the UML side. Ingo ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 18:31 ` Ingo Molnar @ 2009-09-04 18:57 ` H. Peter Anvin 2009-09-04 19:04 ` H. Peter Anvin 2009-09-11 21:16 ` Jeff Dike 0 siblings, 2 replies; 15+ messages in thread From: H. Peter Anvin @ 2009-09-04 18:57 UTC (permalink / raw) To: Ingo Molnar, Jeff Dike, Sam Ravnborg Cc: David Woodhouse, linux-kernel, mingo, sam, tglx, David.Woodhouse, linux-tip-commits On 09/04/2009 11:31 AM, Ingo Molnar wrote: > > * H. Peter Anvin <hpa@zytor.com> wrote: > >> On 09/04/2009 08:33 AM, David Woodhouse wrote: >>> On Fri, 4 Sep 2009, Ingo Molnar wrote: >>> >>> >>>>> x86: Don't silently override CONFIG_64BIT in 'make oldconfig' >>>> >>>> -tip testing found that this commit broke the UML build: >>>> >>>> /home/mingo/tip/arch/um/Makefile:52: >>>> /home/mingo/tip/arch/um/Makefile-x86: No such file or directory >>>> make[1]: *** No rule to make target >>>> `/home/mingo/tip/arch/um/Makefile-x86'. Stop. >>>> make: *** [sub-make] Error 2 >>> >>> Hm, doesn't that mean that UML has always been broken for ARCH=x86? >> >> Quite possible. ARCH=x86 hasn't exactly been widely used. > > Note, i used 'make ARCH=um' so this commit cannot be pushed upwards > until this problem is fixed. It could very well be some missing > changes on the UML side. > Okay, the problem is the following: UM treats i386 and x86-64 as separate architectures, and it gets very unhappy with SUBARCH=x86. A trivial attempt to fix it: diff --git a/arch/um/Makefile b/arch/um/Makefile index 0728def..b1cc9cf 100644 --- a/arch/um/Makefile +++ b/arch/um/Makefile @@ -12,6 +12,17 @@ OS := $(shell uname -s) # features. SHELL := /bin/bash +# +# i386 and x86_64 are separate architectures to the UM build. +# +ifeq ($(SUBARCH),x86) +ifeq ($(CONFIG_64BIT),y) +SUBARCH := x86_64 +else +SUBARCH := i386 +endif +endif + filechk_gen_header = $< core-y += $(ARCH_DIR)/kernel/ \ ... didn't fix it, as "make defconfig" promptly made a 32-bit configuration on my 64-bit system, and the build failed. The "obvious" change of allowing SUBARCH to take values like i386 and x86_64 is also wrong (and possibly have a DEFAULT_ARCH which can be different than SUBARCH), because we have several instances of: ifneq ($(SUBARCH),$(ARCH)) ... in the build tree, and even have ugliness like: [scripts/tags.h] # Support um (which uses SUBARCH) if [ "${ARCH}" = "um" ]; then if [ "$SUBARCH" = "i386" ]; then archinclude=x86 elif [ "$SUBARCH" = "x86_64" ]; then archinclude=x86 else archinclude=${SUBARCH} fi fi Anyway... it sounds like we need to drop this commit for now and re-merge it when there is a fix for UM. Jeff, Sam, I would appreciate your suggestions as how best to fix this kind of stuff... -hpa ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 18:57 ` H. Peter Anvin @ 2009-09-04 19:04 ` H. Peter Anvin 2009-09-11 21:16 ` Jeff Dike 1 sibling, 0 replies; 15+ messages in thread From: H. Peter Anvin @ 2009-09-04 19:04 UTC (permalink / raw) To: Ingo Molnar, Jeff Dike, Sam Ravnborg Cc: David Woodhouse, linux-kernel, mingo, tglx, David.Woodhouse, linux-tip-commits On 09/04/2009 11:57 AM, H. Peter Anvin wrote: > > Anyway... it sounds like we need to drop this commit for now and > re-merge it when there is a fix for UM. > > Jeff, Sam, I would appreciate your suggestions as how best to fix this > kind of stuff... > On that note, this issue isn't just limited to UM. klibc has to deal with this issue too: a single kernel architecture may correspond to more than one user-space architecture, and as far as userspace is concerned, they are completely different. -hpa ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-04 18:57 ` H. Peter Anvin 2009-09-04 19:04 ` H. Peter Anvin @ 2009-09-11 21:16 ` Jeff Dike 2009-09-14 23:29 ` H. Peter Anvin 1 sibling, 1 reply; 15+ messages in thread From: Jeff Dike @ 2009-09-11 21:16 UTC (permalink / raw) To: H. Peter Anvin Cc: Ingo Molnar, Sam Ravnborg, David Woodhouse, linux-kernel, mingo, tglx, David.Woodhouse, linux-tip-commits On Fri, Sep 04, 2009 at 11:57:58AM -0700, H. Peter Anvin wrote: > The "obvious" change of allowing SUBARCH to take values like i386 and > x86_64 is also wrong (and possibly have a DEFAULT_ARCH which can be > different than SUBARCH), because we have several instances of: > > ifneq ($(SUBARCH),$(ARCH)) > > ... in the build tree, and even have ugliness like: > > [scripts/tags.h] > # Support um (which uses SUBARCH) > if [ "${ARCH}" = "um" ]; then > if [ "$SUBARCH" = "i386" ]; then > archinclude=x86 > elif [ "$SUBARCH" = "x86_64" ]; then > archinclude=x86 > else > archinclude=${SUBARCH} > fi > fi > > Anyway... it sounds like we need to drop this commit for now and > re-merge it when there is a fix for UM. > > Jeff, Sam, I would appreciate your suggestions as how best to fix this > kind of stuff... I figured out how to make this work - see below. Jeff -- Work email - jdike at linux dot intel dot com commit 370d4e326094c053b7f602178a4303f958471136 Author: Jeff Dike <jdike@kvm.user-mode-linux.org> Date: Fri Sep 11 17:07:36 2009 -0400 Make UML build with SUBARCH=x86 This patch makes UML build with David Woodhouse's commit 5a8a2d13b1526e306ff2a9fe12dc9d5878d355f9 in x86/linux-2.6-tip. I start with Peter Anvin's patch to arch/um/Makefile: +# +# i386 and x86_64 are separate architectures to the UM build. +# +ifeq ($(SUBARCH),x86) +ifeq ($(CONFIG_64BIT),y) +SUBARCH := x86_64 +else +SUBARCH := i386 +endif +endif and fix a couple of things. CONFIG_64BIT was defined in terms of SUBARCH, so this patch is a bit circular in making SUBARCH depend on CONFIG_64BIT. I do like x86 and change the test to ifeq ($(shell uname -m),x86_64) SUBARCH is used in other places, so in order to just confine changes to UML, I define UML_SUBARCH to be either x86_64 or i386 and heavy-handedly do s/SUBARCH/UML_SUBARCH in arch/um/Makefile and arch/um/os-Linux/Makefile. This makes the UML build descend through the existing -i386 and -x86_64 directories. CONFIG_64BIT needs to be defined correctly. This used to be done by seeing if SUBARCH = "x86_64". I replace SUBARCH with UML_SUBARCH, which is exported from the Makefile. This induces a build loop by causing include/config/auto.conf.cmd to see an unexpected mismatch between UML_SUBARCH and x86_64 because the arch Makefile hasn't been included yet, so UML_SUBARCH hasn't been set yet. Somehow, this causes infinite Makefile updates and make restarts. I fixed this by moving the include of the arch Makefile above the auto.conf stuff. UML is fine with this, but maybe other arches won't be. diff --git a/Makefile b/Makefile index 60de4ef..c25406f 100644 --- a/Makefile +++ b/Makefile @@ -473,6 +473,8 @@ libs-y := lib/ core-y := usr/ endif # KBUILD_EXTMOD +include $(srctree)/arch/$(SRCARCH)/Makefile + ifeq ($(dot-config),1) # Read in config -include include/config/auto.conf @@ -524,7 +526,6 @@ else KBUILD_CFLAGS += -O2 endif -include $(srctree)/arch/$(SRCARCH)/Makefile ifneq ($(CONFIG_FRAME_WARN),0) KBUILD_CFLAGS += $(call cc-option,-Wframe-larger-than=${CONFIG_FRAME_WARN}) diff --git a/arch/um/Kconfig.common b/arch/um/Kconfig.common index 0d207e7..686c7b0 100644 --- a/arch/um/Kconfig.common +++ b/arch/um/Kconfig.common @@ -75,3 +75,7 @@ config HZ config SUBARCH string option env="SUBARCH" + +config UML_SUBARCH + string + option env="UML_SUBARCH" diff --git a/arch/um/Kconfig.x86 b/arch/um/Kconfig.x86 index 5ee3280..e915680 100644 --- a/arch/um/Kconfig.x86 +++ b/arch/um/Kconfig.x86 @@ -13,7 +13,7 @@ config UML_X86 config 64BIT bool - default SUBARCH = "x86_64" + default UML_SUBARCH = "x86_64" config X86_32 def_bool !64BIT diff --git a/arch/um/Makefile b/arch/um/Makefile index 0728def..28419ef 100644 --- a/arch/um/Makefile +++ b/arch/um/Makefile @@ -12,6 +12,18 @@ OS := $(shell uname -s) # features. SHELL := /bin/bash +UML_SUBARCH = $(SUBARCH) +# +# i386 and x86_64 are separate architectures to the UM build. +# +ifeq ($(SUBARCH),x86) +ifeq ($(shell uname -m),x86_64) + UML_SUBARCH := x86_64 +else + UML_SUBARCH := i386 +endif +endif + filechk_gen_header = $< core-y += $(ARCH_DIR)/kernel/ \ @@ -24,11 +36,11 @@ include $(srctree)/$(ARCH_DIR)/Makefile-skas SHARED_HEADERS := $(ARCH_DIR)/include/shared ARCH_INCLUDE := -I$(srctree)/$(SHARED_HEADERS) -ARCH_INCLUDE += -I$(srctree)/$(ARCH_DIR)/sys-$(SUBARCH)/shared +ARCH_INCLUDE += -I$(srctree)/$(ARCH_DIR)/sys-$(UML_SUBARCH)/shared ifneq ($(KBUILD_SRC),) ARCH_INCLUDE += -I$(SHARED_HEADERS) endif -KBUILD_CPPFLAGS += -I$(srctree)/$(ARCH_DIR)/sys-$(SUBARCH) +KBUILD_CPPFLAGS += -I$(srctree)/$(ARCH_DIR)/sys-$(UML_SUBARCH) # -Dvmap=kernel_vmap prevents anything from referencing the libpcap.o symbol so # named - it's a common symbol in libpcap, so we get a binary which crashes. @@ -38,7 +50,7 @@ KBUILD_CPPFLAGS += -I$(srctree)/$(ARCH_DIR)/sys-$(SUBARCH) # # These apply to USER_CFLAGS to. -KBUILD_CFLAGS += $(CFLAGS) $(CFLAGS-y) -D__arch_um__ -DSUBARCH=\"$(SUBARCH)\" \ +KBUILD_CFLAGS += $(CFLAGS) $(CFLAGS-y) -D__arch_um__ -DSUBARCH=\"$(UML_SUBARCH)\" \ $(ARCH_INCLUDE) $(MODE_INCLUDE) -Dvmap=kernel_vmap \ -Din6addr_loopback=kernel_in6addr_loopback \ -Din6addr_any=kernel_in6addr_any @@ -49,7 +61,7 @@ USER_CFLAGS = $(patsubst $(KERNEL_DEFINES),,$(patsubst -D__KERNEL__,,\ $(patsubst -I%,,$(KBUILD_CFLAGS)))) $(ARCH_INCLUDE) $(MODE_INCLUDE) \ $(filter -I%,$(CFLAGS)) -D_FILE_OFFSET_BITS=64 -include $(srctree)/$(ARCH_DIR)/Makefile-$(SUBARCH) +include $(srctree)/$(ARCH_DIR)/Makefile-$(UML_SUBARCH) #This will adjust *FLAGS accordingly to the platform. include $(srctree)/$(ARCH_DIR)/Makefile-os-$(OS) @@ -99,7 +111,7 @@ CFLAGS_NO_HARDENING := $(call cc-option, -fno-PIC,) $(call cc-option, -fno-pic,) CONFIG_KERNEL_STACK_ORDER ?= 2 STACK_SIZE := $(shell echo $$[ 4096 * (1 << $(CONFIG_KERNEL_STACK_ORDER)) ] ) -CPPFLAGS_vmlinux.lds = -U$(SUBARCH) -DSTART=$(START) -DELF_ARCH=$(ELF_ARCH) \ +CPPFLAGS_vmlinux.lds = -U$(UML_SUBARCH) -DSTART=$(START) -DELF_ARCH=$(ELF_ARCH) \ -DELF_FORMAT="$(ELF_FORMAT)" -DKERNEL_STACK_SIZE=$(STACK_SIZE) # The wrappers will select whether using "malloc" or the kernel allocator. @@ -129,8 +141,8 @@ archclean: # Generated files -$(ARCH_DIR)/sys-$(SUBARCH)/user-offsets.s: FORCE - $(Q)$(MAKE) $(build)=$(ARCH_DIR)/sys-$(SUBARCH) $@ +$(ARCH_DIR)/sys-$(UML_SUBARCH)/user-offsets.s: FORCE + $(Q)$(MAKE) $(build)=$(ARCH_DIR)/sys-$(UML_SUBARCH) $@ define filechk_gen-asm-offsets (set -e; \ @@ -145,7 +157,7 @@ define filechk_gen-asm-offsets echo ""; ) endef -$(SHARED_HEADERS)/user_constants.h: $(ARCH_DIR)/sys-$(SUBARCH)/user-offsets.s +$(SHARED_HEADERS)/user_constants.h: $(ARCH_DIR)/sys-$(UML_SUBARCH)/user-offsets.s $(call filechk,gen-asm-offsets) $(SHARED_HEADERS)/kern_constants.h: @@ -153,3 +165,4 @@ $(SHARED_HEADERS)/kern_constants.h: $(Q)echo '#include "../../../../include/asm/asm-offsets.h"' >$@ export SUBARCH USER_CFLAGS CFLAGS_NO_HARDENING OS HEADER_ARCH DEV_NULL_PATH +export UML_SUBARCH diff --git a/arch/um/os-Linux/Makefile b/arch/um/os-Linux/Makefile index d66f038..2b23018 100644 --- a/arch/um/os-Linux/Makefile +++ b/arch/um/os-Linux/Makefile @@ -5,13 +5,13 @@ obj-y = aio.o elf_aux.o execvp.o file.o helper.o irq.o main.o mem.o process.o \ registers.o sigio.o signal.o start_up.o time.o tty.o uaccess.o \ - umid.o tls.o user_syms.o util.o drivers/ sys-$(SUBARCH)/ skas/ + umid.o tls.o user_syms.o util.o drivers/ sys-$(UML_SUBARCH)/ skas/ USER_OBJS := $(user-objs-y) aio.o elf_aux.o execvp.o file.o helper.o irq.o \ main.o mem.o process.o registers.o sigio.o signal.o start_up.o time.o \ tty.o tls.o uaccess.o umid.o util.o -CFLAGS_user_syms.o += -DSUBARCH_$(SUBARCH) +CFLAGS_user_syms.o += -DSUBARCH_$(UML_SUBARCH) HAVE_AIO_ABI := $(shell [ -r /usr/include/linux/aio_abi.h ] && \ echo -DHAVE_AIO_ABI ) ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-11 21:16 ` Jeff Dike @ 2009-09-14 23:29 ` H. Peter Anvin 2009-09-15 3:33 ` Jeff Dike 0 siblings, 1 reply; 15+ messages in thread From: H. Peter Anvin @ 2009-09-14 23:29 UTC (permalink / raw) To: Jeff Dike Cc: Ingo Molnar, Sam Ravnborg, David Woodhouse, linux-kernel, mingo, tglx, David.Woodhouse, linux-tip-commits On 09/11/2009 02:16 PM, Jeff Dike wrote: > > I figured out how to make this work - see below. > > Jeff > Jeff, could you re-send this as a proper patch with an SOB line? -hpa ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-14 23:29 ` H. Peter Anvin @ 2009-09-15 3:33 ` Jeff Dike 2009-09-25 9:03 ` H. Peter Anvin 0 siblings, 1 reply; 15+ messages in thread From: Jeff Dike @ 2009-09-15 3:33 UTC (permalink / raw) To: H. Peter Anvin Cc: Ingo Molnar, Sam Ravnborg, David Woodhouse, linux-kernel, mingo, tglx, David.Woodhouse, linux-tip-commits On Mon, Sep 14, 2009 at 04:29:38PM -0700, H. Peter Anvin wrote: > Jeff, could you re-send this as a proper patch with an SOB line? I could - I wanted to get it out for comment first in case anyone had any conniptions, especially about moving the include of the arch Makefile. I'll take another look at it tomorrow and send it out properly... Jeff ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-15 3:33 ` Jeff Dike @ 2009-09-25 9:03 ` H. Peter Anvin 2009-09-28 4:31 ` Sam Ravnborg 0 siblings, 1 reply; 15+ messages in thread From: H. Peter Anvin @ 2009-09-25 9:03 UTC (permalink / raw) To: Jeff Dike Cc: Ingo Molnar, Sam Ravnborg, David Woodhouse, linux-kernel, mingo, tglx, David.Woodhouse Jeff Dike wrote: > On Mon, Sep 14, 2009 at 04:29:38PM -0700, H. Peter Anvin wrote: >> Jeff, could you re-send this as a proper patch with an SOB line? > > I could - I wanted to get it out for comment first in case anyone had > any conniptions, especially about moving the include of the arch > Makefile. > > I'll take another look at it tomorrow and send it out properly... > > Jeff Ping on this? Also, Sam, it would be great if you could comment on the patch: http://marc.info?i=20090911211654.GA11399@c2.user-mode-linux.org -hpa ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' 2009-09-25 9:03 ` H. Peter Anvin @ 2009-09-28 4:31 ` Sam Ravnborg 0 siblings, 0 replies; 15+ messages in thread From: Sam Ravnborg @ 2009-09-28 4:31 UTC (permalink / raw) To: H. Peter Anvin Cc: Jeff Dike, Ingo Molnar, David Woodhouse, linux-kernel, mingo, tglx, David.Woodhouse On Fri, Sep 25, 2009 at 02:03:47AM -0700, H. Peter Anvin wrote: > Jeff Dike wrote: >> On Mon, Sep 14, 2009 at 04:29:38PM -0700, H. Peter Anvin wrote: >>> Jeff, could you re-send this as a proper patch with an SOB line? >> >> I could - I wanted to get it out for comment first in case anyone had >> any conniptions, especially about moving the include of the arch >> Makefile. >> >> I'll take another look at it tomorrow and send it out properly... >> >> Jeff > > Ping on this? Also, Sam, it would be great if you could comment on the > patch: > > http://marc.info?i=20090911211654.GA11399@c2.user-mode-linux.org I took a quick look at it this weekend. My first reaction was that um is now again doing it best to workaround something rather than to use something. But I have not had time to come up with any alternative proposal. I may give it a spin the coming weekend. Sam ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2009-09-28 4:31 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <tip-5a8a2d13b1526e306ff2a9fe12dc9d5878d355f9@git.kernel.org>
2009-09-04 14:48 ` [tip:x86/kbuild] x86: Don't silently override CONFIG_64BIT in 'make oldconfig' Ingo Molnar
2009-09-04 15:33 ` David Woodhouse
2009-09-04 16:27 ` H. Peter Anvin
2009-09-04 16:38 ` Randy Dunlap
2009-09-04 16:53 ` H. Peter Anvin
2009-09-04 21:40 ` David Woodhouse
2009-09-04 22:54 ` H. Peter Anvin
2009-09-04 18:31 ` Ingo Molnar
2009-09-04 18:57 ` H. Peter Anvin
2009-09-04 19:04 ` H. Peter Anvin
2009-09-11 21:16 ` Jeff Dike
2009-09-14 23:29 ` H. Peter Anvin
2009-09-15 3:33 ` Jeff Dike
2009-09-25 9:03 ` H. Peter Anvin
2009-09-28 4:31 ` Sam Ravnborg
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox