WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] tools: disable linker --as-needed option

To: Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] tools: disable linker --as-needed option
From: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Date: Tue, 1 Feb 2011 09:27:23 +0000
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Nathan March <nathan@xxxxxx>
Delivery-date: Tue, 01 Feb 2011 01:28:28 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D47DEC1020000780002FA3C@xxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Citrix Systems, Inc.
References: <a3fb60cc3be3dc6596ed.1296551385@xxxxxxxxxxxxxxxxxxxxx> <4D47DEC1020000780002FA3C@xxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Tue, 2011-02-01 at 09:21 +0000, Jan Beulich wrote:
> >>> On 01.02.11 at 10:09, Ian Campbell <ian.campbell@xxxxxxxxxx> wrote:
> > # HG changeset patch
> > # User Ian Campbell <ian.campbell@xxxxxxxxxx>
> > # Date 1296550939 0
> > # Node ID a3fb60cc3be3dc6596ed8395e0eb70514bdfcc74
> > # Parent  5b6663ba2bb2c54e8fa6745afa16297ebe43328d
> > tools: disable linker --as-needed option.
> > 
> > The Xen build system is not currently compatible with the --as-needed
> > linker option. The proper fix for this is turning out to be rather
> > invasive to the build system so simply disable for now with the
> > intention of revisiting for the 4.2 release.
> > 
> > The $(COMMA) trick allows "," in macro arguments.
> > 
> > Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > Reported/Tested-by: Nathan March <nathan@xxxxxx>
> > 
> > diff -r 5b6663ba2bb2 -r a3fb60cc3be3 tools/Rules.mk
> > --- a/tools/Rules.mk        Mon Jan 31 09:14:52 2011 +0000
> > +++ b/tools/Rules.mk        Tue Feb 01 09:02:19 2011 +0000
> > @@ -55,6 +55,10 @@ LDFLAGS += $(shell getconf LFS_LDFLAGS)
> >  LDFLAGS += $(shell getconf LFS_LDFLAGS)
> >  endif
> >  
> > +# Xen tools build is currently incompatible with ld --as-needed
> > +COMMA := ,
> > +LDFLAGS += $(call cc-option,$(CC),-Wl$(COMMA)--no-as-needed)
> 
> I don't think this actually works (Did you try with a linker not
> supporting this option?), as cc-option is implemented passing
> -S to the compiler (i.e. the linker won't even get invoked).

Er, no. I think you are probably right, I only tested with a linker
which has the option defaulting to off and Nathan tested the default on
case but noone actually tested the not available option.

Any idea how we can sensibly do this test? I'm not even sure when
--{no-,}as-needed came along, perhaps we don't actually need to worry?

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel