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] x86: force out-of-line instances of inline funct

To: Jan Beulich <JBeulich@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] x86: force out-of-line instances of inline functions into .init.text in init-only code
From: Keir Fraser <keir@xxxxxxx>
Date: Fri, 11 Mar 2011 16:16:32 +0000
Cc:
Delivery-date: Fri, 11 Mar 2011 08:17:24 -0800
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:sender:user-agent:date:subject:from:to :message-id:thread-topic:thread-index:in-reply-to:mime-version :content-type:content-transfer-encoding; bh=AoZ9/U1rcv9dj3XwDaDr8/88wNJwZ420RtdzC+LHdrU=; b=VYp48awEIf8k8SlDABvCnr+QdWv0m7yQDOcMFGvNRkikpkqKp0QTcw4LOWalwT4MOJ Ynmmbpb65l/0ZndqzzBFYxyi43SdwFX39aBP6PPcRen7o2Jv/zgcDPswE8KZpqzExCYe qTQpEJbXso70oAOVk3Zf3sJUiRJ53jwNFmL4Y=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:user-agent:date:subject:from:to:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; b=knm6KfclKfrsq4vLNAPqSHidYoOg73rOwCEnfodfbRP/L6vEC8WwKw60GEWrG2SAH6 QYsycGHuqhUgTrvDqf9ypG9HDPxilPx6wf2MMENgz+8sXT3OPZFdvu4Fz0ELlFlkI/oQ rLL6Jee+eM0071/J6WO71E5uCCFuSdsnwF4h4=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <4D7A571E0200007800035EF2@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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcvgB69DtehNeAIQsUijBjDMIpUfsg==
Thread-topic: [Xen-devel] [PATCH] x86: force out-of-line instances of inline functions into .init.text in init-only code
User-agent: Microsoft-Entourage/12.28.0.101117
On 11/03/2011 16:08, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:

> Some compiler versions may choose to not inline certain functions,
> but the check introduced in c/s 23003:768269c43914 and applying to
> domain_build.o as of 23011:be7e54d86c57 wants .text to be empty.

Isn't this a possible problem for any file compiled under the rules of
obj-bin-y? If so, below should be defined for all such source files, perhaps
-D a macro def on $CC command line in that case (e.g., some obvious textual
macro name) and then pick up on that in <xen/compiler.h> to suitably
re-define inline and always_inline (and explain why in a code comment).

 -- Keir

> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
> 
> --- 2011-03-09.orig/xen/arch/x86/domain_build.c
> +++ 2011-03-09/xen/arch/x86/domain_build.c
> @@ -5,6 +5,7 @@
>   */
>  
>  #include <xen/config.h>
> +#define __inline__ __inline__ __init
>  #include <xen/init.h>
>  #include <xen/lib.h>
>  #include <xen/ctype.h>
> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



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