On Wed, Mar 28, 2007 at 03:13:28PM +0100, Daniel P. Berrange wrote:
On Wed, Mar 28, 2007 at 07:07:32AM -0400, Daniel Veillard wrote:
If the fact of being inactive is only represented internally by having id == -1 then we probably made a mistake, we should have domain->flags & DOMAIN_IS_DEFINED bit set.
We already have a state flag for it
VIR_DOMAIN_SHUTOFF
which is set in virDomainInfoPtr, we can fix virt-manager not to call get_id if the domain is shutdown.
It would be nice though if each time you look whether the domain is defined or not, libvirt could give teh answer directly instead of going though the potentially expensive Info call. This would also require to double check invalidation of the _virDomain cached data at the proper places. Daniel -- 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/