Hi Andrea,
We've implemented a demo of KVM live memory snapshot based on the userfaultfd
write-protect series in your private
tree(https://git.kernel.org/pub/scm/linux/kernel/git/andrea/aa.git/log/). We did
a little hack on that series to make the demo works.
Zhang had discussed with you about th
(Add Zhanghailiang and Gonglei)
Hi Andrea,
We've implemented a demo of KVM live memory snapshot based on the userfaultfd
write-protect series in your private
tree(https://git.kernel.org/pub/scm/linux/kernel/git/andrea/aa.git/log/). We did
a little hack on that series to make the demo works.
Zha
Hi Peter,
Here are the first set of OpenBIOS updates for 2.11. Please pull.
ATB,
Mark.
The following changes since commit fcea73709b966a7ded9efa7b106ea50c7fe9025c:
Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging
(2017-09-08 16:04:42 +0100)
are available in the g
On Mon, 11 Sep 2017 04:35:08 +0200
Thomas Huth wrote:
> On 10.09.2017 05:24, David Gibson wrote:
> > On Wed, Aug 30, 2017 at 03:59:07PM +0100, Peter Maydell wrote:
> >> On 30 August 2017 at 15:53, Philippe Mathieu-Daudé
> >> wrote:
> >>> On 08/30/2017 10:39 AM, Thomas Huth wrote:
> T
On Tue, 29 Aug 2017 20:13:36 +0200
Thomas Huth wrote:
> Broken with commit b4ba67d9a7025 ("libqos: Change PCI accessors to take
> opaque BAR handle") a while ago, but nobody noticed since the tests are
> only run in SPEED=slow mode: The msix_pba_bar is not correctly initialized
> anymore if bir_p
This commit introduces a new vhost-user device for block, it uses a
chardev to connect with the backend, same with Qemu virito-blk device,
Guest OS still uses the virtio-blk frontend driver.
To use it, start Qemu with command line like this:
qemu-system-x86_64 \
-chardev socket,id=char0,path=
Although virtio scsi specification was designed as a replacement for virtio_blk,
there are still many users using virtio_blk. Qemu 2.9 introduced a new device
vhost user scsi which can process I/O in user space for virtio_scsi, this commit
introduces a new vhost user block host device, which can su
Enable VHOST_USER_GET_CONFIG/VHOST_USER_SET_CONFIG/VHOST_USER_SET_CONFIG_FD
messages in libvhost-user library, users can implement their own I/O target
based on the library. This enable the virtio config space delivered between
Qemu host device and the I/O target, also event notifier is added in ca
Add VHOST_USER_GET_CONFIG/VHOST_USER_SET_CONFIG messages which can be
used for live migration for vhost user devices, also vhost user devices
can benefit from the messages to get/set virtio config space from/to the
I/O target besides Qemu. For the purpose to support virtio config space
change, VHOS
This commit introcudes a vhost-user-blk backend device, it uses UNIX
domain socket to communicate with Qemu. The vhost-user-blk sample
application should be used with Qemu vhost-user-blk-pci device.
To use it, complie with:
make vhost-user-blk
and start like this:
vhost-user-blk -b /dev/sdb -s /p
On 07/09/2017 11:20, Alexey Kardashevskiy wrote:
>
> /* Accessed via RCU. */
> struct FlatView *current_map;
>
> int ioeventfd_nb;
> struct MemoryRegionIoeventfd *ioeventfds;
> -struct AddressSpaceDispatch *dispatch;
> -struct AddressSpaceDispatch *next_dispatch;
>
On Sun, 10 Sep 2017 15:37:33 +0100
Mark Cave-Ayland wrote:
> Commit a90db15 "target-ppc: Convert ppc cpu savevm to VMStateDescription"
> appears to drop the internal CPU IRQ state from the migration stream. Whilst
> testing migration on g3beige/mac99 machines, test images would randomly fail
> t
On Fri, Sep 08, 2017 at 10:38:00AM -0700, Brandon Carpenter wrote:
> Keep pings and gratuitous pongs generated by web browsers from killing
> websocket connections.
>
> Signed-off-by: Brandon Carpenter
> ---
> io/channel-websock.c | 21 +
> 1 file changed, 17 insertions(+), 4
On Fri, Sep 08, 2017 at 10:38:01AM -0700, Brandon Carpenter wrote:
> Add an immediate ping reply (pong) to the outgoing stream when a ping
> is received. Unsolicited pongs are ignored.
>
> Signed-off-by: Brandon Carpenter
> ---
> io/channel-websock.c | 50
On Mon, Sep 11, 2017 at 09:38:46AM +0100, Daniel P. Berrange wrote:
> On Fri, Sep 08, 2017 at 10:38:00AM -0700, Brandon Carpenter wrote:
> > Keep pings and gratuitous pongs generated by web browsers from killing
> > websocket connections.
> >
> > Signed-off-by: Brandon Carpenter
> > ---
> > io/c
On 11/09/17 17:40, Paolo Bonzini wrote:
> On 07/09/2017 11:20, Alexey Kardashevskiy wrote:
>>
>> /* Accessed via RCU. */
>> struct FlatView *current_map;
>>
>> int ioeventfd_nb;
>> struct MemoryRegionIoeventfd *ioeventfds;
>> -struct AddressSpaceDispatch *dispatch;
>> -
Richard Henderson writes:
> On 09/08/2017 06:10 AM, Alex Bennée wrote:
>> Also this commit breaks RISU:
>>
>> qemu-aarch64 build/aarch64-linux-gnu/risu
>> testcases.aarch64/insn_ANDSi_RES8_ANDS_RES_ANDv_ASRV__INC.risu.bin \
>> -t
>> testcases.aarch64/insn_ANDSi_RES8_ANDS_RES_ANDv_ASRV_
Richard Henderson writes:
> On 09/07/2017 09:58 AM, Alex Bennée wrote:
>>> +switch (size + 4 * is_u) {
>>
>> Hmm I find this switch a little too magical. I mean I can see that the
>> encoding abuses size for the final opcode when I look at the manual but
>> it reads badly.
>>
>>> +case 0
On 30/08/2017 18:37, Stefan Hajnoczi wrote:
>
> The case statements asssume sz has a certain minimum value. I didn't
> see a check anywhere that guarantees this. It may be easier to hide the
> client's sz value and instead use sizeof(client->data). The caller can
> worry about sz.
Makes sense.
On 9/8/2017 2:34 PM, Markus Armbruster wrote:
Pradeep Jagadeesh writes:
On 9/8/2017 12:02 PM, Markus Armbruster wrote:
Pradeep Jagadeesh writes:
[...]
diff --git a/qmp.c b/qmp.c
index b86201e..eed91e5 100644
--- a/qmp.c
+++ b/qmp.c
@@ -130,6 +130,20 @@ void qmp_cpu_add(int64_t id, Error **
* Greg Kurz (gr...@kaod.org) wrote:
> On Sun, 10 Sep 2017 15:37:33 +0100
> Mark Cave-Ayland wrote:
>
> > Commit a90db15 "target-ppc: Convert ppc cpu savevm to VMStateDescription"
> > appears to drop the internal CPU IRQ state from the migration stream. Whilst
> > testing migration on g3beige/mac9
On Fri, 8 Sep 2017 17:24:43 +0200
Halil Pasic wrote:
> The architecture says that channel-data check is indicating that
> an uncorrected storage (memory) error has been detected in regard
> to the data residing in main storage (memory) that is currently
> used for an I/O operation. The described
On 11/09/2017 11:06, Alexey Kardashevskiy wrote:
> On 11/09/17 17:40, Paolo Bonzini wrote:
>> On 07/09/2017 11:20, Alexey Kardashevskiy wrote:
>>>
>>> /* Accessed via RCU. */
>>> struct FlatView *current_map;
>>>
>>> int ioeventfd_nb;
>>> struct MemoryRegionIoeventfd *ioeve
On Fri, 8 Sep 2017 17:24:44 +0200
Halil Pasic wrote:
> Back then in the time of df1fe5bb49 ("s390: Virtual channel subsystem
> support.", 2013-01-24) -EIO used to map to a channel-program check (via
> the default label of the switch statement). Then 2dc95b4cac
> ("s390x/3270: 3270 data stream h
On Fri, 8 Sep 2017 17:24:45 +0200
Halil Pasic wrote:
s/dubious/unused/
> The case in question actually never happens. Let us get rid of the dead
> code.
I had tried to be complete in my initial implementation. With the
current implementation, it can never happen, yes. We can easily
resurrect i
On 07/08/2017 10:56, Ladi Prosek wrote:
> This is the QEMU part of the changes required for nested Hyper-V to read
> timestamps with RDTSC + TSC page. Without exposing the frequency MSRs,
> Windows with the Hyper-V role enabled use the much slower
> HV_X64_MSR_TIME_REF_COUNT (0x4020) RDMSR to r
On 18/08/2017 20:47, Alistair Francis wrote:
> As discussed with Paolo and Markus let's combine libqemustub.a into
> libqemuutil.a to avoid circular dependencies.
>
> Alistair Francis (2):
> Makefile: Remove libqemustub.a
> Convert remaining single line fprintf() to warn_report()
>
> Makefil
On Fri, 8 Sep 2017 17:24:46 +0200
Halil Pasic wrote:
> We report incorrect length via SCSW program check instead of incorrect
> length check (SCWS word 2 bit 10 instead of bit 9). Since we have there
> is no fitting errno for incorrect length, and since I don't like what we
> do with the errno's
On 10 September 2017 at 17:18, Laurent Vivier wrote:
> The following changes since commit fcea73709b966a7ded9efa7b106ea50c7fe9025c:
>
> Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging
> (2017-09-08 16:04:42 +0100)
>
> are available in the git repository at:
>
> git:/
Added a qemu task, this seems to be a use case affected by the file locking.
In particular a (formerly working) case for multipath tests that use the same
image files multiple times intentionally.
So far the workaround is to set file.locking=off for all these, which might be
just the right thing
Thanks Ryan for filing this bug, as I said in IRC all the file locking is
rather new and I think this is for upstream qemu to address.
Might just be a missed use case for them as well.
I'll be adding an upstream qemu task to get their attention.
@Rayn - It would be nice if - from the multipath te
---
Greg Kurz (2):
spapr_pci: convert sprintf() to g_strdup_printf()
spapr_pci: don't create 64-bit MMIO window if we don't need to
hw/ppc/spapr_pci.c | 37 +++--
1 file changed, 23 insertions(+), 14 deletions(-)
In order to follow a QEMU common practice.
Signed-off-by: Greg Kurz
---
hw/ppc/spapr_pci.c | 21 -
1 file changed, 12 insertions(+), 9 deletions(-)
diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c
index 75cd9392233e..7d84b9766ed2 100644
--- a/hw/ppc/spapr_pci.c
+++ b/h
When running a pseries-2.2 or older machine type, we get the following
lines in info mtree:
address-space: memory
...
- (prio 0, i/o): alias
pci@8002000.mmio64-alias @p...@8002000.mmio
-
address-space: cpu-memory
.
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [Qemu-devel] [PATCH 0/2] spapr_pci: some more cleanup
Message-id: 150512483616.8642.15831240906912354238.st...@bahia.lan
Type: series
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
tot
On 10/09/2017 00:07, Eduardo Habkost wrote:
> On Fri, Sep 08, 2017 at 02:46:36PM +0200, David Hildenbrand wrote:
>> On 08.09.2017 06:21, Thomas Huth wrote:
>>> On 07.09.2017 22:13, David Hildenbrand wrote:
Implemented in sclp.c, so let's move it to the right include file.
Fix up one inclu
On 10/09/2017 19:15, Philippe Mathieu-Daudé wrote:
> and update maintainer email address
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> Hannes Reinecke replied:
> I'm currently on vacation with no internet access, but will be happy to
> answer your mail once I'm back at work on Sep, 12th.
>
On 07/09/2017 11:30, Thomas Huth wrote:
> On 13.07.2017 13:35, Paolo Bonzini wrote:
>> On 12/07/2017 07:52, Thomas Huth wrote:
>>> On 11.07.2017 20:55, Philippe Mathieu-Daudé wrote:
Signed-off-by: Philippe Mathieu-Daudé
---
same as Thomas previous patch. this file had no entry in MA
Dne 8.9.2017 v 17:55 Thomas Huth napsal(a):
> On 08.09.2017 13:54, Kevin Wolf wrote:
>> Am 08.09.2017 um 13:24 hat Cornelia Huck geschrieben:
>>> On Fri, 8 Sep 2017 13:04:25 +0200
>>> Kevin Wolf wrote:
>>>
Am 05.09.2017 um 17:16 hat Cornelia Huck geschrieben:
> The default cpu model on s3
On 06/09/2017 07:26, Thomas Huth wrote:
> You asked for opinions, so here's mine: I agree with you, please do
> *not* add a new QEMU-specific construct here. assert() should be a
> well-known C construct that every programmer should have understood. You
> also need it for other projects. If you hav
On Fri, Sep 08, 2017 at 01:49:41PM +0200, Markus Armbruster wrote:
> "Dr. David Alan Gilbert" writes:
>
> > * Markus Armbruster (arm...@redhat.com) wrote:
> >> "Dr. David Alan Gilbert" writes:
> >>
> >> > * Markus Armbruster (arm...@redhat.com) wrote:
> >> >> "Daniel P. Berrange" writes:
> >>
On 03/09/2017 18:33, Kamil Rytarowski wrote:
> SunOS declares struct queue in .
>
> This fixes build on SmartOS (Joyent).
>
> Patch cherry-picked from pkgsrc by jperkin (Joyent).
>
> Signed-off-by: Kamil Rytarowski
> ---
> memory.c | 22 +++---
> 1 file changed, 11 insertions(+
Setting the client architecture DHCP option to 0x001f (s390 Basic) [1]
allows the DHCP server to return a s390-specific bootfile if wanted.
DHCP servers not configured for the option (or not yet recognizing the
option value) will continue to work as they have done before.
[1] https://www.iana.org/
On Mon, Sep 11, 2017 at 06:32:03PM +0800, Peter Xu wrote:
[...]
> I think this OOB solution should work for us, though I'm still trying
> to digest this whole thing. Thanks Markus for this design, much
> appreciated. Meanwhile, sorry to have troubled you on this. I really
> didn't mean to!
>
>
On 11 September 2017 at 11:30, Paolo Bonzini wrote:
> On 06/09/2017 07:26, Thomas Huth wrote:
>> You asked for opinions, so here's mine: I agree with you, please do
>> *not* add a new QEMU-specific construct here. assert() should be a
>> well-known C construct that every programmer should have und
On Fri, Sep 08, 2017 at 01:49:41PM +0200, Markus Armbruster wrote:
> > I also see the other problem as keeping the management level
> > understanding of which commands are asynchronous; Dan's suggestion is
> > that command where the management layer specifies which commands it
> > expects to be asy
Fam Zheng writes:
> On Fri, 09/08 16:22, Alex Bennée wrote:
>>
>> Fam Zheng writes:
>>
>> > This is the common code to implement a "VM test" to
>> >
>> > 1) Download and initialize a pre-defined VM that has necessary
>> > dependencies to build QEMU and SSH access.
>> >
>> > 2) Archive $SR
On 09/11/2017 12:33 PM, Viktor Mihajlovski wrote:
> Setting the client architecture DHCP option to 0x001f (s390 Basic) [1]
> allows the DHCP server to return a s390-specific bootfile if wanted.
> DHCP servers not configured for the option (or not yet recognizing the
> option value) will continue to
On Sun, Sep 10, 2017 at 03:37:34PM +0100, Mark Cave-Ayland wrote:
> This is referenced in cpu_ppc_handle_mmu_fault() and so should be included
> in the migration stream.
That is not, on its own, sufficient reason.
> Note: the vmstate_ppc version number has already been bumped by the previous
> pa
On Mon, Sep 11, 2017 at 10:30:33AM +0100, Dr. David Alan Gilbert wrote:
> * Greg Kurz (gr...@kaod.org) wrote:
> > On Sun, 10 Sep 2017 15:37:33 +0100
> > Mark Cave-Ayland wrote:
> >
> > > Commit a90db15 "target-ppc: Convert ppc cpu savevm to VMStateDescription"
> > > appears to drop the internal C
On Mon, Sep 11, 2017 at 09:32:07AM +0200, Cornelia Huck wrote:
> On Tue, 29 Aug 2017 20:13:36 +0200
> Thomas Huth wrote:
>
> > Broken with commit b4ba67d9a7025 ("libqos: Change PCI accessors to take
> > opaque BAR handle") a while ago, but nobody noticed since the tests are
> > only run in SPEED=
On Tue, 5 Sep 2017 09:17:06 +0800
Dou Liyang wrote:
> Hi Eduardo,
>
> At 09/04/2017 09:08 PM, Eduardo Habkost wrote:
> [...]
> >> In my opinion, this may also add the hotpluggable memory, and see the
> >> following commemts.
> >>
> >> /*
> >> * Entry is required for Windows to enable me
Commit fd5d23babf (hmp: fix "dump-quest-memory" segfault)
fixes the problem for i386, do the same for ppc.
Running QEMU with
qemu-system-ppc64 -M none -nographic -m 256
and executing
dump-guest-memory /dev/null 0 8192
results in segfault
Fix by checking if we have CPU.
Signed-off-by: Lau
On Mon, 11 Sep 2017 03:17:45 -0700 (PDT)
no-re...@patchew.org wrote:
> Hi,
>
> This series seems to have some coding style problems. See output below for
> more information:
>
> Subject: [Qemu-devel] [PATCH 0/2] spapr_pci: some more cleanup
> Message-id: 150512483616.8642.15831240906912354238.st
On Mon, Sep 11, 2017 at 01:00:37PM +0200, Laurent Vivier wrote:
> Commit fd5d23babf (hmp: fix "dump-quest-memory" segfault)
> fixes the problem for i386, do the same for ppc.
What about all the other targets QEMU supports ? Have you checked if they
are similarly affected, as we don't want to wait
In order to clean-up some hacks in qapi (having to unregister commands
at runtime), I proposed a "[PATCH v5 02/20] qapi.py: add a simple #ifdef
condition"
(see http://lists.gnu.org/archive/html/qemu-devel/2016-08/msg03106.html).
However, we decided to drop that patch from the series and solve th
Replace the generated json string with a literal qobject. The later is
easier to deal with, at run time as well as compile time: adding #if
conditionals will be easier than in a json string.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-introspect.py | 83 ++--
Built-in objects remain unconditional. Explicitly defined objects
use the condition specified in the schema. Implicitly defined
objects inherit their condition from their users. For most of them,
there is exactly one user, so the condition to use is obvious. The
exception is the wrapper types g
Instanciate a QObject* form a literal QLitObject.
Signed-off-by: Marc-André Lureau
---
include/qapi/qmp/qlit.h | 2 ++
qobject/qlit.c | 36
tests/check-qlit.c | 26 ++
3 files changed, 64 insertions(+)
diff --git a/incl
Signed-off-by: Marc-André Lureau
---
scripts/qapi2texi.py | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py
index a317526e51..73cfb01727 100755
--- a/scripts/qapi2texi.py
+++ b/scripts/qapi2texi.py
@@ -136,10 +136,9 @@ def texi_
Signed-off-by: Marc-André Lureau
---
tests/Makefile.include | 3 +++
tests/qapi-schema/bad-if-empty-list.err | 1 +
tests/qapi-schema/bad-if-empty-list.exit | 1 +
tests/qapi-schema/bad-if-empty-list.json | 3 +++
tests/qapi-schema/bad-if-empty-list.out | 0
tests/qapi-schema/
The following patch is going to break list entries with #if/#endif, so
they should have the trailing ',' as suffix.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-introspect.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/scripts/qapi-introspect.py b/scripts/qapi-int
Accept 'if' key in top-level elements, accepted as string or list of
string type. The following patches will modify the test visitor to
check the value is correctly saved, and generate #if/#endif code (as a
single #if/endif line or a series for a list).
Example of 'if' key:
{ 'struct': 'TestIfStru
Add helpers to wrap generated code with #if/#endif lines.
Add a function decorator that will be used to wrap visitor methods.
The decorator will check if code was generated before adding #if/#endif
lines. Used in the following patches.
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py | 55 +
Modify the test visitor to check correct passing of values.
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py| 31 +--
scripts/qapi-commands.py | 2 +-
scripts/qapi-event.py | 2 +-
scripts/qapi-introspect.py
The generator now accepts (obj, condition) tuples to wrap generated
QLit objects for 'obj' with #if/#endif conditions.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-introspect.py | 7 +++
1 file changed, 7 insertions(+)
diff --git a/scripts/qapi-introspect.py b/scripts/qapi-introspect.p
Add 'ifcond' condition to top-level QLit objects.
to_qlit() handles the (obj, ifcond) tuples in previous patch.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-introspect.py | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/scripts/qapi-introspect.py b/
Signed-off-by: Marc-André Lureau
---
tests/Makefile.include | 3 +++
tests/qapi-schema/enum-dict-member-invalid.err | 1 +
tests/qapi-schema/enum-dict-member-invalid.exit | 1 +
tests/qapi-schema/enum-dict-member-invalid.json | 2 ++
tests/qapi-schema/enum-dict-membe
Skip preprocessor lines when adding indentation, since that would
likely result in invalid code.
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/qapi.py b/scripts/qapi.py
index f2b5a7e131..2a8e60e975 100644
--- a/sc
The C standard has the initial value at 0 and the subsequent values
incremented by 1. No need to set this explicitely.
This will prevent from artificial "gaps" when compiling out some enum
values and having unnecessarily large MAX values & enums arrays.
Signed-off-by: Marc-André Lureau
---
scri
Generate variants objects outside gen_object(). This will allow to
easily wrap gen_object() with ifcond_decorator in the following patch.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-types.py | 37 +++--
1 file changed, 23 insertions(+), 14 deletions(-)
diff
The documentation is generated only once, and doesn't know C
pre-conditions. Add 'If:' sections for top-level entities.
Signed-off-by: Marc-André Lureau
---
scripts/qapi2texi.py | 24 +---
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/scripts/qapi2texi.py b/
Wrap generated code with #if/#endif using the ifcond_decorator.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-commands.py | 2 ++
tests/test-qmp-commands.c | 4 ++--
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/scripts/qapi-commands.py b/scripts/qapi-commands.py
index 669a
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py | 4
1 file changed, 4 insertions(+)
diff --git a/scripts/qapi.py b/scripts/qapi.py
index 1535de9ce7..df2a304e8f 100644
--- a/scripts/qapi.py
+++ b/scripts/qapi.py
@@ -1989,11 +1989,13 @@ const QEnumLookup %(c_name)s_lookup = {
''',
Signed-off-by: Marc-André Lureau
---
scripts/qapi2texi.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py
index 7327245a60..bb819856ef 100755
--- a/scripts/qapi2texi.py
+++ b/scripts/qapi2texi.py
@@ -138,9 +138,10 @@ def texi_membe
This will allow to add and access more properties associated with enum
values/members, like the associated 'if' condition. We may want to
have a specialized type QAPISchemaEnumMember, for now this will do.
Suggested-by: Markus Armbruster
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py
Wrap generated code with #if/#endif using the ifcond_decorator.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-event.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/scripts/qapi-event.py b/scripts/qapi-event.py
index dda496e824..bef301dfe9 100644
--- a/scripts/qapi-event.py
+++ b/scr
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py | 9 -
tests/qapi-schema/qapi-schema-test.json | 6 +-
tests/qapi-schema/qapi-schema-test.out | 8 +++-
3 files changed, 20 insertions(+), 3 deletions(-)
diff --git a/scripts/qapi.py b/scripts/qapi.py
i
Add #if defined(CONFIG_REPLICATION) in generated code, and adjust the
code accordingly.
Made conditional:
* xen-set-replication, query-xen-replication-status,
xen-colo-do-checkpoint
Before the patch, we first register the commands unconditionally in
generated code (requires a stub), then c
Add condition to QAPIEvent enum members based on the event 'if'.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-event.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/qapi-event.py b/scripts/qapi-event.py
index 38f4264817..60c6f7030d 100644
--- a/scripts/qapi-eve
Types & visitors are coupled and must be handled together to avoid
temporary build regression.
Wrap generated types/visitor code with #if/#endif using the
ifcond_decorator & helpers.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-types.py | 18 ++
scripts/qapi-visit.py | 5 +
This will let the caller add several list of commands.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-commands.py | 2 --
monitor.c | 1 +
qga/main.c| 1 +
tests/test-qmp-commands.c | 1 +
4 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/scripts/
check_type() will now accept a DICT { 'type': TYPENAME, 'if': ... }
instead of a TYPENAME. This is the case in various situations where
implicit object types are allowed such as commands/events arguments
and return type, base and branches of union & alternate.
Signed-off-by: Marc-André Lureau
---
The following patches are going to introduce per-target #ifdef, and
but the introspection data is generated only once, and must thus be
built with the target.
Drop "do_test_visitor_in_qmp_introspect(&&qmp_schema_qlit)" since it
is no longer in a common object, and covered by "query-qmp-schema
test
This schema is going to contain target-specific commands/events &
types, that can be conditionnally guarded with poisoned defines. To
filter it out by default, set the unit name to 'target'.
And new rules to compile this unit generated files per-target.
Use the "-u all" options for the introspect
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py | 39 -
tests/Makefile.include | 1 -
tests/qapi-schema/enum-dict-member.err | 1 -
tests/qapi-schema/enum-dict-member.exit | 1 -
tests/qapi-schema/enum-dict-member.j
Signed-off-by: Marc-André Lureau
---
scripts/qapi.py | 15 ++-
tests/qapi-schema/qapi-schema-test.json | 7 ++-
tests/qapi-schema/qapi-schema-test.out | 8
tests/qapi-schema/test-qapi.py | 5 -
4 files changed, 28 insertions(+), 7
Signed-off-by: Marc-André Lureau
---
qapi/target.json | 6 ++
1 file changed, 6 insertions(+)
diff --git a/qapi/target.json b/qapi/target.json
index 3b73a9202f..12002b54d5 100644
--- a/qapi/target.json
+++ b/qapi/target.json
@@ -77,6 +77,9 @@
# an unknown cpu definition name, unkno
It depends on TARGET_PPC || TARGET_ARM || TARGET_I386 || TARGET_S390X.
Signed-off-by: Marc-André Lureau
---
qapi-schema.json| 11 ---
qapi/target.json| 12
include/sysemu/arch_init.h | 1 -
monitor.c | 22 --
qm
Signed-off-by: Marc-André Lureau
---
tests/Makefile.include| 2 ++
tests/qapi-schema/struct-if-invalid.err | 1 +
tests/qapi-schema/struct-if-invalid.exit | 1 +
tests/qapi-schema/struct-if-invalid.json | 3 +++
tests/qapi-schema/struct-if-invalid.out | 0
tests/qap
Signed-off-by: Marc-André Lureau
---
qapi-schema.json | 38 -
qapi/target.json | 39 ++
include/sysemu/arch_init.h | 3 ---
monitor.c | 3 ---
qmp
Signed-off-by: Marc-André Lureau
---
scripts/qapi2texi.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py
index 150e7045d2..7327245a60 100755
--- a/scripts/qapi2texi.py
+++ b/scripts/qapi2texi.py
@@ -130,7 +130,9 @@ def texi_body(
Signed-off-by: Marc-André Lureau
---
scripts/qapi-introspect.py | 2 ++
scripts/qapi-types.py | 2 ++
scripts/qapi-visit.py | 2 ++
3 files changed, 6 insertions(+)
diff --git a/scripts/qapi-introspect.py b/scripts/qapi-introspect.py
index 32a58cf879..0e43e470bb 100644
--- a/scripts/qa
The correct way to query whether file locking is supported is 'query-
qmp-schema', which will expose the 'locking' option for the 'file'
branch of the 'blockdev-add' command then.
As a first comment, in your setup, completely disabling file locking is
probably a too big hammer. It is preferable to
Signed-off-by: Marc-André Lureau
---
qapi-schema.json | 43 ---
qapi/target.json | 45 +
monitor.c| 11 ---
target/arm/monitor.c | 2 +-
4 files changed, 46 insertions(+), 55 deletions
Add #if defined(CONFIG_SPICE) in generated code, and adjust the
qmp/hmp code accordingly.
Signed-off-by: Marc-André Lureau
---
qapi/char.json | 10 ++
qapi/ui.json | 30 --
monitor.c | 3 ---
qmp.c | 16
4 files changed, 26 in
Signed-off-by: Marc-André Lureau
---
scripts/qapi-introspect.py | 3 ++-
scripts/qapi-types.py | 2 ++
scripts/qapi-visit.py | 4
3 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/scripts/qapi-introspect.py b/scripts/qapi-introspect.py
index 0e43e470bb..ef2d5577db 10064
On 11/09/2017 13:06, Daniel P. Berrange wrote:
> On Mon, Sep 11, 2017 at 01:00:37PM +0200, Laurent Vivier wrote:
>> Commit fd5d23babf (hmp: fix "dump-quest-memory" segfault)
>> fixes the problem for i386, do the same for ppc.
>
> What about all the other targets QEMU supports ? Have you checked i
* Peter Xu (pet...@redhat.com) wrote:
> On Thu, Aug 24, 2017 at 08:27:24PM +0100, Dr. David Alan Gilbert (git) wrote:
> > From: "Dr. David Alan Gilbert"
> >
> > This message is sent just before the end of postcopy to get the
> > client to stop using userfault since we wont respond to any more
> >
Mostly covered by previous patches already.
Signed-off-by: Marc-André Lureau
---
scripts/qapi-introspect.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/qapi-introspect.py b/scripts/qapi-introspect.py
index ef2d5577db..d6194ff702 100644
--- a/scripts/qapi-introspec
1 - 100 of 440 matches
Mail list logo
