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

Re: [libvirt] [PATCH v2 0/9] Introduce virDomainGetState API



On Tue, May 10, 2011 at 15:39:02 +0200, Jiri Denemark wrote:
> This new API solves several problems:
> 
> - calling virDomainGetInfo for just getting domain status is an overkill since
>   it may result in sending requests to guest OS
> - since virDomainGetInfo can hang when guest OS is not responding and it is
>   used by virsh list, listing domains can hang
> - virDomainGetState provides additional info about what action led to domain's
>   current state, which can be used instead of listening to domain events
> 
> Version 2:
> - rebased to current HEAD
> - unsigned int flags parameter
> - updated version info in public.syms
> - noGetState renamed as useGetInfo in virsh.c
> - simplified implementation in esx driver per Matthias' suggestion
> - call internal xen drivers directly instead of going through xenUnifiedDriver
> - fixed || vs && typo in domain_conf.c
> - new patch 9/9: qemu: Update domain state when reconnecting monitor

I pushed this series now. Thanks for the reviews.

Jirka


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