xenstored.h

xenstored.h

[Magenheimer, Dan (HP Labs Fort Collins)]

Now that xenstore is in the mainline and xenstored.h is
part of the API to xenlinux, should xenstored.h be
moved into xen/include/public instead of being
mkbuildtree'd from tools/xenstore to drivers/xen/xenbus?

Re: xenstored.h

[Rik van Riel]

On Mon, 12 Sep 2005, Magenheimer, Dan (HP Labs Fort Collins) wrote:

> Now that xenstore is in the mainline and xenstored.h is
> part of the API to xenlinux, should xenstored.h be
> moved into xen/include/public instead of being
> mkbuildtree'd from tools/xenstore to drivers/xen/xenbus?

Won't that make it impossible to build xenolinux kernels
outside of the Xen tree - say, kernel RPMs ?

-- 
All Rights Reversed

Re: xenstored.h

[Christian Limpach]

On 9/12/05, Rik van Riel <riel@redhat.com> wrote:
> On Mon, 12 Sep 2005, Magenheimer, Dan (HP Labs Fort Collins) wrote:
> 
> > Now that xenstore is in the mainline and xenstored.h is
> > part of the API to xenlinux, should xenstored.h be
> > moved into xen/include/public instead of being
> > mkbuildtree'd from tools/xenstore to drivers/xen/xenbus?
> 
> Won't that make it impossible to build xenolinux kernels
> outside of the Xen tree - say, kernel RPMs ?

Yeah, you need to include those in your source RPMs - or have some
kind of xen-headers RPM which you'd then require to build kernels...

As Dan pointed out, the same applies for the other files in
xen/include/public...

    christian

Re: xenstored.h

[Chris Wright]

* Magenheimer, Dan (HP Labs Fort Collins) (dan.magenheimer@hp.com) wrote:
> Now that xenstore is in the mainline and xenstored.h is
> part of the API to xenlinux, should xenstored.h be
> moved into xen/include/public instead of being
> mkbuildtree'd from tools/xenstore to drivers/xen/xenbus?

That's basically what my original patch did.

RE: xenstored.h

[Magenheimer, Dan (HP Labs Fort Collins)]

Currently, there are many files in xen/include/public that
are necessary to build xenlinux.  The mkbuildtree process
moves (links?) them to linux/include/asm-xen/xen-public
prior to building linux.

I assume once the API churn settles down, there will need
to be some kind of extra version control to ensure that
these stay sync'd. 

> -----Original Message-----
> From: Rik van Riel [mailto:riel@redhat.com] 
> Sent: Monday, September 12, 2005 1:29 PM
> To: Magenheimer, Dan (HP Labs Fort Collins)
> Cc: xen-devel@lists.xensource.com
> Subject: Re: [Xen-devel] xenstored.h
> 
> On Mon, 12 Sep 2005, Magenheimer, Dan (HP Labs Fort Collins) wrote:
> 
> > Now that xenstore is in the mainline and xenstored.h is
> > part of the API to xenlinux, should xenstored.h be
> > moved into xen/include/public instead of being
> > mkbuildtree'd from tools/xenstore to drivers/xen/xenbus?
> 
> Won't that make it impossible to build xenolinux kernels
> outside of the Xen tree - say, kernel RPMs ?
> 
> -- 
> All Rights Reversed
> 

RE: xenstored.h

[Magenheimer, Dan (HP Labs Fort Collins)]

> From: Chris Wright [mailto:chrisw@osdl.org] 
> (dan.magenheimer@hp.com) wrote:
> > Now that xenstore is in the mainline and xenstored.h is
> > part of the API to xenlinux, should xenstored.h be
> > moved into xen/include/public instead of being
> > mkbuildtree'd from tools/xenstore to drivers/xen/xenbus?
> 
> That's basically what my original patch did.

I'm not sure what "original patch" you mean.  Is it
one that hasn't been applied yet, or has been applied very
recently? 

Re: xenstored.h

[Chris Wright]

* Magenheimer, Dan (HP Labs Fort Collins) (dan.magenheimer@hp.com) wrote:
> > From: Chris Wright [mailto:chrisw@osdl.org] 
> > (dan.magenheimer@hp.com) wrote:
> > > Now that xenstore is in the mainline and xenstored.h is
> > > part of the API to xenlinux, should xenstored.h be
> > > moved into xen/include/public instead of being
> > > mkbuildtree'd from tools/xenstore to drivers/xen/xenbus?
> > 
> > That's basically what my original patch did.
> 
> I'm not sure what "original patch" you mean.  Is it
> one that hasn't been applied yet, or has been applied very
> recently? 

One that got applied about a month ago, just edited from the original
to make it mkbuildtree'd to drivers/xen/xenbus rather than put in public
dir (I chose linux-public rather than xen-public as this is a userspace
interface not a hypervisor interface).