[libvirt] [PATCH 09/12] vcpu: add virsh support
Daniel Veillard
veillard at redhat.com
Fri Oct 1 15:15:08 UTC 2010
On Wed, Sep 29, 2010 at 06:02:13PM -0600, Eric Blake wrote:
> * tools/virsh.c (cmdSetvcpus): Add new flags. Let invalid
> commands through to driver, to ease testing of hypervisor argument
> validation.
> (cmdVcpucount): New command.
> (commands): Add new command.
> * tools/virsh.pod (setvcpus, vcpucount): Document new behavior.
> ---
>
> I know - the typical API addition sequence adds driver support
> first and then virsh support. I can rearrange the patch order
> if desired.
Nahh, fine, actually the entry points are there, so really there
is no problem !
[...]
> /*
> + * "vcpucount" command
> + */
> +static const vshCmdInfo info_vcpucount[] = {
> + {"help", N_("domain vcpu counts")},
> + {"desc", N_("Returns the number of domain virtual CPUs.")},
Ouch "Returns the number of virtual CPUs used by the domain." would
be clearer I think
[...]
> + if (maximum + current + persistent + active == 1) {
> + vshError(ctl,
> + _("when using --%s, either --%s or --%s must be specified"),
> + maximum ? "maximum" : current ? "current"
> + : persistent ? "persistent" : "active",
> + maximum + current ? "persistent" : "maximum",
> + maximum + current ? "active" : "current");
Ouch, headache :-) but but looks right
Okay, code ended up being a bit more complex than I expected but that's
due to the various options and the fallback to old APIs, fine,
ACK,
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel at veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
More information about the libvir-list
mailing list