All of lore.kernel.org
 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 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.