Re: query-command-line-options

2023-05-26 Thread Markus Armbruster
Ján Tomko writes: > On a Friday in 2023, Markus Armbruster wrote: >>> { "sandbox", NULL, QEMU_CAPS_SECCOMP_SANDBOX }, >> >>Does option -sandbox exist? >> >>It does since v1.2. If CONFIG_SECCOMP is off, actually using it is a >>fatal error. Compiling out the option entirely would be more use

Re: query-command-line-options

2023-05-26 Thread Ján Tomko
On a Friday in 2023, Markus Armbruster wrote: { "sandbox", NULL, QEMU_CAPS_SECCOMP_SANDBOX }, Does option -sandbox exist? It does since v1.2. If CONFIG_SECCOMP is off, actually using it is a fatal error. Compiling out the option entirely would be more useful, I guess. Is this probe stil

Re: query-command-line-options

2023-05-26 Thread Markus Armbruster
Stefan, one question regarding your commit 40e07370f21 inline. Peter Krempa writes: [...] > In fact I strive to eliminate it after your suggestions, but we can't > still detect the few outstanding flags from anywhere else. > > List of currently outstanding queries using q

[Stable-8.0.1 39/59] util/async-teardown: wire up query-command-line-options

2023-05-23 Thread Michael Tokarev
From: Claudio Imbrenda Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. Reported-by: Boris Fiuczynski

[PULL 09/21] util/async-teardown: wire up query-command-line-options

2023-05-15 Thread Thomas Huth
From: Claudio Imbrenda Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. Reported-by: Boris Fiuczynski

Re: [PATCH v7 1/1] util/async-teardown: wire up query-command-line-options

2023-05-15 Thread Thomas Huth
On 05/05/2023 14.00, Claudio Imbrenda wrote: Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it

Re: [PATCH v7 1/1] util/async-teardown: wire up query-command-line-options

2023-05-08 Thread Thomas Huth
On 05/05/2023 14.00, Claudio Imbrenda wrote: Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. Reported

[PATCH v7 1/1] util/async-teardown: wire up query-command-line-options

2023-05-05 Thread Claudio Imbrenda
Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. Reported-by: Boris Fiuczynski Fixes: c891c24b1a (&qu

[PATCH v7 0/1] util/async-teardown: appear in query-command-line-options

2023-05-05 Thread Claudio Imbrenda
Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. v6->v7 * move variable declaration inside #ifdef

Re: [PATCH v6 1/1] util/async-teardown: wire up query-command-line-options

2023-04-28 Thread Claudio Imbrenda
On Fri, 28 Apr 2023 14:16:42 +0200 Thomas Huth wrote: > On 28/04/2023 13.12, Claudio Imbrenda wrote: > > Add new -run-with option with an async-teardown=on|off parameter. It is > > visible in the output of query-command-line-options QMP command, so it > > can be discovere

Re: [PATCH v6 1/1] util/async-teardown: wire up query-command-line-options

2023-04-28 Thread Thomas Huth
On 28/04/2023 13.12, Claudio Imbrenda wrote: Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. Reported

[PATCH v6 1/1] util/async-teardown: wire up query-command-line-options

2023-04-28 Thread Claudio Imbrenda
Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. Reported-by: Boris Fiuczynski Fixes: c891c24b1a (&qu

[PATCH v6 0/1] util/async-teardown: appear in query-command-line-options

2023-04-28 Thread Claudio Imbrenda
Add new -run-with option with an async-teardown=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant, deprecate it. v5->v6 * deprecate the old -async-teardown opt

Re: [PATCH v5 1/1] util/async-teardown: wire up query-command-line-options

2023-03-28 Thread Markus Armbruster
Thomas Huth writes: > On 28/03/2023 07.26, Markus Armbruster wrote: >> Paolo Bonzini writes: >> >>> I am honestly not a fan of adding a more complex option,.just because >>> query-command-line-options only returns the square holes whereas here we >>&g

Re: [PATCH v5 1/1] util/async-teardown: wire up query-command-line-options

2023-03-28 Thread Thomas Huth
On 28/03/2023 07.26, Markus Armbruster wrote: Paolo Bonzini writes: I am honestly not a fan of adding a more complex option,.just because query-command-line-options only returns the square holes whereas here we got a round one. Can we imagine another functionality that would be added to

Re: [PATCH v5 1/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Markus Armbruster
Paolo Bonzini writes: > I am honestly not a fan of adding a more complex option,.just because > query-command-line-options only returns the square holes whereas here we > got a round one. > > Can we imagine another functionality that would be added to -teardown? If > not, it&#

Re: [PATCH v5 1/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Paolo Bonzini
I am honestly not a fan of adding a more complex option,.just because query-command-line-options only returns the square holes whereas here we got a round one. Can we imagine another functionality that would be added to -teardown? If not, it's not a good design. If it works, I would

[PATCH v5 0/1] util/async-teardown: appear in query-command-line-options

2023-03-27 Thread Claudio Imbrenda
Add new -teardown option with an async=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant. We'd normally deprecate it and remove it after a grace period, but i

[PATCH v5 1/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Claudio Imbrenda
Add new -teardown option with an async=on|off parameter. It is visible in the output of query-command-line-options QMP command, so it can be discovered and used by libvirt. The option -async-teardown is now redundant. We'd normally deprecate it and remove it after a grace period, but i

Re: [PATCH v4 1/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Markus Armbruster
Claudio Imbrenda writes: > The recently introduced -async-teardown commandline option was not > wired up properly and did not show up in the output of the QMP command > query-command-line-options. This means that libvirt had no way to > discover whether the feature was supported.

[PATCH v4 0/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Claudio Imbrenda
The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported. This patch fixes the issue by adding a new

[PATCH v4 1/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Claudio Imbrenda
The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt had no way to discover whether the feature was supported. This patch fixes the issue by replacing the

Re: [PATCH v3 1/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Thomas Huth
On 24/03/2023 18.45, Claudio Imbrenda wrote: The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported

Re: [PATCH v3 1/1] util/async-teardown: wire up query-command-line-options

2023-03-27 Thread Thomas Huth
-command-line-options. This means that libvirt will have no way to discover whether the feature is supported. This patch fixes the issue by correctly wiring up the commandline option so that it appears in the output of query-command-line-options. Reported-by: Boris Fiuczynski Fixes: c891c24b1a (&qu

Re: [PATCH v3 1/1] util/async-teardown: wire up query-command-line-options

2023-03-24 Thread Claudio Imbrenda
On Fri, 24 Mar 2023 18:56:06 +0100 Thomas Huth wrote: > On 24/03/2023 18.45, Claudio Imbrenda wrote: > > The recently introduced -async-teardown commandline option was not > > wired up properly and did not show up in the output of the QMP command > > query-command-line-opt

Re: [PATCH v3 1/1] util/async-teardown: wire up query-command-line-options

2023-03-24 Thread Thomas Huth
On 24/03/2023 18.45, Claudio Imbrenda wrote: The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported

[PATCH v3 1/1] util/async-teardown: wire up query-command-line-options

2023-03-24 Thread Claudio Imbrenda
The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported. This patch fixes the issue by correctly

[PATCH v3 0/1] util/async-teardown: wire up query-command-line-options

2023-03-24 Thread Claudio Imbrenda
The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported. This patch fixes the issue by adding a new

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-22 Thread Markus Armbruster
rg/archive/html/qemu-devel/2022-08/msg04479.html > > ? > > Anyway, how to continue now here? If I've got that right, we currently need > an option that takes a parameter if we want to make it visible via QAPI, > right? Visible in query-command-line-options, to be precise.

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-22 Thread Thomas Huth
On 22/03/2023 10.28, Daniel P. Berrangé wrote: On Tue, Mar 21, 2023 at 09:47:57PM +0100, Paolo Bonzini wrote: Il lun 20 mar 2023, 16:42 Thomas Huth ha scritto: Would it make sense to add it e.g. to "-action" instead, i.e. something like "-action teardown=async" ? -action is just a wrapper

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-22 Thread Daniel P . Berrangé
On Tue, Mar 21, 2023 at 09:47:57PM +0100, Paolo Bonzini wrote: > Il lun 20 mar 2023, 16:42 Thomas Huth ha scritto: > > > Would it make sense to add it e.g. to "-action" instead, i.e. something > > like > > "-action teardown=async" ? > > > > -action is just a wrapper for the action-set QMP comman

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-21 Thread Paolo Bonzini
Il lun 20 mar 2023, 16:42 Thomas Huth ha scritto: > Would it make sense to add it e.g. to "-action" instead, i.e. something > like > "-action teardown=async" ? > -action is just a wrapper for the action-set QMP command. I don't think it fits very well; its arguments are only guest actions while

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Claudio Imbrenda
>>> wired up properly and did not show up in the output of the QMP command > >>> query-command-line-options. This means that libvirt will have no way to > >>> discover whether the feature is supported. > >> > >> There was nothing imprope

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Markus Armbruster
Thomas Huth writes: > On 20/03/2023 16.31, Markus Armbruster wrote: >> Claudio Imbrenda writes: >> >>> The recently introduced -async-teardown commandline option was not >>> wired up properly and did not show up in the output of the QMP command >>> q

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Thomas Huth
On 20/03/2023 16.31, Markus Armbruster wrote: Claudio Imbrenda writes: The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover

Re: [PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Markus Armbruster
Claudio Imbrenda writes: > The recently introduced -async-teardown commandline option was not > wired up properly and did not show up in the output of the QMP command > query-command-line-options. This means that libvirt will have no way to > discover whether the feature is supported

[PATCH v2 0/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Claudio Imbrenda
The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported. This patch fixes the issue by correctly

[PATCH v2 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Claudio Imbrenda
The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported. This patch fixes the issue by correctly

Re: [PATCH v1 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Claudio Imbrenda
On Mon, 20 Mar 2023 09:56:05 +0100 Thomas Huth wrote: > On 20/03/2023 08.48, Claudio Imbrenda wrote: > > The recently introduced -async-teardown commandline option was not > > wired up properly and did not show up in the output of the QMP command > > query-command-line-opt

Re: [PATCH v1 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Thomas Huth
On 20/03/2023 08.48, Claudio Imbrenda wrote: The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported

[PATCH v1 1/1] util/async-teardown: wire up query-command-line-options

2023-03-20 Thread Claudio Imbrenda
The recently introduced -async-teardown commandline option was not wired up properly and did not show up in the output of the QMP command query-command-line-options. This means that libvirt will have no way to discover whether the feature is supported. This patch fixes the issue by correctly

Re: query-command-line-options (was: [PATCH 1/7] qemu: capabilities: Introduce QEMU_CAPS_MACHINE_ACPI)

2023-03-07 Thread Peter Krempa
> > I think it would be worthwile to check with the QEMU developers and > > make sure that they're aware of this behavior. Is it intended? Is it > > documented anywhere? It certainly seems extremely confusing to me. > > query-command-line-options has... issues. > >

query-command-line-options (was: [PATCH 1/7] qemu: capabilities: Introduce QEMU_CAPS_MACHINE_ACPI)

2023-03-07 Thread Markus Armbruster
x27;re checking whether -acpi itself exists as a >> > top-level option. Which it doesn't, but -no-acpi does and yet it >> > doesn't seem to be advertised in the output of >> > query-command-line-options? >> >> Well, it actually does exist in the o

[PULL v2 09/23] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-12-15 Thread Thomas Huth
The "query-command-line-options" command uses a hand-crafted list of options that should be returned for the "machine" parameter. This is pretty much out of sync with reality, for example settings like "kvm_shadow_mem" or "accel" are not parameters fo

[PULL 09/23] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-12-14 Thread Thomas Huth
The "query-command-line-options" command uses a hand-crafted list of options that should be returned for the "machine" parameter. This is pretty much out of sync with reality, for example settings like "kvm_shadow_mem" or "accel" are not parameters fo

Re: [PATCH v2] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-15 Thread Markus Armbruster
Thomas Huth writes: > On 15/11/2022 08.53, Markus Armbruster wrote: >> Thomas Huth writes: >> >>> On 11/11/2022 15.53, Markus Armbruster wrote: >>>> Thomas Huth writes: >>>> >>>>> The "query-command-line-options" comm

Re: [PATCH v2] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-15 Thread Thomas Huth
On 15/11/2022 08.53, Markus Armbruster wrote: Thomas Huth writes: On 11/11/2022 15.53, Markus Armbruster wrote: Thomas Huth writes: The "query-command-line-options" command uses a hand-crafted list of options that should be returned for the "machine" parameter. This i

Re: [PATCH v2] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-14 Thread Markus Armbruster
Thomas Huth writes: > On 11/11/2022 15.53, Markus Armbruster wrote: >> Thomas Huth writes: >> >>> The "query-command-line-options" command uses a hand-crafted list >>> of options that should be returned for the "machine" parameter. >>

Re: [PATCH v2] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-11 Thread Thomas Huth
On 11/11/2022 15.53, Markus Armbruster wrote: Thomas Huth writes: The "query-command-line-options" command uses a hand-crafted list of options that should be returned for the "machine" parameter. This is pretty much out of sync with reality, for example settings like

Re: [PATCH v2] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-11 Thread Markus Armbruster
Thomas Huth writes: > The "query-command-line-options" command uses a hand-crafted list > of options that should be returned for the "machine" parameter. > This is pretty much out of sync with reality, for example settings > like "kvm_shadow_mem" or

[PATCH v2] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-11 Thread Thomas Huth
The "query-command-line-options" command uses a hand-crafted list of options that should be returned for the "machine" parameter. This is pretty much out of sync with reality, for example settings like "kvm_shadow_mem" or "accel" are not parameters fo

Re: [PATCH] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-11 Thread Thomas Huth
On 09/11/2022 09.44, Thomas Huth wrote: The "query-command-line-options" command uses a hand-crafted list of options that should be returned for the "machine" parameter. This is pretty much out of sync with reality, for example settings like "kvm_shadow_mem" or &quo

[PATCH] util/qemu-config: Fix "query-command-line-options" to provide the right values

2022-11-09 Thread Thomas Huth
The "query-command-line-options" command uses a hand-crafted list of options that should be returned for the "machine" parameter. This is pretty much out of sync with reality, for example settings like "kvm_shadow_mem" or "accel" are not parameters fo

[Qemu-devel] [PULL 20/51] qemu-config: fix leak in query-command-line-options

2018-01-16 Thread Paolo Bonzini
From: Marc-André Lureau Direct leak of 160 byte(s) in 4 object(s) allocated from: #0 0x55ed7678cda8 in calloc (/home/elmarco/src/qq/build/x86_64-softmmu/qemu-system-x86_64+0x797da8) #1 0x7f3f5e725f75 in g_malloc0 /home/elmarco/src/gnome/glib/builddir/../glib/gmem.c:124 #2 0x55ed778a

[Qemu-devel] [PULL 21/52] qemu-config: fix leak in query-command-line-options

2018-01-12 Thread Paolo Bonzini
From: Marc-André Lureau Direct leak of 160 byte(s) in 4 object(s) allocated from: #0 0x55ed7678cda8 in calloc (/home/elmarco/src/qq/build/x86_64-softmmu/qemu-system-x86_64+0x797da8) #1 0x7f3f5e725f75 in g_malloc0 /home/elmarco/src/gnome/glib/builddir/../glib/gmem.c:124 #2 0x55ed778a

[Qemu-devel] [PATCH v3 13/18] qemu-config: fix leak in query-command-line-options

2018-01-04 Thread Marc-André Lureau
Direct leak of 160 byte(s) in 4 object(s) allocated from: #0 0x55ed7678cda8 in calloc (/home/elmarco/src/qq/build/x86_64-softmmu/qemu-system-x86_64+0x797da8) #1 0x7f3f5e725f75 in g_malloc0 /home/elmarco/src/gnome/glib/builddir/../glib/gmem.c:124 #2 0x55ed778aa3a7 in query_option_descs

Re: [Qemu-devel] [PATCH v2 10/13] qemu-config: fix leak in query-command-line-options

2017-12-15 Thread Eric Blake
On 12/15/2017 09:06 AM, Marc-André Lureau wrote: > Direct leak of 160 byte(s) in 4 object(s) allocated from: > #0 0x55ed7678cda8 in calloc > (/home/elmarco/src/qq/build/x86_64-softmmu/qemu-system-x86_64+0x797da8) > #1 0x7f3f5e725f75 in g_malloc0 > /home/elmarco/src/gnome/glib/builddir/../

[Qemu-devel] [PATCH v2 10/13] qemu-config: fix leak in query-command-line-options

2017-12-15 Thread Marc-André Lureau
Direct leak of 160 byte(s) in 4 object(s) allocated from: #0 0x55ed7678cda8 in calloc (/home/elmarco/src/qq/build/x86_64-softmmu/qemu-system-x86_64+0x797da8) #1 0x7f3f5e725f75 in g_malloc0 /home/elmarco/src/gnome/glib/builddir/../glib/gmem.c:124 #2 0x55ed778aa3a7 in query_option_descs

[Qemu-devel] [PATCH 10/12] qemu-config: fix leak in query-command-line-options

2017-12-07 Thread Marc-André Lureau
Direct leak of 160 byte(s) in 4 object(s) allocated from: #0 0x55ed7678cda8 in calloc (/home/elmarco/src/qq/build/x86_64-softmmu/qemu-system-x86_64+0x797da8) #1 0x7f3f5e725f75 in g_malloc0 /home/elmarco/src/gnome/glib/builddir/../glib/gmem.c:124 #2 0x55ed778aa3a7 in query_option_descs

[Qemu-devel] [PULL 04/10] block: Add bdrv_runtime_opts to query-command-line-options

2016-10-07 Thread Kevin Wolf
Recently we moved a few options from QemuOptsLists in blockdev.c to bdrv_runtime_opts in block.c in order to make them accissble using blockdev-add. However, this has the side effect that these options are missing from query-command-line-options now, and libvirt consequently disables the

Re: [Qemu-devel] [PATCH] block: Add bdrv_runtime_opts to query-command-line-options

2016-10-07 Thread Gerd Hoffmann
ts to the list of QemuOptsLists that are > returned in query-command-line-options. For the future, libvirt is > advised to use QMP schema introspection for block device options. > > Reported-by: Michal Privoznik > Signed-off-by: Kevin Wolf Tested-by: Gerd Hoffmann > --- >

Re: [Qemu-devel] [PATCH] block: Add bdrv_runtime_opts to query-command-line-options

2016-10-06 Thread Kevin Wolf
dd. However, this has the side effect that these options are > >missing from query-command-line-options now, and libvirt consequently > >disables the corresponding feature. > > > >This problem was reported as a regression for the 'discard' option, > >introdu

Re: [Qemu-devel] [PATCH] block: Add bdrv_runtime_opts to query-command-line-options

2016-10-06 Thread Martin Kletzander
eturned in query-command-line-options. For the future, libvirt is advised to use QMP schema introspection for block device options. Reported-by: Michal Privoznik Signed-off-by: Kevin Wolf --- block.c | 2 +- include/sysemu/sysemu.h | 1 + util/qemu-config.c |

Re: [Qemu-devel] [PATCH] block: Add bdrv_runtime_opts to query-command-line-options

2016-10-06 Thread Michal Privoznik
On 06.10.2016 11:40, Kevin Wolf wrote: > Recently we moved a few options from QemuOptsLists in blockdev.c to > bdrv_runtime_opts in block.c in order to make them accissble using > blockdev-add. However, this has the side effect that these options are > missing from query-command-line

[Qemu-devel] [PATCH] block: Add bdrv_runtime_opts to query-command-line-options

2016-10-06 Thread Kevin Wolf
Recently we moved a few options from QemuOptsLists in blockdev.c to bdrv_runtime_opts in block.c in order to make them accissble using blockdev-add. However, this has the side effect that these options are missing from query-command-line-options now, and libvirt consequently disables the

[Qemu-devel] [PULL 10/39] qapi: output def_value_str when query command line options

2014-06-16 Thread Stefan Hajnoczi
From: Chunyan Liu Change qapi interfaces to output the newly added def_value_str when querying command line options. Reviewed-by: Stefan Hajnoczi Reviewed-by: Eric Blake Reviewed-by: Leandro Dorileo Signed-off-by: Dong Xu Wang Signed-off-by: Chunyan Liu Signed-off-by: Stefan Hajnoczi ---

[Qemu-devel] [PATCH v28 04/33] qapi: output def_value_str when query command line options

2014-06-05 Thread Chunyan Liu
Change qapi interfaces to output the newly added def_value_str when querying command line options. Reviewed-by: Stefan Hajnoczi Reviewed-by: Eric Blake Reviewed-by: Leandro Dorileo Signed-off-by: Dong Xu Wang Signed-off-by: Chunyan Liu --- qapi-schema.json | 5 - qmp-commands.hx| 2

[Qemu-devel] [PATCH v27 04/33] qapi: output def_value_str when query command line options

2014-05-07 Thread Chunyan Liu
Change qapi interfaces to output the newly added def_value_str when querying command line options. Reviewed-by: Stefan Hajnoczi Reviewed-by: Eric Blake Reviewed-by: Leandro Dorileo Signed-off-by: Dong Xu Wang Signed-off-by: Chunyan Liu --- qapi-schema.json | 5 - qmp-commands.hx| 2

[Qemu-devel] [PATCH v27 04/33] qapi: output def_value_str when query command line options

2014-05-07 Thread Chunyan Liu
Change qapi interfaces to output the newly added def_value_str when querying command line options. Reviewed-by: Stefan Hajnoczi Reviewed-by: Eric Blake Reviewed-by: Leandro Dorileo Signed-off-by: Dong Xu Wang Signed-off-by: Chunyan Liu --- qapi-schema.json | 5 - qmp-commands.hx| 2

Re: [Qemu-devel] [PATCH V26 04/32] qapi: output def_value_str when query command line options

2014-05-06 Thread Stefan Hajnoczi
On Tue, Apr 29, 2014 at 05:10:28PM +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 > --- > Changes to V25: > * update @default description in .json file

Re: [Qemu-devel] [PATCH V26 04/32] qapi: output def_value_str when query command line options

2014-05-01 Thread Leandro Dorileo
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 to V25: > * update

Re: [Qemu-devel] [PATCH V26 04/32] qapi: output def_value_str when query command line options

2014-04-29 Thread Eric Blake
On 04/29/2014 03:10 AM, 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 > --- > Changes to V25: > * update @default description in .json file > > qapi-schema

[Qemu-devel] [PATCH V26 04/32] qapi: output def_value_str when query command line options

2014-04-29 Thread 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 --- Changes to V25: * update @default description in .json file qapi-schema.json | 5 - qmp-commands.hx| 2 ++ util/qemu-config.

[Qemu-devel] [PATCH V26 04/32] qapi: output def_value_str when query command line options

2014-04-29 Thread 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 --- Changes to V25: * update @default description in .json file qapi-schema.json | 5 - qmp-commands.hx| 2 ++ util/qemu-config.

Re: [Qemu-devel] [PATCH v25 03/31] qapi: output def_value_str when query command line options

2014-04-21 Thread Eric Blake
On 04/10/2014 11:53 AM, Chunyan Liu wrote: > Change qapi interfaces to output the newly added def_value_str when querying > command line options. > > Reviewed-by: Eric Blake > Signed-off-by: Dong Xu Wang > Signed-off-by: Chunyan Liu > --- > changes: > * Following Leandro's comment: > upda

[Qemu-devel] [PATCH v25 03/31] qapi: output def_value_str when query command line options

2014-04-10 Thread Chunyan Liu
Change qapi interfaces to output the newly added def_value_str when querying command line options. Reviewed-by: Eric Blake Signed-off-by: Dong Xu Wang Signed-off-by: Chunyan Liu --- changes: * Following Leandro's comment: update description of @default qapi-schema.json | 6 +- qmp

Re: [Qemu-devel] [PATCH v24 03/31] qapi: output def_value_str when query command line options

2014-04-07 Thread Leandro Dorileo
On Thu, Apr 03, 2014 at 05:54:21PM +0800, Chunyan Liu wrote: > Change qapi interfaces to output the newly added def_value_str when querying > command line options. > > Reviewed-by: Eric Blake > Signed-off-by: Dong Xu Wang > Signed-off-by: Chunyan Liu > --- > qapi-schema.json | 6 +- > qm

[Qemu-devel] [PATCH v24 03/31] qapi: output def_value_str when query command line options

2014-04-03 Thread Chunyan Liu
Change qapi interfaces to output the newly added def_value_str when querying command line options. Reviewed-by: Eric Blake 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 inser

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-27 Thread Markus Armbruster
03/05/2014 07:36 PM, Amos Kong wrote: >> >> >> vm_config_groups[] only contains part of the options which have >> >> >> argument, and all options which have no argument aren't added >> >> >> to vm_config_groups[]. Current query-command-line-

Re: [Qemu-devel] [PATCH v6 0/3] fix query-command-line-options

2014-03-27 Thread Amos Kong
On Thu, Mar 27, 2014 at 02:57:00PM +0800, Amos Kong wrote: > This patchset fixed some issues of query-command-line-options: > * some new options that haven't argument can't be queried. (eg: -enable-fips) > * some legacy options that have argument can't be queried. (eg:

[Qemu-devel] [PATCH v6 3/3] query-command-line-options: query all the options in qemu-options.hx

2014-03-26 Thread Amos Kong
vm_config_groups[] only contains part of the options which have parameters, and all options which have no parameter aren't added to vm_config_groups[]. Current query-command-line-options only checks options from vm_config_groups[], so some options will be lost. We have macro in qemu-options.

[Qemu-devel] [PATCH v6 2/3] query-command-line-options: expose implicit parameter name

2014-03-26 Thread Amos Kong
This patch added a new field to expose implicit parameter name, we make it optional for compatibility. Suggested-by: Eric Blake Signed-off-by: Amos Kong --- qapi-schema.json | 6 +- util/qemu-config.c | 24 +++- 2 files changed, 20 insertions(+), 10 deletions(-) diff

[Qemu-devel] [PATCH v6 0/3] fix query-command-line-options

2014-03-26 Thread Amos Kong
This patchset fixed some issues of query-command-line-options: * some new options that haven't argument can't be queried. (eg: -enable-fips) * some legacy options that have argument can't be queried. (eg: -vnc display) More discussion: http://marc.info/?l=qemu-devel&m=

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-26 Thread Amos Kong
> > >>> On 03/05/2014 07:36 PM, Amos Kong wrote: > > >>>> vm_config_groups[] only contains part of the options which have > > >>>> argument, and all options which have no argument aren't added > > >>>> to vm_config_groups[]. Curr

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-26 Thread Amos Kong
gt; >> vm_config_groups[] only contains part of the options which have > >> >> argument, and all options which have no argument aren't added > >> >> to vm_config_groups[]. Current query-command-line-options only > >> >> checks options from vm_

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-26 Thread Markus Armbruster
t, and all options which have no argument aren't added >> >> to vm_config_groups[]. Current query-command-line-options only >> >> checks options from vm_config_groups[], so some options will >> >> be lost. >> >> >> >> We have macro in qemu

Re: [Qemu-devel] [PATCH v5 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-26 Thread Amos Kong
On Tue, Mar 11, 2014 at 06:46:10PM -0600, Eric Blake wrote: > On 03/06/2014 11:09 PM, Amos Kong wrote: > > vm_config_groups[] only contains part of the options which have > > parameters, and all options which have no parameter aren't added > > to vm_config_groups[].

Re: [Qemu-devel] [PATCH v23 03/32] qapi: output def_value_str when query command line options

2014-03-23 Thread Chun Yan Liu
I'll update. All patch series could also be available from: https://github.com/chunyanliu/qemu/commits/QemuOpts >>> On 3/22/2014 at 07:27 AM, in message <532ccadc.40...@redhat.com>, Eric Blake wrote: > On 03/21/2014 04:12 AM, Chunyan Liu wrote: > > Change qapi interfaces to output the newly add

Re: [Qemu-devel] [PATCH 01/26] qapi: output def_value_str when query command line options

2014-03-21 Thread Eric Blake
On 03/20/2014 06:13 PM, Leandro Dorileo wrote: > 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

Re: [Qemu-devel] [PATCH v23 03/32] qapi: output def_value_str when query command line options

2014-03-21 Thread Eric Blake
On 03/21/2014 04:12 AM, Chunyan Liu wrote: > Change qapi interfaces to output the newly added def_value_str when querying > command line options. > > Reviewed-by: Eric Blake > Signed-off-by: Dong Xu Wang > Signed-off-by: Chunyan Liu > --- > qapi-schema.json | 6 +- > qmp-commands.hx|

[Qemu-devel] [PATCH v23 03/32] qapi: output def_value_str when query command line options

2014-03-21 Thread Chunyan Liu
Change qapi interfaces to output the newly added def_value_str when querying command line options. Reviewed-by: Eric Blake 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 inser

[Qemu-devel] [PATCH 01/26] qapi: output def_value_str when query command line options

2014-03-20 Thread Leandro Dorileo
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

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-20 Thread Amos Kong
ions which have > > >> argument, and all options which have no argument aren't added > > >> to vm_config_groups[]. Current query-command-line-options only > > >> checks options from vm_config_groups[], so some options will > > >> be lost. > &g

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-20 Thread Amos Kong
fig_groups[] only contains part of the options which have > >>>> argument, and all options which have no argument aren't added > >>>> to vm_config_groups[]. Current query-command-line-options only > >>>> checks options from vm_config_groups[], so some opt

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-20 Thread Amos Kong
27;t added > >> to vm_config_groups[]. Current query-command-line-options only > >> checks options from vm_config_groups[], so some options will > >> be lost. > >> > >> We have macro in qemu-options.hx to generate a table that > >> contains a

Re: [Qemu-devel] [PATCH v5 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-11 Thread Eric Blake
On 03/06/2014 11:09 PM, Amos Kong wrote: > vm_config_groups[] only contains part of the options which have > parameters, and all options which have no parameter aren't added > to vm_config_groups[]. Current query-command-line-options only > checks options from vm_config_groups[]

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-11 Thread Eric Blake
So your idea of a tri-state (QemuOpts, no argument, or other argument) >> doesn't add anything - any option that takes "other argument" could be >> converted to take QemuOpts, and from the command line, we can't tell the >> difference from whether something

Re: [Qemu-devel] [PATCH v4 2/2] query-command-line-options: query all the options in qemu-options.hx

2014-03-11 Thread Markus Armbruster
ions which have no argument aren't added >>>> to vm_config_groups[]. Current query-command-line-options only >>>> checks options from vm_config_groups[], so some options will >>>> be lost. >>>> > >>Example: -device takes unspecified parameters.

Re: [Qemu-devel] [PATCH v22 02/25] qapi: output def_value_str when query command line options

2014-03-10 Thread Hu Tao
On Mon, Mar 10, 2014 at 03:31:38PM +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 Not a valid email address.

Re: [Qemu-devel] [PATCH v22 02/25] qapi: output def_value_str when query command line options

2014-03-10 Thread Eric Blake
On 03/10/2014 01:31 AM, 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 > --- > qapi-schema.json | 6 +- > qmp-commands.hx| 2 ++ > util/qemu-config.c

  1   2   >