Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755963Ab3GVNuN (ORCPT ); Mon, 22 Jul 2013 09:50:13 -0400 Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:51291 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755700Ab3GVNuL (ORCPT ); Mon, 22 Jul 2013 09:50:11 -0400 Date: Mon, 22 Jul 2013 14:49:34 +0100 From: Mark Rutland To: Julien Grall Cc: "stefano.stabellini@eu.citrix.com" , "konrad.wilk@oracle.com" , "ian.campbell@citrix.com" , "patches@linaro.org" , "linux-kernel@vger.kernel.org" , "xen-devel@lists.xen.org" , "linux-arm-kernel@lists.infradead.org" , catalin.marinas@arm.com Subject: Re: [PATCH v3] xen/arm: enable PV control for ARM Message-ID: <20130722134934.GB32221@e106331-lin.cambridge.arm.com> References: <1374490654-13373-1-git-send-email-julien.grall@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1374490654-13373-1-git-send-email-julien.grall@linaro.org> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1970 Lines: 64 Hi, [Adding Catalin to Cc] On Mon, Jul 22, 2013 at 11:57:34AM +0100, Julien Grall wrote: > Enable power management from the toolstack for ARM guest. > > Signed-off-by: Julien Grall > > --- > Changes in v3: > - Fix condition to compile cpu_hotplug.o > Changes in v2: > - Don't compile xen/cpu_hotplug.o with ARM64 My arm64 cpu hotplug series is going to be dependent on not compiling xen/cpu_hotplug.o. Currently I'm not able to compile an arm64 kernel with my patches and Xen support. How do you intend to push this? I'd like to ensure my cpu hotplug series is able to go in for the next merge window. > --- > drivers/xen/Makefile | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile > index 2bf461a..f185e8d 100644 > --- a/drivers/xen/Makefile > +++ b/drivers/xen/Makefile > @@ -1,9 +1,8 @@ > -ifneq ($(CONFIG_ARM),y) > -obj-y += manage.o > +ifeq ($(filter y, $(CONFIG_ARM) $(CONFIG_ARM64)),) > obj-$(CONFIG_HOTPLUG_CPU) += cpu_hotplug.o > endif Might it make sense to enable this based on a list the architectures that actually need this, rather than disabling it based on the list of architectures that don't? Thanks, Mark. > obj-$(CONFIG_X86) += fallback.o > -obj-y += grant-table.o features.o events.o balloon.o time.o > +obj-y += grant-table.o features.o events.o balloon.o time.o manage.o > obj-y += xenbus/ > > nostackp := $(call cc-option, -fno-stack-protector) > -- > 1.7.10.4 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/