[Qemu-devel] [PULL 08/42] misc: Fix new collection of typos

2015-04-30 Thread Michael Tokarev
From: Stefan Weil s...@weilnetz.de All of them were reported by codespell. Most typos are in comments, one is in an error message. Signed-off-by: Stefan Weil s...@weilnetz.de Reviewed-by: Peter Maydell peter.mayd...@linaro.org Reviewed-by: Stefan Hajnoczi stefa...@redhat.com Signed-off-by:

Re: [Qemu-devel] [PATCH microblaze v1 3/6] mb: cpu: Remote unused cpu_get_pc

2015-04-30 Thread Alex Bennée
Peter Crosthwaite crosthwaitepe...@gmail.com writes: This function is not used by anything. Remove. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- target-microblaze/cpu.h | 5 - 1 file changed, 5 deletions(-) diff --git a/target-microblaze/cpu.h

[Qemu-devel] Question about block driver

2015-04-30 Thread Wen Congyang
Hi, all Some drivers use bdrv_open, while the other dirvers use bdrv_file_open(). What is the difference between bdrv_open() and bdrv_file_open()? Thanks Wen Congyang

Re: [Qemu-devel] [PULL 0/3] VFIO fixes + AMD GPU reset workaround

2015-04-30 Thread Peter Maydell
On 28 April 2015 at 18:52, Alex Williamson alex.william...@redhat.com wrote: The following changes since commit 84cbd63f87c1d246f51ec8eee5367a5588f367fd: Merge remote-tracking branch 'remotes/ehabkost/tags/x86-pull-request' into staging (2015-04-28 12:22:20 +0100) are available in the git

Re: [Qemu-devel] [PATCH V2] vhost: logs sharing

2015-04-30 Thread Jason Wang
On Thu, Apr 30, 2015 at 4:09 PM, Michael S. Tsirkin m...@redhat.com wrote: On Thu, Apr 30, 2015 at 04:05:09PM +0800, Jason Wang wrote: On Tue, Apr 28, 2015 at 6:30 PM, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Apr 28, 2015 at 05:58:28PM +0800, Jason Wang wrote: On Tue,

[Qemu-devel] [PULL 29/42] microblaze: cpu: Renumber EXCP_* constants to close gap

2015-04-30 Thread Michael Tokarev
After removal of EXCP_NMI there's a gap in EXCP_* numbering. Let's remove it. Signed-off-by: Michael Tokarev m...@tls.msk.ru Reviewed-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-microblaze/cpu.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[Qemu-devel] [PULL 15/42] pci: Remove unused function ich9_d2pbr_init()

2015-04-30 Thread Michael Tokarev
From: Thomas Huth th...@linux.vnet.ibm.com The function ich9_d2pbr_init() is completely unused and thus can be deleted. Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com Reviewed-by: Michael S. Tsirkin m...@redhat.com Signed-off-by: Michael Tokarev m...@tls.msk.ru ---

[Qemu-devel] [PULL 36/42] i440fx-test: remove ARRAY_SIZE redefinition

2015-04-30 Thread Michael Tokarev
From: Emilio G. Cota c...@braap.org It's defined in osdep.h and shouldn't be redefined here. Signed-off-by: Emilio G. Cota c...@braap.org Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com Reviewed-by: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Michael Tokarev m...@tls.msk.ru ---

[Qemu-devel] [PULL 34/42] coroutine: remove unnecessary parentheses in qemu_co_queue_empty

2015-04-30 Thread Michael Tokarev
From: Emilio G. Cota c...@braap.org Signed-off-by: Emilio G. Cota c...@braap.org Signed-off-by: Michael Tokarev m...@tls.msk.ru --- qemu-coroutine-lock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-coroutine-lock.c b/qemu-coroutine-lock.c index e4860ae..6b49033

[Qemu-devel] [PULL 33/42] qemu-char: remove unused list node from FDCharDriver

2015-04-30 Thread Michael Tokarev
From: Emilio G. Cota c...@braap.org Signed-off-by: Emilio G. Cota c...@braap.org Signed-off-by: Michael Tokarev m...@tls.msk.ru --- qemu-char.c | 1 - 1 file changed, 1 deletion(-) diff --git a/qemu-char.c b/qemu-char.c index a405d76..d0c1564 100644 --- a/qemu-char.c +++ b/qemu-char.c @@ -973,7

[Qemu-devel] [PULL 31/42] cpus: use first_cpu macro instead of QTAILQ_FIRST(cpus)

2015-04-30 Thread Michael Tokarev
From: Emilio G. Cota c...@braap.org Signed-off-by: Emilio G. Cota c...@braap.org Signed-off-by: Michael Tokarev m...@tls.msk.ru --- cpus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cpus.c b/cpus.c index e6dcae3..62d157a 100644 --- a/cpus.c +++ b/cpus.c @@ -1016,7

[Qemu-devel] [PULL 06/42] ui/console : remove 'struct' from 'typedef struct' type

2015-04-30 Thread Michael Tokarev
From: Chih-Min Chao cmc...@gmail.com Signed-off-by: Chih-Min Chao cmc...@gmail.com Reviewed-by: Gerd Hoffmann kra...@redhat.com Signed-off-by: Michael Tokarev m...@tls.msk.ru --- ui/console.c | 4 ++-- ui/spice-display.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

[Qemu-devel] [PULL 28/42] microblaze: cpu: Delete EXCP_NMI

2015-04-30 Thread Michael Tokarev
From: Peter Crosthwaite crosthwaitepe...@gmail.com This define is unused. Remove. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com Reviewed-by: Edgar E. Iglesias edgar.igles...@xilinx.com Signed-off-by: Michael Tokarev m...@tls.msk.ru --- target-microblaze/cpu.h | 1 - 1 file

[Qemu-devel] [PULL 37/42] qmp-commands: Fix typo

2015-04-30 Thread Michael Tokarev
From: John Snow js...@redhat.com Just a trivial patch to correct a QMP example in qmp-commands.hx. Signed-off-by: John Snow js...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com Signed-off-by: Michael Tokarev m...@tls.msk.ru --- qmp-commands.hx | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [Qemu-devel] [PATCH v4 0/4] scripts: qmp-shell: add transaction support

2015-04-30 Thread Kashyap Chamarthy
On Wed, Apr 29, 2015 at 03:14:00PM -0400, John Snow wrote: The qmp-shell is a little rudimentary, but it can be hacked to give us some transactional support without too much difficulty. (1) Prep. (2) Add support for serializing json arrays and improve the robustness of QMP parsing (3)

Re: [Qemu-devel] [PATCH v14 09/10] sysbus: add irq_routing_notifier

2015-04-30 Thread Peter Crosthwaite
On Wed, Apr 29, 2015 at 7:52 AM, Eric Auger eric.au...@linaro.org wrote: Add a new connect_irq_notifier notifier in the SysBusDeviceClass. This notifier, if populated, is called after sysbus_connect_irq. This mechanism is used to setup VFIO signaling once VFIO platform devices get attached to

Re: [Qemu-devel] [PATCH V2] vhost: logs sharing

2015-04-30 Thread Michael S. Tsirkin
On Thu, Apr 30, 2015 at 04:05:09PM +0800, Jason Wang wrote: On Tue, Apr 28, 2015 at 6:30 PM, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Apr 28, 2015 at 05:58:28PM +0800, Jason Wang wrote: On Tue, Apr 28, 2015 at 5:37 PM, Michael S. Tsirkin m...@redhat.com wrote: On Fri, Apr

[Qemu-devel] [PATCH] openrisc: cpu: Remove unused cpu_get_pc

2015-04-30 Thread Peter Crosthwaite
This function is not used by anything. Remove. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- target-openrisc/cpu.h | 5 - 1 file changed, 5 deletions(-) diff --git a/target-openrisc/cpu.h b/target-openrisc/cpu.h index b25324b..9e23cd0 100644 --- a/target-openrisc/cpu.h

[Qemu-devel] [PULL 23/42] libcacard: do not use full paths for include files in the same dir

2015-04-30 Thread Michael Tokarev
Signed-off-by: Michael Tokarev m...@tls.msk.ru Reviewed-by: Paolo Bonzini pbonz...@redhat.com --- libcacard/vcard_emul_nss.c | 2 +- libcacard/vcardt.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c index

[Qemu-devel] [PULL 04/42] ui/vnc : fix coding style

2015-04-30 Thread Michael Tokarev
From: Chih-Min Chao cmc...@gmail.com reported by checkpatch.pl Signed-off-by: Chih-Min Chao cmc...@gmail.com Reviewed-by: Gerd Hoffmann kra...@redhat.com Signed-off-by: Michael Tokarev m...@tls.msk.ru --- ui/vnc-auth-vencrypt.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-)

Re: [Qemu-devel] [REBASE PATCH v5 1/2] machine: add default_ram_size to machine class

2015-04-30 Thread Alexander Graf
On 30.04.15 06:41, Nikunj A Dadhania wrote: Hi Paolo, Paolo Bonzini pbonz...@redhat.com writes: On 29/04/2015 11:06, Nikunj A Dadhania wrote: so David can push both patches. But isn't 1G a bit too much? At least on x86 you can easily boot with 512M. I understood this number as not

[Qemu-devel] [PULL 35/42] linux-user/elfload: use QTAILQ_FOREACH instead of open-coding it

2015-04-30 Thread Michael Tokarev
From: Emilio G. Cota c...@braap.org Signed-off-by: Emilio G. Cota c...@braap.org Signed-off-by: Michael Tokarev m...@tls.msk.ru --- linux-user/elfload.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/linux-user/elfload.c b/linux-user/elfload.c index 399c021..0ba9706 100644

[Qemu-devel] [PULL 32/42] input: remove unused mouse_handlers list

2015-04-30 Thread Michael Tokarev
From: Emilio G. Cota c...@braap.org Signed-off-by: Emilio G. Cota c...@braap.org Signed-off-by: Michael Tokarev m...@tls.msk.ru --- ui/input-legacy.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ui/input-legacy.c b/ui/input-legacy.c index 2d4ca19..3e9bb38 100644 --- a/ui/input-legacy.c

[Qemu-devel] [PULL 39/42] vhost-user: remove superfluous '\n' around error_report()

2015-04-30 Thread Michael Tokarev
From: Gonglei arei.gong...@huawei.com Signed-off-by: Gonglei arei.gong...@huawei.com Reviewed-by: Andreas Färber afaer...@suse.de Signed-off-by: Michael Tokarev m...@tls.msk.ru --- hw/virtio/vhost-user.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git

[Qemu-devel] [PULL 24/42] defconfigs: Piggyback microblazeel on microblaze

2015-04-30 Thread Michael Tokarev
From: Peter Crosthwaite crosthwaitepe...@gmail.com Theres no difference in defconfig. Going forward microblazeel should superset microblaze so use an include. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com Reviewed-by: Edgar E. Iglesias edgar.igles...@xilinx.com Signed-off-by:

[Qemu-devel] [PULL 11/42] tpm: fix coding style

2015-04-30 Thread Michael Tokarev
From: Stefan Berger stef...@linux.vnet.ibm.com Fix coding style in one instance. Signed-off-by: Stefan Berger stef...@linux.vnet.ibm.com Signed-off-by: Michael Tokarev m...@tls.msk.ru --- hw/tpm/tpm_tis.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/tpm/tpm_tis.c

[Qemu-devel] [PULL 22/42] libcacard: stop including qemu-common.h

2015-04-30 Thread Michael Tokarev
From: Paolo Bonzini pbonz...@redhat.com This is a small step towards making libcacard standalone. Signed-off-by: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Michael Tokarev m...@tls.msk.ru --- libcacard/cac.c| 5 - libcacard/card_7816.c | 4 +++- libcacard/event.c

Re: [Qemu-devel] [PATCH] openrisc: cpu: Remove unused cpu_get_pc

2015-04-30 Thread Alex Bennée
Peter Crosthwaite crosthwaitepe...@gmail.com writes: This function is not used by anything. Remove. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- target-openrisc/cpu.h | 5 - 1 file changed, 5 deletions(-) diff --git a/target-openrisc/cpu.h b/target-openrisc/cpu.h

Re: [Qemu-devel] [PATCH V2] vhost: logs sharing

2015-04-30 Thread Jason Wang
On Tue, Apr 28, 2015 at 6:30 PM, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Apr 28, 2015 at 05:58:28PM +0800, Jason Wang wrote: On Tue, Apr 28, 2015 at 5:37 PM, Michael S. Tsirkin m...@redhat.com wrote: On Fri, Apr 10, 2015 at 05:33:35PM +0800, Jason Wang wrote: Currently we

Re: [Qemu-devel] [RFC PATCH 6/8] tap: Drop tap_can_send

2015-04-30 Thread Jason Wang
On Wed, Apr 29, 2015 at 6:37 PM, Fam Zheng f...@redhat.com wrote: This callback is called by main loop before polling s-fd, if it returns false, the fd will not be polled in this iteration. This is redundant with checks inside read callback. After this patch, the data will be sent to peer

[Qemu-devel] [PULL 21/42] docs/atomics.txt: fix two typos

2015-04-30 Thread Michael Tokarev
From: Laszlo Ersek ler...@redhat.com Cc: Paolo Bonzini pbonz...@redhat.com Signed-off-by: Laszlo Ersek ler...@redhat.com Signed-off-by: Michael Tokarev m...@tls.msk.ru --- docs/atomics.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/atomics.txt b/docs/atomics.txt

Re: [Qemu-devel] [PULL 00/10] Ide patches

2015-04-30 Thread Peter Maydell
On 29 April 2015 at 00:25, John Snow js...@redhat.com wrote: The following changes since commit a9392bc93c8615ad1983047e9f91ee3fa8aae75f: Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging (2015-04-28 16:55:03 +0100) are available in the git repository at:

Re: [Qemu-devel] Question about block driver

2015-04-30 Thread Wen Congyang
On 04/30/2015 05:33 PM, Kevin Wolf wrote: [Cc: qemu-block] Am 30.04.2015 um 11:11 hat Wen Congyang geschrieben: Some drivers use bdrv_open, while the other dirvers use bdrv_file_open(). What is the difference between bdrv_open() and bdrv_file_open()? bdrv_file_open() is used by protocol

[Qemu-devel] [PATCH 4/6] qcow2: remove qcow2_cache_find_entry_to_replace()

2015-04-30 Thread Alberto Garcia
A cache miss means that the whole array was traversed and the entry we were looking for was not found, so there's no need to traverse it again in order to select an entry to replace. Signed-off-by: Alberto Garcia be...@igalia.com --- block/qcow2-cache.c | 45

Re: [Qemu-devel] Question about block driver

2015-04-30 Thread Kevin Wolf
Am 30.04.2015 um 11:43 hat Wen Congyang geschrieben: On 04/30/2015 05:33 PM, Kevin Wolf wrote: [Cc: qemu-block] Am 30.04.2015 um 11:11 hat Wen Congyang geschrieben: Some drivers use bdrv_open, while the other dirvers use bdrv_file_open(). What is the difference between bdrv_open() and

[Qemu-devel] [PATCH 5/6] qcow2: use a hash to look for entries in the L2 cache

2015-04-30 Thread Alberto Garcia
The current cache algorithm traverses the array starting always from the beginning, so the average number of comparisons needed to perform a lookup is proportional to the size of the array. By using a hash of the offset as the starting point, lookups are faster and independent from the array

[Qemu-devel] [PATCH 3/6] qcow2: use an LRU algorithm to replace entries from the L2 cache

2015-04-30 Thread Alberto Garcia
The current algorithm to evict entries from the cache gives always preference to those in the lowest positions. As the size of the cache increases, the chances of the later elements of being removed decrease exponentially. In a scenario with random I/O and lots of cache misses, entries in

[Qemu-devel] [PATCH 0/6] qcow2 L2/refcount cache improvements

2015-04-30 Thread Alberto Garcia
Here are some improvements to the qcow2 L2/refcount cache code. The first one is that all cache tables are now allocated using a single memory block, as we discussed last week. Apart from a more efficient use of memory, this allows some additional optimizations so I took the chance to make other

Re: [Qemu-devel] [REBASE PATCH v5 1/2] machine: add default_ram_size to machine class

2015-04-30 Thread Thomas Huth
On Thu, 30 Apr 2015 11:18:05 +0200 Alexander Graf ag...@suse.de wrote: On 30.04.15 06:41, Nikunj A Dadhania wrote: Hi Paolo, Paolo Bonzini pbonz...@redhat.com writes: On 29/04/2015 11:06, Nikunj A Dadhania wrote: so David can push both patches. But isn't 1G a bit too much?

Re: [Qemu-devel] [RFC PATCH 6/8] tap: Drop tap_can_send

2015-04-30 Thread Paolo Bonzini
On 30/04/2015 10:59, Jason Wang wrote: +/* If !can_send, we will want to disable the read poll, but we still + * need the send completion callback to enable it again, which is a + * sign of peer becoming ready. So call the send function + * regardlessly

[Qemu-devel] [PATCH 1/6] qcow2: use one single memory block for the L2/refcount cache tables

2015-04-30 Thread Alberto Garcia
The qcow2 L2/refcount cache contains one separate table for each cache entry. Doing one allocation per table adds unnecessary overhead and it also requires us to store the address of each table separately. Since the size of the cache is constant during its lifetime, it's better to have an array

[Qemu-devel] [PATCH 6/6] qcow2: style fixes in qcow2-cache.c

2015-04-30 Thread Alberto Garcia
Fix pointer declaration to make it consistent with the rest of the code. Signed-off-by: Alberto Garcia be...@igalia.com --- block/qcow2-cache.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/block/qcow2-cache.c b/block/qcow2-cache.c index c0e0278..dd591ef 100644 ---

[Qemu-devel] [PATCH 2/6] qcow2: simplify qcow2_cache_put() and qcow2_cache_entry_mark_dirty()

2015-04-30 Thread Alberto Garcia
Since all tables are now stored together, it is possible to obtain the position of a particular table directly from its address, so the operation becomes O(1). Signed-off-by: Alberto Garcia be...@igalia.com --- block/qcow2-cache.c | 22 +- 1 file changed, 5 insertions(+), 17

Re: [Qemu-devel] Question about block driver

2015-04-30 Thread Kevin Wolf
[Cc: qemu-block] Am 30.04.2015 um 11:11 hat Wen Congyang geschrieben: Some drivers use bdrv_open, while the other dirvers use bdrv_file_open(). What is the difference between bdrv_open() and bdrv_file_open()? bdrv_file_open() is used by protocol drivers that don't need any other driver to work

Re: [Qemu-devel] [PATCH V2] vhost: logs sharing

2015-04-30 Thread Michael S. Tsirkin
On Thu, Apr 30, 2015 at 05:22:33PM +0800, Jason Wang wrote: On Thu, Apr 30, 2015 at 4:09 PM, Michael S. Tsirkin m...@redhat.com wrote: On Thu, Apr 30, 2015 at 04:05:09PM +0800, Jason Wang wrote: On Tue, Apr 28, 2015 at 6:30 PM, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Apr

Re: [Qemu-devel] [PULL 00/42] Trivial patches for 2015-04-30

2015-04-30 Thread Peter Maydell
On 30 April 2015 at 06:08, Michael Tokarev m...@tls.msk.ru wrote: Hello. This is the first pull request for trivial-patches tree since 2.3 has been released. During the freeze many patches has been accumulated, and even more has been received after 2.4 developmnent has been opened. So here

[Qemu-devel] [PULL 42/42] openrisc: cpu: Remove unused cpu_get_pc

2015-04-30 Thread Michael Tokarev
From: Peter Crosthwaite crosthwaitepe...@gmail.com This function is not used by anything. Remove. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com Reviewed-by: Alex Bennée alex.ben...@linaro.org Signed-off-by: Michael Tokarev m...@tls.msk.ru --- target-openrisc/cpu.h | 5 - 1

[Qemu-devel] [PATCH] MAINTAINERS: Add qemu-block list where missing

2015-04-30 Thread Kevin Wolf
Signed-off-by: Kevin Wolf kw...@redhat.com --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 25fd2b5..0b67c48 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -735,12 +735,14 @@ F: backends/rng*.c nvme M: Keith Busch keith.bu...@intel.com

[Qemu-devel] [PATCH v4 1/7] vmport.c: Fix vmport_cmd_ram_size

2015-04-30 Thread Don Slutz
Based on https://sites.google.com/site/chitchatvmback/backdoor and testing on ESXi, this should be in MB not bytes. Signed-off-by: Don Slutz dsl...@verizon.com --- hw/misc/vmport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/misc/vmport.c b/hw/misc/vmport.c index

[Qemu-devel] [PATCH v4 2/7] vmport_rpc: Add the object vmport_rpc

2015-04-30 Thread Don Slutz
This is the 1st part of Add limited support of VMware's hyper-call rpc. This patch uses existing infrastructure used by vmmouse.c (provided by vmport.c) to handle the VMware backdoor command 30. One of the better on-line references is: https://sites.google.com/site/chitchatvmback/backdoor More

[Qemu-devel] [PATCH v4 4/7] vmport_rpc: Add QMP access to vmport_rpc object.

2015-04-30 Thread Don Slutz
This adds one new inject command: inject-vmport-action And three guest info commands: vmport-guestinfo-set vmport-guestinfo-get query-vmport-guestinfo More details in qmp-commands.hx Signed-off-by: Don Slutz dsl...@verizon.com --- hw/misc/vmport_rpc.c | 269

[Qemu-devel] [PATCH v4 3/7] vmport_rpc: Add limited support of VMware's hyper-call rpc

2015-04-30 Thread Don Slutz
The support included is enough to allow VMware tools to install in a guest and provide guestinfo support. guestinfo support is provided by what is known as VMware RPC support. If the guest is running VMware tools, then the build version of the tools is also available via the property

Re: [Qemu-devel] [RFC PATCH 02/15] qdev: store DeviceState's canonical path to use when unparenting

2015-04-30 Thread Paolo Bonzini
On 29/04/2015 21:20, Michael Roth wrote: If the parent is finalized as a result of object_unparent(), it will still be attached to the composition tree at the time any children are unparented as a result of that same call to object_unparent(). However, in some cases, object_unparent() will

[Qemu-devel] [PULL 05/10] s390x/mmu: Use access type definitions instead of magic values

2015-04-30 Thread Cornelia Huck
From: Thomas Huth th...@linux.vnet.ibm.com Since there are now proper definitions for the MMU access type, let's use them in the s390x MMU code, too, instead of the hard-to-understand magic values. Signed-off-by: Thomas Huth th...@linux.vnet.ibm.com Reviewed-by: Jens Freimann

[Qemu-devel] [PULL 08/10] s390x/mmu: Use ioctl for reading and writing from/to guest memory

2015-04-30 Thread Cornelia Huck
From: Thomas Huth th...@linux.vnet.ibm.com Add code to make use of the new ioctl for reading from / writing to virtual guest memory. By using the ioctl, the memory accesses are now protected with the so-called ipte-lock in the kernel. [CH: moved error message into kvm_s390_mem_op()]

[Qemu-devel] [PULL 06/10] linux-headers: update

2015-04-30 Thread Cornelia Huck
This updates linux-headers against master 4.1-rc1 (commit b787f68c36d49bb1d9236f403813641efa74a031). Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- include/standard-headers/linux/virtio_balloon.h | 28 +++- include/standard-headers/linux/virtio_ids.h | 1 +

Re: [Qemu-devel] [PATCH v2 2/3] qobject: Add a special null QObject

2015-04-30 Thread Markus Armbruster
Eric Blake ebl...@redhat.com writes: From: Markus Armbruster arm...@redhat.com I'm going to fix the JSON parser to recognize null. The obvious representation of JSON null as (QObject *)NULL doesn't work, because the parser already uses it as an error value. Perhaps we should change it to

[Qemu-devel] [PATCH v15 05/10] hw/arm/sysbus-fdt: enable vfio-calxeda-xgmac dynamic instantiation

2015-04-30 Thread Eric Auger
This patch allows the instantiation of the vfio-calxeda-xgmac device from the QEMU command line (-device vfio-calxeda-xgmac,host=device). A specialized device tree node is created for the guest, containing compat, dma-coherent, reg and interrupts properties. Signed-off-by: Eric Auger

[Qemu-devel] [PATCH v15 04/10] hw/vfio/platform: calxeda xgmac device

2015-04-30 Thread Eric Auger
The platform device class has become abstract. This patch introduces a calxeda xgmac device that derives from it. Signed-off-by: Eric Auger eric.au...@linaro.org Reviewed-by: Alex Benneealex.ben...@linaro.org --- v10 - v11: - add Alex Reviewed-by - move virt modifications in a separate patch v8

[Qemu-devel] [PATCH v4 5/7] vmport_rpc: Add migration

2015-04-30 Thread Don Slutz
Signed-off-by: Don Slutz dsl...@verizon.com --- hw/misc/vmport_rpc.c | 250 +++ trace-events | 8 +- 2 files changed, 255 insertions(+), 3 deletions(-) diff --git a/hw/misc/vmport_rpc.c b/hw/misc/vmport_rpc.c index 0ba3319..a147561 100644

[Qemu-devel] [PULL 09/10] s390x/kvm: Support access register mode for KVM_S390_MEM_OP ioctl

2015-04-30 Thread Cornelia Huck
From: Alexander Yarygin yary...@linux.vnet.ibm.com Access register mode is one of the modes that control dynamic address translation. In this mode the address space is specified by values of the access registers. The effective address-space-control element is obtained from the result of the

[Qemu-devel] [PULL 01/10] virtio-ccw: sort into categories

2015-04-30 Thread Cornelia Huck
Sort the various virtio-ccw devices into the same categories as their virtio-pci counterparts. Reviewed-by: David Hildenbrand d...@linux.vnet.ibm.com Acked-by: Christian Borntraeger borntrae...@de.ibm.com Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- hw/s390x/virtio-ccw.c | 9

[Qemu-devel] [PULL 02/10] s390-virtio: sort into categories

2015-04-30 Thread Cornelia Huck
Sort the various s390-virtio devices into the same categories as their virtio-pci counterparts. Reviewed-by: David Hildenbrand d...@linux.vnet.ibm.com Acked-by: Christian Borntraeger borntrae...@de.ibm.com Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- hw/s390x/s390-virtio-bus.c | 9

[Qemu-devel] [PATCH v15 02/10] hw/vfio/platform: vfio-platform skeleton

2015-04-30 Thread Eric Auger
Minimal VFIO platform implementation supporting register space user mapping but not IRQ assignment. Signed-off-by: Kim Phillips kim.phill...@linaro.org Signed-off-by: Eric Auger eric.au...@linaro.org --- v14 - v15: - vfio_platform_compute_needs_reset now returns true while

[Qemu-devel] [PATCH v15 00/10] KVM platform device passthrough

2015-04-30 Thread Eric Auger
This series aims at enabling KVM platform device passthrough. On kernel side, the vfio platform driver is needed, available from 4.1-rc1 onwards. This series now only relies on the following QEMU series, for dynamic instantiation of the VFIO platform device from qemu command line: [1] [PATCH

[Qemu-devel] [PATCH v15 06/10] kvm: rename kvm_irqchip_[add, remove]_irqfd_notifier with gsi suffix

2015-04-30 Thread Eric Auger
Anticipating for the introduction of new add/remove functions taking a qemu_irq parameter, let's rename existing ones with a gsi suffix. Signed-off-by: Eric Auger eric.au...@linaro.org --- hw/s390x/virtio-ccw.c | 8 hw/vfio/pci.c | 6 +++--- hw/virtio/virtio-pci.c | 4 ++--

Re: [Qemu-devel] [REBASE PATCH v5 1/2] machine: add default_ram_size to machine class

2015-04-30 Thread Paolo Bonzini
On 30/04/2015 11:40, Thomas Huth wrote: On 29/04/2015 11:06, Nikunj A Dadhania wrote: so David can push both patches. But isn't 1G a bit too much? At least on x86 you can easily boot with 512M. I understood this number as not the _minimum memory_ to boot the VM. And this will only come

Re: [Qemu-devel] [PULL 00/42] Trivial patches for 2015-04-30

2015-04-30 Thread Michael Tokarev
30.04.2015 15:58, Peter Maydell wrote: [] Hi. I'm afraid this fails to build the tests on OSX: CCtests/i440fx-test.o /Users/pm215/src/qemu/tests/i440fx-test.c:229:21: warning: implicit declaration of function 'ARRAY_SIZE' is invalid in C99 So much for trivial ;) Let's remove

[Qemu-devel] [v2 1/2] pci: GPEX: Add a function to determine interrupt number for INTx routing

2015-04-30 Thread Pranavkumar Sawargaonkar
This patch adds a PCI bus specific function pointer route_intx_to_irq for GPEX. This is used in detemining PCI INTx number from pin. Signed-off-by: Pranavkumar Sawargaonkar pranavku...@linaro.org Signed-off-by: Tushar Jagad tushar.ja...@linaro.org --- hw/pci-host/gpex.c | 12

[Qemu-devel] [v2 2/2] arm: hw: virt: Store information about GPEX legacy interrupt numbers

2015-04-30 Thread Pranavkumar Sawargaonkar
This patch stores information about assigned legacy interrupt numbers in GPEX host structure. This is used during GPEX INTx number determination from a pin during INTx routing. Signed-off-by: Pranavkumar Sawargaonkar pranavku...@linaro.org Signed-off-by: Tushar Jagad tushar.ja...@linaro.org ---

[Qemu-devel] [PULL 00/10] first pile of s390x patches for 2.4

2015-04-30 Thread Cornelia Huck
repository at: git://github.com/cohuck/qemu tags/s390x-20150430 for you to fetch changes up to 2c80e996e427ae31982f3405a762859578a6261d: kvm: better advice for failed s390x startup (2015-04-30 13:21:42 +0200) First pile of s390x

[Qemu-devel] [PULL 10/10] kvm: better advice for failed s390x startup

2015-04-30 Thread Cornelia Huck
If KVM_CREATE failed on s390x, we print a hint to enable the switch_amode kernel parameter. This only applies to old kernels, and only if the error was -EINVAL. Moreover, with new kernels, the most likely reason for -EINVAL is that pgstes were not enabled. Let's update the error message to give a

Re: [Qemu-devel] [PATCH v2 0/3] parse 'null' literal in QMP

2015-04-30 Thread Luiz Capitulino
On Thu, 30 Apr 2015 14:21:02 +0200 Markus Armbruster arm...@redhat.com wrote: Eric Blake ebl...@redhat.com writes: Here's my attempt to merge the best points of Markus' approach [1] (patches 16-18 of that series - benefit of smaller patches and fewer malloc calls) and my approach [2]

[Qemu-devel] [PATCH v15 03/10] hw/vfio/platform: add irq assignment

2015-04-30 Thread Eric Auger
This patch adds the code requested to assign interrupts to a guest. The interrupts are mediated through user handled eventfds only. Signed-off-by: Eric Auger eric.au...@linaro.org --- v13 - v14: - remove virtualID field in header v12 - v13: - start user-side eventfd handling at realize time -

[Qemu-devel] [PATCH v15 01/10] linux-headers: update

2015-04-30 Thread Eric Auger
From: Cornelia Huck cornelia.h...@de.ibm.com This updates linux-headers against master 4.1-rc1 (commit b787f68c36d49bb1d9236f403813641efa74a031). Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- include/standard-headers/linux/virtio_balloon.h | 28 +++-

[Qemu-devel] [PATCH v15 07/10] kvm-all.c: add qemu_irq/gsi hash table and utility routines

2015-04-30 Thread Eric Auger
VFIO platform device needs to setup irqfd but it does not know the gsi corresponding to the device qemu_irq. This series proposes to store a hash table in kvm_state using the qemu_irq as key and the gsi as a value. kvm_irqchip_set_qemuirq_gsi allows to insert such a pair. The interrupt controller

Re: [Qemu-devel] [PULL v2 00/22] Memory, TCG, NBD, build system changes for 2015-04-27

2015-04-30 Thread Peter Maydell
On 29 April 2015 at 09:35, Paolo Bonzini pbonz...@redhat.com wrote: The following changes since commit e1a5476354d396773e4c555f126d752d4ae58fa9: Open 2.4 development tree (2015-04-25 22:05:07 +0100) are available in the git repository at: git://github.com/bonzini/qemu.git

[Qemu-devel] [PATCH v15 10/10] hw/vfio/platform: add irqfd support

2015-04-30 Thread Eric Auger
This patch aims at optimizing IRQ handling using irqfd framework. Instead of handling the eventfds on user-side they are handled on kernel side using - the KVM irqfd framework, - the VFIO driver virqfd framework. the virtual IRQ completion is trapped at interrupt controller This removes the need

[Qemu-devel] [PATCH v4 0/7] Add limited support of VMware's hyper-call rpc

2015-04-30 Thread Don Slutz
Changes v4 to v4: Paolo Bonzini on vmort_rpc: Add QMP access to vmport_rpc Does this compile on non-x86 targets? Nope. Fixed. Changes v2 to v3: s/2.3/2.4 Changes v1 to v2: Added live migration code. Adjust data structures for migration. Switch to GHashTable. Eric

[Qemu-devel] [v2 0/2] Generic PCIe host bridge INTx determination for INTx routing

2015-04-30 Thread Pranavkumar Sawargaonkar
This patch adds a routine for GPEX to implement PCI bus specific function pointer route_intx_to_irq which is used during INTx routing. ChangeLog: V2: - Drop a patch about adding an API to get irq number from qemu_irq - Store a GPEX INTx information from board specific code (virt.c) V1: -

[Qemu-devel] [PULL 03/10] sclp: sort into categories

2015-04-30 Thread Cornelia Huck
Sort the sclp consoles into the input category, just as virtio-serial. Various other sclp devices don't have an obvious category, sort them into misc. Reviewed-by: David Hildenbrand d...@linux.vnet.ibm.com Acked-by: Christian Borntraeger borntrae...@de.ibm.com Signed-off-by: Cornelia Huck

[Qemu-devel] [PULL 04/10] s390x/ipl: sort into categories

2015-04-30 Thread Cornelia Huck
The s390 ipl device has no real home (it's not really a storage device), so let's sort it into the misc category. Reviewed-by: David Hildenbrand d...@linux.vnet.ibm.com Acked-by: Christian Borntraeger borntrae...@de.ibm.com Signed-off-by: Cornelia Huck cornelia.h...@de.ibm.com --- hw/s390x/ipl.c

[Qemu-devel] [PULL 07/10] s390x/kvm: Put vm name, extended name and UUID into STSI322 SYSIB

2015-04-30 Thread Cornelia Huck
From: Ekaterina Tumanova tuman...@linux.vnet.ibm.com KVM prefills the SYSIB, returned by STSI 3.2.2. This patch allows userspace to intercept execution, and fill in the values, that are known to qemu: machine name (8 chars), extended machine name (256 chars), extended machine name encoding

Re: [Qemu-devel] [PATCH v2 0/3] parse 'null' literal in QMP

2015-04-30 Thread Markus Armbruster
Eric Blake ebl...@redhat.com writes: Here's my attempt to merge the best points of Markus' approach [1] (patches 16-18 of that series - benefit of smaller patches and fewer malloc calls) and my approach [2] (benefit of a testsuite addition and more detailed commit messages), while fixing the

[Qemu-devel] [PATCH v15 09/10] sysbus: add irq_routing_notifier

2015-04-30 Thread Eric Auger
Add a new connect_irq_notifier notifier in the SysBusDeviceClass. This notifier, if populated, is called after sysbus_connect_irq. This mechanism is used to setup VFIO signaling once VFIO platform devices get attached to their platform bus, on a machine init done notifier. Signed-off-by: Eric

[Qemu-devel] [PATCH v15 08/10] intc: arm_gic_kvm: set the qemu_irq/gsi mapping

2015-04-30 Thread Eric Auger
The arm_gic_kvm now calls kvm_irqchip_set_qemuirq_gsi to build the hash table storing qemu_irq/gsi mappings. From that point on irqfd can be setup directly from the qemu_irq using kvm_irqchip_add_irqfd_notifier. Signed-off-by: Eric Auger eric.au...@linaro.org --- v2 - v3: -

[Qemu-devel] [PULL 00/42] Trivial patches for 2015-04-30

2015-04-30 Thread Michael Tokarev
Another attempt, now without the problematic ARRAY_SIZE removal, but with additional patch removing the unused cpu_get_pc(), and with additional Reviewed-by. Resending only the newly added patch, not whole series. Thanks, /mjt The following changes since commit

Re: [Qemu-devel] [PATCH 0/5] MAINTAINERS: split block layer

2015-04-30 Thread Kevin Wolf
Am 29.04.2015 um 16:13 hat Stefan Hajnoczi geschrieben: Kevin and I have been maintaining the block layer together. We take weekly turns reviewing/merging patches. The volume of traffic is so high that we struggle to give timely code reviews. This series adjusts MAINTAINERS to reflect how

[Qemu-devel] [PATCH v4 6/7] vmport: Add VMware all ring hack

2015-04-30 Thread Don Slutz
This is done by adding a new machine property vmware-port-ring3 that needs to be enabled to have any effect. It only effects accel=tcg mode. It is needed if you want to use VMware tools in accel=tcg mode. Signed-off-by: Don Slutz dsl...@verizon.com (cherry picked from commit

Re: [Qemu-devel] [REBASE PATCH v5 1/2] machine: add default_ram_size to machine class

2015-04-30 Thread Alexander Graf
Am 30.04.2015 um 11:40 schrieb Thomas Huth th...@redhat.com: On Thu, 30 Apr 2015 11:18:05 +0200 Alexander Graf ag...@suse.de wrote: On 30.04.15 06:41, Nikunj A Dadhania wrote: Hi Paolo, Paolo Bonzini pbonz...@redhat.com writes: On 29/04/2015 11:06, Nikunj A Dadhania wrote: so

[Qemu-devel] [PATCH] nvme: support NVME_VOLATILE_WRITE_CACHE feature

2015-04-30 Thread Christoph Hellwig
The SCSI emulation in the Linux NVMe driver really wants to know if a device has a volatile write cache. Given that qemu has moved away from a model where we report the backing store WCE bit to one where the WCE bit is supposed to be part of the migratable guest-visible state we always return 1

Re: [Qemu-devel] [PATCH 4/7] qom: add object_new_propv / object_new_proplist constructors

2015-04-30 Thread Paolo Bonzini
On 30/04/2015 16:42, Daniel P. Berrange wrote: + * object_new_propv: + * @typename: The name of the type of the object to instantiate. + * @path: the object path to register under + * @id: The unique ID of the object + * @errp: pointer to error object + * @...: list of property names and

Re: [Qemu-devel] [PATCH 4/7] qom: add object_new_propv / object_new_proplist constructors

2015-04-30 Thread Daniel P. Berrange
On Thu, Apr 30, 2015 at 04:53:27PM +0200, Paolo Bonzini wrote: On 30/04/2015 16:42, Daniel P. Berrange wrote: + * object_new_propv: + * @typename: The name of the type of the object to instantiate. + * @path: the object path to register under + * @id: The unique ID of the object +

Re: [Qemu-devel] [PATCH COLO v3 01/14] docs: block replication's description

2015-04-30 Thread Stefan Hajnoczi
On Wed, Apr 29, 2015 at 04:37:49PM +0800, Gonglei wrote: On 2015/4/29 16:29, Paolo Bonzini wrote: On 27/04/2015 11:37, Stefan Hajnoczi wrote: But it's only for the failover case. Quorum (or a new block/colo.c driver or filter) is fine for normal colo operation. Perhaps this

Re: [Qemu-devel] [PATCH 4/7] qom: add object_new_propv / object_new_proplist constructors

2015-04-30 Thread Paolo Bonzini
On 30/04/2015 16:42, Daniel P. Berrange wrote: +propname = va_arg(vargs, char *); +while (propname != NULL) { +const char *value = va_arg(vargs, char *); + +g_assert(value != NULL); +object_property_parse(obj, value, propname, errp); +if (*errp) {

Re: [Qemu-devel] [PATCH 4/7] qom: add object_new_propv / object_new_proplist constructors

2015-04-30 Thread Daniel P. Berrange
On Thu, Apr 30, 2015 at 05:00:28PM +0200, Paolo Bonzini wrote: On 30/04/2015 16:42, Daniel P. Berrange wrote: +propname = va_arg(vargs, char *); +while (propname != NULL) { +const char *value = va_arg(vargs, char *); + +g_assert(value != NULL); +

Re: [Qemu-devel] [PATCH v4 0/3] block: Fix unaligned bdrv_aio_write_zeroes

2015-04-30 Thread Kevin Wolf
Am 27.04.2015 um 15:18 hat Fam Zheng geschrieben: An unaligned zero write causes NULL deferencing in bdrv_co_do_pwritev. That path is reachable from bdrv_co_write_zeroes and bdrv_aio_write_zeroes. You can easily trigger through the former with qemu-io, as the test case added by 61815d6e0aa.

Re: [Qemu-devel] [PATCH] Enable NVMe start controller for Windows guest.

2015-04-30 Thread Kevin Wolf
[Cc: qemu-block] Am 24.04.2015 um 21:19 hat Keith Busch geschrieben: On Fri, 24 Apr 2015, Daniel Stekloff wrote: Windows seems to send two separate calls to NVMe controller configuration. The first sends configuration info and the second the enable bit. I couldn't enable the Windows 8.1

Re: [Qemu-devel] [PATCH 0/7] virtio: inline private qdev properties into virtio devices

2015-04-30 Thread Cornelia Huck
On Wed, 29 Apr 2015 23:24:02 +0800 Shannon Zhao shannon.z...@linaro.org wrote: The private qdev properties of virtio devices are only used by themselves. As Peter suggested and like what virtio-blk has done, we should move the private qdev properties into devices and don't expose them to

Re: [Qemu-devel] [PULL 00/10] first pile of s390x patches for 2.4

2015-04-30 Thread Peter Maydell
-20150428.0' into staging (2015-04-28 18:58:15 +0100) are available in the git repository at: git://github.com/cohuck/qemu tags/s390x-20150430 for you to fetch changes up to 2c80e996e427ae31982f3405a762859578a6261d: kvm: better advice for failed s390x startup (2015-04-30 13:21:42 +0200

  1   2   3   >