[libvirt] [PATCH 1/6] virsh: vcpuinfo: Report vcpu number from the structure rather than it's position
Jiri Denemark
jdenemar at redhat.com
Fri Aug 26 02:21:35 UTC 2016
On Thu, Aug 25, 2016 at 18:42:45 -0400, Peter Krempa wrote:
> virVcpuInfo contains the vcpu number that the data refers to. Report
> what's returned by the daemon rather than the sequence number as with
> sparse vcpu topologies they won't match.
> ---
> tools/virsh-domain.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
> index de2a22c..90d2543 100644
> --- a/tools/virsh-domain.c
> +++ b/tools/virsh-domain.c
> @@ -6315,8 +6315,8 @@ cmdVcpuinfo(vshControl *ctl, const vshCmd *cmd)
> }
>
> for (n = 0; n < ncpus; n++) {
> - vshPrint(ctl, "%-15s %d\n", _("VCPU:"), n);
> if (cpuinfo) {
> + vshPrint(ctl, "%-15s %d\n", _("VCPU:"), cpuinfo[n].number);
> vshPrint(ctl, "%-15s %d\n", _("CPU:"), cpuinfo[n].cpu);
> vshPrint(ctl, "%-15s %s\n", _("State:"),
> virshDomainVcpuStateToString(cpuinfo[n].state));
> @@ -6328,6 +6328,7 @@ cmdVcpuinfo(vshControl *ctl, const vshCmd *cmd)
> vshPrint(ctl, "%-15s %.1lfs\n", _("CPU time:"), cpuUsed);
> }
> } else {
> + vshPrint(ctl, "%-15s %d\n", _("VCPU:"), n);
> vshPrint(ctl, "%-15s %s\n", _("CPU:"), _("N/A"));
> vshPrint(ctl, "%-15s %s\n", _("State:"), _("N/A"));
> vshPrint(ctl, "%-15s %s\n", _("CPU time"), _("N/A"));
ACK
Jirka
More information about the libvir-list
mailing list