Re: [Xen-devel] [PATCH 13/28] libxl: idl: add domain_type field to libxl_dominfo struct

[Rob Hoes <Rob.Hoes@xxxxxxxxxx>]

> > One reason is simply to report to the user what kind of VMs they have
> > running.
> 
> Does xapi/xenopsd not already track this in the DB?
> 
> >  Xenopsd also uses this information internally in some places, e.g.
> > when inserting a CD, it uses libxl_cdrom_insert for HVMs, and does a
> > normal VBD hotplug for PV.
> 
> xenopsd doesn't already know which to use?

It is probably not strictly necessary, because xenopsd would remember whether 
it has started a VM as PV or HVM. But I think that it is in general just a 
little safer to ask Xen what the state of the system is rather than relying on 
internal state in the daemon.

Rob

> I wonder if it is/was a mistake to not have libxl_cdrom_insert do the
> obvious PV thing for PV guests.
> 
> Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel