All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: "Richard W.M. Jones" <rjones@redhat.com>
Cc: "seabios@seabios.org" <seabios@seabios.org>,
	qemu-devel@nongnu.org, Max Filippov <jcmvbkbc@gmail.com>,
	Avi Kivity <avi@redhat.com>, Amit Shah <amit.shah@redhat.com>,
	Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [Qemu-devel] insmod virtio-blk is broken in qemu 1.0
Date: Sat, 17 Dec 2011 10:24:07 -0600	[thread overview]
Message-ID: <4EECC227.4060904@codemonkey.ws> (raw)
In-Reply-To: <20111217152514.GM2520@amd.home.annexia.org>

On 12/17/2011 09:25 AM, Richard W.M. Jones wrote:
> On Sat, Dec 17, 2011 at 09:22:45AM -0600, Anthony Liguori wrote:
>> I've even further narrowed it down to the presents or lack of '-vga
>> cirrus'.  If you add '-vga cirrus' to the above command line, the
>> guest will boot successfully.
>
> Confirmed: Adding -vga cirrus to the command line cures it too.
>
> That's a strange one :-)

vga sticks out a bit because it's one of the few places where we treat device 
memory as ram as a performance optimization.

The only time vga has been touched in between v0.15 and v1.0 was during the 
introduction of the memory API.

It's this commit:

commit d67c3f2cd92aed2247bfa8a9da61a902b7b2ff09
Author: Gerd Hoffmann <kraxel@redhat.com>
Date:   Wed Aug 10 17:34:13 2011 +0200

     seabios: update to master

     commit 8e301472e324b6d6496d8b4ffc66863e99d7a505

     user visible changes in seabios:
       * ahci is enabled by default (and thus in this build).
       * bootorder support for ahci.
       * two-pass pci allocator (orders bars by size for better packing).

     Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

:040000 040000 76eb0c81b76563b55cb2bb5c484ccd48b8cfcded 
5ec0d65d3a763a5566fe1f4c86269cad6d671020 M	pc-bios
:040000 040000 a5a7ea6e297c1e7490b0a2c28a06ce56e5be9449 
78adb664d3ea82f1a4dd5ec239887ac5b0168a7f M	roms

It can be reproduced by using virtio and -vga none with a number of PCI devices. 
  The line below is what I used to bisect and reproduce 100% of the time.  It's 
a 64-bit Fedora 15 guest.

$ qemu-system-x86_64 -drive 
file=/home/anthony/images/fedora.img,if=none,snapshot=on,id=hd0 -device 
virtio-balloon-pci,addr=03.0 -device virtio-blk-pci,addr=04.0,drive=hd0 -kernel 
~/vmlinuz-2.6.38.6-26.rc1.fc15.x86_64 -initrd 
~/initramfs-2.6.38.6-26.rc1.fc15.x86_64.img -append 
"root=/dev/mapper/VolGroup-lv_root rd_LVM_LV=VolGroup/lv_root 
rd_LVM_LV=VolGroup/lv_swap ro console=ttyS0 selinux=0" -nographic -nodefconfig 
-m 1G -no-reboot -no-hpet -device virtio-serial -chardev 
socket,path=/tmp/foo.sock,id=channel0,server,nowait -device 
virtserialport,chardev=channel0,name=org.libguestfs.channel.0 -nodefaults 
-serial stdio -enable-kvm

My guess it that it has something to do with the changes to the PCI allocator. 
I've confirmed reverting this commit fixes the problem.

Regards,

Anthony Liguori

>
> Rich.
>

  reply	other threads:[~2011-12-17 16:24 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-30 13:11 [Qemu-devel] qemu.git hangs booting Linux after insmod virtio_blk.ko Richard W.M. Jones
2011-09-30 16:51 ` [Qemu-devel] git-bisect results (was: Re: qemu.git hangs booting Linux after insmod virtio_blk.ko) Richard W.M. Jones
2011-12-16 22:19   ` Richard W.M. Jones
2011-12-17  0:07     ` [Qemu-devel] insmod virtio-blk is broken in qemu 1.0 (was: Re: git-bisect results (was: Re: qemu.git hangs booting Linux after insmod virtio_blk.ko)) Richard W.M. Jones
2011-12-17  0:26       ` Max Filippov
2011-12-17  0:53         ` Max Filippov
2011-12-17  1:44           ` [Qemu-devel] insmod virtio-blk is broken in qemu 1.0 (was: Re: git-bisect results Anthony Liguori
2011-12-17  8:33             ` Richard W.M. Jones
2011-12-17 15:13               ` [Qemu-devel] insmod virtio-blk is broken in qemu 1.0 Anthony Liguori
2011-12-17 15:22                 ` Richard W.M. Jones
2011-12-17 15:22                 ` Anthony Liguori
2011-12-17 15:25                   ` Richard W.M. Jones
2011-12-17 16:24                     ` Anthony Liguori [this message]
2011-12-17 16:35                       ` Richard W.M. Jones
2011-12-17 16:49                       ` Kevin O'Connor
2011-12-17 17:03                         ` Anthony Liguori
2011-12-17 17:30                         ` Richard W.M. Jones
2011-12-19 10:31                         ` Daniel P. Berrange
2011-12-19 17:23                           ` [Qemu-devel] [SeaBIOS] " Fred .
2011-12-19 17:26                           ` Fred .
2011-12-19 17:34                           ` [Qemu-devel] " Anthony Liguori
2011-12-19 17:43                             ` Daniel P. Berrange
2011-12-19 18:02                               ` Anthony Liguori
2011-12-19 19:04                                 ` Richard W.M. Jones
2011-12-19 19:16                                   ` Daniel P. Berrange
2011-12-19 19:40                                     ` Richard W.M. Jones
2011-12-19 19:42                                       ` Richard W.M. Jones
2011-12-19 19:44                                       ` Anthony Liguori
2011-12-19 19:19                                 ` Daniel P. Berrange
2011-12-19 19:26                                   ` Anthony Liguori
2011-12-20  3:38                                 ` Kevin O'Connor
2011-12-20  8:13                                   ` Gleb Natapov
2011-12-17 13:16             ` [Qemu-devel] insmod virtio-blk is broken in qemu 1.0 (was: Re: git-bisect results Richard W.M. Jones
2011-12-17  1:39       ` Anthony Liguori
2011-12-17  1:43       ` Anthony Liguori

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=4EECC227.4060904@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=amit.shah@redhat.com \
    --cc=avi@redhat.com \
    --cc=jcmvbkbc@gmail.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rjones@redhat.com \
    --cc=seabios@seabios.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.