[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/4] qapi: Create module 'control'
|
From: |
Markus Armbruster |
|
Subject: |
Re: [PATCH v3 2/4] qapi: Create module 'control' |
|
Date: |
Thu, 30 Jan 2020 08:31:50 +0100 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Kevin Wolf <address@hidden> writes:
> Am 29.01.2020 um 17:35 hat Markus Armbruster geschrieben:
>> Kevin Wolf <address@hidden> writes:
>>
>> > misc.json contains definitions that are related to the system emulator,
>> > so it can't be used for other tools like the storage daemon. This patch
>> > moves basic functionality that is shared between all tools (and mostly
>> > related to the monitor itself) into a new control.json, which could be
>> > used in tools as well.
>> >
>> > Signed-off-by: Kevin Wolf <address@hidden>
>> > Reviewed-by: Markus Armbruster <address@hidden>
>>
>> I'd like to retitle this patch and the next one:
>>
>> qapi: Split control.json off misc.json
>> monitor: Collect "control" command handlers in qmp-cmds.control.c
>>
>> Okay?
>
> Sure, why not.
Queued, thanks!