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

[Xen-devel] Re: [POWERPC/IA64] Updates required due to loader changes

To: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [POWERPC/IA64] Updates required due to loader changes
From: Horms <horms@xxxxxxxxxxxx>
Date: Sat, 26 Aug 2006 17:56:23 +0900
Cc: Jimi Xenidis <jimix@xxxxxxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Alex Williamson <alex.williamson@xxxxxx>, Hollis Blanchard <hollisb@xxxxxxxxxx>
Delivery-date: Tue, 29 Aug 2006 02:04:26 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1156498945.12949.180.camel@xxxxxxxxxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20060825032151.C2647340AE@xxxxxxxxxxxxxxxxx> <1156498945.12949.180.camel@xxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.13 (2006-08-11)
On Fri, Aug 25, 2006 at 10:42:25AM +0100, Ian Campbell wrote:
> On Fri, 2006-08-25 at 12:21 +0900, Horms wrote:
> > On Thu, 24 Aug 2006 15:29:48 -0500, Hollis Blanchard wrote:
> > > On Thu, 2006-08-24 at 14:16 -0600, Alex Williamson wrote:
> > >> 
> > >> diff -r d5eb5205ff35 tools/xcutils/readnotes.c
> > >> --- a/tools/xcutils/readnotes.c       Thu Aug 24 16:25:49 2006 +0100
> > >> +++ b/tools/xcutils/readnotes.c       Thu Aug 24 14:12:45 2006 -0600
> > >> @@ -22,7 +22,7 @@ typedef Elf32_Nhdr Elf_Nhdr;
> > >>  typedef Elf32_Nhdr Elf_Nhdr;
> > >>  typedef Elf32_Half Elf_Half;
> > >>  typedef Elf32_Word Elf_Word;
> > >> -#elif defined(__x86_64__)
> > >> +#elif defined(__x86_64__) || defined(__ia64__)
> > >>  typedef Elf64_Ehdr Elf_Ehdr;
> > >>  typedef Elf64_Nhdr Elf_Nhdr;
> > >>  typedef Elf64_Half Elf_Half;
> > > 
> > > Are we doing this again? Ian, please use ELFSIZE, as seen in the
> > > following files:
> > >        tools/libxc/xg_private.h
> > >        tools/libxc/xc_elf.h
> > >        tools/libxc/xc_load_elf.c
> > >        config/powerpc64.mk
> > > 
> > 
> > Is the patch below what you had in mind?
> 
> Nearly, we need to define ELFSIZE if it isn't already since not all
> architectures define it.

I was wondering about that. Perhaps the defination you propose
could be moved somewhere central, like elf.h.

-- 
Horms
  H: http://www.vergenet.net/~horms/
  W: http://www.valinux.co.jp/en/


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