qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Kevin Wolf <kwolf@redhat.com>
Cc: qemu-devel <qemu-devel@nongnu.org>
Subject: [Qemu-devel] Strange virtio regression on mainline and stable-0.10
Date: Tue, 05 May 2009 12:52:52 +0300	[thread overview]
Message-ID: <4A000C74.5020907@redhat.com> (raw)

Running the Fedora 10 installer on a virtio disk on current master and 
on v0.10.3 will cause the installer to complain when mounting the 
freshly formatted filesystems.  A bisect (on stable-0.10) yielded

4df8f71ee5276fb22a810bbb67db3a4288f5cad4 is first bad commit
commit 4df8f71ee5276fb22a810bbb67db3a4288f5cad4
Author: aliguori <aliguori@c046a42c-6fe2-441c-8c8c-71466251a162>
Date:   Fri Apr 17 20:44:41 2009 +0000

    qcow2 corruption: Fix alloc_cluster_link_l2 (Kevin Wolf)
   
    This patch fixes a qcow2 corruption bug introduced in SVN Rev 5861. 
L2 tables
    are big endian, so entries must be converted before being passed to 
functions.
   
    This bug is easy to trigger. The following script will create and 
destroy a
    qcow2 image (the header is gone after three loop iterations):
   
        #!/bin/bash
        qemu-img create -f qcow2 test.qcow 1M
        for i in $(seq 1 10); do
        qemu-system-x86_64 -hda test.qcow -monitor stdio > /dev/null 
2>&1 <<EOF
        savevm test-$i
        quit
        EOF
        done
   
    Signed-off-by: Kevin Wolf <kwolf@redhat.com>
    Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

As the guilty commit.

I don't understand it, as free_any_clusters() shouldn't be called while 
formatting.  Any ideas?

-- 
error compiling committee.c: too many arguments to function

             reply	other threads:[~2009-05-05  9:53 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-05  9:52 Avi Kivity [this message]
2009-05-05 10:03 ` [Qemu-devel] Re: Strange virtio regression on mainline and stable-0.10 Avi Kivity
     [not found] ` <4A001563.1020604@redhat.com>
2009-05-05 11:46   ` Avi Kivity
2009-05-05 16:18 ` [Qemu-devel] " Avi Kivity
2009-05-05 18:18   ` Avi Kivity
2009-05-06  8:14   ` Kevin Wolf
2009-05-06  8:34     ` Avi Kivity

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=4A000C74.5020907@redhat.com \
    --to=avi@redhat.com \
    --cc=kwolf@redhat.com \
    --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 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).