[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount()
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount() |
Date: |
Sat, 18 Apr 2009 00:29:06 +0200 |
User-agent: |
KMail/1.9.5 |
Am Samstag, 18. April 2009 00:07 schrieben Sie:
> Every patch I commit gets tested. I have various tests that I run
> depending on which subsystem the patch touches. Right now, for qcow2, I
> don't have nearly enough. I was hoping that Christoph had something
> laying around that I could use since it looks like qemu-io would be a
> great harness for qcow2 changes.
It is, definitely. qemu-io is the other part which I needed. But qemu-io alone
won't make you happy. It can exercise a lot of code, but it can't tell you if
your image is broken in the end. Except for reading back written data, of
course, but breakage is often enough more subtle.
> But I can write a pretty easy script myself on top of qemu-io so it's no
> big deal. I'm not suggesting holding up development.
Heh, so everyone is writing his own scripts. What would you think about
bringing the test cases you use into the repository?
> > This is even more true for changes which are actually made for testing
> > and debugging purposes like these. This series is what helped me to find
> > the corruption bug.
> >
> > What we should do is to make sure that qcow2 patches (especially those
> > touching the core) are given a thorough review before committing.
>
> In theory, r5006 did. That wasn't enough.
I know. It also was tested a fair amount and it wasn't enough. We need both,
but currently we don't have any systematic tests for it.
> >>> And even though I think that this series can't break anything, we
> >>> definitely could use a strong test suite. I'm almost sure that there is
> >>> at least one bug left (the one Jamie Lokier saw from 5006 on, but
> >>> nobody ever found it).
> >>
> >> You don't think that was Nolan's fix?
> >
> > Hm, I haven't look very much in detail at it. But according to the commit
> > log only qcow_is_allocated() was affected, and I can't see how booting
> > Jamie's Windows guest would call this function.
>
> The bug was in get_cluster_offset() so it could have caused much more
> subtle breakages.
Maybe. I wouldn't count on that bug being fixed, but it might be me who is
overly skeptical.
Kevin
- [Qemu-devel] [PATCH 0/5] Add qemu-img check subcommand, Kevin Wolf, 2009/04/17
- [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Kevin Wolf, 2009/04/17
- [Qemu-devel] [PATCH 2/5] Introduce bdrv_check, kwolf, 2009/04/17
- [Qemu-devel] [PATCH 3/5] Introduce qemu-img check subcommand, kwolf, 2009/04/17
- [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), kwolf, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Anthony Liguori, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Kevin Wolf, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Anthony Liguori, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Kevin Wolf, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Anthony Liguori, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(),
Kevin Wolf <=
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Anthony Liguori, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Ryan Harper, 2009/04/17
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Christoph Hellwig, 2009/04/20
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Kevin Wolf, 2009/04/21
- Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Christoph Hellwig, 2009/04/20
Re: [Qemu-devel] [PATCH 1/5] qcow2: Fix warnings in check_refcount(), Anthony Liguori, 2009/04/21
[Qemu-devel] [PATCH 4/5] qcow2: Refcount checking code cleanup, kwolf, 2009/04/17
[Qemu-devel] [PATCH 5/5] qcow2: Add plausibility check for L1/L2 entries, kwolf, 2009/04/17
Re: [Qemu-devel] [PATCH 0/5] Add qemu-img check subcommand, Gleb Natapov, 2009/04/20