According to PowerISA, the PIR register should be readable in privileged
mode also, not only in hypervisor privileged mode.
The following patch fixes this:
--- qemu-2.12.0-original/target/ppc/translate_init.c2018-04-24
13:30:47.0 -0300
+++ qemu-2.12.0/target/ppc/translate_init.c 2
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 fr
On Wed, Apr 30, 2014 at 10:28:26AM +0200, Markus Armbruster wrote:
> Leandro Dorileo writes:
>
> > Hi Andreas,
> >
> > On Mon, Apr 28, 2014 at 09:02:12PM +0200, Andreas Färber wrote:
> >> Am 28.04.2014 20:55, schrieb Leandro Dorileo:
> >> > ping?
>
On Fri, May 02, 2014 at 10:40:33AM +0200, Stefan Hajnoczi wrote:
> On Tue, Mar 25, 2014 at 10:27:19AM -0300, Leandro Dorileo wrote:
> > Cover basic aspects and API usage for QemuOpt. The current implementation
> > covers the API's planned to be changed b
Cover basic aspects and API usage for QemuOpt. The current implementation
covers the API's planned to be changed by Chunyan Liu in his QEMUOptionParameter
replacement/cleanup job.
Other APIs should be covered in future improvements.
Signed-off-by: Leandro Dorileo
---
Notes:
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 :
> > >
> > > >
On Mon, Mar 24, 2014 at 02:06:15PM +0800, Deepak Kathayat wrote:
>
> Signed-off-by: Deepak Kathayat
Reviewed-by: Leandro Dorileo
> ---
> block/qcow2.h |1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/block/qcow2.h b/block/qcow2.h
> index 0b0eac8..25663d4 10
, "leaks")) {
> (gdb) bt
> 0 img_check (argc=6, argv=0x7fffab9b8a10) at qemu-img.c:588
> 1 __libc_start_main () from /lib/x86_64-linux-gnu/libc.so.6
> 2 _start ()
> (gdb)
>
> Signed-off-by: Prasad Joshi
Patch looks good to me.
Reviewed-by: Leandro Dorileo
> --
Cover basic aspects and API usage for QemuOpt. The current implementation
covers the API's planned to be changed by Chunyan Liu in his QEMUOptionParameter
replacement/cleanup job.
Other APIs should be covered in future improvements.
Signed-off-by: Leandro Dorileo
Reviewed-by: Eric
Cover basic aspects and API usage for QemuOpt. The current implementation
covers the API's planned to be changed by Chunyan Liu in his QEMUOptionParameter
replacement/cleanup job.
Other APIs should be covered in future improvements.
Signed-off-by: Leandro Dorileo
Reviewed-by: Eric
Cover basic aspects and API usage for QemuOpt. The current implementation
covers the API's planned to be changed by Chunyan Liu in his QEMUOptionParameter
replacement/cleanup job.
Other APIs should be covered in future improvements.
Signed-off-by: Leandro Dorileo
Reviewed-by: Eric
> qemu_opt_append function
> qemu_opt_get_*_del functions
> I think we could add tests for these functions later based on
> Leandro Dorileo's test suite patches:
> https://lists.gnu.org/archive/html/qemu-devel/2014-03/msg03282.html
>
>
> Chunyan Li
ptsList *params_to_opts(QEMUOptionParameter *list)
> -{
> -QemuOptsList *opts = NULL;
> -size_t num_opts, i = 0;
> -
> -if (!list
pattern.
I may be missing something completely obvious here but if you really need this
change
could you introduce a test in the test-opts-visitor.c test suite so we can
understand
the problem you're fixing here?
Regards...
--
Leandro Dorileo
>
> const QemuOptDesc *desc;
>
On Fri, Mar 21, 2014 at 06:12:18PM +0800, Chunyan Liu wrote:
> print_option_help takes QEMUOptionParameter as parameter, add
> qemu_opts_print_help to take QemuOptsList as parameter for later
> replace work.
>
> Signed-off-by: Dong Xu Wang
> Signed-off-by: Chunyan Liu
Re
On Fri, Mar 21, 2014 at 06:12:20PM +0800, Chunyan Liu wrote:
> For later merge .create_opts of drv and proto_drv in qemu-img commands.
>
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> Changes:
> * Following Eric's suggestion, qemu_opts_append(
; if (ret < 0) {
> qerror_report_err(local_err);
> error_free(local_err);
> @@ -2955,6 +2962,8 @@ static int enable_write_target(BDRVVVFATState *s)
> return 0;
>
> err:
> +qemu_opts_del(opts);
> +qemu_opts_free(create_opts);
> g_free(s->qcow_filename);
> s->qcow_filename = NULL;
> return ret;
> --
> 1.7.12.4
>
>
--
Leandro Dorileo
preallocation")) {
> -fprintf(stderr, "Cannot change preallocation mode.\n");
> -return -ENOTSUP;
> +/* Cannot change preallocation mode. Ignore it. */
You're ignoring/silencing an informed option, I think it's fear enough to
noti
printf("%s=%s ", desc->name, value);
> }
> }
> -fprintf(stderr, "\n");
> +printf("\n");
This new line is breaking most of the io tests since it changes
the expected output.
Regards...
--
Leandro Dorileo
> }
>
> static int opts_do_parse(QemuOpts *opts, const char *params,
> --
> 1.7.12.4
>
>
On Tue, Mar 25, 2014 at 06:09:40PM +, Leandro Dorileo wrote:
> On Fri, Mar 21, 2014 at 06:12:11PM +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.
> &
t; - "show CPU state before block translation" },
> + "show CPU registers before each executed TB (lots of logs)" },
what about s/lots of logs/hight volume log/ or maybe s/lots of logs/hight
volume/ ?
Regards
--
Leandro Dorileo
> { CPU_LOG_PCALL, "p
atch_simple("^[^%]+%d[^%]+$", filename, 0, 0)) {
> +logfilename = g_strdup_printf(filename, getpid());
> +} else {
> +g_error("Bad logfile format: %s", filename);
> +}
> +} else {
> +logfilename = g_strdup(filename);
> +}
> qemu_log_close();
> qemu_set_log(qemu_loglevel);
> }
Looks good to me.
--
Leandro Dorileo
On Thu, Mar 27, 2014 at 09:59:21AM +, Alex Bennée wrote:
>
> Leandro Dorileo writes:
>
> > On Wed, Mar 26, 2014 at 02:37:12PM +, alex.ben...@linaro.org wrote:
> >> From: Alex Bennée
> >>
> >> When debugging stuff that occurs over several
d(list->name)) {
> +error_report("Didn't find a matched option definition, "
> + "group name (%s) of option table must match
> with "
> + "defined option name (Since 2.0)", list->
ce QEMUOptionParameter with QemuOpts
> vpc.c: replace QEMUOptionParameter with QemuOpts
> cleanup QEMUOptionParameter
> QemuOpts: cleanup tmp 'allocated' member from QemuOptsList
block/nfs.c is missing conversion. Have you tested your own patches?
A simple git am &&
On Thu, Apr 03, 2014 at 05:54:19PM +0800, Chunyan Liu wrote:
> Reviewed-by: Eric Blake
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> util/qemu-option.c | 28 ++--
> 1 file changed, 14 insertions(+), 14 deletions(-)
>
> d
gt; +fprintf(stderr, "%s=\"%s\" ", opt->name, opt->str);
> +}
> +return;
> +}
> +for (; desc && desc->name; desc++) {
> +const char *value;
> +QemuOpt *opt = qemu_opt_find(opts, desc->name);
> +
> +value = opt ? opt->str : desc->def_value_str;
> +if (!value) {
> +continue;
> +}
> +if (desc->type == QEMU_OPT_STRING) {
> +fprintf(stderr, "%s='%s' ", desc->name, value);
> +} else {
> +fprintf(stderr, "%s=%s ", desc->name, value);
> +}
> }
> fprintf(stderr, "\n");
I see you removed this fprintf() in the commit
afe65f5c92694d551aaebc128233b623e8665d5e
Please do this change in this commit and avoid breaking the block tests in
further commits.
--
Leandro Dorileo
> -return 0;
> }
>
> static int opts_do_parse(QemuOpts *opts, const char *params,
> --
> 1.7.12.4
>
On Thu, Apr 03, 2014 at 05:54:25PM +0800, Chunyan Liu wrote:
> print_option_help takes QEMUOptionParameter as parameter, add
> qemu_opts_print_help to take QemuOptsList as parameter for later
> replace work.
>
> Signed-off-by: Dong Xu Wang
> Signed-off-by: Chunyan Liu
Re
ult value - set if option not informed
(since 2.1).
But if you feel comfortable with the current text you can add:
Reviewed-by: Leandro Dorileo
--
Leandro Dorileo
> # Since 1.5
> ##
> { 'type': 'CommandLineParameterInfo',
>'data': { 'name&#
using the correct type and malloced strings everywhere, the
> usage of this struct becomes clearer.
>
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> Changes:
> * Add explaination to this change.
> * Remove extra space before char *name, char *str
gt; Reviewed-by: Eric Blake
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> util/qemu-option.c | 16
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/util/qemu-option.c b/util/qemu-option.c
> index eab5102..25abd
vent_throttle(QEVENT_QUORUM_REPORT_BAD, 1000);
> +monitor_protocol_event_throttle(QEVENT_QUORUM_FAILURE, 1000);
> }
>
> /**
> diff --git a/qapi-schema.json b/qapi-schema.json
> index 7cfb5e5..990d0c5 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -4352,6 +43
On Tue, Feb 18, 2014 at 06:48:27PM +0100, Benoît Canet wrote:
> The Tuesday 18 Feb 2014 à 17:37:08 (+), Leandro Dorileo wrote :
> > On Tue, Feb 18, 2014 at 01:11:26PM +0100, Beno??t Canet wrote:
> > > From: Beno??t Canet
> > >
> > > Example of command
set = cg3_reset;
> +dc->vmsd = &vmstate_cg3;
> +dc->props = cg3_properties;
> +}
> +
> +static const TypeInfo cg3_info = {
> +.name = TYPE_CG3,
> +.parent= TYPE_SYS_BUS_DEVICE,
> +.instance_size = sizeof(CG3State),
> +.class_init= cg3_class_init,
> +};
> +
> +static void cg3_register_types(void)
> +{
> +type_register_static(&cg3_info);
> +}
> +
> +type_init(cg3_register_types)
> diff --git a/pc-bios/QEMU,cgthree.bin b/pc-bios/QEMU,cgthree.bin
> new file mode 100644
> index
> ..6fec9462070bf9a0f88024c7bf66f444d1d11d61
> GIT binary patch
> literal 850
> zcmZ{i&2HL25XX10!NXT-76XB#Rk_y^s6|SoR%&la53Q7_>KX6`yTvxLO)GA>_ch=J
> zP_0Ed5~+vw1^OC&f^?RT1C@IDd*?SZ|5@Af2Y zPDC*9!r1=eIU=;bdQRdZo4=yUd6i|Ci{<)%MIyz_ir4;eaD_K=6J(UtR=nVdN#fcA
> zFNrruCp7i~VGm}B*rM#FYA_wy$!sE!rI=f#Xh;N$ zu)fC(Rdr#9vwOTXDN4*eUYqPSqhX~xGG|XyEYsmuks~^k)Zx)xil*!=AoCkEsCJ zoLnnXbubf1(BxVqMqm=>vIAO|=luS}_JT~FP*rk6h2b=zc%GuQBB{~))g@X_rt6=%
> zVK@$>Ha6q}>z8kpvySz{2N@kpEMXb>Jz5ksB_3_=s6dTCOX4v$*W4J65;qbe1Ke=D
> zcrxzKpvBAAAKra@*6Vcb?u%{@nkk;p^!ZDR`PfpU9u9?JLjiUu4_oRe`bNojC4ddA
> z-NOJr!DrG6H~Nkfi8uxm4a5uZ+ZQly!#DLmP9;_lsVKMI5>-P{cC&R96e!56_1J6&
> zm}1hD{<>+03P;w8TyOmF(b
> yWEu%F;rYw!_h)ClbGu8)^3d%^loP5i*^VudTX8sz;xLL`?}lgvPvCTor|d5SYsmWm
>
> literal 0
> HcmV?d1
>
> diff --git a/pc-bios/README b/pc-bios/README
> index f190068..5914200 100644
> --- a/pc-bios/README
> +++ b/pc-bios/README
> @@ -11,8 +11,8 @@
>firmware implementation. The goal is to implement a 100% IEEE
>1275-1994 (referred to as Open Firmware) compliant firmware.
>The included images for PowerPC (for 32 and 64 bit PPC CPUs),
> - Sparc32 (including QEMU,tcx.bin) and Sparc64 are built from OpenBIOS SVN
> - revision 1246.
> + Sparc32 (including QEMU,tcx.bin and QEMU,cgthree.bin) and Sparc64 are built
> + from OpenBIOS SVN revision 1246.
>
> - SLOF (Slimline Open Firmware) is a free IEEE 1275 Open Firmware
>implementation for certain IBM POWER hardware. The sources are at
> --
> 1.7.10.4
>
>
--
Leandro Dorileo
On Wed, Feb 19, 2014 at 09:39:09PM +, Mark Cave-Ayland wrote:
> On 19/02/14 13:35, Leandro Dorileo wrote:
>
> Hi Leandro,
>
> >>+static void cg3_realizefn(DeviceState *dev, Error **errp)
> >>+{
> >>+SysBusDevice *sbd = SYS_BUS_DEVICE(dev);
> &
On Thu, Apr 10, 2014 at 11:36:05AM +0800, Chunyan Liu wrote:
> 2014-04-08 9:31 GMT+08:00 Leandro Dorileo :
>
> > On Thu, Apr 03, 2014 at 05:54:20PM +0800, Chunyan Liu wrote:
> > > Add def_value_str (default value) to QemuOptDesc, to replace function of
> >
On Thu, Apr 10, 2014 at 11:20:46AM +0800, Chunyan Liu wrote:
> 2014-04-08 9:12 GMT+08:00 Leandro Dorileo :
>
> > On Thu, Apr 03, 2014 at 05:54:18PM +0800, Chunyan Liu wrote:
> > > This patch series is to replace QEMUOptionParameter with QemuOpts, so
> > that only
>
ping?
On Tue, Mar 25, 2014 at 10:27:19AM -0300, Leandro Dorileo wrote:
> Cover basic aspects and API usage for QemuOpt. The current implementation
> covers the API's planned to be changed by Chunyan Liu in his
> QEMUOptionParameter
> replacement/cleanup job.
>
> Other AP
Hi Andreas,
On Mon, Apr 28, 2014 at 09:02:12PM +0200, Andreas Färber wrote:
> Am 28.04.2014 20:55, schrieb Leandro Dorileo:
> > ping?
> >
> >
> > On Tue, Mar 25, 2014 at 10:27:19AM -0300, Leandro Dorileo wrote:
> >> Cover basic aspects and API usage for
Hi Chunyan,
I haven't found enough time the last versions you've sent for review. I've buit
your
patch series and the final result builds nicely, but it fails the test 061, I
haven't
had the time to investigate it yet.
I'm leaving on vacation next Friday so I'll try to review all you patches
b
On Tue, Apr 29, 2014 at 05:08:10PM +0800, Chunyan Liu wrote:
> Reviewed-by: Stefan Hajnoczi
> Reviewed-by: Eric Blake
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> util/qemu-option.c | 28 ++--
> 1 file changed, 14 insertion
g Xu Wang
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> Changes to V25:
> * split v25 patch into two: this one and next one.
> * this patch is the same as v22 which is reviewed-by Eric.
>
> include/qemu/option.h | 3 ++-
> util/qemu-option.
tent with
> print_option_parameters, remove last printf and print size/number with
> opt->value.uint instead of opt->str.
>
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> util/qemu-option.c | 10 ++
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> d
On Tue, Apr 29, 2014 at 05:08:13PM +0800, Chunyan Liu wrote:
> Change qapi interfaces to output the newly added def_value_str when querying
> command line options.
>
> Signed-off-by: Dong Xu Wang
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> Changes
using the correct type and malloced strings everywhere, the
> usage of this struct becomes clearer.
>
> Reviewed-by: Eric Blake
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> include/qemu/option_int.h | 4 ++--
> qapi/opts-visitor.c | 10 ++
gt; Reviewed-by: Eric Blake
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> util/qemu-option.c | 16
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/util/qemu-option.c b/util/qemu-option.c
> index 69cdf3f..4d2d4
is (const char *).
>
> Reviewed-by: Eric Blake
> Signed-off-by: Dong Xu Wang
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> include/qemu/option.h | 6 +++
> util/qemu-option.c| 116
> --
>
d-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> include/qemu/option.h | 1 +
> util/qemu-option.c| 13 +
> 2 files changed, 14 insertions(+)
>
> diff --git a/include/qemu/option.h b/include/qemu/option.h
> index 6653e43..fbf5dc2 100644
> ---
gt; QemuOpts.
>
> Reviewed-by: Stefan Hajnoczi
> Reviewed-by: Eric Blake
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> include/qemu/option.h | 9 +++
> util/qemu-option.c| 153
> ++
> 2 fil
On Tue, Apr 29, 2014 at 05:08:20PM +0800, Chunyan Liu wrote:
> To simplify later using of qemu_opts_del, accept NULL input.
>
> Reviewed-by: Stefan Hajnoczi
> Reviewed-by: Eric Blake
> Signed-off-by: Chunyan Liu
Reviewed-by: Leandro Dorileo
> ---
> util/qemu-option
o out;
> }
>
> -if (encryption && encryption->value.n) {
> +if (encryption) {
> error_report("Compression and encryption not supported at "
> "the same time");
> ret =
ed-off-by: Dong Xu Wang
> Signed-off-by: Chunyan Liu
Did you send the patch series twice? I commented in the first v26 emails...
--
Leandro Dorileo
> ---
> Changes to V25:
> * fix Eric's comments:
> * update bdrv_create_co_entry and bdrv_amend_options code, to le
The following patchset introduces a general purpose argument parser and migrates
qemu-img to make use of it. qemu-img is just the first user of it, if we see a
good feedback here I move forward and migrate all the other possible users.
Leandro Dorileo (2):
qemu-arg: introduce a general purpose
nd parsing decorated arguments ("--argument value" and
"--argument=value"
are valid)
Signed-off-by: Leandro Dorileo
---
include/qemu/qemu-arg.h | 287
util/Makefile.objs | 1 +
util/qemu-arg.c | 887 ++
Hi Peter,
On Sat, Mar 08, 2014 at 06:55:50PM +, Peter Maydell wrote:
> On 8 March 2014 18:47, Leandro Dorileo wrote:
> > The following patchset introduces a general purpose argument parser and
> > migrates
> > qemu-img to make use of it. qemu-img is just the first user
Hi Paolo,
On Sun, Mar 09, 2014 at 08:30:28AM +0100, Paolo Bonzini wrote:
> Il 08/03/2014 19:47, Leandro Dorileo ha scritto:
> >Remove the arg parsing implementations using getopt and use qemu-arg.
> >Also remove the qemu-img-cmds.hx since it's now generated on building time,
On Sun, Mar 09, 2014 at 01:03:12PM +, Peter Maydell wrote:
> On 9 March 2014 12:37, Leandro Dorileo wrote:
> > Hi Paolo,
> >
> > On Sun, Mar 09, 2014 at 08:30:28AM +0100, Paolo Bonzini wrote:
> >> Il 08/03/2014 19:47, Leandro Dorileo ha scritto:
> >>
Hi Andreas,
On Sun, Mar 09, 2014 at 05:32:17PM +0100, Andreas Färber wrote:
> Am 08.03.2014 19:47, schrieb Leandro Dorileo:
> > The following patchset introduces a general purpose argument parser and
> > migrates
> > qemu-img to make use of it. qemu-img is just the fi
Hi Kevin,
On Tue, Mar 11, 2014 at 12:06:16PM +0100, Kevin Wolf wrote:
> Am 08.03.2014 um 19:47 hat Leandro Dorileo geschrieben:
> > The following patchset introduces a general purpose argument parser and
> > migrates
> > qemu-img to make use of it. qemu-img is just the fi
e' type. So the last version silently won, which was not the
> > behavior I had predicted.
> >
> > Post-patch, I get a compilation error (so how did you test your patch?):
> >
>
> Mostly tested ./qemu-img commands where QEMUOptionParameter is used.
> I really didn't think of test QemuOpts fully, and about the test suite, I
> have no full
> knowledge about how many things need to be tested, how many things need to
> be
> covered?
The testsuite should test the QemuOpts implementation, not the current users.
Regards...
--
Leandro Dorileo
Cover basic aspects and API usage for QemuOpt. The current implementation
covers the API's planed to be changed by Chunyan Liu in his QEMUOptionParameter
replacement/cleanup job.
Other APIs should be covered in future improvements.
Signed-off-by: Leandro Dorileo
---
tests/Mak
Hi Chunyan,
On Tue, Mar 11, 2014 at 09:00:04PM +, Leandro Dorileo wrote:
> Hi,
>
> On Tue, Mar 11, 2014 at 03:26:51PM +0800, Chunyan Liu wrote:
> > 2014-03-11 5:21 GMT+08:00 Eric Blake :
> >
> > > On 03/10/2014 02:29 PM, Eric Blake wrote:
> > >
>
Hi Erick,
On Tue, Mar 11, 2014 at 09:22:54AM -0600, Eric Blake wrote:
> On 03/11/2014 08:09 AM, Leandro Dorileo wrote:
> > Hi Kevin,
> >
> > On Tue, Mar 11, 2014 at 12:06:16PM +0100, Kevin Wolf wrote:
> >> Am 08.03.2014 um 19:47 hat Leandro Dorileo geschrieben
On Mon, Mar 17, 2014 at 10:16:12AM -0600, Eric Blake wrote:
> On 03/16/2014 03:18 PM, Leandro Dorileo wrote:
> > Cover basic aspects and API usage for QemuOpt. The current implementation
> > covers the API's planed to be changed by Chunyan Liu in his
> > QEMUOptionParam
2 wrong assertions).
I'll dig it a little deeper - when I find some time to - and comment in the
proper patches.
Regards...
--
Leandro Dorileo
Hi,
On Mon, Mar 10, 2014 at 03:31:41PM +0800, Chunyan Liu wrote:
> Add some qemu_opt functions to replace the same functionality of
> QEMUOptionParameter handling.
>
> Signed-off-by: Dong Xu Wang
> Signed-off-by: Chunyan Liu
> ---
> include/qemu/option.h | 9 +++
> util/qemu-option.c| 18
Hi,
On Mon, Mar 17, 2014 at 07:29:29PM +, Leandro Dorileo wrote:
> Hi,
>
> On Mon, Mar 10, 2014 at 03:32:01PM +0800, Chunyan Liu wrote:
> > Now all places using QEMUOptionParameter could use QemuOpts too, remove
> > QEMUOptionParameter related code.
> >
>
Hi,
On Mon, Mar 10, 2014 at 03:31:39PM +0800, Chunyan Liu wrote:
> Improve opt_get and opt_set group of functions. For opt_get, check and handle
> NULL input; for opt_set, when set to an existing option, rewrite the option
> with new value.
>
> Signed-off-by: Dong Xu Wang
> Signed-off-by: Chunya
Cover basic aspects and API usage for QemuOpt. The current implementation
covers the API's planned to be changed by Chunyan Liu in his QEMUOptionParameter
replacement/cleanup job.
Other APIs should be covered in future improvements.
Signed-off-by: Leandro Dorileo
Changes:
v2:
+
eenend.org.uk/~sgtatham/coroutines.html
[2] -
http://tia.mat.br/blog/html/2012/09/29/asynchronous_i_o_in_c_with_coroutines.html
Regards...
--
Leandro Dorileo
.
I'm
sending the job's resulting series to the mailing list so I can show you what
I mean and have some more room for discussion. It doesn't mean I want to overlap
you work, I just needed to have a little more input on that.
Regards
--
Leandro Dorileo
From: Chunyan Liu
Add def_value_str (default value) to QemuOptDesc, to replace function of the
default value in QEMUOptionParameter. And improved related functions.
Signed-off-by: Dong Xu Wang
Signed-off-by: Chunyan Liu
---
include/qemu/option.h | 3 +-
util/qemu-option.c| 84 +++
From: Chunyan Liu
The qemu_opts_append() function is intended to merge to different
QemuOptsList's. The resulting list must be freed by its user.
Signed-off-by: Chunyan Liu
Signed-off-by: Leandro Dorileo
---
include/qemu/option.h | 1 +
util/qemu-option.c
Do the directly migration from QemuOptionParameter to QemuOpts on
vpc block driver.
Signed-off-by: Leandro Dorileo
---
block/vpc.c | 54 +++---
1 file changed, 27 insertions(+), 27 deletions(-)
diff --git a/block/vpc.c b/block/vpc.c
index 82bf248
Use a pointer to a structure holding the primitive types and avoid
parsing the default value representation.
Signed-off-by: Leandro Dorileo
---
include/qemu/option.h | 20 +++-
util/qemu-config.c| 4 ++--
util/qemu-option.c| 45
Finally remove all the remaining QEMUOptionParameter bits.
Signed-off-by: Leandro Dorileo
---
include/qemu/option.h | 40 ---
util/qemu-option.c| 296 --
2 files changed, 336 deletions(-)
diff --git a/include/qemu/option.h b/include/qemu
Do the directly migration from QemuOptionParameter to QemuOpts on
gluster block driver.
Signed-off-by: Leandro Dorileo
---
block/gluster.c | 68 +++--
1 file changed, 37 insertions(+), 31 deletions(-)
diff --git a/block/gluster.c b/block
Analogous to print_option_help(QEMUOptionParameter *list) this function
displays the available key and help for each described QemuOptDesc in
QemuOptList.
Signed-off-by: Leandro Dorileo
---
include/qemu/option.h | 1 +
util/qemu-option.c| 12
2 files changed, 13 insertions
Do the directly migration from QemuOptionParameter to QemuOpts on
cow block driver.
Signed-off-by: Leandro Dorileo
---
block/cow.c | 44 ++--
1 file changed, 22 insertions(+), 22 deletions(-)
diff --git a/block/cow.c b/block/cow.c
index 30deb88..811f7f7
Do the directly migration from QemuOptionParameter to QemuOpts on
qed block driver.
Signed-off-by: Leandro Dorileo
---
block/qed.c | 79 +++--
1 file changed, 35 insertions(+), 44 deletions(-)
diff --git a/block/qed.c b/block/qed.c
index
Do the directly migration from QemuOptionParameter to QemuOpts on
raw-posix block driver.
Signed-off-by: Leandro Dorileo
---
block/raw-posix.c | 50 +++---
1 file changed, 23 insertions(+), 27 deletions(-)
diff --git a/block/raw-posix.c b/block/raw
Do the directly migration from QemuOptionParameter to QemuOpts on
iscsi block driver.
Signed-off-by: Leandro Dorileo
---
block/iscsi.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/block/iscsi.c b/block/iscsi.c
index b490e98..85252e7
Do the directly migration from QemuOptionParameter to QemuOpts on
nfs block driver.
Signed-off-by: Leandro Dorileo
---
block/nfs.c | 11 ++-
1 file changed, 2 insertions(+), 9 deletions(-)
diff --git a/block/nfs.c b/block/nfs.c
index 98aa363..c01f109 100644
--- a/block/nfs.c
+++ b
Do the directly migration from QemuOptionParameter to QemuOpts on
vhdx block driver.
Signed-off-by: Leandro Dorileo
---
block/vhdx.c | 88
1 file changed, 41 insertions(+), 47 deletions(-)
diff --git a/block/vhdx.c b/block/vhdx.c
Do the directly migration from QemuOptionParameter to QemuOpts on
qcow2 block driver.
Signed-off-by: Leandro Dorileo
---
block/qcow2.c | 263 --
1 file changed, 128 insertions(+), 135 deletions(-)
diff --git a/block/qcow2.c b/block/qcow2
Do the directly migration from QemuOptionParameter to QemuOpts on
qemu-img.
Signed-off-by: Leandro Dorileo
---
qemu-img.c | 166 +++--
1 file changed, 95 insertions(+), 71 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index 2e40cc1
Do the directly migration from QemuOptionParameter to QemuOpts on
raw-win32 block driver.
Signed-off-by: Leandro Dorileo
---
block/raw-win32.c | 29 +
1 file changed, 13 insertions(+), 16 deletions(-)
diff --git a/block/raw-win32.c b/block/raw-win32.c
index 48cb2c2
Do the directly migration from QemuOptionParameter to QemuOpts on
raw_bsd block driver.
Signed-off-by: Leandro Dorileo
---
block/raw_bsd.c | 21 -
1 file changed, 12 insertions(+), 9 deletions(-)
diff --git a/block/raw_bsd.c b/block/raw_bsd.c
index 01ea692..5090b4e 100644
Do the directly migration from QemuOptionParameter to QemuOpts on
vmdk block driver.
Signed-off-by: Leandro Dorileo
---
block/vmdk.c | 105 ++-
1 file changed, 53 insertions(+), 52 deletions(-)
diff --git a/block/vmdk.c b/block/vmdk.c
Do the directly migration from QemuOptionParameter to QemuOpts on
vvfat block driver.
Signed-off-by: Leandro Dorileo
---
block/vvfat.c | 26 +-
1 file changed, 21 insertions(+), 5 deletions(-)
diff --git a/block/vvfat.c b/block/vvfat.c
index f966ea5..7aefba3 100644
unyan's authorship.
--
Dorileo
Chunyan Liu (3):
qapi: output def_value_str when query command line options
add def_value_str to QemuOptDesc
QemuOpt: introduce qemu_opts_append()
Leandro Dorileo (23):
QemuOpt: improve default value
QemuOpt: add qemu_opt_print_help()
block: migrate block la
Do the directly migration from QemuOptionParameter to QemuOpts on
sheepdog block driver.
Signed-off-by: Leandro Dorileo
---
block/sheepdog.c | 104 ---
1 file changed, 53 insertions(+), 51 deletions(-)
diff --git a/block/sheepdog.c b/block
From: Chunyan Liu
Change qapi interfaces to output the newly added def_value_str when querying
command line options.
Signed-off-by: Dong Xu Wang
Signed-off-by: Chunyan Liu
---
qapi-schema.json | 6 +-
qmp-commands.hx| 2 ++
util/qemu-config.c | 4
3 files changed, 11 insertions
Do the directly migration from QemuOptionParameter to QemuOpts on
qcow block driver.
Signed-off-by: Leandro Dorileo
---
block/qcow.c | 59 +--
1 file changed, 29 insertions(+), 30 deletions(-)
diff --git a/block/qcow.c b/block/qcow.c
Do the directly migration from QemuOptionParameter to QemuOpts on
vdi block driver.
Signed-off-by: Leandro Dorileo
---
block/vdi.c | 73 -
1 file changed, 34 insertions(+), 39 deletions(-)
diff --git a/block/vdi.c b/block/vdi.c
index
Do the directly migration from QemuOptionParameter to QemuOpts on
ssh block driver.
Signed-off-by: Leandro Dorileo
---
block/ssh.c | 29 +
1 file changed, 13 insertions(+), 16 deletions(-)
diff --git a/block/ssh.c b/block/ssh.c
index aa63c9d..b55c518 100644
--- a
Do the directly migration from QemuOptionParameter to QemuOpts on
block layer.
Signed-off-by: Leandro Dorileo
---
block.c | 133 --
include/block/block.h | 7 ++-
include/block/block_int.h | 8 ++-
3 files changed, 86
Do the directly migration from QemuOptionParameter to QemuOpts on
rbd block driver.
Signed-off-by: Leandro Dorileo
---
block/rbd.c | 60 +---
1 file changed, 29 insertions(+), 31 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index
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 QEMUOpt
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 Qem
1 - 100 of 219 matches
Mail list logo
