From: Paolo Bonzini <pbonzini@redhat.com>
To: Pavel Hrdina <phrdina@redhat.com>
Cc: kwolf@redhat.com, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2] scsi: reset cdrom tray statuses on scsi_disk_reset
Date: Mon, 20 May 2013 13:21:23 +0200 [thread overview]
Message-ID: <519A0733.3070703@redhat.com> (raw)
In-Reply-To: <c7f0c18a40b18d59ac6ecc0b6c256755c900c24b.1369047799.git.phrdina@redhat.com>
Il 20/05/2013 13:06, Pavel Hrdina ha scritto:
> Tray statuses should be also reset. Some guests may lock the tray and
> right after resetting the guest it should be unlocked and closed. This
> is done on power-on, reset and resume from suspend/hibernate on bare-metal.
>
> This fix is already committed for IDE CD.
> Check the commit a7f3d65b65b8c86a5ff0c0abcfefb45e2ec6fe4c.
>
> Test results on bare-metal:
> - on reset/power-on the CD-ROM tray is closed even before the monitor
> is turned on
> - on resume from suspend/hibernate the tray is also closed before
> the monitor is turned on
>
> From test results it seems that this behavior is OS and probably BIOS
> independent.
>
> Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
Applied to scsi-next branch, thanks.
Paolo
> ---
> hw/scsi/scsi-disk.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
> index c8d2a99..02733dc 100644
> --- a/hw/scsi/scsi-disk.c
> +++ b/hw/scsi/scsi-disk.c
> @@ -1984,6 +1984,9 @@ static void scsi_disk_reset(DeviceState *dev)
> nb_sectors--;
> }
> s->qdev.max_lba = nb_sectors;
> + /* reset tray statuses */
> + s->tray_locked = 0;
> + s->tray_open = 0;
> }
>
> static void scsi_destroy(SCSIDevice *dev)
>
prev parent reply other threads:[~2013-05-20 11:23 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-20 11:06 [Qemu-devel] [PATCH v2] scsi: reset cdrom tray statuses on scsi_disk_reset Pavel Hrdina
2013-05-20 11:21 ` Paolo Bonzini [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=519A0733.3070703@redhat.com \
--to=pbonzini@redhat.com \
--cc=kwolf@redhat.com \
--cc=phrdina@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.