Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure

[Pradeep Jagadeesh]

On 8/8/2017 1:30 PM, Alberto Garcia wrote:
> On Mon 07 Aug 2017 04:48:38 PM CEST, Markus Armbruster wrote:
> > > > Awkward question for a v7, but here goes anyway: why is IOThrottle
> > > > worth its very own .json file?
> > > I feel this is a common throttle structure that is used by block
> > > devices as well as fsdev, so moved to a separate file.
> > I'm not sure that's a good idea.  Kevin, Berto, what do you think?
>
> Mmm... I don't have a very strong opinion, but if there's no actual need
> to move it to a separate file I'd prefer to leave it where it is.
The segregation is the solid reason. Because throttling is a feature 
that is used by fsdev, block may many more in future. I do not see 
moving it back to block does it make any sense?

Regards,
Pradeep

> Berto