[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] EFI: fix tool chain capabilities detection


  • To: Jan Beulich <JBeulich@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir.xen@xxxxxxxxx>
  • Date: Mon, 09 Sep 2013 04:18:46 -0700
  • Cc: Tim Deegan <tim@xxxxxxx>
  • Delivery-date: Mon, 09 Sep 2013 11:19:19 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>
  • Thread-index: Ac6tTllx6vu69Fh/y0aOJ2h32ho68g==
  • Thread-topic: [PATCH] EFI: fix tool chain capabilities detection

On 26/08/2013 14:55, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:

> Commit f5a54e92 ("xen: move some arch CFLAGS into the common Rules.mk")
> transformed CFLAGS assignments to CFLAGS-y ones, which collides with
> the was xen/arch/x86/efi/Makefile determines whether the tol chain is
> usable for an EFI build. Transform the block back to using CFLAGS.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: Keir Fraser <keir@xxxxxxx>

> --- a/xen/Rules.mk
> +++ b/xen/Rules.mk
> @@ -41,13 +41,12 @@ ALL_OBJS-y               += $(BASEDIR)/x
>  ALL_OBJS-y               += $(BASEDIR)/arch/$(TARGET_ARCH)/built_in.o
>  ALL_OBJS-$(x86)          += $(BASEDIR)/crypto/built_in.o
>  
> -CFLAGS-y                += -fno-builtin -fno-common
> -CFLAGS-y                += -Werror -Wredundant-decls -Wno-pointer-arith
> -CFLAGS-y                += -pipe
> -CFLAGS-y                += -g -D__XEN__ -include
> $(BASEDIR)/include/xen/config.h
> +CFLAGS += -fno-builtin -fno-common
> +CFLAGS += -Werror -Wredundant-decls -Wno-pointer-arith
> +CFLAGS += -pipe -g -D__XEN__ -include $(BASEDIR)/include/xen/config.h
>  # Solaris puts stdarg.h &c in the system include directory.
>  ifneq ($(XEN_OS),SunOS)
> -CFLAGS-y                += -nostdinc -iwithprefix include
> +CFLAGS += -nostdinc -iwithprefix include
>  endif
>  
>  CFLAGS-$(XSM_ENABLE)    += -DXSM_ENABLE
> 
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.