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

Re: [Xen-devel] [PATCH] x86: force out-of-line instances of inline functions into .init.text in init-only code


  • To: Jan Beulich <JBeulich@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • 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
  • 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=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcvgB69DtehNeAIQsUijBjDMIpUfsg==
  • Thread-topic: [Xen-devel] [PATCH] x86: force out-of-line instances of inline functions into .init.text in init-only code

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


 


Rackspace

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