[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 34/39] block: Inquire tray state before tray-
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v7 34/39] block: Inquire tray state before tray-moved events |
Date: |
Fri, 23 Oct 2015 16:16:09 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 19.10.2015 um 17:53 hat Max Reitz geschrieben:
> blk_dev_change_media_cb() is called for all potential tray movements;
> however, it is possible to request closing the tray but nothing actually
> happening (on a floppy disk drive without a medium).
>
> Thus, the actual tray status should be inquired before sending a
> tray-moved event (and an event should be sent whenever the status
> changed).
>
> Checking @load is now superfluous; it was necessary because it was
> possible to change a medium without having explicitly opened the tray
> and closed it again (or it might have been possible, at least). This is
> no longer possible, though.
>
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Kevin Wolf <address@hidden>
- Re: [Qemu-devel] [PATCH v7 31/39] blockdev: Add blockdev-insert-medium, (continued)
- [Qemu-devel] [PATCH v7 29/39] blockdev: Add blockdev-close-tray, Max Reitz, 2015/10/19
- [Qemu-devel] [PATCH v7 32/39] blockdev: Implement eject with basic operations, Max Reitz, 2015/10/19
- [Qemu-devel] [PATCH v7 33/39] blockdev: Implement change with basic operations, Max Reitz, 2015/10/19
- [Qemu-devel] [PATCH v7 34/39] block: Inquire tray state before tray-moved events, Max Reitz, 2015/10/19
- Re: [Qemu-devel] [PATCH v7 34/39] block: Inquire tray state before tray-moved events,
Kevin Wolf <=
- [Qemu-devel] [PATCH v7 35/39] qmp: Introduce blockdev-change-medium, Max Reitz, 2015/10/19
- [Qemu-devel] [PATCH v7 37/39] blockdev: read-only-mode for blockdev-change-medium, Max Reitz, 2015/10/19
- [Qemu-devel] [PATCH v7 36/39] hmp: Use blockdev-change-medium for change command, Max Reitz, 2015/10/19
- [Qemu-devel] [PATCH v7 38/39] hmp: Add read-only-mode option to change command, Max Reitz, 2015/10/19