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] [Mini-OS] Drop useless -fpic flag

To: Samuel Thibault <samuel.thibault@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [Mini-OS] Drop useless -fpic flag
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Wed, 28 Nov 2007 16:42:38 +0000
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 28 Nov 2007 08:43:35 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20071128163501.GK4083@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acgx3a9h7cp6CJ3QEdyqDgAX8io7RQ==
Thread-topic: [Xen-devel] [Mini-OS] Drop useless -fpic flag
User-agent: Microsoft-Entourage/11.3.6.070618
Ah yes, I think the -fpic was inherited from Xen's Makefile. Xen has to be
built -fpic because it is not linked sufficiently close to address 0x0 to
use absolute addressing the 32-bit displacements. This isn't the case for
minios.

 -- Keir

On 28/11/07 16:35, "Samuel Thibault" <samuel.thibault@xxxxxxxxxxxxx> wrote:

> Keir Fraser, le Tue 27 Nov 2007 12:33:23 +0000, a écrit :
>> If it builds without it then it should be fine, I believe.
> 
> And nobody raised his hand, so here is a patch:
> 
> Drop useless -fpic flag.
> 
> Signed-off-by: Samuel Thibault <samuel.thibault@xxxxxxxxxx>
> 
> diff -r c555a5f97982 extras/mini-os/arch/x86/arch.mk
> --- a/extras/mini-os/arch/x86/arch.mk Wed Nov 28 13:36:56 2007 +0000
> +++ b/extras/mini-os/arch/x86/arch.mk Wed Nov 28 16:33:37 2007 +0000
> @@ -17,7 +17,7 @@ endif
>  endif
>  
>  ifeq ($(TARGET_ARCH),x86_64)
> -ARCH_CFLAGS := -m64 -mno-red-zone -fpic -fno-reorder-blocks
> +ARCH_CFLAGS := -m64 -mno-red-zone -fno-reorder-blocks
>  ARCH_CFLAGS += -fno-asynchronous-unwind-tables
>  ARCH_ASFLAGS := -m64
>  ARCH_LDFLAGS := -m elf_x86_64



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

<Prev in Thread] Current Thread [Next in Thread>