On 05/23/14 14:45, Peter Krempa wrote: > On 05/23/14 14:37, Ján Tomko wrote: >> QEMU does not join vcpu threads after vcpu hotunplug. >> The thread might stay pinned when we hotplug the cpu again. >> >> We were already pinning the hotplugged vcpus when a per-domain >> pinning was specified. >> >> Explicitly pin the hotplugged vcpu to all cpus if there is no >> cpuset specified in domain XML, to make sure the vcpupin >> stays accurate. >> >> https://bugzilla.redhat.com/show_bug.cgi?id=1099836 >> --- >> src/qemu/qemu_driver.c | 20 ++++++++++++++++++++ >> 1 file changed, 20 insertions(+) >> > > The statements about qemu are true only for the downstream version. > Upstream qemu promises that it will clean up the threads once they > implement vCPU hot unplug so we will not need to execute the code you've > proposed in upstream. Hmmm, re-reading the code ... pinning the currently hotplugged vcpu thread to all cpus isn't silly on the other hand. The stuff is that qemu upstream doesn't do hot unplug and thus the statements about qemu shouldn't be in libvirt's code. I'll respond to the original mail again to clarify my intent. > > NACK for merging this upstream. > > Peter > Peter > > > > -- > libvir-list mailing list > libvir-list@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/libvir-list >
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list