Re: [libvirt PATCH 1/8] daemons: Support --timeout 0

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

 



On Thu, 2020-04-02 at 13:05 +0100, Daniel P. Berrangé wrote:
> On Wed, Apr 01, 2020 at 08:53:38PM +0200, Andrea Bolognani wrote:
> > -    if (timeout != -1) {
> > +    if (timeout > 0) {
> >          VIR_DEBUG("Registering shutdown timeout %d", timeout);
> >          virNetDaemonAutoShutdown(lockDaemon->dmn,
> >                                   timeout);
> 
> This conditional can be removed because virNetDaemonAutoShutdown
> treats "timeout == 0" as meaning no timeout.

Can do, although to be honest I kinda like the current form, with the
debug message only printed when a timeout is actually configured, a
bit better.

-- 
Andrea Bolognani / Red Hat / Virtualization





[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