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

Re: [Xen-devel] [PATCH V2 01/12] Create efi-shared.[ch], and move string functions



On Mon, 2014-07-28 at 17:00 +0100, Jan Beulich wrote:
> >>> On 28.07.14 at 17:56, <Ian.Campbell@xxxxxxxxxx> wrote:
> > On Mon, 2014-07-28 at 16:52 +0100, Jan Beulich wrote:
> >> > Even if not it looks like ~20K of mostly __init stuff, which doesn't
> >> > seem like the end of the world, especially given that more and more
> >> > toolstacks do support EFI with time.
> >> 
> >> Right now - with the runtime code not moved over yet - it's
> >> mostly __init. Plus (with the linker not being able to discard that
> >> code) it carries the risk of having references to symbols that
> >> don't exist in the non-EFI build.
> > 
> > Perhaps we can put the relevant code into efi specific sections and DTRT
> > in xen.lds.S?
> 
> Maybe it could be made work, but I'd be wary of linker version issues
> then.

Rather than arch .c files including common .c (or .inc) files how about
making xen/arch/*/efi/Makefile link xen/commmon/efi/built-in.o into it's
own built-in.o instead of having xen/common/Makefile do it like would
normally happen?

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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