[Qemu-devel] [PULL 08/14] virtio: add virtqueue_rewind()

2016-09-09 Thread Michael S. Tsirkin
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 VirtQueueElement. This will allow virtio-balloon to access element again after migr

[Qemu-devel] [PULL 09/14] virtio-balloon: fix stats vq migration

2016-09-09 Thread Michael S. Tsirkin
From: Ladi Prosek 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 element (s->stats_vq_elem is NULL on the destination QEMU

[Qemu-devel] [PULL 10/14] virtio-pci: error out when both legacy and modern modes are disabled

2016-09-09 Thread Michael S. Tsirkin
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: Cornelia Huck Signed-off-by: Greg Kurz R

[Qemu-devel] [PULL 13/14] vhost-vsock: add virtio sockets device

2016-09-09 Thread Michael S. Tsirkin
From: Stefan Hajnoczi Implement the new virtio sockets device for host<->guest communication using the Sockets API. Most of the work is done in a vhost kernel driver so that virtio-vsock can hook into the AF_VSOCK address family. The QEMU vhost-vsock device handles configuration and live migrati

[Qemu-devel] [PULL 12/14] virtio-pci: minor refactoring

2016-09-09 Thread Michael S. Tsirkin
!legacy && !modern is shorter than !(legacy || modern). I also perfer this (less ()s) as a matter of taste. Cc: Greg Kurz Signed-off-by: Michael S. Tsirkin --- hw/virtio/virtio-pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pc

Re: [Qemu-devel] [PATCH v3] migrate: Fix bounds check for migration parameters in migration.c

2016-09-09 Thread Eric Blake
On 09/09/2016 03:37 PM, Ashijeet Acharya wrote: > This patch fixes the out-of-bounds check of migration parameters in > qmp_migrate_set_parameters() for cpu-throttle-initial and > cpu-throttle-increment by adding a return statement for both as they > were broken since their introduction in 2.5 via

Re: [Qemu-devel] [PATCH for-2.8 2/2] vhost-vsock: add virtio sockets device

2016-09-09 Thread Michael S. Tsirkin
On Tue, Aug 16, 2016 at 01:27:22PM +0100, Stefan Hajnoczi wrote: > Implement the new virtio sockets device for host<->guest communication > using the Sockets API. Most of the work is done in a vhost kernel > driver so that virtio-vsock can hook into the AF_VSOCK address family. > The QEMU vhost-vs

Re: [Qemu-devel] [PULL 00/14] virtio,vhost,pc: fixes and updates

2016-09-09 Thread Michael S. Tsirkin
On Fri, Sep 09, 2016 at 11:34:13PM +0300, Michael S. Tsirkin wrote: > The following changes since commit c2a57aae9a1c3dd7de77daf5478df10379aeeebf: > > Merge remote-tracking branch 'remotes/famz/tags/docker-pull-request' into > staging (2016-09-09 12:49:41 +0100) > > are available in the git re

[Qemu-devel] [PATCH v4] migrate: Fix bounds check for migration parameters in migration.c

2016-09-09 Thread Ashijeet Acharya
This patch fixes the out-of-bounds check of migration parameters in qmp_migrate_set_parameters() for cpu-throttle-initial and cpu-throttle-increment by adding a return statement for both as they were broken since their introduction in 2.5 via commit 1626fee. Due to the missing return statements, pa

[Qemu-devel] [PATCH] MAINTAINERS: Add include/sysemu/cpus.h

2016-09-09 Thread Thomas Huth
This header seems to belong to the guest CPU section since it contains prototypes for cpus.c. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index cc69977..9d17cb3 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -83,6 +83,7

[Qemu-devel] [PATCH] MAINTAINERS: Add include/hw/tricore/ to TriCore section

2016-09-09 Thread Thomas Huth
get_maintainer.pl now properly recognizes that the file in include/hw/tricore/ belongs to TriCore. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7196dba..dad82e7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -228,6

[Qemu-devel] [PATCH] MAINTAINERS: Add include/hw/unicore32/ to UniCore32 section

2016-09-09 Thread Thomas Huth
get_maintainer.pl now properly recognizes that the file in include/hw/unicore32/ belongs to UniCore32. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9d17cb3..7196dba 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -20

[Qemu-devel] [PATCH] MAINTAINERS: Add include/hw/sh4/ to SH4 section

2016-09-09 Thread Thomas Huth
get_maintainer.pl now properly recognizes that the files in include/hw/sh4/ belong to SH4. Signed-off-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index dad82e7..ad70868 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -189,6 +189,7 @

Re: [Qemu-devel] kvm test warnings

2016-09-09 Thread Michael S. Tsirkin
On Fri, Sep 09, 2016 at 09:24:45PM +0200, Thomas Huth wrote: > On 09.09.2016 21:03, Michael S. Tsirkin wrote: > > I am running make test on a box without kvm, and I am seeing these warnings: > > > > GTESTER check-qtest-ppc64 > > "kvm" accelerator not found. > > "kvm" accelerator not found. > > GTE

[Qemu-devel] [PULL v2 00/14] virtio,vhost,pc: fixes and updates

2016-09-09 Thread Michael S. Tsirkin
The following changes since commit c2a57aae9a1c3dd7de77daf5478df10379aeeebf: Merge remote-tracking branch 'remotes/famz/tags/docker-pull-request' into staging (2016-09-09 12:49:41 +0100) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upst

[Qemu-devel] [PULL v2 01/14] virtio: Tell the user what went wrong when event_notifier_init failed

2016-09-09 Thread Michael S. Tsirkin
From: Thomas Huth event_notifier_init() can fail in real life, for example when there are not enough open file handles available (EMFILE) when using a lot of devices. So instead of leaving the average user with a cryptic error number only, print out a proper error message with strerror() instead,

[Qemu-devel] [PULL v2 02/14] virtio-pci: use size from correct structure

2016-09-09 Thread Michael S. Tsirkin
PIO MR registration should use size from the correct notify struct. Doesn't affect any visible behaviour because the field values are the same (both are 4). Signed-off-by: Michael S. Tsirkin --- hw/virtio/virtio-pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/virtio/

[Qemu-devel] [PULL v2 03/14] pc: Add 2.8 machine

2016-09-09 Thread Michael S. Tsirkin
From: "Longpeng(Mike)" This will used by the next patch. Signed-off-by: Longpeng(Mike) Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/i386/pc.h | 3 +++ hw/i386/pc_piix.c| 16 +--- hw/i386/pc_q35.c | 13 +++-- 3 files changed, 27

[Qemu-devel] [PULL v2 06/14] virtio: zero vq->inuse in virtio_reset()

2016-09-09 Thread Michael S. Tsirkin
From: Stefan Hajnoczi 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 leaked!). In practice, it is difficult to achieve vq

[Qemu-devel] [PULL v2 05/14] virtio-pci: reduce modern_mem_bar size

2016-09-09 Thread Michael S. Tsirkin
From: Marcel Apfelbaum Currently each VQ Notification Virtio Capability is allocated on a different page. The idea is to enable split drivers within guests, however there are no known plans to do that. The allocation will result in a 8MB BAR, more than various guest firmwares pre-allocates for PC

[Qemu-devel] [PULL v2 04/14] target-i386: present virtual L3 cache info for vcpus

2016-09-09 Thread Michael S. Tsirkin
From: "Longpeng(Mike)" Some software algorithms are based on the hardware's cache info, for example, for x86 linux kernel, when cpu1 want to wakeup a task on cpu2, cpu1 will trigger a resched IPI and told cpu2 to do the wakeup if they don't share low level cache. Oppositely, cpu1 will access cpu2

[Qemu-devel] [PULL v2 13/14] tests/acpi: speedup acpi tests

2016-09-09 Thread Michael S. Tsirkin
From: Marcel Apfelbaum Use kvm acceleration if available. Disable kernel-irqchip and use qemu64 cpu for both kvm and tcg cases. Using kvm acceleration saves about a second and disabling kernel-irqchip has no visible performance impact. Acked-by: Michael S. Tsirkin Signed-off-by: Marcel Apfelba

[Qemu-devel] [PULL v2 07/14] virtio-balloon: discard virtqueue element on reset

2016-09-09 Thread Michael S. Tsirkin
From: Ladi Prosek 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 that its buffers are unmapped and the inuse counter

[Qemu-devel] [PULL v2 10/14] virtio-pci: error out when both legacy and modern modes are disabled

2016-09-09 Thread Michael S. Tsirkin
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: Cornelia Huck Signed-off-by: Greg Kurz R

[Qemu-devel] [PULL v2 14/14] vhost-vsock: add virtio sockets device

2016-09-09 Thread Michael S. Tsirkin
From: Stefan Hajnoczi Implement the new virtio sockets device for host<->guest communication using the Sockets API. Most of the work is done in a vhost kernel driver so that virtio-vsock can hook into the AF_VSOCK address family. The QEMU vhost-vsock device handles configuration and live migrati

[Qemu-devel] [PULL v2 11/14] vhost: don't set vring call if no vector

2016-09-09 Thread Michael S. Tsirkin
From: Jason Wang We used to set vring call fd unconditionally even if guest driver does not use MSIX for this vritqueue at all. This will cause lots of unnecessary userspace access and other checks for drivers does not use interrupt at all (e.g virtio-net pmd). So check and clean vring call fd if

[Qemu-devel] [PULL v2 09/14] virtio-balloon: fix stats vq migration

2016-09-09 Thread Michael S. Tsirkin
From: Ladi Prosek 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 element (s->stats_vq_elem is NULL on the destination QEMU

[Qemu-devel] [PULL v2 12/14] virtio-pci: minor refactoring

2016-09-09 Thread Michael S. Tsirkin
!legacy && !modern is shorter than !(legacy || modern). I also perfer this (less ()s) as a matter of taste. Cc: Greg Kurz Signed-off-by: Michael S. Tsirkin --- hw/virtio/virtio-pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pc

[Qemu-devel] [PULL v2 08/14] virtio: add virtqueue_rewind()

2016-09-09 Thread Michael S. Tsirkin
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 VirtQueueElement. This will allow virtio-balloon to access element again after migr

Re: [Qemu-devel] [PATCH v2 3/3] target-ppc: tlbie should have global effect

2016-09-09 Thread Benjamin Herrenschmidt
On Fri, 2016-09-09 at 18:44 +0530, Nikunj A Dadhania wrote: > +static inline void tlb_clear_flag(CPUState *cs) > +{ > +    PowerPCCPU *cpu = POWERPC_CPU(cs); > +    CPUPPCState *env = &cpu->env; > + > +    env->tlb_need_flush = 0; > +} What is the point of making this a separate function ? Also I

Re: [Qemu-devel] [PATCH RFC v1 1/3] target-ppc: add TLB_NEED_LOCAL_FLUSH flag

2016-09-09 Thread Benjamin Herrenschmidt
On Fri, 2016-09-09 at 15:07 +0100, Alex Bennée wrote: > Nikunj A Dadhania writes: > > I think we need a little more detail here. In fact when you post the > next version of the series could you please include a cover letter to > cover what the series is trying to achieve? In the meantime, for th

<    1   2   3