Re: [RFC PATCH 0/8 v2] Summary on block IO throttle
|[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]|
On 10/27/2011 11:45 PM, Eric Blake wrote:
On 10/27/2011 03:12 AM, Lei Li wrote:1) Enable the blkio throttling in xml when guest is starting up. Add blkio throttling in xml as follows: <disk type='file' device='disk'> <driver name='qemu' type='raw'/> <source file='/var/lib/libvirt/images/kvm-one.img'/> <target dev='vda' bus='virtio'/><address type='pci' domain='0x0000' bus='0x00' slot='0x05' function='0x0'/><iotune bps='n'.../> </disk> 2) Enable blkio throttling setting at guest running time.virsh blkiothrottle<domain> <device> [--bps<number>] [--bps_rd<number>] \ [--bps_wr<number>] [--iops<number>] [--iops_rd<number>] [--iops_wr<number>]3) The support to get the current block i/o throttling for a device - HMP/QMP.virsh blkiothrottle<domain> <device>Given that the XML is named <iotune> under <disk>, we should probably name the virsh command 'blkiotune' or 'disk-iotune', not 'blkiothrottle'.
Hi Eric, we used<iothrottle> first, I changed it since Daniel P. Berrange proposed<iotune> for per-disk element instead of<iothrottle> when we discussed at RFC V1. The command 'blkiotune' already exist, supported the cgroups blkio-controller, which handles proportional shares and throughput/iops limits on host block devices, global to the domain, but blkio throttling is specified per-disk and can vary across multiple disks. They are different two mechanism. So how about use<iothrottle> again? :)
Also, <iotune bps='n'.../> doesn't look right. This should be similar to the top-level XML. Here, taking into account the other proposal for per-block-device values (which can't be tied to individual <disk>):<domain...> <blkiotune> <weight>800</weight> <device> <path>/path/to/device</path> <weight>200</weight> </device> </blkiotune> <devices> <disk ...> <iotune> <bps>800</bps> ... </iotune> </disk> </devices> </domain>
OK. I will do it at v3.
daemon/remote.c | 85 +++++++++++++++++ include/libvirt/libvirt.h.in | 25 +++++Missing changes in docs/formatdomain.html.in (to describe the new XML), docs/schemas/domaincommon.rng (to validate the new xml), and tests/ (probably qemuxml2argvdata), to test it.
-- Lei -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
[Virt Tools] [Libvirt Users] [Fedora Users] [Fedora Legacy] [Fedora Maintainers] [Fedora Desktop] [Fedora SELinux] [Big List of Linux Books] [Yosemite News] [Yosemite Photos] [KDE Users] [Fedora Tools]