[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



>>> On 11.03.11 at 17:16, Keir Fraser <keir@xxxxxxx> wrote:
> 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).

It's not tied to obj-bin-y, but rather to the .init.o rule, but yes, the
problem would possibly affect any such object. My problem is that
so far I wasn't able to think of a (clean and maintenance free) way
to force in a -D for those very files, since the compilation step is
the same as for "normal" .o-s. Hence, rather than waiting for
people to start asking what the resulting error message means, I
thought we should at least fix it in the place where the problem
was observed in reality (which happened to me when I put the
new bits on a system that I don't update that regularly).

Jan


_______________________________________________
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®.