On Mon, Mar 19, 2007 at 12:51:53PM +0900, Horms wrote:
> On Fri, Mar 16, 2007 at 10:10:24AM +0000, Ian Campbell wrote:
> > This is necessary when running Xen with a 64 bit hypervisor and 32 bit
> > domain 0 since the CPU crash notes will be 64 bit.
> > 
> > Detecting the hypervisor archiecture requires libxenctrl and therefore this
> > support is optional and disabled by default.

I have queued up this patch. Thanks :-)

> > 
> > Signed-off-by: Ian Campbell <[EMAIL PROTECTED]>
> > ---
> > 
> >  configure.ac          |    9 +++++++++
> >  kexec/crashdump-elf.c |    2 +-
> >  kexec/crashdump-xen.c |   42 ++++++++++++++++++++++++++++++++++++++++++
> >  kexec/crashdump.c     |    8 ++++++++
> >  kexec/crashdump.h     |    3 +++
> >  5 files changed, 63 insertions(+), 1 deletions(-)
> > 
> > diff --git a/configure.ac b/configure.ac
> > index 98afd5e..df8dfa3 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -72,6 +72,9 @@ AC_ARG_WITH([gamecube], 
> > AC_HELP_STRING([--with-gamecube],[enable gamecube suppor
> >  AC_ARG_WITH([zlib], AC_HELP_STRING([--without-zlib],[disable zlib 
> > support]),
> >     [ with_zlib="$withval"], [ with_zlib=yes ] )
> >  
> > +AC_ARG_WITH([xen], AC_HELP_STRING([--with-xen],[enable extended xen 
> > support]),
> > +   [ with_xen="$withval"], [ with_xen=no ] )
> > +
> 
> I think that it is reasinable to make with_xen=yes the default,
> as the preseance of the required header is checked below.

I will post a separate patch for this minor change.

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

_______________________________________________
fastboot mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/fastboot

Reply via email to