Re: [PATCH v2] qemu: Introduce state_lock_timeout toqemu.conf

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

 



> On 08/27/2018 12:57 PM, Yi Wang wrote:
> > When doing some job holding state lock for a long time,
> > we may come across error:
> >  #swtpm_user = "tss"
> >  #swtpm_group = "tss"
> > +
> > +# The timeout (in seconds) waiting for acquiring state lock.
>
> This is rather sparse description. I know that state change lock is,
> because I'm a libvirt devel. However, I don't expect our users to know
> that. How about adding the following description:
>
>   When two or more threads want to work with the same domain they use a
>   job lock to mutually exclude each other. However, waiting for the lock
>   is limited up to state_lock_timeout seconds.
That's much better, thanks.

>
> Also, could you move this close to max_queued variable since they both
> refer to the same area?
Of course, I will adjust it.

> > +    if (virConfGetValueInt(conf, "state_lock_timeout", &cfg->stateLockTimeout) < 0)
> > +        goto cleanup;
> > +
>
> Almost, you need to check if cfg->stateLockTimeout is not zero. Such
> code could go into virQEMUDriverConfigValidate().

Thanks again for your time and patience, Michal.

---
Best wishes
Yi Wang
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux