Re: [PATCH v3 0/9] Introduce display of IOThreads Information

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

 




On 02/24/2015 07:01 AM, John Ferlan wrote:
> 
> 
> On 02/17/2015 04:03 PM, John Ferlan wrote:
>> v2 here:
>> http://www.redhat.com/archives/libvir-list/2015-February/msg00562.html
>>
>> Changes over v2:
>> Patches 1-4
>>  * Add "resources" and "nresources" in order to return the resources that
>>    are using an IOThread
>>  * Fixed memory leak - neglected to free the qemuMonitorIOThreadsInfoPtr
>>    'iothreads' in patch 3
>>
>> Patches 5-9 (NEW)
>>  * Patches 5 & 6 implement the API/remote protocol for virDomainSetIOThreads
>>  * Patch 7 implements a couple of helper function - copies of existing
>>    vcpupin functions
>>  * Patch 8 implements the qemu backend qemuDomainSetIOThreads in order to
>>    allow setting IOThread pin data either for live or config. Heavily lifted
>>    from existing vcpu code
>>  * Patch 9 implements the virsh command options in order to set the pinning
>>    on a specific thread. As a side effect/benefit one can now display just one
>>    IOThread if so desired.
>>
>> All the changes are sufficient to satisfy the following bz:
>>
>> https://bugzilla.redhat.com/show_bug.cgi?id=1135491
>>
>> John Ferlan (9):
>>   Implement public API for virDomainGetIOThreadsInfo
>>   remote: Implement the remote plumbing for virDomainGetIOThreadsInfo
>>   qemu: Implement the qemu driver fetch for IOThreads
>>   virsh: Add 'iothreads' command
>>   Implement public API for virDomainSetIOThreads
>>   remote: Implement remote plumbing for virDomainSetIOThreads
>>   domain: Introduce virDomainIOThreadsPin{Add|Del}
>>   qemu: Add qemuDomainSetIOThreads
>>   virsh: Allow setting of the iothread pin
>>
>>  daemon/remote.c                  |  99 +++++++-
>>  include/libvirt/libvirt-domain.h |  48 +++-
>>  src/conf/domain_conf.c           |  64 +++++
>>  src/conf/domain_conf.h           |  10 +
>>  src/driver-hypervisor.h          |  16 +-
>>  src/libvirt-domain.c             | 149 ++++++++++-
>>  src/libvirt_private.syms         |   2 +
>>  src/libvirt_public.syms          |   7 +
>>  src/qemu/qemu_driver.c           | 519 +++++++++++++++++++++++++++++++++++++++
>>  src/remote/remote_driver.c       |  94 ++++++-
>>  src/remote/remote_protocol.x     |  48 +++-
>>  src/remote_protocol-structs      |  33 +++
>>  src/rpc/gendispatch.pl           |   1 +
>>  tools/virsh-domain.c             | 154 ++++++++++++
>>  tools/virsh.pod                  |  20 ++
>>  15 files changed, 1257 insertions(+), 7 deletions(-)
>>
> 
> ping - although probably now not a 1.2.13 candidate with the freeze - if
> a review can be completed I could get it in early to 1.2.14...

Now that 1.2.13 is out - I know I have to change to using 1.2.14, but
hopefully someone can review, so I can start on the hotplug options of
iothreads.

tks

John

--
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]