All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Hrdina <phrdina@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: kwolf@redhat.com, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] scsi: reset cdrom tray statuses on scsi_disk_reset
Date: Fri, 17 May 2013 13:57:51 +0200	[thread overview]
Message-ID: <51961B3F.3090503@redhat.com> (raw)
In-Reply-To: <5195053B.1070507@redhat.com>

On 16.5.2013 18:11, Paolo Bonzini wrote:
> Il 16/05/2013 17:48, Pavel Hrdina ha scritto:
>>
>>  From what I just tested it is the OS-independent HW behavior.
>>
>> Tested on desktop:
>> 1. Turn off the computer.
>> 2. Turn on the computer and after few second open the CD-ROM tray before
>> any OS system is loaded and holding the power button turn the computer
>> off again. The CD-ROM tray should stay opened.
>> 3. Turn on the computer and the CD-ROM tray is automatically closed.
>
> When?  Compared to the BIOS splash screen, for example.

Immediately after the computer is turned on. On HP Z400 workstation it 
means before the monitor is turned on.

>
>> Next time I'll mention this test in the commit message.
>
> It could mean it is done in the firmware.
>
> What happens when you suspend the QEMU machine to RAM?  I believe the
> tray should remain open or, if closed, should remain locked.  Can you
> try the same test on bare-metal (hint: boot to emergency mode to bypass
> anything that udev can do with locking the tray, then use sg_prevent to
> toggle the state).
>
> Paolo
>

I've booted to init 1, then ejected the CD-ROM tray and run the 
pm-suspend. The computer has suspended successfully. And again 
immediately after the computer is turned on before the screen is turned 
on the CD-ROM tray is closed.

I don't have SCSI CD-ROM device to use the sg_prevent.

Pavel

  reply	other threads:[~2013-05-17 11:57 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-16 15:34 [Qemu-devel] [PATCH] scsi: reset cdrom tray statuses on scsi_disk_reset Pavel Hrdina
2013-05-16 15:42 ` Eric Blake
2013-05-16 15:48   ` Pavel Hrdina
2013-05-16 16:11     ` Paolo Bonzini
2013-05-17 11:57       ` Pavel Hrdina [this message]
2013-05-17 12:02         ` Paolo Bonzini
2013-05-17 12:11           ` Pavel Hrdina
2013-05-17 12:15             ` Paolo Bonzini
2013-05-17 12:34               ` Pavel Hrdina

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=51961B3F.3090503@redhat.com \
    --to=phrdina@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=pbonzini@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.