[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure |
Date: |
Tue, 08 Aug 2017 17:18:36 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Alberto Garcia <address@hidden> 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.
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Pradeep Jagadeesh, 2017/08/07
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Markus Armbruster, 2017/08/07
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Alberto Garcia, 2017/08/08
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Pradeep Jagadeesh, 2017/08/08
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Alberto Garcia, 2017/08/08
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Pradeep Jagadeesh, 2017/08/10
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Eric Blake, 2017/08/10
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Markus Armbruster, 2017/08/10
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Markus Armbruster, 2017/08/16
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Eric Blake, 2017/08/16
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Markus Armbruster, 2017/08/16
- Message not available
- Message not available
- Re: [Qemu-devel] [PATCH v7 2/6] qmp: Create IOThrottle structure, Pradeep Kiruvale, 2017/08/18