24 messages in com.xensource.lists.xen-develRe: [Xen-devel] PATCH: 0/10: Merge xe...
FromSent OnAttachments
Daniel P. Berrange15 Aug 2007 13:00 
Daniel P. Berrange15 Aug 2007 13:02 
Daniel P. Berrange15 Aug 2007 13:03 
Daniel P. Berrange15 Aug 2007 13:04 
Daniel P. Berrange15 Aug 2007 13:05 
Daniel P. Berrange15 Aug 2007 13:06 
Daniel P. Berrange15 Aug 2007 13:07 
Daniel P. Berrange15 Aug 2007 13:08 
Daniel P. Berrange15 Aug 2007 13:09 
Daniel P. Berrange15 Aug 2007 13:10 
Daniel P. Berrange15 Aug 2007 13:11 
Keir Fraser16 Aug 2007 00:29 
Daniel P. Berrange16 Aug 2007 05:49 
Keir Fraser16 Aug 2007 05:53 
Daniel P. Berrange16 Aug 2007 08:33 
Keir Fraser16 Aug 2007 09:03 
Pat Campbell16 Aug 2007 09:12 
Keir Fraser16 Aug 2007 09:17 
Daniel P. Berrange16 Aug 2007 09:21 
Mark Williamson16 Aug 2007 12:43 
Ben Guthro16 Aug 2007 13:07.patch
Daniel P. Berrange17 Aug 2007 13:12 
Daniel P. Berrange17 Aug 2007 13:13 
Keir Fraser18 Aug 2007 03:10 
Subject:Re: [Xen-devel] PATCH: 0/10: Merge xenfb & xenconsoled into qemu-dm
From:Keir Fraser (Keir@cl.cam.ac.uk)
Date:08/18/2007 03:10:57 AM
List:com.xensource.lists.xen-devel

On 17/8/07 21:12, "Daniel P. Berrange" <berr@redhat.com> wrote:

Thinking about this, I think I can easily re-work the last two patches so that xenconsoled will continue to process the guest consoles, if-and-only-if the guest doesn't have a QEMU instance already doing it. That would give us choice between both deployment scenarios per-guest.

I've got this working ok - since both xenconsoled and qemu-dm are already reading the /local/domain/<id>/console/ path in xenstore, I simply added an extra field 'use-device-model' which can be 1 or 0. If XenD sees that a PV domain has a device model then it sets it to 1, causing xenconsoled to ignore that guest. I'll post patches to this effect over the weekend

Sounds basically good, but a field called 'type' in the backend xenstore directory would be more flexible and more consistent with other backend directory contents (at least netback's). We could use values 'ioemu' and 'xenconsoled', and absence of the field implies xenconsoled. Both ioemu and xenconsoled should check the field.

-- Keir