[Qemu-devel] Re: [PATCH 08/15] xen: Read and write the state of the VM in xenstore

[Stefano Stabellini]

On Mon, 16 Aug 2010, Paolo Bonzini wrote:
> On 08/16/2010 01:15 PM, Stefano Stabellini wrote:
> > On Sun, 15 Aug 2010, Paolo Bonzini wrote:
> >> On 08/13/2010 02:53 PM, Anthony Liguori wrote:
> >>> On 08/12/2010 09:09 AM, address@hidden wrote:
> >>>> From: Anthony PERARD<address@hidden>
> >>>>
> >>>> Introduce functions to read and write the state of the VM in xenstore.
> >>>
> >>> This basically creates a new management interface for QEMU via the
> >>> xenstore.
> >>>
> >>> Our management interface is QMP. If you want to maintain compatibility,
> >>> you'll need to write a QMP ->  xenstore daemon that maps events
> >>> appropriately.
> >>
> >> This would belong in xl/libxl.
> >
> > Yes, but considering that we don't want a xenstore-based management
> > interface I would gladly do without the compatibility daemon.
> > We'll try to reduce all the xenstore interaction to the bare minimum,
> > and use QMP everywhere else.
> 
> Yes, I was just pointing out that you don't need a new compatibility 
> daemon.  Since xl is already daemonizing itself to handle on_crash and 
> friends, it could also set up all the watches it cares about, and 
> convert them to QMP commands.  It's all more code that needs to be 
> written of course, and boring stuff even. :)

Well yes, but after all you cannot write interesting code all the time
:)