[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH 1/2] xen/generic: Disable fallback build on ARM.
On Wed, Nov 07, 2012 at 08:38:44AM +0000, Jan Beulich wrote: > >>> On 07.11.12 at 08:19, Ian Campbell <ian.campbell@xxxxxxxxxx> wrote: > > On Tue, 2012-11-06 at 22:13 +0000, Konrad Rzeszutek Wilk wrote: > >> As there is no need for it (the fallback code is for older > >> hypervisors and they won't run under ARM), > > > > I think more specifically they won't run on anything other than x86. > > > > [...] > >> diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile > >> index 46de6cd..273d2b9 100644 > >> --- a/drivers/xen/Makefile > >> +++ b/drivers/xen/Makefile > >> @@ -1,8 +1,8 @@ > >> ifneq ($(CONFIG_ARM),y) > >> -obj-y += manage.o balloon.o > >> +obj-y += manage.o balloon.o fallback.o > >> obj-$(CONFIG_HOTPLUG_CPU) += cpu_hotplug.o > >> endif > > > > I think : > > obj-$(CONFIG_X86) += fallback.o > > would better reflect what is going on here. > > Yes please. >From 6bf926ddd44ddc67edbeb28d4069f207f2c6e07e Mon Sep 17 00:00:00 2001 From: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> Date: Tue, 6 Nov 2012 15:49:27 -0500 Subject: [PATCH 1/2] xen/generic: Disable fallback build on ARM. As there is no need for it (the fallback code is for older hypervisors and they only run under x86), and also b/c we get: drivers/xen/fallback.c: In function 'xen_event_channel_op_compat': drivers/xen/fallback.c:10:19: error: storage size of 'op' isn't known drivers/xen/fallback.c:15:2: error: implicit declaration of function '_hypercall1' [-Werror=implicit-function-declaration] drivers/xen/fallback.c:15:19: error: expected expression before 'int' drivers/xen/fallback.c:18:7: error: 'EVTCHNOP_close' undeclared (first use in this function) drivers/xen/fallback.c:18:7: note: each undeclared identifier is reported only once for each function it appears in .. and more [v1: Moved the enablement to be covered by CONFIG_X86 per Ian's suggestion] Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> --- drivers/xen/Makefile | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile index 46de6cd..7435470 100644 --- a/drivers/xen/Makefile +++ b/drivers/xen/Makefile @@ -2,7 +2,8 @@ ifneq ($(CONFIG_ARM),y) obj-y += manage.o balloon.o obj-$(CONFIG_HOTPLUG_CPU) += cpu_hotplug.o endif -obj-y += grant-table.o features.o events.o fallback.o +obj-$(CONFIG_X86) += fallback.o +obj-y += grant-table.o features.o events.o obj-y += xenbus/ nostackp := $(call cc-option, -fno-stack-protector) -- 1.7.7.6 > > Jan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |