[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 00/12] cleanup qemu-iotests
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-block] [PATCH 00/12] cleanup qemu-iotests |
Date: |
Mon, 11 Sep 2017 18:31:55 -0400 (EDT) |
----- Original Message -----
> From: "John Snow" <address@hidden>
> To: "Paolo Bonzini" <address@hidden>, address@hidden
> Cc: address@hidden, address@hidden
> Sent: Tuesday, September 12, 2017 12:22:26 AM
> Subject: Re: [Qemu-block] [PATCH 00/12] cleanup qemu-iotests
>
>
>
> On 08/09/2017 05:54 PM, Paolo Bonzini wrote:
> > The purpose of this series is to separate the "check" sources from
> > the tests. After these patches, common.config is reduced to simple
> > shell initialization, and common.rc is only included by the tests.
> >
> > Along the way, a lot of dead code is removed too.
> >
> > Paolo
> >
> > Paolo Bonzini (12):
> > qemu-iotests: remove dead code
> > qemu-iotests: get rid of AWK_PROG
> > qemu-iotests: move "check" code out of common.rc
> > qemu-iotests: limit non-_PROG-suffixed variables to common.rc
> > qemu-iotests: do not include common.rc in "check"
> > qemu-iotests: do not do useless search for QEMU_*_PROG
> > qemu-iotests: disintegrate more parts of common.config
> > qemu-iotests: fix uninitialized variable
> > qemu-iotests: do not search for binaries in the current directory
> > qemu-iotests: get rid of $iam
> > qemu-iotests: include common.env and common.config early
> > qemu-iotests: merge "check" and "common"
> >
> > tests/qemu-iotests/039.out | 10 +-
> > tests/qemu-iotests/061.out | 4 +-
> > tests/qemu-iotests/137.out | 2 +-
> > tests/qemu-iotests/check | 551
> > ++++++++++++++++++++++++++++++++++-----
> > tests/qemu-iotests/common | 459 --------------------------------
> > tests/qemu-iotests/common.config | 205 +--------------
> > tests/qemu-iotests/common.qemu | 1 +
> > tests/qemu-iotests/common.rc | 225 ++++++++--------
> > 8 files changed, 615 insertions(+), 842 deletions(-)
> > delete mode 100644 tests/qemu-iotests/common
> >
>
> Stale? Patchset is >1 month old now, doesn't look like it got merged.
Yup, didn't get reviewed by Kevin either... I was waiting for that
before sending v2.
Paolo