[libvirt] [PATCH 2/3] Introduce two new methods for triggering controlled shutdown/reboot

Eric Blake eblake at redhat.com
Fri Nov 30 15:22:56 UTC 2012

> > Alas, you are missing documentation of the new flags in
> > src/libvirt.c (not that the existing flags were documented
> > there yet),
> Well the docs refer to the corresponding enum, whcih will
> be included in the docs with its descriptions. So isn't
> that better than duplicating the enum descriptions again.

I suppose that's okay, if you think the enum itself has enough details
about what each flag means.
But there's still the question of semantics, on whether we
have properly captured the fact that you can now specify
more than one flag, and if so, it is hypervisor choice on
which order to attempt the flags.

At any rate, ACK to this patch, now that you've done a separate
patch for moving the mutual exclusion of flags into the
qemu driver.

More information about the libvir-list mailing list