Re: [Xen-devel] [patch 1/5] add libelf: an ELF binary parser library.

["Christoph Egger" <Christoph.Egger@xxxxxxx>]

On Tuesday 23 January 2007 15:53, Gerd Hoffmann wrote:

[...]

> Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxx>
> ---
>  tools/libxc/xc_elf.h                |  525
> ----------------------------------- xen/arch/x86/boot/mkelf32.c         |  
>  2
>  xen/common/Makefile                 |    2
>  xen/common/libelf/Makefile          |    4
>  xen/common/libelf/README            |    1
>  xen/common/libelf/libelf-dominfo.c  |  420 ++++++++++++++++++++++++++++
>  xen/common/libelf/libelf-loader.c   |  156 ++++++++++
>  xen/common/libelf/libelf-private.h  |   51 +++
>  xen/common/libelf/libelf-relocate.c |  345 +++++++++++++++++++++++
>  xen/common/libelf/libelf-tools.c    |  225 +++++++++++++++
>  xen/include/public/elfstructs.h     |  527

[...]

How about putting libelf into xen/lib/libelf ?
When we also move the libc files from xen/common/
to xen/lib/libc, then we have this structure:

xen/lib/libelf
xen/lib/libc

This is a better structured IMO.

Christoph Egger



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