From: Jamie Lokier <jamie@shareable.org>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel][PATCH] Send unit_attention on cd-rom not_ready to ready transition
Date: Mon, 16 Jun 2008 23:34:23 +0100 [thread overview]
Message-ID: <20080616223423.GB29899@shareable.org> (raw)
In-Reply-To: <18518.40135.80334.281645@mariner.uk.xensource.com>
> > Patch sends a UNIT_ATTENTION(6), MEDIUM_MAY_HAVE_CHANGED(0x28) sense
> > when cdrom transitions from not ready to ready.
> ...
> > Patch applies to SVN revision 4456.
>
> I have checked that this code compiles, and it seems to make sense
> (although I haven't double-checked the SCSI specification to check
> whether this is the right place to send that UNIT ATTENTION message).
Is this expected to fix the following?
Running Windows XP guest, in the QEMU monitor I do "change ide1-cd0
none", then a little while later, "change ide1-cd0 newfile.iso".
Interestingly, Windows XP _does_ show the CD (drive D:) change from
having contents to being not present (after the first change command).
But after the second change command, Windows XP shows it changes to
having contents again - but the _old_ contents of the drive!
Using the "eject" QEMU monitor command doesn't help.
However, using the eject menu option from within Windows XP itself,
then changing the CD image QEMU's monitor - that does work and it
shows the new contents.
I'm thinking perhaps XP caches the disk contents, and even though it
sees the CD transition from present to not-present and back, it does
not flush it's cache for some (bizarre) reason.
Is this patch expected to fix this, so issuing the "change" command in
QEMU's monitor will cause XP to show the new contents? Does that
imply _all_ CD drives issue the error shown in this patch?
Thanks,
-- Jamie
prev parent reply other threads:[~2008-06-16 22:34 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-14 14:01 [Qemu-devel][PATCH] Send unit_attention on cd-rom not_ready to ready transition Pat Campbell
2008-05-20 14:09 ` Ian Jackson
2008-06-16 17:03 ` Ian Jackson
2008-06-16 22:34 ` Jamie Lokier [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=20080616223423.GB29899@shareable.org \
--to=jamie@shareable.org \
--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.