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 Wed, Mar 19, 2025 at 03:00:47 +0100, Ján Tomko wrote:
> 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>

Thanks; pushed now.

Harikumar; please give it a test on your side as well. You can also
provide NEWS update.




[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