From c4a8ca71b10ebb3131b4dd13f9a1a41ba52047d3 Mon Sep 17 00:00:00 2001 From: Taku Izumi Date: Fri, 10 Jun 2011 15:39:38 +0900 Subject: [PATCH] vcpupin: add reset option to virsh vcpupin command When resetting vcpupin setting, we have to specify all host physical cpus as a cpulist parameter of virsh vcpupin command. It's a little tedious. This patch changes to allow to receive the special keyword 'r' as a cpulist parameter of virsh vcpupin command when resetting vcpupin setting. If you set the following: # virsh vcpupin VM 0 r the vcpu0 will be pinned to all physical cpus. Signed-off-by: Taku Izumi --- tools/virsh.c | 7 ++++++- tools/virsh.pod | 2 ++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/tools/virsh.c b/tools/virsh.c index 92faffc02c..e04c9fcae6 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -3062,8 +3062,13 @@ cmdVcpupin(vshControl *ctl, const vshCmd *cmd) /* Parse cpulist */ cur = cpulist; - if (*cur == 0) + if (*cur == 0) { goto parse_error; + } else if (*cur == 'r') { + for (cpu = 0; cpu < maxcpu; cpu++) + VIR_USE_CPU(cpumap, cpu); + *cur = 0; + } while (*cur != 0) { diff --git a/tools/virsh.pod b/tools/virsh.pod index b90f37eab7..e7dcd41d82 100644 --- a/tools/virsh.pod +++ b/tools/virsh.pod @@ -836,6 +836,8 @@ Pin domain VCPUs to host physical CPUs. The I number must be provided and I is a list of physical CPU numbers. Its syntax is a comma separated list and a special markup using '-' and '^' (ex. '0-4', '0-3,^2') can also be allowed. The '-' denotes the range and the '^' denotes exclusive. +If you want to reset vcpupin setting, that is, to pin vcpu all physical cpus, +simply specify 'r' as a cpulist. If I<--live> is specified, affect a running guest. If I<--config> is specified, affect the next boot of a persistent guest. If I<--current> is specified, affect the current guest state.