qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>, Gonglei <arei.gonglei@huawei.com>,
	qemu-devel@nongnu.org, Stefan Hajnoczi <stefanha@redhat.com>
Subject: Re: [Qemu-devel] win7 boot failure with commit f68ec83 - ide: Set BSY bit during FLUSH
Date: Tue, 09 Jul 2013 22:29:33 +0200	[thread overview]
Message-ID: <51DC72AD.2090209@suse.de> (raw)
In-Reply-To: <20130709202700.GA30219@redhat.com>

Hi,

Am 09.07.2013 22:27, schrieb Michael S. Tsirkin:
> I am using this command line:
> qemu-system-x86_64 -m 512 -enable-kvm -drive
> id=disk,file=./images/win7-x86.qcow2,if=none -device ahci,id=ahci
> -device ide-drive,drive=disk,bus=ahci.0 -cdrom
> ./images/en_windows_7_ultimate_x86_dvd_x15-65921.iso  -monitor stdio
> -cpu kvm64 -snapshot
> 
> and windows 7 does not boot.

The same was recently reported for q35 - Kevin found the cause but I
haven't noticed a proposed solution yet.

Andreas

> This happens starting from this commit:
> 
> commit f68ec8379e88502b4841a110c070e9b118d3151c
> Author: Andreas Färber <afaerber@suse.de>
> Date:   Wed Jun 5 15:17:57 2013 +0200
> 
>     ide: Set BSY bit during FLUSH
>     
>     The implementation of the ATA FLUSH command invokes a flush at the block
>     layer, which may on raw files on POSIX entail a synchronous fdatasync().
>     This may in some cases take so long that the SLES 11 SP1 guest driver
>     reports I/O errors and filesystems get corrupted or remounted read-only.
>     
>     Avoid this by setting BUSY_STAT, so that the guest is made aware we are
>     in the middle of an operation and no ATA commands are attempted to be
>     processed concurrently.
>     
>     Addresses BNC#637297.
>     
>     Suggested-by: Gonglei (Arei) <arei.gonglei@huawei.com>
>     Signed-off-by: Andreas Färber <afaerber@suse.de>
>     Signed-off-by: Kevin Wolf <kwolf@redhat.com>
>     Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> 
> 
> guest starts booting but never completes boot - it just keeps flashing
> the flag image.  Boots fine with previous commit:
> commit c27d565604038c1572b16dd1cd06e277e6ef02e2
> 
> However, reverting f68ec8379e88502b4841a110c070e9b118d3151c on
> current master still has the hang on boot
> behaviour - possibly a separate issue, I'll have to redo the bisect
> to find out.
> 


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

      reply	other threads:[~2013-07-09 20:29 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-09 20:27 [Qemu-devel] win7 boot failure with commit f68ec83 - ide: Set BSY bit during FLUSH Michael S. Tsirkin
2013-07-09 20:29 ` Andreas Färber [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=51DC72AD.2090209@suse.de \
    --to=afaerber@suse.de \
    --cc=arei.gonglei@huawei.com \
    --cc=kwolf@redhat.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).