[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: |
Thu, 10 Aug 2017 18:25:21 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 08/10/2017 09:06 AM, Pradeep Jagadeesh wrote:
>
>>>> 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?
>
> No.
>
> If more than one .json file would benefit by including the definition,
> then put it in a separate file that both .json include from.
This is the case.
Your opinion is incompatible with mine, stated above.
> But if only one .json file would be including a new file, then just
> inline the struct directly into that one original file (in this case,
> block-core.json) instead of creating a separate file (so no to needing
> iothrottle.json), or putting the code in yet a different file than the
> one that is using the struct (so no to putting it in common.json).
This is no longer the case.
Conclusion: no consensus, yet.
- 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, 2017/08/08
- 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 <=
- 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