Re: [PATCHv2 1/1] Remove contiguous CPU indexes assumption

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2013年03月15日 17:42, Daniel P. Berrange wrote:
On Fri, Mar 15, 2013 at 05:25:09PM +0800, Li Zhang wrote:
From: Li Zhang<zhlcindy@xxxxxxxxxxxxxxxxxx>

When getting CPUs' information, it assumes that CPU indexes
are not contiguous. But for ppc64 platform, CPU indexes are not
contiguous because SMT is needed to be disabled, so CPU information
is not right on ppc64 and vpuinfo, vcpupin can't work corretly.

This patch is to remove the assumption to be compatible with ppc64.

Test:
    4 vcpus are assigned to one VM and execute vcpuinfo command.

    Without patch: There is only one vcpu informaion can be listed.
    With patch: All vcpus' information can be listed correctly.

Signed-off-by: Li Zhang<zhlcindy@xxxxxxxxxxxxxxxxxx>
---
  v2 ->  v1:
   * Remove unused cpu variable suggested by Daniel P.Berrange
   * Remove contigous assumption in qemu_monitor_text.c suggested by Daniel P.Berrange

  src/qemu/qemu_monitor_json.c |   14 --------------
  src/qemu/qemu_monitor_text.c |   13 ++-----------
  2 files changed, 2 insertions(+), 25 deletions(-)

ACK

Daniel

Pushed.

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]