[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 04/10] block: Accept device model name for block
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH 04/10] block: Accept device model name for blockdev-open/close-tray |
Date: |
Mon, 19 Sep 2016 17:10:02 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 15.09.2016 um 17:51 hat Eric Blake geschrieben:
> On 09/15/2016 03:35 AM, Kevin Wolf wrote:
>
> >>>
> >>> -- "device": block device name (json-string)
> >>> +- "device": block device name (deprecated, use @id instead)
> >>> + (json-string, optional)
> >>> +- "id": the name or QOM path of the guest device (json-string, optional)
> >>> - "force": if false (the default), an eject request will be sent to the
> >>> guest if
> >>> it has locked the tray (and the tray will not be opened
> >>> immediately);
> >>> if true, the tray will be opened regardless of whether it is
> >>> locked
> >>
> >> Are there any example code snippets that should be updated alongside
> >> this? If not, should we be thinking of adding an example?
> >
> > We could probably update the examples to avoid deprecated fields in
> > them. Though the old examples still work, so is it worth changing all
> > examples if we're goig to remove qmp-commands.hx anyway? Or will the
> > examples be moved to somewhere else?
>
> Marc-Andre is moving the examples into qapi-schema.json (and friends),
> over the course of multiple commits. It's going to be a lot of churn
> and potential merge conflicts, based on what changes go into
> qmp-commands.hx after his work starts, but hopefully we can avoid things
> slipping through the cracks, without too much rebase pain on either
> Marc-Andre or other developers.
Ok, I'm changing the examples from 'device' to 'id' now. And actually
the first one immediately made me notice that the DEVICE_TRAY_MOVED
event needs to be changed, too (because its 'device' field can be empty
now). Matter for another series, but I put it on my list.
Kevin
pgp2XgidctCxs.pgp
Description: PGP signature