Re: [Qemu-devel] [PATCH v2 3/3] qapi: Split host-qmp into quit and system-reset

[Dominik Csapak]

On 12/3/18 10:38 PM, Eric Blake wrote:
> On 12/3/18 3:26 AM, Dominik Csapak wrote:
> > It is interesting to know whether the shutdown cause was 'quit' or
> > 'reset', especially when using "--no-reboot". In that case, a management
> > layer can now determine if the guest wanted a reboot or shutdown, and
> > can act accordingly.
> >
> > Changes the ouput of the reason in the iotests from 'host-qmp' to
>
> s/ouput/output/
>
> > 'host-qmp-quit'.
>
> May be worth adding that this change is not a compatibility break, 
> because the field itself was also added in 4.0.

will do in v3. thanks for the review @all

> > Signed-off-by: Dominik Csapak <address@hidden>
> > ---
> >   qapi/run-state.json        | 10 ++++++----
>
> Otherwise, series is looking good to me.