Re: [libvirt PATCH 9/9] util: add logging about node suspend availability

On Wed, Jun 21, 2023 at 14:32:32 +0100, Daniel P. Berrangé wrote: > Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com&gt; > --- > src/util/virnodesuspend.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/util/virnodesuspend.c b/src/util/virnodesuspend.c > index 01f387d6fa..91a7f10eb9 100644 > --- a/src/util/virnodesuspend.c > +++ b/src/util/virnodesuspend.c > @@ -249,6 +249,7 @@ virNodeSuspendSupportsTargetPMUtils(unsigned int target, bool *supported) > * (i.e., the PM capability is supported) > */ > *supported = (status == 0); > + VIR_DEBUG("Node systemd pm-utils target %d: %d", target, *supported); ... > > return 0; > } > @@ -257,6 +258,7 @@ static int > virNodeSuspendSupportsTargetPMUtils(unsigned int target G_GNUC_UNUSED, > bool *supported G_GNUC_UNUSED) > { > + VIR_DEBUG("Node systemd pm-utils target %d: unsupported platform", target); ... extra 'systemd' ? > return -2; > } > #endif /* ! WITH_PM_UTILS */ > @@ -282,6 +284,7 @@ virNodeSuspendSupportsTargetSystemd(unsigned int target, bool *supported) > return ret; > } > > + VIR_DEBUG("Node systemd systemd target %d: %d", target, ret); systemd systemd?

> return ret; > } Reviewed-by: Peter Krempa <pkrempa(a)redhat.com&gt;