Re: [PATCH 0/3] qemu: Don't call qemuMonitorGetCPUInfo to update vCPU halted state

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

 



On 21.11.2016 16:30, Peter Krempa wrote:
> The original implementation reused qemuMonitorGetCPUInfo to update the halted
> state. The function is very complex and should not be called all the time just
> to update a trivial parameter.
> 
> Add infrastructure to properly update the state without the need to match in
> hotplug parameters.
> 
> Peter Krempa (3):
>   qemu: monitor: Extract qemu cpu id along with other data
>   qemu: monitor: Extract halted state to a bitmap indexed by cpu id
>   qemu: domain:  Refresh vcpu halted state using qemuMonitorGetCpuHalted
> 
>  src/qemu/qemu_domain.c       | 20 ++++++++------------
>  src/qemu/qemu_domain.h       |  1 +
>  src/qemu/qemu_monitor.c      | 40 ++++++++++++++++++++++++++++++++++++++++
>  src/qemu/qemu_monitor.h      |  3 +++
>  src/qemu/qemu_monitor_json.c |  3 +++
>  src/qemu/qemu_monitor_text.c | 11 +++++++++++
>  tests/qemumonitorjsontest.c  |  8 ++++----
>  7 files changed, 70 insertions(+), 16 deletions(-)
> 
Could you please hold off pushing? I've just run a sniff test on s390x
and see erratic values for halted. I'll try to investigate and get back
to you.

-- 

Mit freundlichen Grüßen/Kind Regards
   Viktor Mihajlovski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martina Köderitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294

--
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]
  Powered by Linux