On Fri, 2017-05-26 at 15:23 +1000, David Gibson wrote:
> Migrating between different CPU versions is a bit complicated for
> ppc.
> A long time ago, we ensured identical CPU versions at either end by
> checking the PVR had the same value. However, this breaks under KVM
> HV, because we always have
On Wed, May 31, 2017 at 12:39:25PM +0100, Dr. David Alan Gilbert wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
> > Everything else assumes that we always load a device from its own
> > savevm handler.
>
> Except for the title (it's the version_id you're removing not
> section_id) but now I
On Fri, 2017-05-26 at 15:23 +1000, David Gibson wrote:
> Server class POWER CPUs have a "compat" property, which is used to
> set the
> backwards compatibility mode for the processor. However, this only
> makes
> sense for machine types which don't give the guest access to
> hypervisor
> privilege
On Wed, May 31, 2017 at 11:25:41PM -0500, Michael Roth wrote:
> Quoting Bharata B Rao (2017-05-31 23:06:46)
> > On Thu, Jun 01, 2017 at 11:52:14AM +1000, David Gibson wrote:
> > > The code managing DRCs[0] has quite a few things that are more
> > > complicated than they need to be. In particular t
Wed, May 31, 2017 at 02:54:05PM +0200, Paolo Bonzini wrote:
>
>
> On 31/05/2017 08:28, Yang Zhong wrote:
> > Move the tcg related files into tcg/ subdirectory, which will make
> > the code more clean in qemu. Next step, we will base on those changes
> > to disable tcg.
>
> You should also move
On Wed, May 31, 2017 at 04:56:44PM +0530, Bharata B Rao wrote:
> Add a "no HPT" encoding (using value -1) to the HTAB migration
> stream (in the place of HPT size) when the guest doesn't allocate HPT.
> This will help the target side to match target HPT with the source HPT
> and thus enable success
On Thu, Jun 01, 2017 at 09:36:46AM +0530, Bharata B Rao wrote:
> On Thu, Jun 01, 2017 at 11:52:14AM +1000, David Gibson wrote:
> > The code managing DRCs[0] has quite a few things that are more
> > complicated than they need to be. In particular the object
> > representing a DRC has a bunch of met
Quoting Bharata B Rao (2017-05-31 23:06:46)
> On Thu, Jun 01, 2017 at 11:52:14AM +1000, David Gibson wrote:
> > The code managing DRCs[0] has quite a few things that are more
> > complicated than they need to be. In particular the object
> > representing a DRC has a bunch of method pointers, despi
On Thu, Jun 01, 2017 at 11:52:14AM +1000, David Gibson wrote:
> The code managing DRCs[0] has quite a few things that are more
> complicated than they need to be. In particular the object
> representing a DRC has a bunch of method pointers, despite the fact
> that there are currently no subclasses
On Wed, May 31, 2017 at 02:57:46PM +0200, Paolo Bonzini wrote:
> If msi_init fails, the thread has already been created and the
> mutex/condvar are not destroyed. Initialize everything only
> after the point where pci_edu_realize cannot fail.
>
> Reported-by: Markus Armbruster
> Cc: Peter Xu
>
On Wed, May 31, 2017 at 02:55:59PM +0200, Paolo Bonzini wrote:
>
>
> On 31/05/2017 11:50, Peter Xu wrote:
> > On Wed, May 31, 2017 at 05:42:34PM +0800, Peter Xu wrote:
> >> A whitelist for it does not really makes sense. Let's remove it and
> >> then we can introduce a blacklist when really neede
There are some places that binded "return path" with postcopy. Let's be
prepared for its usage even without postcopy. This patch mainly did this
on source side.
Signed-off-by: Peter Xu
---
v2
- s/s->to_dst_file/s->rp_state.from_dst_file/, also add some comment for
it [Dave]
---
migration/migra
Hi, Markus
On 05/31/2017 07:07 PM, Markus Armbruster wrote:
This is cleanup, not a bug fix, so:
pci: Clean up error checking in pci_add_capability()
Mao Zhongyi writes:
The return value of pci_add_capability2() is only 2 cases, positive
on success, nagetive on failure and set error mess
> From: Wang, Zhi A
> Sent: Tuesday, May 30, 2017 1:31 AM
>
> This patch series are aim to refine the IGD stolen memory support in QEMU.
>
> Some features of Intel HW relies on the stolen memory to work, but some of
> registers used to indicate the base address of stolen memory can only be
> prog
On Wed, May 31, 2017 at 08:33:01PM +0200, Juan Quintela wrote:
> "Dr. David Alan Gilbert" wrote:
> > * Peter Xu (pet...@redhat.com) wrote:
> >> There are some places that binded "return path" with postcopy. Let's be
> >> prepared for its usage even without postcopy. This patch mainly did this
> >>
Currently implementations of the RTAS calls related to DRCs are in
spapr_rtas.c. They belong better in spapr_drc.c - that way they're closer
to related code, and we'll be able to make some more things local.
spapr_rtas.c was intended to contain the RTAS infrastructure and core calls
that don't be
The DRConnectorClass includes a get_fdt method. However
* There's only one implementation, and there's only likely to ever be one
* Both callers are local to spapr_drc
* Each caller only uses one half of the actual implementation
So abolish get_fdt() entirely, and just open-code what we nee
These two methods only have one implementation, and the spec they're
implementing means any other implementation is unlikely, verging on
impossible.
So replace them with simple functions.
Signed-off-by: David Gibson
---
hw/ppc/spapr.c | 13 +++---
hw/ppc/spapr_drc.c | 61
The code managing DRCs[0] has quite a few things that are more
complicated than they need to be. In particular the object
representing a DRC has a bunch of method pointers, despite the fact
that there are currently no subclasses, and even if there were the
method implementations would be unlikely
DRConnectorClass has a set_configured method, however:
* There is only one implementation, and only ever likely to be one
* There's exactly one caller, and that's (now) local
* The implementation is very straightforward
So abolish the method entirely, and just open-code what we need. We als
On 05/30/2017 04:17 AM, Vladimir Sementsov-Ogievskiy wrote:
> It will be needed in following commits for persistent bitmaps.
> If bitmap is loaded from read-only storage (and we can't mark it
> "in use" in this storage) corresponding BdrvDirtyBitmap should be
> read-only.
>
> Signed-off-by: Vlad
On 05/19/2017 07:02 PM, John Snow wrote:
>
>
> On 05/03/2017 08:25 AM, Vladimir Sementsov-Ogievskiy wrote:
>> It will be needed in following commits for persistent bitmaps.
>> If bitmap is loaded from read-only storage (and we can't mark it
>> "in use" in this storage) corresponding BdrvDirtyBi
adj_len_to_page doesn't return the correct result when the address
is already page aligned and the length is bigger than a page. Fix that.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/mem_helper.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Ping.
-Original Message-
From: Qemu-devel [mailto:qemu-devel-bounces+zhi.a.wang=intel@nongnu.org] On
Behalf Of Zhi Wang
Sent: Monday, May 29, 2017 4:29 PM
To: qemu-devel@nongnu.org
Cc: Tian, Kevin ; daniel.vet...@ffwll.ch;
joonas.lahti...@linux.intel.com; zhen...@linux.intel.com; Ale
And at the same time make IPTE SMP aware.
Reviewed-by: Thomas Huth
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 2 +-
target/s390x/mem_helper.c | 21 +
target/s390x/translate.c | 6 +-
3 files changed, 19 insertions(+),
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 12
target/s390x/translate.c | 8
4 files changed, 23 insertions(+)
diff --git a/target/s390x/helper.h b/
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/insn-data.def | 3 +++
target/s390x/translate.c | 10 ++
2 files changed, 13 insertions(+)
diff --git a/target/s390x/insn-data.def b/target/s390x/insn-data.def
index 0f70acea5c..170b50ef2e 100644
--- a/targ
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 12
target/s390x/translate.c | 6 ++
4 files changed, 21 insertions(+)
diff --git a/target/s390x/helper.h b/target/s390x/helper.h
index ca78d1
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 4
target/s390x/mem_helper.c | 31 +++
target/s390x/translate.c | 8
4 files changed, 44 insertions(+)
diff --git a/tar
These functions differ from COMPARE by generating an exception for a
QNaN input. Use the non quiet version of floatXX_compare.
Signed-off-by: Aurelien Jarno
---
target/s390x/fpu_helper.c | 27 +++
target/s390x/helper.h | 3 +++
target/s390x/insn-data.def | 6 +
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 13 +
target/s390x/translate.c | 7 +++
4 files changed, 23 insertions(+)
diff --git a/target/s390x/helper.h b/target/s390x/helper.h
index c6fb
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 5 +
target/s390x/mem_helper.c | 37 +
target/s390x/translate.c | 8
4 files changed, 51 insertions(+)
diff --gi
There are multiple issues with the COMPARE LOGICAL LONG EXTENDED
instruction:
- The test between the two operands is inverted, leading to an inversion
of the cc values 1 and 2.
- The address and length of an operand continue to be decreased after
reaching the end of this operand. These values a
As MVCL and MVCLE only differ by their operands, use a common
do_mvcl helper. Optimize it calling fast_memmove and fast_memset.
Correctly write back addresses. Check that r1 and r2/r3 registers
are even.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/mem_helper.c
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 13 +
target/s390x/translate.c | 8
4 files changed, 24 insertions(+)
diff --git a/target/s390x/helper.h b
Improve fix_address to also handle the 24-bit mode. Rename fix_address
to wrap_address to better explain what is changed.
Replace the calls to get_address with x2 = 0 and b2 = 0 by
call to wrap_address, leading to the removal of this function. Rename
get_address_31fix into get_address.
Reviewed-b
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 13 +
target/s390x/translate.c | 8
4 files changed, 24 insertions(+)
diff --git a/target/s390x/helper.h b
This patchset tries to improve the s390x emulation by fixing and
improving some instructions. It implement some more instructions, from
the zArchitecture base or from the Extended-Translation Facility 2. The
last patch updates the maximum TCG CPU to z800, as the ETF2 and
Long-Displacement Facility
Reviewed-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Aurelien Jarno
---
target/s390x/translate.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/target/s390x/translate.c b/target/s390x/translate.c
index 97ca639a34..f7598184a6 100644
--- a/tar
Signed-off-by: Aurelien Jarno
---
target/s390x/cpu.h| 2 ++
target/s390x/mem_helper.c | 17 ++---
target/s390x/mmu_helper.c | 2 --
3 files changed, 12 insertions(+), 9 deletions(-)
diff --git a/target/s390x/cpu.h b/target/s390x/cpu.h
index 79235cfa45..ba3d50b8e0 100644
---
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 35 +++
target/s390x/translate.c | 16
4 files changed, 54 insertions(+)
diff -
As CLCL and CLCLE mostly differ by their operands, use a common do_clcl
helper. Another difference is that CLCL is not interruptible.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c
At the same time fix the TCG version of get_max_cpu_model to return the
maximum model like on KVM. Remove the ETF2 and long-displacement
facilities from the additional features as it is included in the z800.
Signed-off-by: Aurelien Jarno
---
target/s390x/cpu_models.c | 13 ++---
1 file c
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 49 --
target/s390x/translate.c | 18 +
4 files changed, 60 insertion
For that we need to make program_interrupt available to qemu-user.
Fortunately there is almost nothing to change as both kvm_enabled and
CONFIG_KVM evaluate to false in that case.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/in
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/insn-data.def | 3 +++
target/s390x/translate.c | 10 ++
2 files changed, 13 insertions(+)
diff --git a/target/s390x/insn-data.def b/target/s390x/insn-data.def
index f818437069..0f70acea5c 100644
--- a/targ
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/mem_helper.c | 37 ++---
1 file changed, 30 insertions(+), 7 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 2425bfc984..2113494983 100644
--- a/ta
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/mem_helper.c | 47 +--
1 file changed, 33 insertions(+), 14 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 2113494983..98a7aa22d3 1006
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 9 +
target/s390x/mem_helper.c | 45 +
target/s390x/translate.c | 30 ++
4 files changed, 85 insertions(+)
diff --
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 3 +++
target/s390x/mem_helper.c | 23 +++
target/s390x/translate.c | 9 +
4 files changed, 36 insertions(+)
diff --git a/target/s390
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 47 --
target/s390x/translate.c | 21 +
4 files changed, 65 inser
Signed-off-by: Aurelien Jarno
---
target/s390x/cpu_models.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
index e5e005a430..fc3cb25cc3 100644
--- a/target/s390x/cpu_models.c
+++ b/target/s390x/cpu_models.c
@@ -668,8 +668,10 @@ static v
Use a common helper with PACK ASCII as the differences are limited to
the stride of the source operand.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 30 +--
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 51 ++
target/s390x/translate.c | 17
4 files changed, 71 insertions
On 05/31/2017 11:56 AM, Maurício Almeida wrote:
> The Traffic of Virtio was 956 MB/s while e1000 Emulated Driver was 192
> MB/s, both using Packet size of 1518 Bytes.
The quick answer - yes, that's the expected results, because paravirt is
SOOO much more direct than fully-emulated. Everything do
Public bug reported:
Using qemu-kvm as shipped with Ubuntu 16.04, I cannot get a passed-
through USB Host Keyboard to work at boot-up using the Q35 platform.
My minimal example to verify this bug is the following:
qemu-system-x86_64 -M q35 -m 128 -cdrom mini.iso -usb -usbdevice
host:04ca:005a
On Wed, 2017-05-31 at 14:09 -0300, Maurício Almeida wrote:
> My name is Maurício, we are making a study about Kvm Paravirtualized
> and Full virtualized at our University, we are from University Federal
> of Santa Maria.
> Can you give us some papers about how the Kvm or Qemu executes the I/O
> Net
Hello Guys!
My name is Maurício, we are making a study about Kvm Paravirtualized and
Full virtualized at our University, we are from University Federal of Santa
Maria.
Can you give us some papers about how the Kvm or Qemu executes the I/O Network
with one emulated Device?
We executed some tests u
On 05/31/2017 04:43 AM, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini
> ---
> block/nbd-client.c | 30 +-
> 1 file changed, 9 insertions(+), 21 deletions(-)
>
> diff --git a/block/nbd-client.c b/block/nbd-client.c
> index 1e2952fdae..43e0292ac1 100644
> --- a/bl
On Sun, May 28, 2017 at 12:01:14PM +0530, Niels de Vos wrote:
> Add missing support for "preallocation=falloc" to the Gluster block
> driver. This change bases its logic on that of block/file-posix.c and
> removed the gluster_supports_zerofill() and qemu_gluster_zerofill()
> functions in favour of
On 05/31/2017 04:43 AM, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini
> ---
> block/vvfat.c | 8 +++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
Reviewed-by: Eric Blake
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.
On 05/31/2017 04:43 AM, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini
> ---
> block/vpc.c | 20 ++--
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> @@ -705,10 +700,13 @@ static int64_t coroutine_fn
> vpc_co_get_block_status(BlockDriverState *bs,
>
On 05/31/2017 04:43 AM, Paolo Bonzini wrote:
> The VirtualBox driver is using a mutex to order all allocating writes,
> but it is not protecting accesses to the bitmap because they implicitly
> happen under the AioContext mutex. Change this to use a CoRwlock
> explicitly.
>
> Signed-off-by: Paolo
On 05/31/2017 04:43 AM, Paolo Bonzini wrote:
> These functions are more efficient in the presence of contention.
> qemu_co_rwlock_downgrade also guarantees not to block, which may
> be useful in some algorithms too.
>
> Signed-off-by: Paolo Bonzini
> ---
> include/qemu/coroutine.h | 19 +++
Hi,
This series seems to have some coding style problems. See output below for
more information:
Message-id: 20170531135709.345-1-marcandre.lur...@redhat.com
Type: series
Subject: [Qemu-devel] [PATCH v2 00/45] qobject/qapi: add uint type
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total
On Wed, May 31, 2017 at 03:28:36PM -0400, John Snow wrote:
> To quiet patchew, add an assert for fwrite's return value.
>
> Signed-off-by: John Snow
> ---
> tests/ide-test.c | 8 ++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/tests/ide-test.c b/tests/ide-test.c
> in
We should properly forward the CPUID format bit. CPU model
detection on systems without IBC might select a model with a wrong
CPU type (esp. selecting the business class (BC) model on an enterprise
class (EC) machine). So let's improve that, too.
I am also preparing a patch to properly indicate th
Let's also properly forward that bit. It should always be set. I
verified it under z/VM, it seems to be always set there. For now,
zKVM guests never get that bit set when the CPU model is active.
The PoP mentiones, that z800 + z900 (HW generation 7) always set this
bit to 0, so let's take care of
Currently, under z/VM on a 0x2827, QEMU will detect a 0x2828 if no
IBC value is provided. QEMU will simply take the last model of that HW
generation, which happens to be the BC version.
Let's improve our search for that case by selecting the latest CPU
definition that matches the CPU type. This mi
To quiet patchew, add an assert for fwrite's return value.
Signed-off-by: John Snow
---
tests/ide-test.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tests/ide-test.c b/tests/ide-test.c
index 139ebc0..bfd79dd 100644
--- a/tests/ide-test.c
+++ b/tests/ide-test.c
@@
* Peter Xu (pet...@redhat.com) wrote:
> On Thu, May 25, 2017 at 09:28:57AM +0300, Alexey Perevalov wrote:
> > On 05/25/2017 02:30 AM, Peter Xu wrote:
> > >On Wed, May 24, 2017 at 03:16:23PM +0300, Alexey Perevalov wrote:
> > >>On 05/24/2017 03:01 PM, Peter Xu wrote:
> > >>>On Wed, May 24, 2017 at 1
Hi
On Tue, May 30, 2017 at 6:15 PM Anton Nefedov
wrote:
> Signed-off-by: Anton Nefedov
> ---
> tests/test-char.c | 68
> +++
> 1 file changed, 68 insertions(+)
>
> diff --git a/tests/test-char.c b/tests/test-char.c
> index ed6b18f..cd54f88 10
On Tue, May 30, 2017 at 6:15 PM Anton Nefedov
wrote:
> makes it possible to test the existing chardev-file
>
> Signed-off-by: Anton Nefedov
> ---
> tests/test-char.c | 127
> +-
> 1 file changed, 79 insertions(+), 48 deletions(-)
>
> diff --gi
On Tue, May 30, 2017 at 6:02 PM Anton Nefedov
wrote:
> makes it possible to test the existing chardev-udp
>
> Signed-off-by: Anton Nefedov
>
Reviewed-by: Marc-André Lureau
> ---
> tests/test-char.c | 58
> +++
> 1 file changed, 41 insertion
On Tue, May 30, 2017 at 5:59 PM Anton Nefedov
wrote:
> frontends should avoid accessing CharDriver struct where possible
>
> Signed-off-by: Anton Nefedov
> Reviewed-by: Vladimir Sementsov-Ogievskiy
>
Reviewed-by: Marc-André Lureau
> ---
> chardev/char.c | 5 +
> gdbstub.c
On Tue, May 30, 2017 at 6:06 PM Anton Nefedov
wrote:
> this is only not a problem if the test is last in a suite,
> otherwise it makes the following main_loop() calls to fail
>
> Signed-off-by: Anton Nefedov
>
Reviewed-by: Marc-André Lureau
> ---
> tests/test-char.c | 2 ++
> 1 file changed,
On Tue, May 30, 2017 at 6:13 PM Anton Nefedov
wrote:
> qemu_chr_fe_get_driver() is unsafe, frontends with hotswap support
> should not access CharDriver ptr directly as CharDriver might change.
>
> Signed-off-by: Anton Nefedov
> Reviewed-by: Vladimir Sementsov-Ogievskiy
>
Reviewed-by: Marc-And
Hi
On Tue, May 30, 2017 at 6:12 PM Anton Nefedov
wrote:
> This patch adds a possibility to change a char device without a frontend
> removal.
>
> 1. Ideally, it would have to happen transparently to a frontend, i.e.
> frontend would continue its regular operation.
> However, backends are not sta
On Tue, May 30, 2017 at 6:09 PM Anton Nefedov
wrote:
> parse function will be used by the following patch
>
> Signed-off-by: Anton Nefedov
> Reviewed-by: Vladimir Sementsov-Ogievskiy
>
Reviewed-by: Marc-André Lureau
---
> chardev/char.c | 70
> --
* Alexey Perevalov (a.pereva...@samsung.com) wrote:
> On 05/24/2017 02:33 PM, Peter Xu wrote:
> > On Wed, May 24, 2017 at 09:45:48AM +0300, Alexey wrote:
> >
> > [...]
> >
> > > > > return false;
> > > > > }
> > > > > -ioctl_mask = (__u64)1 << _UFFDIO_REGISTER |
> > > > > -
On 31/05/2017 18:24, Laurent Desnogues wrote:
> All source files have an explicit BSD license except tcg-runtime.h
> which has none and aarch64 files which are licensed under GPL. So we
> are already not following the rule expressed in LICENSE.
>
> tcg-runtime.c is BSD
> tci.c is GPL
> translat
Halil Pasic wrote:
> Although we have recently vmstatified the migration of some css
> infrastructure, for some css entities there is still state to be
> migrated left, because the focus was keeping migration stream
> compatibility (that is basically everything as-is).
>
> Let us add vmstate help
On 05/31/2017 11:55 AM, Vladimir Sementsov-Ogievskiy wrote:
> Functions nbd_negotiate_{read,write,drop_sync} were introduced in
> 1a6245a5b, when nbd_rwv (was nbd_wr_sync) was working through
> qemu_co_sendv_recvv (the path is nbd_wr_sync -> qemu_co_{recv/send} ->
> qemu_co_send_recv -> qemu_co_sen
On 05/31/2017 11:55 AM, Vladimir Sementsov-Ogievskiy wrote:
> Following commit will reuse it for nbd server too.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy
> ---
> nbd/client.c | 26 --
> nbd/common.c | 26 ++
> nbd/nbd-internal.h
On 05/31/2017 11:55 AM, Vladimir Sementsov-Ogievskiy wrote:
> Rename
> nbd_wr_syncv -> nbd_rwv
> read_sync -> nbd_read
> read_sync_eof -> nbd_read_eof
> write_sync -> nbd_write
> drop_sync -> nbd_drop
>
> 1. nbd_ prefix
>read_sync and write_sync are already shared, so it is good to h
On Wed, 31 May 2017 19:40:46 +0200
Claudio Imbrenda wrote:
> On Wed, 31 May 2017 19:06:29 +0200
> Greg Kurz wrote:
>
> > On Wed, 31 May 2017 18:51:06 +0200
> > Claudio Imbrenda wrote:
> > [...]
> > > > >
> > > > > This is strange. cpu_index() is defined as:
> > > > >
> > > > > static inlin
"Dr. David Alan Gilbert" wrote:
> * Peter Xu (pet...@redhat.com) wrote:
>> There are some places that binded "return path" with postcopy. Let's be
>> prepared for its usage even without postcopy. This patch mainly did this
>> on source side.
>>
>> Signed-off-by: Peter Xu
>> ---
>> standalone pat
Claudio Imbrenda writes:
> On Wed, 31 May 2017 19:06:29 +0200
> Greg Kurz wrote:
>
>> On Wed, 31 May 2017 18:51:06 +0200
>> Claudio Imbrenda wrote:
>> [...]
>> > > >
>> > > > This is strange. cpu_index() is defined as:
>> > > >
>> > > > static inline int cpu_index(CPUState *cpu)
>> > > > {
>>
Halil Pasic wrote:
> Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for
> flexibility (extending using subsections) and for fun.
>
> To achieve this we need to hack the config_vector, which is VirtIODevice
> (that is common virtio) state, in the middle of the VirtioCcwDevice sta
* Alexey Perevalov (a.pereva...@samsung.com) wrote:
> That tiny refactoring is necessary to be able to set
> UFFD_FEATURE_THREAD_ID while requesting features, and then
> to create downtime context in case when kernel supports it.
>
> Signed-off-by: Alexey Perevalov
> ---
> migration/migration.c
On Mon, May 29, 2017 at 11:25:03AM +0200, Markus Armbruster wrote:
> Eduardo Habkost writes:
> > On Tue, May 23, 2017 at 04:52:47PM +0200, Markus Armbruster wrote:
[...]
> >> > +ERROR_WHITELIST = [
> >> > + # Machines that won't work out of the box:
> >> > + # MACHINE
On Wed, 31 May 2017 19:06:29 +0200
Greg Kurz wrote:
> On Wed, 31 May 2017 18:51:06 +0200
> Claudio Imbrenda wrote:
> [...]
> > > >
> > > > This is strange. cpu_index() is defined as:
> > > >
> > > > static inline int cpu_index(CPUState *cpu)
> > > > {
> > > > #if defined(CONFIG_USER_ONLY)
> >
On 31.05.2017 19:10, Richard W.M. Jones wrote:
On Wed, May 31, 2017 at 06:54:33PM +0300, Pavel Butsykin wrote:
It is assumed that the user has already made a preparatory with the
image:
1. freeing space at the end of the image
2. reducing the last partition on the disk
3. rebuilding fs
Only aft
Claudio Imbrenda writes:
> On Wed, 31 May 2017 18:33:24 +0200
> Greg Kurz wrote:
>
>> On Wed, 31 May 2017 18:17:37 +0200
>> Claudio Imbrenda wrote:
>>
>> > On Wed, 31 May 2017 16:09:33 +0100
>> > Alex Bennée wrote:
>> >
>> > > The thread-id of 0 means any CPU but we then ignore the fact we
>>
The next commit is going to make alternate members of type 'str'
conflict with other scalar types. Would break a few test cases that
don't actually require 'str'. Flip them from 'str' to 'bool' or
'EnumOne'.
Signed-off-by: Markus Armbruster
Message-Id: <1495471335-23707-4-git-send-email-arm...@
The QObject input visitor can produce only finite numbers when its
input comes out of the JSON parser, because the the JSON parser
implements RFC 7159, which provides no syntax for infinity and NaN.
However, it can produce infinity and NaN when its input comes out of
keyval_parse(), because we par
Alternates are sum types like unions, but use the JSON type on the
wire / QType in QObject instead of an explicit tag. That's why we
require alternate members to have distinct QTypes.
The recently introduced keyval_parse() (commit d454dbe) can only
produce string scalars. The qobject_input_visit
It's already documented in keyval.c (commit 0ee9ae7), but visitor.h
can use a note, too.
Signed-off-by: Markus Armbruster
Message-Id: <1495471335-23707-3-git-send-email-arm...@redhat.com>
Reviewed-by: Eric Blake
Reviewed-by: Marc-André Lureau
---
include/qapi/visitor.h | 4
1 file changed
On Wed, 31 May 2017 18:51:06 +0200
Claudio Imbrenda wrote:
[...]
> > >
> > > This is strange. cpu_index() is defined as:
> > >
> > > static inline int cpu_index(CPUState *cpu)
> > > {
> > > #if defined(CONFIG_USER_ONLY)
> > > return cpu->host_tid;
> > > #else
> > > return cpu->cpu_index
Markus Armbruster writes:
> "Dr. David Alan Gilbert" writes:
>
>> I notice this pair of patches doesn't seem to have gone anywhere.
>> WHile it's labelled as a monitor fix, it's all QOM stuff, so I don't
>> think it should be going via me.
>
> It's somewhere between QOM and QemuOpts. Andreas, p
1 - 100 of 414 matches
Mail list logo