On 07/15/2011 01:02 AM, Taku Izumi wrote:
>
> This patch adds the --current option to "virsh setvcpus"
> command. Currently "virsh setvcpus" command supports
> "--live" and "--config" , but "--current" option.
>>From view of consistency, it's reasonable to support
> "--current" option too.
>
> When --current is specified, it affects a "current"
> domain.
>
> Signed-off-by: Taku Izumi <izumi.taku@xxxxxxxxxxxxxx>
> ---
> tools/virsh.c | 26 ++++++++++++++++++++++----
> tools/virsh.pod | 6 ++++--
> 2 files changed, 26 insertions(+), 6 deletions(-)
> @@ -3294,6 +3310,8 @@ cmdSetvcpus(vshControl *ctl, const vshCm
> if (maximum) {
> vshDebug(ctl, VSH_ERR_DEBUG, "--maximum flag was given\n");
>
> + flags |= VIR_DOMAIN_VCPU_MAXIMUM;
This is a bit separated from the rest of flags settings, so it took me
two reads to make sure you didn't lose the bit, but your logic is valid.
So no changes needed.
>
> =item B<setvcpus> I<domain-id> I<count> optional I<--maximum> I<--config>
> -I<--live>
> +I<--live> I<--current>
Oh fun - merge conflict with my patch to tweak the use of "optional" in
virsh.pod:
https://www.redhat.com/archives/libvir-list/2011-July/msg00868.html
>
> Change the number of virtual CPUs active in a guest domain. By default,
> this command works on active guest domains. To change the settings for an
> @@ -758,7 +758,9 @@ If I<--live> is specified, the guest dom
> takes place immediately. Both the I<--config> and I<--live> flags may be
> specified together if supported by the hypervisor.
>
> -When neither the I<--config> nor I<--live> flags are given, the I<--live>
> +If I<--current> is specified, affect the current guest state.
> +
> +When neither flags are given, the I<--live>
s/neither/no/
> flag is assumed and the guest domain must be active. In this situation it
> is up to the hypervisor whether the I<--config> flag is also assumed, and
> therefore whether the XML configuration is adjusted to make the change
ACK with the one word nit fixed, so series pushed.
--
Eric Blake eblake@xxxxxxxxxx +1-801-349-2682
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list