On 8/8/2017 5:18 PM, Markus Armbruster wrote:
Alberto Garcia <be...@igalia.com> writes:
On Tue 08 Aug 2017 02:30:43 PM CEST, Pradeep Jagadeesh wrote:
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?
It's not "moving it back", it's keeping it where it is. But I see no big
problem with moving it to a common file either.
I'd rather not put every struct shared across subsystem boundaries in
its own file.
We can keep it right where it is for now. Bonus: more readable diff.
If we start sharing more throttle-related material than just a struct,
we can reconsider.
We could also move it to the existing file for common stuff:
qapi/common.json. Not a great fit, though.
So, the final conclusion is to move to common.json?
Regards,
Pradeep
