[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 35/40] job: Add JOB_STATUS_CHANGE QMP event
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH v2 35/40] job: Add JOB_STATUS_CHANGE QMP event |
Date: |
Wed, 23 May 2018 20:02:50 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 05/18/2018 09:21 AM, Kevin Wolf wrote:
> This adds a QMP event that is emitted whenever a job transitions from
> one status to another.
>
> Signed-off-by: Kevin Wolf <address@hidden>
That's a lot of events, and a lot are redundant to what we already
transmitted under block jobs; it also has the effect of making internal
state changes explicit behavior that we're responsible for maintaining
for external clients.
Is that what we want here?
(I mean, the answer is probably "Yes" because you're here writing the
patch, but I was hoping to find your motivation.)
- [Qemu-devel] [PATCH v2 37/40] job: Add query-jobs QMP command, (continued)
- [Qemu-devel] [PATCH v2 37/40] job: Add query-jobs QMP command, Kevin Wolf, 2018/05/18
- [Qemu-devel] [PATCH v2 33/40] job: Move progress fields to Job, Kevin Wolf, 2018/05/18
- [Qemu-devel] [PATCH v2 39/40] iotests: Move qmp_to_opts() to VM, Kevin Wolf, 2018/05/18
- [Qemu-devel] [PATCH v2 38/40] blockjob: Remove BlockJob.driver, Kevin Wolf, 2018/05/18
- [Qemu-devel] [PATCH v2 40/40] qemu-iotests: Test job-* with block jobs, Kevin Wolf, 2018/05/18
- [Qemu-devel] [PATCH v2 35/40] job: Add JOB_STATUS_CHANGE QMP event, Kevin Wolf, 2018/05/18
Re: [Qemu-devel] [PATCH v2 00/40] Generic background jobs, no-reply, 2018/05/18
Re: [Qemu-devel] [PATCH v2 00/40] Generic background jobs, Dr. David Alan Gilbert, 2018/05/18
Re: [Qemu-devel] [PATCH v2 00/40] Generic background jobs, Kevin Wolf, 2018/05/23