Re: [PATCH v9 reviewed 00/17] qemu: block: Support block disk along with throttle filters

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

 



On a Tuesday in 2025, Peter Krempa via Devel wrote:
v9 of the throttle filtering series with my reviews and R-b tags
applied.

Requires

[PATCH 0/5] qemu: Two block job fixes
https://lists.libvirt.org/archives/list/devel@xxxxxxxxxxxxxxxxx/message/TX2YAFHOFO46GR7WVQSPURWHL6O2KCDZ/

to be applied to work properly.

Posting for tracking and possibly final review of API. I'll push it
before the freeze if there won't be further comments.

Changes (most recorded in commit messages):
- removed leftover code after deletion of some of the APIs
- fixed numerous memleaks
- simplified virsh code
- dropped test driver pach

Chun Feng Wu (16):
 schema: Add new domain elements to support multiple throttle groups
 schema: Add new domain elements to support multiple throttle filters
 config: Introduce ThrottleGroup and corresponding XML parsing
 config: Introduce ThrottleFilter and corresponding XML parsing
 qemu: monitor: Add support for ThrottleGroup operations
 tests: Test qemuMonitorJSONGetThrottleGroup and
   qemuMonitorJSONUpdateThrottleGroup
 remote: New APIs for ThrottleGroup lifecycle management
 qemu: Refactor qemuDomainSetBlockIoTune to extract common methods
 qemu: Implement qemu driver for throttle API
 qemu: helper: throttle filter nodename and preparation processing
 qemu: block: Support block disk along with throttle filters
 config: validate: Verify iotune, throttle group and filter
 qemuxmlconftest: Add 'throttlefilter' tests
 virsh: Refactor iotune options for re-use
 virsh: Add support for throttle group operations
 virsh: Add option "throttle-groups" to "attach_disk"

Harikumar Rajkumar (1):
 qemustatusxml2xmldata: Add 'throttlefilter' tests

Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

Jano

Attachment: signature.asc
Description: PGP signature


[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