[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libvir] virDomainConfigureDevice?

On Wed, Jul 18, 2007 at 10:40:48AM +0100, Richard W.M. Jones wrote:
> Hugh Brock wrote:
> >I'm looking at ways to replicate xm block-configure at the libvirt 
> >level. xm block-configure is useful in that it allows you to change the 
> >back device for an HVM guest while the guest is still running; this 
> >permits you for example to change the CDROM without shutting down your 
> >guest or poking around in xenstore. Thus we can have an "eject" button 
> >on cdrom devices in virt-manager (or "eject" and "load" buttons, I 
> >guess), which we really need.
> >
> >The issue of course is that xm block-configure is specifically intended 
> >for block devices and takes arguments for the back device and the front 
> >device (among others). We would naturally prefer to accept a block of 
> >XML just as virDomainAttachDevice does and then parse it, determine if 
> >the device in question is of a type we can actually edit, determine if 
> >the new backdev is appropriate, and so on.
> Definitely this will be useful for USB ...
> I think the question is: Should we have another entry point 
> (virDomainConfigureDevice), or should we just modify 
> virDomainAttachDevice so that if it sees XML for a device which is 
> already attached it just modifies the device?

  It was raised previously that it's okay to recreate an existing domain
to modify its configuration, then allowing virDomainAttachDevice to
override an existing definition would be consistent with this. Another
bonus point is one less entry point in the library and drivers.


Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]