From: Avi Kivity <avi@qumranet.com>
To: Alex Williamson <alex.williamson@hp.com>
Cc: kvm-devel <kvm@vger.kernel.org>
Subject: Re: [PATCH] qemu: fix ATAPI read drive structure command
Date: Sun, 25 May 2008 12:11:04 +0300 [thread overview]
Message-ID: <48392D28.3020103@qumranet.com> (raw)
In-Reply-To: <1211511513.21166.135.camel@bling>
Alex Williamson wrote:
> I believe this fixes bug 1967368. Windows Vista hangs if you try to run
> diskpart.exe with a DVD image loaded. This does not occur on Xen
> because their version of Qemu doesn't try to emulate the
> GPCMD_READ_DVD_STRUCTURE command. If I comment out the version in KVM,
> diskpart.exe works. Digging through the versions of the MMC-6 spec I
> can find online, I believe the problem is that we're ignoring the
> allocation length field. This specifies the maximum number of bytes
> that may be returned by the drive. Instead we're returning the maximum
> possible table size each time. I also found that we seem to be using
> the wrong field for the format request. Byte 2 is MSB of the address
> field, we want byte 7. I also added a few missing comments for the
> fields as we fill them in. I've only tested this with Vista since it's
> the only thing that I know using this command. Thanks,
>
This seems to fix a bug in upstream qemu, so the preferred path for this
is to post the patch on qemu-devel, and kvm will get it on the next merge.
--
error compiling committee.c: too many arguments to function
prev parent reply other threads:[~2008-05-25 9:11 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-23 2:58 [PATCH] qemu: fix ATAPI read drive structure command Alex Williamson
2008-05-25 9:11 ` Avi Kivity [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=48392D28.3020103@qumranet.com \
--to=avi@qumranet.com \
--cc=alex.williamson@hp.com \
--cc=kvm@vger.kernel.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.