Re: PATCH, was Re: [Xen-ia64-devel] Re: PATCH: enable Xen compilation with debug=y

[Tristan Gingold <Tristan.Gingold@xxxxxxxx>]

Le Jeudi 15 Juin 2006 16:53, Alex Williamson a Ãcrit :
> On Thu, 2006-06-15 at 10:07 +0200, Tristan Gingold wrote:
> > Le Jeudi 15 Juin 2006 00:24, Alex Williamson a Ãcrit :
> > > On Wed, 2006-06-14 at 11:48 +0200, Tristan Gingold wrote:
> > > > Hi,
> > > >
> > > > this patch fixes a few issue, so that make debug=y works (in Xen).
> > > > I think it is expected by Fedora.
> > > >
> > > > Tested by booting dom0 (debug=y).
> > > > Tested by compiling without debug=y.
> > >
> > > Hi Tristan,
> > >
> > >    This doesn't build for me, there are still things including
> > > asm/hardirq.h.  Did you need me to make a hardirq.h in linux-null?
> >
> > Sorry, I forget an hg add.
> >
> > Here is the updated patch.
>
> Hi Tristan,
>
>    I'm still a little confused about the hardirq.h stuff.  Why drop
> include/asm-ia64/linux/asm/hardirq.h only to split it out into
> include/asm-ia64/linux-xen/linux/hardirq.h and a new
> include/asm-ia64/hardirq.h? 
Xen expects an asm/hardirq.h.  The later must have some definitions of 
linux/hardirq.h
Linux headers expect a linux/hardirq.h

You might prefer the attached patch: it does not modify 
linux-xen/linux/hardirq.h
(tested by compiling with and without debug=y)

> BTW, this latest patch doesn't remove
> include/asm-ia64/linux/asm/hardirq.h, but I assume that's still
> intended.  Thanks,
Yes! (sorry)

Tristan.

Attachment: xen-debug2.diffs
Description: Text Data

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