[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V26 00/32] replace QEMUOptionParameter with Qemu
From: |
Leandro Dorileo |
Subject: |
Re: [Qemu-devel] [PATCH V26 00/32] replace QEMUOptionParameter with QemuOpts |
Date: |
Mon, 19 May 2014 18:11:22 -0300 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Sun, May 18, 2014 at 09:02:45PM -0600, Chun Yan Liu wrote:
>
>
> >>> On 5/6/2014 at 09:26 PM, in message
> <address@hidden>, Stefan Hajnoczi
> <address@hidden> wrote:
> > On Tue, Apr 29, 2014 at 05:10:24PM +0800, Chunyan Liu wrote:
> > > This patch series is to replace QEMUOptionParameter with QemuOpts, so
> > > that
> > only
> > > one Qemu Option structure is kept in QEMU code.
> > >
> > > ---
> > > Changes to v25:
> > > * split v25 2/31 (add def_value_str to QemuOptDesc) into two patches:
> > > 1st patch adds def_value_str, 2nd patch repurpose qemu_opts_print.
> > > * update 4/32 qapi command line description.
> > > * update 12/32 (change block layer to support both) according to
> > > Eric's comments.
> > > * small update to gluster.c
> > > * rebase to latest code
> > >
> > > All patches are also available from:
> > > https://github.com/chunyanliu/qemu/commits/QemuOpts
> >
> > I looked through the comments from Leandro and Eric. A respin is
> > necessary to address the test failures that Leandro found. Please also
> > address the comments Eric made.
> >
> > We're almost there now!
> >
> >
>
> Hi, Stefan, Leandro & Eric,
>
> Could you help to have a look at the new update? I've sent it about 10
> days ago. It addressed all Eric's comments in last version and two
> memory free fix to address iotest issue. Hope it's the final work and
> won't take you more time :-)
>
> The thread has been cut into two, since the git-send-email died in the
> middle at the 1st time sending. Please refer to:
I'll take a look at it, thanks.
--
Leandro Dorileo