Hello Edgar,
On 07/09/2016 18:02, Edgar E. Iglesias wrote:
On Wed, Sep 07, 2016 at 05:39:08PM +0200, Christian Pinto wrote:
On 07/09/2016 09:51, Edgar E. Iglesias wrote:
On Wed, Sep 07, 2016 at 09:24:39AM +0200, Christian Pinto wrote:
Hello Edgar,
thanks for your comments.
Thanks for the
On Wed, Sep 07, 2016 at 05:20:48PM +0200, Ladi Prosek wrote:
> From: Stefan Hajnoczi
>
> virtqueue_discard() requires a VirtQueueElement but virtio-balloon does
> not migrate its in-use element. Introduce a new function that is
> similar to virtqueue_discard() but doesn't require a VirtQueueElem
Hi All,
As discussed before, upstream are working about PCIE instead of PCI in
AArch64.
Thanks for your efforts about this on AArch64 :-)
If it convenient, could you tell me when we plan to finish this task ?
and which qemu version will support this functions in the future?
Big Thanks
Greg Kurz writes:
> Calling assert() really makes sense when hitting a genuine bug, which calls
> for a fix in QEMU. However, when something goes wrong because the guest
> sends a malformed message, it is better to write down a more meaningul
> error message and exit.
>
> Signed-off-by: Greg Kurz
Signed-off-by: Gerd Hoffmann
---
hw/display/virtio-vga.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/hw/display/virtio-vga.c b/hw/display/virtio-vga.c
index 5b510a1..f77b401 100644
--- a/hw/display/virtio-vga.c
+++ b/hw/display/virtio-vga.c
@@ -122,6 +122,17 @@ static void vi
Greg Kurz writes:
> From: Greg Kurz
>
> Without presuming if we got there because of a user mistake or some
> more subtle bug in the tooling, it really does not make sense to
> implement a non-functional device.
>
> Signed-off-by: Greg Kurz
> Reviewed-by: Marcel Apfelbaum
> Reviewed-by: Cornel
On 09/07/2016 11:48 PM, Benjamin Herrenschmidt wrote:
> On Wed, 2016-09-07 at 14:13 +0200, Cédric Le Goater wrote:
>> On 09/07/2016 01:08 PM, Benjamin Herrenschmidt wrote:
>>>
>>> On Wed, 2016-09-07 at 12:50 +0200, Cédric Le Goater wrote:
This is a bit broader than Ben's patch which used
We can't hotplug display adapters in qemu, tag virtio-gpu-pci
accordingly (virtio-vga already has this).
Signed-off-by: Gerd Hoffmann
---
hw/display/virtio-gpu-pci.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/display/virtio-gpu-pci.c b/hw/display/virtio-gpu-pci.c
index 34a724c..ef92c
Make use of memory barrier TCG opcode in MIPS front end.
Signed-off-by: Leon Alrae
---
This patch complements the following series:
https://lists.nongnu.org/archive/html/qemu-devel/2016-07/msg03283.html
---
target-mips/translate.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff -
Hi,
> I had understood that the xhci could be a legacy PCI device or a PCI
> Express device depending on the socket it was plugged into (or was that
> possibly just someone doing some hand-waving over the fact that
> obscuring the PCI Express capabilities effectively turns it into a
> legacy
On 09/08/2016 12:56 AM, Alex Williamson wrote:
> On Wed, 07 Sep 2016 14:42:58 +0800
> Jike Song wrote:
>
>> On 09/07/2016 11:38 AM, Neo Jia wrote:
>>> On Wed, Sep 07, 2016 at 10:22:26AM +0800, Jike Song wrote:
On 09/02/2016 11:03 PM, Alex Williamson wrote:
> On Fri, 2 Sep 2016 16:1
Hi,
Your series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.
Subject: [Qemu-devel] [PATCH] virtio-vga: adapt to page-per-vq=off
Type: series
Message-id: 1473319012-27560-1-git-send-email-
On Thu, Sep 8, 2016 at 3:33 AM, Eric Blake wrote:
> On 09/06/2016 08:39 AM, Ashijeet Acharya wrote:
>> Mark old-commands for speed and downtime as deprecated.
>> Move max-bandwidth and downtime-limit into migrate-set-parameters for
>> setting maximum migration speed and expected downtime limit par
On 08/09/2016 04:04, David Gibson wrote:
> On Tue, Sep 06, 2016 at 03:17:56PM +0200, Laurent Vivier wrote:
>> And add support for ppc64.
>>
>> Signed-off-by: Laurent Vivier
>
> Some of my coments may be obsoleted by the discussion with Greg.
>
>> ---
>> v2:
>> - remove useless parenthesis, inl
On Tue, 6 Sep 2016 15:17:55 +0200
Laurent Vivier wrote:
> Signed-off-by: Laurent Vivier
> ---
FWIW,
Reviewed-by: Greg Kurz
> v4:
> - add this patch in the series to change all strtoXX() in qtest.c
>
> qtest.c | 49 ++---
> 1 file changed, 26 ins
On 08.09.2016 09:32, Cédric Le Goater wrote:
> From: Benjamin Herrenschmidt
>
> Power ISA 2.x has deleted the rfi instruction and rfid shoud be used
> instead on cpus following this instruction set or later.
>
> This will raise an invalid exception when rfi is used on such
> processors: Book3S 6
On Thu, Sep 8, 2016 at 8:44 AM, Roman Kagan wrote:
> On Wed, Sep 07, 2016 at 05:20:48PM +0200, Ladi Prosek wrote:
>> From: Stefan Hajnoczi
>>
>> virtqueue_discard() requires a VirtQueueElement but virtio-balloon does
>> not migrate its in-use element. Introduce a new function that is
>> similar
Hi,
> > Good point, maybe libvirt can avoid adding switches unless the user
> > explicitly
> > asked for them. I checked and it a actually works fine in QEMU.
> So, *is* there any downside to doing this?
I don't think so.
The only issue I can think of when it comes to multifunction is hotplug
From: Benjamin Herrenschmidt
Power ISA 2.x has deleted the rfi instruction and rfid shoud be used
instead on cpus following this instruction set or later.
This will raise an invalid exception when rfi is used on such
processors: Book3S 64-bit processors.
Signed-off-by: Benjamin Herrenschmidt
R
On Wed, 7 Sep 2016 16:36:12 -0500
Eric Blake wrote:
> On 09/07/2016 07:39 AM, Pradeep wrote:
> > Uses throttling APIs to limit I/O bandwidth and number of operations on the
> > devices which use 9p-local driver.
> >
> > Signed-off-by: Pradeep
> > fsdev/file-op-9p.h | 3 +
> > fsdev/qem
On Wed, Sep 7, 2016 at 5:54 PM, Stefan Hajnoczi wrote:
> On Wed, Sep 7, 2016 at 11:51 AM, Stefan Hajnoczi wrote:
>> vq->inuse must be zeroed upon device reset like most other virtqueue
>> fields.
>>
>> In theory, virtio_reset() just needs assert(vq->inuse == 0) since
>> devices must clean up in-f
On 09/08/2016 05:28 AM, Pranith Kumar wrote:
> FYI,
>
> The KVM Forum 2016 videos are now online on youtube. You can find them here:
>
> https://www.youtube.com/playlist?list=PLW3ep1uCIRfzQoZ0SlniYE8nz1ZRobjH7
The videos are not complete yet, some more still to come as far as I can see.
At 2016-09-03 16:44:47, "Chen Hanxiao" wrote:
>From: Chen Hanxiao
>
>If backend(such as dpdk) lack this feature,
>don't assume it and mark it in vring_enable.
>Or we may fail in vhost_net_start,
>then we can't use vhost net.
>This will bring compat issue with old version backend.
>
>Signed-off-
On Wed, 7 Sep 2016 11:51:25 -0400
Stefan Hajnoczi wrote:
> vq->inuse must be zeroed upon device reset like most other virtqueue
> fields.
>
> In theory, virtio_reset() just needs assert(vq->inuse == 0) since
> devices must clean up in-flight requests during reset (requests cannot
> not be leake
On Wed, Sep 07, 2016 at 05:23:09PM -0400, Stefan Hajnoczi wrote:
> Add a parameter to associate an iSCSI block driver instance with an -iscsi
> object. This is more powerful than relying on the implicit target IQN naming
> convention since that only allows one -iscsi object per target.
I'm inclin
Define spapr_alloc_init()/spapr_alloc_init_flags()/spapr_alloc_uninit()
to allocate and use SPAPR guest memory
Define qtest_spapr_vboot()/qtest_spapr_boot()/qtest_spapr_shutdown()
to start SPAPR guest with QOSState initialized for it (memory management)
Move qtest_irq_intercept_in() from ge
This series allows to call RTAS commands from the qtest framework,
and defines a first test to call RTAS command "get-time-of-day"
to validate the protocol and test RTAS.
RTAS command parameters are passed to the guest via the
guest memory, so we also need to implement the guest memory
management
Check the result of qemu_strtoXX() and assert
if the string cannot be converted.
Signed-off-by: Laurent Vivier
Reviewed-by: David Gibson
---
v5:
- update log message about result checking
- add David's Rb
v4:
- add this patch in the series to change all strtoXX() in qtest.c
qtest.c | 49 +
On 08/25/2016 11:53 AM, Kirti Wankhede wrote:
> Design for Mediated Device Driver:
> Main purpose of this driver is to provide a common interface for mediated
> device management that can be used by different drivers of different
> devices.
>
> This module provides a generic interface to create th
The series first restore the GET_FEATURES call in vhost_set_mem_table(),
as the hang it introduced was due to the use of TCG in the vhost-user-test,
mode which is no more used since commit cdafe929 ("vhost-user-test: Use libqos
instead of pxe-virtio.rom").
Second patch aims at being as close as po
The goal of this patch is to only request a sync (reply_ack,
or get_features) in set_mem_table only when necessary.
It should not be necessary the first time we set the table,
or when we add a new regions which hadn't been merged with an
existing ones.
Suggested-by: Michael S. Tsirkin
Cc: Prerna
Add a first test to validate the protocol:
- rtas/get-time-of-day compares the time
from the guest with the time from the host.
Signed-off-by: Laurent Vivier
---
v5:
- use qtest_spapr_boot() instead of machine_alloc_init()
v4:
- use qemu_strtoXXX() instead strtoXX()
v3:
- use mktimegm() inst
Richard Henderson writes:
> This is really just a placeholder for an actual
> command-line switch for mttcg.
> ---
> translate-all.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/translate-all.c b/translate-all.c
> index 99ae7f9..a10fa06 100644
> --- a/translate-all.c
On 09/07/2016 11:53 PM, Benjamin Herrenschmidt wrote:
> On Wed, 2016-09-07 at 17:47 +0200, Cédric Le Goater wrote:
>>
>> +static uint64_t pnv_lpc_xscom_mr_read(void *opaque, hwaddr addr,
>> unsigned size)
>> +{
>> +XScomDevice *xd = XSCOM_DEVICE(opaque);
>> +uint64_t val = 0;
>> +
>> +
This reverts commit 94c9cb31c04737f86be29afefbff401cd23bc24d.
Analysis of the race shows that it would happen only when QEMU relies
on TCG.
Since commit cdafe929 ("vhost-user-test: Use libqos instead of
pxe-virtio.rom"), vhost-user-test don't use TCG, so the race no more
appear.
Cc: Michael S. T
On Thu, Sep 08, 2016 at 10:43:19AM +0200, Kevin Wolf wrote:
> Am 08.09.2016 um 00:32 hat ashish mittal geschrieben:
> > >> +
> > >> +void vxhs_set_acb_buffer(void *ptr, void *buffer)
> > >> +{
> > >> +VXHSAIOCB *acb = ptr;
> > >> +
> > >> +acb->buffer = buffer;
> > >> +}
> > >> +
> > >
> >
Am 08.09.2016 um 00:32 hat ashish mittal geschrieben:
> >> +
> >> +void vxhs_set_acb_buffer(void *ptr, void *buffer)
> >> +{
> >> +VXHSAIOCB *acb = ptr;
> >> +
> >> +acb->buffer = buffer;
> >> +}
> >> +
> >
> > Unused function?
>
> This is called from within libqnio.
Wait, you mean the li
Allows one to specify a destroy function for the test data.
Add a fallback using glib g_test_add_vtable() internal function, whose
signature changed over time. Tested with glib 2.22, 2.26 and 2.48, which
according to git log should be enough to cover all variations.
Signed-off-by: Marc-André Lure
Richard Henderson writes:
> When we cannot emulate an atomic operation within a parallel
> context, this exception allows us to stop the world and try
> again in a serial context.
>
> Signed-off-by: Richard Henderson
> ---
> cpu-exec-common.c | 6 +
> cpu-exec.c | 23 ++
On Fri, Aug 19, 2016 at 08:40:32PM +0100, Paul Burton wrote:
> On 19/08/16 20:25, no-re...@patchew.org wrote:
> > Hi,
> >
> > Your series failed automatic build test. Please find the testing commands
> > and
> > their output below. If you have docker installed, you can probably
> > reproduce it
On Wed, 07 Sep 2016 19:19:24 +0200
Greg Kurz wrote:
> Calling assert() really makes sense when hitting a genuine bug, which calls
> for a fix in QEMU. However, when something goes wrong because the guest
> sends a malformed message, it is better to write down a more meaningul
> error message and
On 08/09/2016 07:43, Stefan Weil wrote:
> Am 07.09.2016 um 23:29 schrieb Peter Maydell:
>> On 7 September 2016 at 20:13, Stefan Weil wrote:
>>> The old log messages are implemented by conditional compilation
>>> and not available by default.
>>>
>>> The new log messages can be enabled either by
On Thu, Sep 08, 2016 at 11:29:41AM +0200, Kevin Wolf wrote:
> Am 08.09.2016 um 10:49 hat Daniel P. Berrange geschrieben:
> > On Thu, Sep 08, 2016 at 10:43:19AM +0200, Kevin Wolf wrote:
> > > Am 08.09.2016 um 00:32 hat ashish mittal geschrieben:
> > > > >> +
> > > > >> +void vxhs_set_acb_buffer(void
On Thu, Sep 8, 2016 at 11:29 AM Gerd Hoffmann wrote:
> We can't hotplug display adapters in qemu, tag virtio-gpu-pci
> accordingly (virtio-vga already has this).
>
> Signed-off-by: Gerd Hoffmann
>
Reviewed-by: Marc-André Lureau
---
> hw/display/virtio-gpu-pci.c | 1 +
> 1 file changed, 1 in
On Thu, 8 Sep 2016 10:59:26 +0200
Cornelia Huck wrote:
> On Wed, 07 Sep 2016 19:19:24 +0200
> Greg Kurz wrote:
>
> > Calling assert() really makes sense when hitting a genuine bug, which calls
> > for a fix in QEMU. However, when something goes wrong because the guest
> > sends a malformed mess
Jhash will be used by colo-compare and filter-rewriter
to save and lookup net connection info
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
include/qemu/jhash.h | 59
net/colo.h | 1 +
2 files
On Thu, 08 Sep 2016 09:14:05 +0200
Markus Armbruster wrote:
> Greg Kurz writes:
>
> > Calling assert() really makes sense when hitting a genuine bug, which calls
> > for a fix in QEMU. However, when something goes wrong because the guest
> > sends a malformed message, it is better to write down
COLO-proxy is a part of COLO project. COLO project is
composed of COLO-frame, COLO-proxy and block-replication.
It is used to compare the network package to help COLO
decide whether to do checkpoint. With COLO-proxy's help,
COLO greatly improves the performance.
The filter-redirector, filter-mirro
We will rewrite tcp packet secondary received and sent.
When colo guest is a tcp server.
Firstly, client start a tcp handshake. the packet's seq=client_seq,
ack=0,flag=SYN. COLO primary guest get this pkt and mirror(filter-mirror)
to secondary guest, secondary get it use filter-redirector.
Then,pr
On Wed, Sep 07, 2016 at 04:41:54PM +1000, David Gibson wrote:
> On Wed, Sep 07, 2016 at 02:34:19PM +0800, Peter Xu wrote:
> > On Wed, Sep 07, 2016 at 03:44:19PM +1000, David Gibson wrote:
> > > > For "CHANGE", it sounds like a unmap() + a map(). However I'd say
> > > > "ADDITION" is nowhere better.
On Thu, 08 Sep 2016 09:15:28 +0200
Markus Armbruster wrote:
> Greg Kurz writes:
>
> > From: Greg Kurz
> >
> > Without presuming if we got there because of a user mistake or some
> > more subtle bug in the tooling, it really does not make sense to
> > implement a non-functional device.
> >
> >
On Thu, Sep 08, 2016 at 11:29:41AM +0200, Kevin Wolf wrote:
> Am 08.09.2016 um 10:49 hat Daniel P. Berrange geschrieben:
> > On Thu, Sep 08, 2016 at 10:43:19AM +0200, Kevin Wolf wrote:
> > > Am 08.09.2016 um 00:32 hat ashish mittal geschrieben:
> > > > >> +
> > > > >> +void vxhs_set_acb_buffer(void
Am 08.09.2016 um 11:43 hat Daniel P. Berrange geschrieben:
> On Thu, Sep 08, 2016 at 11:29:41AM +0200, Kevin Wolf wrote:
> > Am 08.09.2016 um 10:49 hat Daniel P. Berrange geschrieben:
> > > On Thu, Sep 08, 2016 at 10:43:19AM +0200, Kevin Wolf wrote:
> > > > Am 08.09.2016 um 00:32 hat ashish mittal
If primary packet is same with secondary packet,
we will send primary packet and drop secondary
packet, otherwise notify COLO frame to do checkpoint.
If primary packet comes but secondary packet does not,
after REGULAR_PACKET_CHECK_MS milliseconds we set
the primary packet as old_packet,then do a c
We add TCP,UDP,ICMP packet comparison to replace
IP packet comparison. This can increase the
accuracy of the package comparison.
Less checkpoint more efficiency.
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
net/colo-compare.c | 147 +++
Add the conformance targets and clauses for
virtio-crypto device.
Signed-off-by: Gonglei
---
conformance.tex | 31 +++
1 file changed, 31 insertions(+)
diff --git a/conformance.tex b/conformance.tex
index f59e360..039f44e 100644
--- a/conformance.tex
+++ b/conformanc
1. Default cache size is 64MB.
2. Semantics correction.
Signed-off-by: Cao jin
---
docs/xbzrle.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/xbzrle.txt b/docs/xbzrle.txt
index 52c8511..c0a7dfd 100644
--- a/docs/xbzrle.txt
+++ b/docs/xbzrle.txt
@@ -42,7 +42,7 @@
add Zhang Chen and Li zhijian as co-maintainers of COLO-proxy.
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
MAINTAINERS | 9 +
1 file changed, 9 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index b6fb84e..4781f9f 100644
--- a/MAINTAINERS
Right after main_loop ends, we release various things but keep iothread
alive. The latter is not prepared to the sudden change of resources.
Specifically, after bdrv_close_all(), virtio-scsi dataplane get a
surprise at the empty BlockBackend:
(gdb) bt
at /usr/src/debug/qemu-2.6.0/hw/scsi/virt
On 08/09/2016 11:28, Fam Zheng wrote:
> Right after main_loop ends, we release various things but keep iothread
> alive. The latter is not prepared to the sudden change of resources.
>
> Specifically, after bdrv_close_all(), virtio-scsi dataplane get a
> surprise at the empty BlockBackend:
>
>
In this patch we use kernel jhash table to track
connection, and then enqueue net packet like this:
+ CompareState ++
| |
+---+ +---+ +---+
|conn list +--->conn +->conn |
+---+ +-
On 09/05/2016 09:53 AM, Longpeng(Mike) wrote:
> +++ b/include/hw/compat.h
> @@ -1,6 +1,8 @@
> #ifndef HW_COMPAT_H
> #define HW_COMPAT_H
>
> +#define HW_COMPAT_2_7
> +
This is already defined (due to the s390 2.8 machine)
Introduce the design of COLO-proxy, and how to use it.
Signed-off-by: Zhang Chen
---
docs/colo-proxy.txt | 188
1 file changed, 188 insertions(+)
create mode 100644 docs/colo-proxy.txt
diff --git a/docs/colo-proxy.txt b/docs/colo-proxy.txt
n
Am 08.09.2016 um 10:49 hat Daniel P. Berrange geschrieben:
> On Thu, Sep 08, 2016 at 10:43:19AM +0200, Kevin Wolf wrote:
> > Am 08.09.2016 um 00:32 hat ashish mittal geschrieben:
> > > >> +
> > > >> +void vxhs_set_acb_buffer(void *ptr, void *buffer)
> > > >> +{
> > > >> +VXHSAIOCB *acb = ptr;
>
This a COLO net ascii figure:
Primary qemu
Secondary qemu
+--+
++
| +--
From: Prasad J Pandit
When processing svga command DEFINE_CURSOR in vmsvga_fifo_run,
the computed BITMAP and PIXMAP size are checked against the
'cursor.mask[]' and 'cursor.image[]' array sizes in bytes.
Correct these checks to avoid OOB memory access.
Reported-by: Qinghao Tang
Reported-by: Li
On 08/09/16 09:57, Leon Alrae wrote:
> On Fri, Aug 19, 2016 at 08:40:32PM +0100, Paul Burton wrote:
>> On 19/08/16 20:25, no-re...@patchew.org wrote:
>>> Hi,
>>>
>>> Your series failed automatic build test. Please find the testing commands
>>> and
>>> their output below. If you have docker install
On Thu, Sep 08, 2016 at 04:09:39PM +0800, Jike Song wrote:
> On 08/25/2016 11:53 AM, Kirti Wankhede wrote:
> > +
> > +/**
> > + * struct parent_ops - Structure to be registered for each parent device to
> > + * register the device to mdev module.
> > + *
> > + * @owner: The module owner.
>
On Thu, Sep 08, 2016 at 09:42:33AM +0200, Christian Borntraeger wrote:
> On 09/08/2016 05:28 AM, Pranith Kumar wrote:
> > FYI,
> >
> > The KVM Forum 2016 videos are now online on youtube. You can find them here:
> >
> > https://www.youtube.com/playlist?list=PLW3ep1uCIRfzQoZ0SlniYE8nz1ZRobjH7
>
>
Paolo Bonzini writes:
> From: Sergey Fedorov
>
> This is a small clean up. tb_find_fast() is a final consumer of this
> variable so no need to pass it by reference. 'last_tb' is always updated
> by subsequent cpu_loop_exec_tb() in cpu_exec().
>
> @@ -621,7 +620,7 @@ int cpu_exec(CPUState *cpu)
Add qemu_chr_add_handlers_full() API, we can use
this API pass in a GMainContext,make handler run
in the context rather than main_loop.
This comments from Daniel P . Berrange.
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
Reviewed-by: Daniel P. Berrange
---
i
Filter-rewriter is a part of COLO project.
It will rewrite some of secondary packet to make
secondary guest's tcp connection established successfully.
In this module we will rewrite tcp packet's ack to the secondary
from primary,and rewrite tcp packet's seq to the primary from
secondary.
usage:
c
On 6 September 2016 at 20:03, Peter Maydell wrote:
> v2 pull:
> * dropped the ast2500 patches
> * fix ast2400 memory controller format string bug
>
> thanks
> -- PMM
>
>
> The following changes since commit 2926375cffce464fde6b4dabaed1e133d549af39:
>
> Merge remote-tracking branch 'remotes/kev
On 08/09/2016 12:18, Kevin Wolf wrote:
> Am 08.09.2016 um 11:43 hat Daniel P. Berrange geschrieben:
>> On Thu, Sep 08, 2016 at 11:29:41AM +0200, Kevin Wolf wrote:
>>> Am 08.09.2016 um 10:49 hat Daniel P. Berrange geschrieben:
On Thu, Sep 08, 2016 at 10:43:19AM +0200, Kevin Wolf wrote:
>
On 09/08/2016 03:40 AM, Marc-André Lureau wrote:
> Allows one to specify a destroy function for the test data.
>
> Add a fallback using glib g_test_add_vtable() internal function, whose
> signature changed over time. Tested with glib 2.22, 2.26 and 2.48, which
> according to git log should be enou
The net/colo.c is used by colo-compare and filter-rewriter.
this can share common data structure like net packet,
and other functions.
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
net/Makefile.objs | 1 +
net/colo-compare.c | 114 ++
On Tue, 09/06 22:05, Sascha Silbe wrote:
> A couple of fixes for issues encountered while trying out the new
> docker test support.
>
> v2→v3:
> - fix non-portable sort -V usage
> - send debootstrap.pre error messages to stderr
> - whitespace changes
>
> v1→v2:
> - found a good place to s
On Wed, Sep 07, 2016 at 08:20:35PM +1000, David Gibson wrote:
> On Wed, Sep 07, 2016 at 03:09:16PM +0800, Peter Xu wrote:
> > On Wed, Sep 07, 2016 at 04:02:39PM +1000, David Gibson wrote:
> > > On Wed, Sep 07, 2016 at 01:32:22PM +0800, Peter Xu wrote:
> > > > IOMMU Notifier list is used for notifyi
On 09/06/2016 02:47 AM, Cornelia Huck wrote:
> From: David Hildenbrand
>
> Let's provide a standardized interface to baseline two CPU models, to
> create a third, compatible one. This is especially helpful when two
> CPU models are not identical, but a CPU model is required that is
> guaranteed t
We use net/colo.h to track connection and parse packet
Signed-off-by: Zhang Chen
Signed-off-by: Li Zhijian
Signed-off-by: Wen Congyang
---
net/colo.c| 14 ++
net/colo.h| 1 +
net/filter-rewriter.c | 50 ++
3 f
On Wed, Sep 07, 2016 at 05:20:49PM +0200, Ladi Prosek wrote:
> The statistics virtqueue is not migrated properly because virtio-balloon
> does not include s->stats_vq_elem in the migration stream.
>
> After migration the statistics virtqueue hangs because the host never
> completes the last elemen
This is the specification about a new virtio crypto device.
Today, I published the Qemu Code and virtio-crypto linux kernel driver
module.
You can get information from the below website:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg395709.html
Please help to review, thanks.
CC: Micha
Signed-off-by: Marc-André Lureau
Reviewed-by: Eric Blake
---
tests/test-qga.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/tests/test-qga.c b/tests/test-qga.c
index dac8fb8..21f44f8 100644
--- a/tests/test-qga.c
+++ b/tests/test-qga.c
@@ -398,6 +398,7 @@ static void test_qga_file_ops
Paolo Bonzini writes:
> From: Sergey Fedorov
>
> In fact, this function does not exactly perform a lookup by physical
> address as it is descibed for comment on get_page_addr_code(). Thus
> it may be a bit confusing to have "physical" in it's name. So rename it
> to tb_htable_lookup() to better
Those functions are only available since glib 2.28.
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
---
include/glib-compat.h | 24
1 file changed, 24 insertions(+)
diff --git a/include/glib-compat.h b/include/glib-compat.h
index 8d5a7f3..8093163 100644
Signed-off-by: Marc-André Lureau
Reviewed-by: Eric Blake
---
qga/guest-agent-command-state.c | 6 ++
qga/main.c | 6 ++
qga/guest-agent-core.h | 1 +
3 files changed, 13 insertions(+)
diff --git a/qga/guest-agent-command-state.c b/qga/guest-agent-command-st
On Thu, Sep 08, 2016 at 03:23:26PM +0200, Lluís Vilanova wrote:
> Daniel P Berrange writes:
>
> > I previously split the global trace-events file up into one file
> > per-subdirectory to avoid merge conflict hell.
> [...]
>
> Sorry, I could not find the message where the infrastructure is modifie
Spotted thanks to ASAN.
Signed-off-by: Marc-André Lureau
Reviewed-by: Eric Blake
---
tests/test-cutils.c | 24
1 file changed, 16 insertions(+), 8 deletions(-)
diff --git a/tests/test-cutils.c b/tests/test-cutils.c
index 64e3e95..20b0f59 100644
--- a/tests/test-cutils.
Richard Henderson writes:
> Add all of cmpxchg, op_fetch, fetch_op, and xchg.
> Handle both endian-ness, and sizes up to 8.
> Handle expanding non-atomically, when emulating in serial.
>
> Signed-off-by: Richard Henderson
> /* For the benefit of TCG generated code, we want to avoid the compli
Found thanks to ASAN.
Signed-off-by: Marc-André Lureau
Reviewed-by: Eric Blake
---
tests/check-qom-proplist.c | 16
1 file changed, 16 insertions(+)
diff --git a/tests/check-qom-proplist.c b/tests/check-qom-proplist.c
index 42defe7..a16cefc 100644
--- a/tests/check-qom-proplis
Hi
- Original Message -
> On 09/08/2016 03:40 AM, Marc-André Lureau wrote:
> > Allows one to specify a destroy function for the test data.
> >
> > Add a fallback using glib g_test_add_vtable() internal function, whose
> > signature changed over time. Tested with glib 2.22, 2.26 and 2.48,
qemu_irq is already a pointer, no need to have an extra pointer level.
Signed-off-by: Marc-André Lureau
Reviewed-by: Eric Blake
---
hw/i386/pc.c | 8
hw/input/pckbd.c | 4 ++--
include/hw/i386/pc.h | 2 +-
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/hw/i3
On Wed, Sep 07, 2016 at 12:54:23PM +0200, Paolo Bonzini wrote:
>
>
> On 07/09/2016 08:05, David Gibson wrote:
> > On Wed, Sep 07, 2016 at 01:32:23PM +0800, Peter Xu wrote:
> >> Considering that we may have multiple IOMMU notifier consumers in the
> >> future, converting iommu_ops.notify_{started|
Make raw_open for POSIX more consistent in handling errors by setting
the error object also when qemu_open fails. The error object was set
generally set in case of errors, but I guess this case was overlooked.
Do the same for win32.
Signed-off-by: Halil Pasic
Reviewed-by: Sascha Silbe
Tested-by:
The free_ranges array is used as a temporary pointer array, the segment
should still be freed, however, it shouldn't free the elements themself.
Signed-off-by: Marc-André Lureau
Tested-by: Marcel Apfelbaum
Reviewed-by: Marcel Apfelbaum
---
hw/i386/acpi-build.c | 4 ++--
1 file changed, 2 inser
On 09/06/2016 02:47 AM, Cornelia Huck wrote:
> From: David Hildenbrand
>
> Let's provide a standardized interface to compare two CPU models.
> "query-cpu-model-compare" takes two models and returns how they compare
> in a specific configuration.
>
> +##
> +# @CpuModelCompareResult:
> +#
> +# A
Further cleanup would need to call qemu_free_irq() at the appropriate
time, but for now this silences ASAN about direct leaks.
Signed-off-by: Marc-André Lureau
Reviewed-by: Markus Armbruster
---
hw/i386/pc_piix.c| 17 -
hw/i386/pc_q35.c | 13 ++---
include/hw/i38
On Wed, Sep 07, 2016 at 05:20:47PM +0200, Ladi Prosek wrote:
> The one pending element is being freed but not discarded on device
> reset, which causes svq->inuse to creep up, eventually hitting the
> "Virtqueue size exceeded" error.
>
> Properly discarding the element on device reset makes sure t
Free the timer allocated in instance_init.
Signed-off-by: Marc-André Lureau
Reviewed-by: Andrew Baumann
---
hw/sd/sd.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/hw/sd/sd.c b/hw/sd/sd.c
index 87c6dc1..8e88e83 100644
--- a/hw/sd/sd.c
+++ b/hw/sd/sd.c
@@ -1876,6 +1876,14 @@ stat
On Thu, 8 Sep 2016 10:30:32 +0200
Laurent Vivier wrote:
> Check the result of qemu_strtoXX() and assert
> if the string cannot be converted.
>
> Signed-off-by: Laurent Vivier
> Reviewed-by: David Gibson
> ---
Reviewed-by: Greg Kurz
> v5:
> - update log message about result checking
> - add
1 - 100 of 306 matches
Mail list logo
