[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RESEND 38/50] blockdev: Add blockdev-open-tray
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH RESEND 38/50] blockdev: Add blockdev-open-tray |
Date: |
Wed, 28 Jan 2015 12:56:28 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
On 01/27/2015 12:46 PM, Max Reitz wrote:
> Signed-off-by: Max Reitz <address@hidden>
> ---
> blockdev.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
> qapi/block-core.json | 21 +++++++++++++++++++++
> qmp-commands.hx | 37 +++++++++++++++++++++++++++++++++++++
> 3 files changed, 106 insertions(+)
>
> +Example (1):
> +
With only one example, the (1) doesn't add much. Minor whether you
leave it in or take it out.
> +-> { "execute": "blockdev-open-tray",
> + "arguments": { "device": "ide1-cd0" } }
> +
> +<- { "timestamp": { "seconds": 1418751016,
> + "microseconds": 716996 },
> + "event": "DEVICE_TRAY_MOVED",
> + "data": { "device": "ide1-cd0",
> + "tray-open": true } }
> +
> +<- { "return": {} }
Nice inclusion of the event alongside the return code!
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH RESEND 31/50] block: Add blk_insert_bs(), (continued)
- [Qemu-devel] [PATCH RESEND 31/50] block: Add blk_insert_bs(), Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 32/50] blockdev: Check BB validity in eject and change, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 34/50] blockdev: Do not create BDS for empty drive, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 36/50] blockdev: Allow more options for BB-less BDS tree, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 38/50] blockdev: Add blockdev-open-tray, Max Reitz, 2015/01/27
- Re: [Qemu-devel] [PATCH RESEND 38/50] blockdev: Add blockdev-open-tray,
Eric Blake <=
- [Qemu-devel] [PATCH RESEND 33/50] blockdev: Respect NULL BDS in do_drive_del(), Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 39/50] blockdev: Add blockdev-close-tray, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 35/50] blockdev: Pull out blockdev option extraction, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 41/50] blockdev: Add blockdev-insert-medium, Max Reitz, 2015/01/27