Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-04-03 Thread Chunyan Liu
2014-03-25 15:15 GMT+08:00 Chunyan Liu : > > > > 2014-03-24 23:00 GMT+08:00 Leandro Dorileo : > > Hi Chunyan, >> >> On Mon, Mar 24, 2014 at 11:02:14AM +0800, Chunyan Liu wrote: >> > 2014-03-21 20:31 GMT+08:00 Leandro Dorileo : >> > >> > > On Fri, Mar 21, 2014 at 06:09:22PM +0800, Chunyan Liu wrote

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-25 Thread Chunyan Liu
2014-03-24 23:00 GMT+08:00 Leandro Dorileo : > Hi Chunyan, > > On Mon, Mar 24, 2014 at 11:02:14AM +0800, Chunyan Liu wrote: > > 2014-03-21 20:31 GMT+08:00 Leandro Dorileo : > > > > > On Fri, Mar 21, 2014 at 06:09:22PM +0800, Chunyan Liu wrote: > > > > 2014-03-21 8:07 GMT+08:00 Leandro Dorileo : >

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-24 Thread Leandro Dorileo
Hi Chunyan, On Mon, Mar 24, 2014 at 11:02:14AM +0800, Chunyan Liu wrote: > 2014-03-21 20:31 GMT+08:00 Leandro Dorileo : > > > On Fri, Mar 21, 2014 at 06:09:22PM +0800, Chunyan Liu wrote: > > > 2014-03-21 8:07 GMT+08:00 Leandro Dorileo : > > > > > > > Hi Chunyan, > > > > > > > > On Mon, Mar 10, 20

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-23 Thread Chunyan Liu
2014-03-21 20:31 GMT+08:00 Leandro Dorileo : > On Fri, Mar 21, 2014 at 06:09:22PM +0800, Chunyan Liu wrote: > > 2014-03-21 8:07 GMT+08:00 Leandro Dorileo : > > > > > Hi Chunyan, > > > > > > On Mon, Mar 10, 2014 at 03:31:36PM +0800, Chunyan Liu wrote: > > > > This patch series is to replace QEMUOpt

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-21 Thread Leandro Dorileo
On Fri, Mar 21, 2014 at 06:09:22PM +0800, Chunyan Liu wrote: > 2014-03-21 8:07 GMT+08:00 Leandro Dorileo : > > > Hi Chunyan, > > > > On Mon, Mar 10, 2014 at 03:31:36PM +0800, Chunyan Liu wrote: > > > This patch series is to replace QEMUOptionParameter with QemuOpts, so > > that only > > > one Qemu

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-21 Thread Leandro Dorileo
Hi Kevin, On Fri, Mar 21, 2014 at 11:34:53AM +0100, Kevin Wolf wrote: > Am 21.03.2014 um 01:07 hat Leandro Dorileo geschrieben: > > Hi Chunyan, > > > > On Mon, Mar 10, 2014 at 03:31:36PM +0800, Chunyan Liu wrote: > > > This patch series is to replace QEMUOptionParameter with QemuOpts, so > > > t

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-21 Thread Kevin Wolf
Am 21.03.2014 um 01:07 hat Leandro Dorileo geschrieben: > Hi Chunyan, > > On Mon, Mar 10, 2014 at 03:31:36PM +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. > > > > > Last nigh

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-21 Thread Chunyan Liu
2014-03-21 8:07 GMT+08:00 Leandro Dorileo : > Hi Chunyan, > > On Mon, Mar 10, 2014 at 03:31:36PM +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. > > > > > Last night I took some time

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-20 Thread Leandro Dorileo
Hi Chunyan, On Mon, Mar 10, 2014 at 03:31:36PM +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. > Last night I took some time do take a deeper look at you series and the required effo

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-11 Thread Stefan Hajnoczi
On Mon, Mar 10, 2014 at 03:31:36PM +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. Hi Chunyan, Please add my Reviewed-by: tags to the commits so future revisions will already have them.

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chunyan Liu
2014-03-11 4:22 GMT+08:00 Stefan Hajnoczi : > On Mon, Mar 10, 2014 at 03:31:36PM +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 v21: > > * update veriso

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Eric Blake
On 03/10/2014 01:31 AM, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, so that > only > one Qemu Option structure is kept in QEMU code. Uggh. The more I learn about QemuOpts while reviewing this patch, the more I KNOW it has lurking bugs waiting to bite u

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Stefan Hajnoczi
On Mon, Mar 10, 2014 at 03:31:36PM +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 v21: > * update verison info in patch 2/25 > * others are not changed except

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chun Yan Liu
The whole patch series could also be get from here: https://github.com/chunyanliu/qemu/commits/QemuOpts -Chunyan >>> On 3/10/2014 at 03:31 PM, in message <1394436721-21812-1-git-send-email-cy...@suse.com>, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, s

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chun Yan Liu
The whole patch series could also be get from here: https://github.com/chunyanliu/qemu/commits/QemuOpts -Chunyan >>> On 3/10/2014 at 03:31 PM, in message <1394436721-21812-1-git-send-email-cy...@suse.com>, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, s

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chun Yan Liu
The whole patch series could also be get from here: https://github.com/chunyanliu/qemu/commits/QemuOpts -Chunyan >>> On 3/10/2014 at 03:31 PM, in message <1394436721-21812-1-git-send-email-cy...@suse.com>, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, s

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chun Yan Liu
The whole patch series could also be get from here: https://github.com/chunyanliu/qemu/commits/QemuOpts -Chunyan >>> On 3/10/2014 at 03:31 PM, in message <1394436721-21812-1-git-send-email-cy...@suse.com>, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, s

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chun Yan Liu
The whole patch series could also be get from here: https://github.com/chunyanliu/qemu/commits/QemuOpts -Chunyan >>> On 3/10/2014 at 03:31 PM, in message <1394436721-21812-1-git-send-email-cy...@suse.com>, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, s

Re: [Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chun Yan Liu
The whole patch series could also be get from here: https://github.com/chunyanliu/qemu/commits/QemuOpts -Chunyan >>> On 3/10/2014 at 03:31 PM, in message <1394436721-21812-1-git-send-email-cy...@suse.com>, Chunyan Liu wrote: > This patch series is to replace QEMUOptionParameter with QemuOpts, s

[Qemu-devel] [PATCH v22 00/25] replace QEMUOptionParameter with QemuOpts

2014-03-10 Thread Chunyan Liu
This patch series is to replace QEMUOptionParameter with QemuOpts, so that only one Qemu Option structure is kept in QEMU code. --- Changes to v21: * update verison info in patch 2/25 * others are not changed except for rebase Chunyan Liu (25): add def_value_str to QemuOptDesc qapi: outpu