On Fri 18 Aug 2017 11:07:22 AM CEST, Manos Pitsidianakis wrote: >>> The driver can be used with the syntax >>> -drive driver=throttle,file.filename=foo.qcow2, \ >>> limits.iops-total=...,throttle-group=bar >> >>I had understood that we would get rid of the limits.* options in this >>driver, or did I get it wrong? >> > I was going to send a patch after this was merged along with adding > ThrottleGroups to the root container, to speed things up. Do you > prefer to do this in this patch?
I'm not sure what's the benefit of adding a complete infrastructure that you are going to remove immediately afterwards :-? > The root container patch probably has to go to the 'remove legacy' > series since adding it here means the name collision errors introduce > error paths in block/block-backend.c that go away in that series, and > that'd be a waste of effort. Ok. Berto