Re: [libvirt PATCH v2 34/44] Deprecate QEMU_CAPS_WAKEUP

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

 



On Mon, 2018-04-09 at 17:20 +0200, Ján Tomko wrote:
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 06b5d1387d..2cfcbacaf0 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -18747,15 +18747,6 @@ qemuDomainPMSuspendForDuration(virDomainPtr dom,
>          goto endjob;
>      }
>  
> -    if (!virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_WAKEUP) &&
> -        (target == VIR_NODE_SUSPEND_TARGET_MEM ||
> -         target == VIR_NODE_SUSPEND_TARGET_HYBRID)) {
> -        virReportError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s",
> -                       _("Unable to suspend domain due to "
> -                         "missing system_wakeup monitor command"));
> -        goto endjob;
> -    }
> -
>      if (vm->def->pm.s3 || vm->def->pm.s4) {
>          if (vm->def->pm.s3 == VIR_TRISTATE_BOOL_NO &&
>              (target == VIR_NODE_SUSPEND_TARGET_MEM ||
> @@ -18816,13 +18807,6 @@ qemuDomainPMWakeup(virDomainPtr dom,
>  
>      priv = vm->privateData;
>  
> -    if (!virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_WAKEUP)) {
> -       virReportError(VIR_ERR_OPERATION_INVALID, "%s",
> -                      _("Unable to wake up domain due to "
> -                        "missing system_wakeup monitor command"));
> -       goto endjob;
> -    }
> -

Now that you removed all uses of 'priv', it doesn't compile
anymore:

  qemu/qemu_driver.c: In function 'qemuDomainPMSuspendForDuration':
  qemu/qemu_driver.c:18699:29: error: variable 'priv' set but not used [-Werror=unused-but-set-variable]
       qemuDomainObjPrivatePtr priv;
                               ^~~~

Simple enough to fix.

Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

-- 
Andrea Bolognani / Red Hat / Virtualization

--
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