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

Re: [libvirt] [PATCH 0/6] parallels: get rid of cached domains list in driver



On 120905 14:13:52, Dmitry Guryanov wrote:
> The goal of this patch series is to get rid of cached domains
> list.
> Someone can create several connections to libvirt or change
> something (start VM or change parameters) using native tools.
> In such case first connection will not get any info about those
> changes.
> There is no way to check, if the cache valid, so remove it and
> always query needed data using prlctl command.
> 

Hello,

Please, disregard this patch series. Driver works too slow with it,
especially with virt-manager.

We decided to add special mode to prlctl, when it will listen for
events and print them to stdout in json format. Then we can keep
one prlctl process in such mode per libvirt connection and update
cached list of domains, when some events occur.

> 
> Dmitry Guryanov (6):
>   parallels: return a new list of domains from parallelsLoadDomains
>   parallels: rename uuidstr to __uuidstr in macro
>   parallels: return up-to-date info about domains
>   parallels: don't track domain state in global domains list
>   parallels: don't use stored domains list in parallelsDomainDefineXML
>   parallels: remove domains list from _parallelsConn structure
> 
>  src/parallels/parallels_driver.c |  423 ++++++++++++++++++--------------------
>  src/parallels/parallels_utils.h  |    1 -
>  2 files changed, 195 insertions(+), 229 deletions(-)
> 


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