Re: [Qemu-devel] [PATCH] Exit on reset for armv7-m

2015-10-08 Thread Peter Crosthwaite
On Thu, Oct 8, 2015 at 6:24 PM, Michael Davidsaver wrote: > On 10/08/2015 04:09 PM, Peter Crosthwaite wrote: >> On Thu, Oct 8, 2015 at 8:40 AM, Michael Davidsaver >> wrote: >>> Implement the SYSRESETREQ bit of the AIRCR register >>> for armv7-m (ie. cortex-m3). >>> >> >> This would serve better a

Re: [Qemu-devel] [PATCH 2/5] hw/scsi/spapr_vscsi: Remove superfluous memset

2015-10-08 Thread David Gibson
On Thu, Oct 08, 2015 at 09:35:13PM +0200, Thomas Huth wrote: > g_malloc0 already clears the memory, so no need for > the additional memset here. > > Cc: Paolo Bonzini > Cc: David Gibson > Cc: Alexander Graf > Signed-off-by: Thomas Huth Thanks, applied to spapr-next. -- David Gibson

Re: [Qemu-devel] [PATCH v2 00/16] block: Get rid of bdrv_swap()

2015-10-08 Thread Fam Zheng
On Thu, 10/01 15:13, Kevin Wolf wrote: > bdrv_swap() has always been an ugly hack that we would rather have > avoided. When it was introduced, we simply didn't have the > infrastructure to update pointers instead of transplanting the contents > of BDS object, so we grudgingly added bdrv_swap() as

Re: [Qemu-devel] [PATCH] Add syscalls for -runas and -chroot to the seccomp sandbox

2015-10-08 Thread namnamc
> ps.: the threads are still being broken by your emails and it's a pain > to track down all of them in order to read. Please fix it. I'm really sorry, I am not able to sign up to Google because I don't have a cell number. I'll try using Sigaint. Does it work now?

Re: [Qemu-devel] [PATCH v2 08/16] block: Manage backing file references in bdrv_set_backing_hd()

2015-10-08 Thread Fam Zheng
On Thu, 10/01 15:13, Kevin Wolf wrote: > This simplifies the code somewhat, especially when dropping whole > backing file subchains. > > The exception is the mirroring code that does adventurous things with > bdrv_swap() and in order to keep it working, I had to duplicate most of > bdrv_set_backin

[Qemu-devel] x86 amd64 singlestepping bug through syscall instruction

2015-10-08 Thread Rudolf Marek
Hi all, I was told on IRC to use this ML to report the following bug. It seems that there is something wrong with QEMU with respect to handle the singlestepping and AMD64 syscall instruction. The AMD "syscall" instruction will clear defined flag in the FMASK MSR. Normally the TF flag is set

Re: [Qemu-devel] [PATCH v2 07/16] block: Convert bs->backing_hd to BdrvChild

2015-10-08 Thread Fam Zheng
On Thu, 10/01 15:13, Kevin Wolf wrote: > This is the final step in converting all of the BlockDriverState > pointers that block drivers use to BdrvChild. > > After this patch, bs->children contains the full list of child nodes > that are referenced by a given BDS, and these children are only > ref

Re: [Qemu-devel] [PATCH] Exit on reset for armv7-m

2015-10-08 Thread Michael Davidsaver
On 10/08/2015 04:09 PM, Peter Crosthwaite wrote: > On Thu, Oct 8, 2015 at 8:40 AM, Michael Davidsaver > wrote: >> Implement the SYSRESETREQ bit of the AIRCR register >> for armv7-m (ie. cortex-m3). >> > > This would serve better as the commit message to the patch (which I > notice is missing a co

Re: [Qemu-devel] [PATCH v3 0/5] qom: more efficient object property handling

2015-10-08 Thread Eric Blake
On 10/08/2015 08:08 AM, Daniel P. Berrange wrote: > This patch series is a combination of my own previous patch to > add support for object properties against classes: > > https://lists.gnu.org/archive/html/qemu-devel/2015-09/msg05953.html > > And Pavel Fedin's patch to use a hash table instead

[Qemu-devel] [PATCH 5/5] linux-user/syscall: Replace g_malloc0 + memcpy with g_memdup

2015-10-08 Thread Thomas Huth
No need to use g_malloc0 to zero the memory if we memcpy to the whole buffer afterwards anyway. Actually, there is even a function which combines both steps, g_memdup, so let's use this function here instead. Cc: Riku Voipio Signed-off-by: Thomas Huth --- linux-user/syscall.c | 3 +-- 1 file ch

[Qemu-devel] [PULL 17/19] linux-user: Use g_new() & friends where that makes obvious sense

2015-10-08 Thread Michael Tokarev
From: Markus Armbruster g_new(T, n) is neater than g_malloc(sizeof(T) * n). It's also safer, for two reasons. One, it catches multiplication overflowing size_t. Two, it returns T * rather than void *, which lets the compiler catch more type errors. This commit only touches allocations with siz

[Qemu-devel] [PATCH 3/5] hw/input/tsc210x: Remove superfluous memset

2015-10-08 Thread Thomas Huth
g_malloc0 already clears the memory, so no need for additional memsets here. And while we're at it, let's also remove the superfluous typecasts for the return values of g_malloc0. Signed-off-by: Thomas Huth --- hw/input/tsc210x.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) dif

Re: [Qemu-devel] [PATCH 2/5] hw/scsi/spapr_vscsi: Remove superfluous memset

2015-10-08 Thread Eric Blake
On 10/08/2015 01:35 PM, Thomas Huth wrote: > g_malloc0 already clears the memory, so no need for > the additional memset here. > > Cc: Paolo Bonzini > Cc: David Gibson > Cc: Alexander Graf > Signed-off-by: Thomas Huth > --- > hw/scsi/spapr_vscsi.c | 1 - > 1 file changed, 1 deletion(-) Revie

Re: [Qemu-devel] [PATCH 6/8] migration: implementation of hook_ram_sync

2015-10-08 Thread Denis V. Lunev
On 10/07/2015 12:44 PM, Paolo Bonzini wrote: On 07/10/2015 08:20, Denis V. Lunev wrote: All calls of this hook will be from ram_save_pending(). At the first call of this hook we need to save the initial size of VM memory and put the migration thread to sleep for decent period (downtime for exa

Re: [Qemu-devel] [PATCH 5/5] linux-user/syscall: Replace g_malloc0 + memcpy with g_memdup

2015-10-08 Thread Eric Blake
On 10/08/2015 01:35 PM, Thomas Huth wrote: > No need to use g_malloc0 to zero the memory if we memcpy to > the whole buffer afterwards anyway. Actually, there is even > a function which combines both steps, g_memdup, so let's use > this function here instead. > > Cc: Riku Voipio > Signed-off-by:

[Qemu-devel] [PULL 11/19] MAINTAINERS: Add NSIS file for W32, W64 hosts

2015-10-08 Thread Michael Tokarev
From: Stefan Weil The NSIS installer configuration is maintained by me. Signed-off-by: Stefan Weil Signed-off-by: Michael Tokarev --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7603ea2..9bde832 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@

[Qemu-devel] [PATCH v5 1/6] fw_cfg: document fw_cfg_modify_iXX() update functions

2015-10-08 Thread Marc Marí
From: "Gabriel L. Somlo" Document the behavior of fw_cfg_modify_iXX() for leak-less updating of integer-type blobs. Currently only fw_cfg_modify_i16() is coded, but 32- and 64-bit versions may be added later if necessary.. Signed-off-by: Gabriel Somlo Signed-off-by: Gerd Hoffmann Reviewed-by:

[Qemu-devel] [PULL 02/19] sdhci: use PRIx64 for uint64_t type

2015-10-08 Thread Michael Tokarev
From: Sai Pavan Boddu Fix compile time warnings, because of type mismatch for unsigned long long type. Signed-off-by: Sai Pavan Boddu Reviewed-by: Peter Crosthwaite Signed-off-by: Michael Tokarev --- hw/sd/sdhci.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/

[Qemu-devel] [PULL 06/19] pci-assign: do not include sys/io.h

2015-10-08 Thread Michael Tokarev
From: Paolo Bonzini This file does not exist on bionic libc and the functions it defines are in fact not used by pci-assign.c. Remove it. Reported-by: Houcheng Lin Signed-off-by: Paolo Bonzini Signed-off-by: Michael Tokarev --- hw/i386/kvm/pci-assign.c | 1 - 1 file changed, 1 deletion(-)

Re: [Qemu-devel] [PATCH v3 4/5] qom: replace object property list with GHashTable

2015-10-08 Thread Pavel Fedin
Hello! > -Original Message- > From: Daniel P. Berrange [mailto:berra...@redhat.com] > Sent: Thursday, October 08, 2015 5:09 PM > To: qemu-devel@nongnu.org > Cc: Andreas Färber; Pavel Fedin; Daniel P. Berrange > Subject: [PATCH v3 4/5] qom: replace object property list with GHashTable > >

[Qemu-devel] [PULL 23/25] vhost-user-test: add live-migration test

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau This test checks that the log fd is given to the migration source, and mark dirty pages during migration. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/vhost-user-test.c | 171 +

Re: [Qemu-devel] [PATCH 1/4] hw/core: Add iommu to machine properties

2015-10-08 Thread Marcel Apfelbaum
On 10/09/2015 05:53 AM, David Kiarie wrote: From: David Add iommu to machine properties in preparation of introducing AMD IOMMU Signed-off-by: David Kiarie --- hw/core/machine.c | 25 + include/hw/boards.h | 2 ++ 2 files changed, 27 insertions(+) diff --gi

Re: [Qemu-devel] [PATCH v2] exec: factor out duplicate mmap code

2015-10-08 Thread Richard Henderson
On 10/09/2015 07:38 AM, Michael S. Tsirkin wrote: Any objections to merging as is, and renaming later as appropriate? None. r~ On Sun, Oct 04, 2015 at 11:38:59AM +0300, Michael S. Tsirkin wrote: On Fri, Oct 02, 2015 at 10:48:13AM +1000, Richard Henderson wrote: On 10/01/2015 10:58 PM, Mic

Re: [Qemu-devel] [PATCH 2/2] Hw: timer: Remove unnecessary variable

2015-10-08 Thread Michael Tokarev
05.10.2015 18:29, Eric Blake wrote: > On 10/03/2015 10:59 AM, Michael Tokarev wrote: >> 25.09.2015 17:36, Shraddha Barke wrote: >>> Compress lines and remove the variable. >> >> Applied to -trivial, removing the Coccinelle script >> from the commit message. > > Why cripple the commit message? The

Re: [Qemu-devel] [PATCH 3/5] hw/input/tsc210x: Remove superfluous memset

2015-10-08 Thread Eric Blake
On 10/08/2015 01:35 PM, Thomas Huth wrote: > g_malloc0 already clears the memory, so no need for additional > memsets here. And while we're at it, let's also remove the > superfluous typecasts for the return values of g_malloc0. > > Signed-off-by: Thomas Huth > --- > hw/input/tsc210x.c | 8 ++---

[Qemu-devel] [PULL 24/25] vhost-user-test: check ownership during migration

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Check that backend source and destination do not have simultaneous ownership during migration. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/vhost-user-test.c | 38 ++ 1

Re: [Qemu-devel] [PATCH v2 5/5] xlnx-ep108: Connect the SPI Flash

2015-10-08 Thread Alistair Francis
On Wed, Oct 7, 2015 at 5:01 PM, Peter Crosthwaite wrote: > On Wed, Oct 7, 2015 at 2:34 PM, Alistair Francis > wrote: >> Connect the sst25wf080 SPI flash to the EP108 board. >> >> Signed-off-by: Alistair Francis >> --- >> V2: >> - Use sst25wf080 instead of m25p80 >> >> hw/arm/xlnx-ep108.c | 20

[Qemu-devel] [PULL 04/19] imx_serial: Generate interrupt on tx empty if enabled

2015-10-08 Thread Michael Tokarev
From: Guenter Roeck Generate an interrupt if the tx buffer is empty and the tx empty interrupt is enabled. This fixes a problem seen when running a Linux image since Linux commit 55c3cb1358e ("serial: imx: remove unneeded imx_transmit_buffer() from imx_start_tx()"). Linux now waits for the tx emp

Re: [Qemu-devel] [PATCH 1/5] hw/dma/pxa2xx: Remove superfluous memset

2015-10-08 Thread Eric Blake
On 10/08/2015 01:35 PM, Thomas Huth wrote: > g_malloc0 already clears the memory, so no need for > the additional memset here. > > Signed-off-by: Thomas Huth > --- > hw/dma/pxa2xx_dma.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/hw/dma/pxa2xx_dma.c b/hw/dma/pxa2xx_dma.c > index d45

Re: [Qemu-devel] [PATCH v5 3/6] Implement fw_cfg DMA interface

2015-10-08 Thread Laszlo Ersek
On 10/08/15 17:02, Marc Marí wrote: > Based on the specifications on docs/specs/fw_cfg.txt > > This interface is an addon. The old interface can still be used as usual. > > Based on Gerd Hoffman's initial implementation. > > Signed-off-by: Marc Marí > --- > hw/arm/virt.c | 2 +- >

Re: [Qemu-devel] [PATCH] target-mips: Add enum for BREAK32

2015-10-08 Thread Aurelien Jarno
On 2015-10-02 17:50, Yongbok Kim wrote: > Add enum for BREAK32 > > Signed-off-by: Yongbok Kim > --- > target-mips/translate.c |3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) Reviewed-by: Aurelien Jarno -- Aurelien Jarno GPG: 4096R/1DDD8C9B aurel...@aur

[Qemu-devel] [PATCH 1/5] hw/dma/pxa2xx: Remove superfluous memset

2015-10-08 Thread Thomas Huth
g_malloc0 already clears the memory, so no need for the additional memset here. Signed-off-by: Thomas Huth --- hw/dma/pxa2xx_dma.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/dma/pxa2xx_dma.c b/hw/dma/pxa2xx_dma.c index d4501fb..3f0a720 100644 --- a/hw/dma/pxa2xx_dma.c +++ b/hw/dma/pxa

Re: [Qemu-devel] [PATCH] tests: Unique test path for /string-visitor/output

2015-10-08 Thread Andreas Färber
Am 05.10.2015 um 13:04 schrieb Dr. David Alan Gilbert (git): > From: "Dr. David Alan Gilbert" > > Newer GLib's want unique test paths, and thus moan at dupes. > (Seen on Fedora 23 which has glib 2.46) > > Uniqueify the paths. > > Signed-off-by: Dr. David Alan Gilbert > --- > tests/test-string

Re: [Qemu-devel] [PATCH v3 3/6] tcg/mips: Add use_mips32r6_instructions definition

2015-10-08 Thread Aurelien Jarno
On 2015-10-02 13:24, James Hogan wrote: > Add definition use_mips32r6_instructions to the MIPS TCG backend which > is constant 1 when built for MIPS release 6. This will be used to decide > between pre-R6 and R6 instruction encodings. > > Signed-off-by: James Hogan > Reviewed-by: Richard Henderso

Re: [Qemu-devel] [PATCH] target-tilegx: Let prefetch nop instructions return before allocating dest temporary register

2015-10-08 Thread Chen Gang
On 10/7/15 18:17, Chen Gang wrote: > On 10/7/15 17:19, Richard Henderson wrote: >> On 10/04/2015 10:15 PM, Chen Gang wrote: From 40ec3f1c75b4c97e3e0495c9e465be898f48a652 Mon Sep 17 00:00:00 2001 >>> From: Chen Gang >>> Date: Sun, 4 Oct 2015 17:34:17 +0800 >>> Subject: [PATCH] target-tilegx: Le

[Qemu-devel] [PATCH v5 6/6] fw_cfg: Define a static signature to be returned on DMA port reads

2015-10-08 Thread Marc Marí
From: Kevin O'Connor Return a static signature ("QEMU CFG") if the guest does a read to the DMA address io register. Signed-off-by: Kevin O'Connor Reviewed-by: Laszlo Ersek Reviewed-by: Stefan Hajnoczi --- docs/specs/fw_cfg.txt | 3 +++ hw/nvram/fw_cfg.c | 14 -- 2 files cha

[Qemu-devel] [PULL 22/25] vhost-user-test: learn to tweak various qemu arguments

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Add a new macro to make the qemu command line with other values of memory size, and specific chardev id. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/vhost-user-test.c | 25 +++-- 1 fi

[Qemu-devel] [PATCH 4/5] tests/i44fx-test: No need for zeroing memory before memset

2015-10-08 Thread Thomas Huth
Change a g_malloc0 into g_malloc since the following memset fills the whole buffer anyway. Cc: Laszlo Ersek Signed-off-by: Thomas Huth --- tests/i440fx-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/i440fx-test.c b/tests/i440fx-test.c index d0bc8de..7fa1709 100

Re: [Qemu-devel] [PATCH] Exit on reset for armv7-m

2015-10-08 Thread Peter Crosthwaite
On Thu, Oct 8, 2015 at 8:40 AM, Michael Davidsaver wrote: > Implement the SYSRESETREQ bit of the AIRCR register > for armv7-m (ie. cortex-m3). > This would serve better as the commit message to the patch (which I notice is missing a commit blurb). > A small patch to see if I have the submission

[Qemu-devel] [PULL 08/25] vhost-user: add vhost_user_requires_shm_log()

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Check if the backend has VHOST_USER_PROTOCOL_F_LOG_SHMFD feature and require a shared log. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/virtio/vhost-backend.h | 4 hw/virtio/vhost-user.c

[Qemu-devel] [PULL 06/25] vhost: document log resizing

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/virtio/vhost.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c index c0ed5b2..67e09fe 100644 --- a/hw/virtio/vhost.c ++

[Qemu-devel] [PULL 07/25] vhost: add vhost_set_log_base op

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Split VHOST_SET_LOG_BASE call in a seperate function callback, so that type safety works and more arguments can be added in the next patches. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/virtio/vhost

Re: [Qemu-devel] Qemu - Install Windows 7 64bits hang on startup with Blue Screen of Death

2015-10-08 Thread Stefan Weil
Am 07.10.2015 um 13:41 schrieb Aaron Elkins: > Hi, > > With this command line: > > qemu-system-x86_64 -boot d -cdrom /dev/disk1 -hda windows7.img -m 1024 > > and I have my windows 7 cd in the cd rom, and the installation process > hang on startup, here is > the screen shot: > > https://www.dropbox

[Qemu-devel] [PULL 00/25] virtio,pc features, fixes

2015-10-08 Thread Michael S. Tsirkin
The following changes since commit 1d27b91723c252d9a97151dc1959cfd89c5816cb: Merge remote-tracking branch 'remotes/awilliam/tags/vfio-update-20151007.0' into staging (2015-10-08 16:50:34 +0100) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/f

[Qemu-devel] [PULL 17/25] vhost-user: use an enum helper for features mask

2015-10-08 Thread Michael S. Tsirkin
From: Thibaut Collet The VHOST_USER_PROTOCOL_FEATURE_MASK will be automatically updated when adding new features to the enum. Signed-off-by: Thibaut Collet [Adapted from mailing list discussion - Marc-André] Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Micha

[Qemu-devel] [PULL 21/25] vhost-user-test: wrap server in TestServer struct

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau In the coming patches, a test will use several servers simultaneously. Wrap the server in a struct, out of the global scope. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/vhost-user-test.c | 139 ++

Re: [Qemu-devel] [PATCH v3 5/9] target-arm: Add ARMMMUFaultInfo

2015-10-08 Thread Peter Maydell
On 8 October 2015 at 21:06, Edgar E. Iglesias wrote: > On Wed, Oct 07, 2015 at 05:24:27PM +0100, Alex Bennée wrote: >> >> Edgar E. Iglesias writes: >> >> > +/** >> > + * ARMMMUFaultInfo: Information describing an ARM MMU Fault >> > + * @s2addr: Address that caused a fault at stage 2 >> > + * @sta

[Qemu-devel] [PULL 19/25] vhost-user-test: move wait_for_fds() out

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau This function is a precondition for most vhost-user tests. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/vhost-user-test.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --

[Qemu-devel] [PULL 11/25] vhost-user: add a migration blocker

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau If VHOST_USER_PROTOCOL_F_LOG_SHMFD is not announced, block vhost-user migration. The blocker is removed in vhost_dev_cleanup(). Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/virtio/vhost-user.c | 9 +

[Qemu-devel] [PULL 04/25] util: add memfd helpers

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Add qemu_memfd_alloc/free() helpers. The function helps to allocate and seal shared memory. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/qemu/memfd.h | 4 +++ util/memfd.c | 75 +++

[Qemu-devel] [PULL 01/25] configure: probe for memfd

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Check if memfd_create() is part of system libc. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- configure | 19 +++ 1 file changed, 19 insertions(+) diff --git a/configure b/configure index 2d2a49

[Qemu-devel] [PULL 03/25] util: add linux-only memfd fallback

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Implement memfd_create() fallback if not available in system libc. memfd_create() is still not included in glibc today, atlhough it's been available since Linux 3.17 in Oct 2014. memfd has numerous advantages over traditional shm/mmap for ipc memory sharing with fd handle

[Qemu-devel] [PULL 05/25] util: add fallback for qemu_memfd_alloc()

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Add an open/unlink/mmap fallback for system that do not support memfd (only available since 3.17, ~1y ago). This patch may require additional SELinux policies to work for enforced systems, but should fail gracefully in this case. Signed-off-by: Marc-André Lureau Reviewe

[Qemu-devel] [PULL 20/25] vhost-user-test: remove useless static check

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/vhost-user-test.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/vhost-user-test.c b/tests/vhost-user-test.c index 827c0c4..4be5583 100644 --- a/tests

[Qemu-devel] [PULL 16/25] vhost user: add rarp sending after live migration for legacy guest

2015-10-08 Thread Michael S. Tsirkin
From: Thibaut Collet A new vhost user message is added to allow QEMU to ask to vhost user backend to broadcast a fake RARP after live migration for guest without GUEST_ANNOUNCE capability. This new message is sent only if the backend supports the new VHOST_USER_PROTOCOL_F_RARP protocol feature.

[Qemu-devel] [PULL 15/25] vhost user: add support of live migration

2015-10-08 Thread Michael S. Tsirkin
From: Thibaut Collet Some vhost user backends are able to support live migration. To provide this service the following features must be added: 1. Add the VIRTIO_NET_F_GUEST_ANNOUNCE capability to vhost-net when netdev backend is vhost-user. 2. Provide a nop receive callback to vhost-user.

[Qemu-devel] [PULL 18/25] vhost: add migration block if memfd failed

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/qemu/memfd.h | 2 ++ hw/virtio/vhost.c| 3 +++ util/memfd.c | 22 ++ 3 files changed, 27 insertions(+) diff --git a/include

[Qemu-devel] [PULL 13/25] vhost-user: document migration log

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- docs/specs/vhost-user.txt | 48 +-- 1 file changed, 46 insertions(+), 2 deletions(-) diff --git a/docs/specs/vhost-user.t

[Qemu-devel] [PULL 14/25] net: add trace_vhost_user_event

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Replace error_report() and use tracing instead. It's not an error to get a connection or a disconnection, so silence this and trace it instead. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- net/vhost-user.c | 4

[Qemu-devel] [PATCH] target-arm: Fix "no 64-bit EL2" assumption in arm_excp_unmasked()

2015-10-08 Thread Peter Maydell
The code in arm_excp_unmasked() suppresses the ability of PSTATE.AIF to mask exceptions from a lower EL targeting EL2 or EL3 if the CPU is 64-bit. This is correct for a target of EL3, but not correct for targeting EL2. Further, we go to some effort to calculate scr and hcr values which are not used

[Qemu-devel] [PULL 10/25] vhost-user: send log shm fd along with log_base

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Send the shm for the dirty pages logging if the backend supports VHOST_USER_PROTOCOL_F_LOG_SHMFD. Wait for a reply to make sure the old log is no longer used. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- includ

[Qemu-devel] [PULL 09/25] vhost: alloc shareable log

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau If the backend is requires it, allocate shareable memory. vhost_log_get() now uses 2 globals "vhost_log" and "vhost_log_shm", that way there is a common non-shareable log and a common shareable one. Signed-off-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Signe

[Qemu-devel] [PULL 12/25] vhost: use a function for each call

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau Replace the generic vhost_call() by specific functions for each function call to help with type safety and changing arguments. While doing this, I found that "unsigned long long" and "uint64_t" were used interchangeably and causing compilation warnings, using uint64_t ins

[Qemu-devel] [PULL 25/25] intel_iommu: Add support for translation for devices behind bridges

2015-10-08 Thread Michael S. Tsirkin
From: Knut Omang - Use a hash table indexed on bus pointers to store information about buses instead of using the bus numbers. Bus pointers are stored in a new VTDBus struct together with the vector of device address space pointers indexed by devfn. - The bus number is still used for lookup

[Qemu-devel] [PULL 02/25] linux-headers: add unistd.h

2015-10-08 Thread Michael S. Tsirkin
From: Marc-André Lureau New syscalls are not yet widely distributed. Add them to qemu linux-headers include directory. Update based on v4.3-rc3 kernel headers. Exclude mips for now, which is more problematic due to extra header inclusion and probably unnecessary here. Signed-off-by: Marc-André

Re: [Qemu-devel] [PATCH 03/17] spec: add qcow2-dirty-bitmaps specification

2015-10-08 Thread Denis V. Lunev
On 10/08/2015 11:28 PM, John Snow wrote: On 10/07/2015 03:05 PM, Denis V. Lunev wrote: On 10/07/2015 07:47 PM, Max Reitz wrote: On 05.09.2015 18:43, Vladimir Sementsov-Ogievskiy wrote: Persistent dirty bitmaps will be saved into qcow2 files. It may be used as 'internal' bitmaps (for qcow2 dri

Re: [Qemu-devel] [PATCH v7 00/24] vhost-user: add migration support

2015-10-08 Thread Michael S. Tsirkin
On Thu, Oct 01, 2015 at 07:23:44PM +0200, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Hi, > > The following series implement shareable log for vhost-user to support > memory tracking during live migration. On qemu-side, the solution is > fairly straightfoward since vhost alr

Re: [Qemu-devel] [PATCH v2 1/2] target-arm: Fix GDB breakpoint handling

2015-10-08 Thread Peter Maydell
On 28 September 2015 at 11:07, Sergey Fedorov wrote: > GDB breakpoints have higher priority so they have to be checked first. > Should GDB breakpoint match, just return from the debug exception > handler. > > Signed-off-by: Sergey Fedorov > --- > target-arm/op_helper.c | 9 + > 1 file ch

Re: [Qemu-devel] [PATCH v2] exec: factor out duplicate mmap code

2015-10-08 Thread Michael S. Tsirkin
Any objections to merging as is, and renaming later as appropriate? On Sun, Oct 04, 2015 at 11:38:59AM +0300, Michael S. Tsirkin wrote: > On Fri, Oct 02, 2015 at 10:48:13AM +1000, Richard Henderson wrote: > > On 10/01/2015 10:58 PM, Michael S. Tsirkin wrote: > > >Anonymous and file-backed RAM allo

Re: [Qemu-devel] [PATCH 03/17] spec: add qcow2-dirty-bitmaps specification

2015-10-08 Thread John Snow
On 10/07/2015 03:05 PM, Denis V. Lunev wrote: > On 10/07/2015 07:47 PM, Max Reitz wrote: >> On 05.09.2015 18:43, Vladimir Sementsov-Ogievskiy wrote: >>> Persistent dirty bitmaps will be saved into qcow2 files. It may be used >>> as 'internal' bitmaps (for qcow2 drives) or as 'external' bitmaps fo

Re: [Qemu-devel] [PATCH 4/5] tests/i44fx-test: No need for zeroing memory before memset

2015-10-08 Thread Laszlo Ersek
On 10/08/15 21:35, Thomas Huth wrote: > Change a g_malloc0 into g_malloc since the following > memset fills the whole buffer anyway. > > Cc: Laszlo Ersek > Signed-off-by: Thomas Huth > --- > tests/i440fx-test.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/i440

Re: [Qemu-devel] [PATCH v3 5/9] target-arm: Add ARMMMUFaultInfo

2015-10-08 Thread Edgar E. Iglesias
On Wed, Oct 07, 2015 at 05:24:27PM +0100, Alex Bennée wrote: > > Edgar E. Iglesias writes: > > > From: "Edgar E. Iglesias" > > > > Introduce ARMMMUFaultInfo to propagate MMU Fault information > > across the MMU translation code path. This is in preparation for > > adding State-2 translation. >

[Qemu-devel] [PULL 01/19] Add .dir-locals.el file to configure emacs coding style

2015-10-08 Thread Michael Tokarev
From: "Daniel P. Berrange" Some default emacs setups indent by 2 spaces and uses tabs which is counter to the QEMU coding style rules. Adding a .dir-locals.el file in the top level of the GIT repo will inform emacs about the QEMU coding style, and so assist contributors in avoiding common style m

Re: [Qemu-devel] [PATCH v3 1/9] target-arm: Add HPFAR_EL2

2015-10-08 Thread Edgar E. Iglesias
On Thu, Oct 08, 2015 at 10:14:08AM +0100, Alex Bennée wrote: > > Edgar E. Iglesias writes: > > > From: "Edgar E. Iglesias" > > > > Signed-off-by: Edgar E. Iglesias > > Now Peter has pointed out I can't read ;-) > > Reviewed-by: Alex Bennée Thanks for all the clarifications, I'll add your

Re: [Qemu-devel] [PATCH v3 5/5] qom: allow properties to be registered against classes

2015-10-08 Thread Eric Blake
On 10/08/2015 08:09 AM, Daniel P. Berrange wrote: > When there are many instances of a given class, registering > properties against the instance is wasteful of resources. The > majority of objects have a statically defined list of possible > properties, so most of the properties are easily registe

[Qemu-devel] [PATCH v3 5/5] qom: allow properties to be registered against classes

2015-10-08 Thread Daniel P. Berrange
When there are many instances of a given class, registering properties against the instance is wasteful of resources. The majority of objects have a statically defined list of possible properties, so most of the properties are easily registerable against the class. Only those properties which are c

Re: [Qemu-devel] [PATCH] target-arm: Avoid calling arm_el_is_aa64() function for unimplemented EL

2015-10-08 Thread Peter Maydell
On 6 October 2015 at 18:43, Sergey Sorokin wrote: > That is ok. Thanks; I've now applied your patch to target-arm.next and sent out the patch to fix the no-64-bit-EL2 assumptions in the code. -- PMM

Re: [Qemu-devel] [PATCH v3 2/9] target-arm: Add computation of starting level for S2 PTW

2015-10-08 Thread Edgar E. Iglesias
On Wed, Oct 07, 2015 at 01:24:27PM +0100, Alex Bennée wrote: > > Edgar E. Iglesias writes: > > > From: "Edgar E. Iglesias" > > > > The starting level for S2 pagetable walks is computed > > differently from the S1 starting level. Implement the S2 > > variant. > > > > Signed-off-by: Edgar E. Igle

[Qemu-devel] [PATCH 2/5] hw/scsi/spapr_vscsi: Remove superfluous memset

2015-10-08 Thread Thomas Huth
g_malloc0 already clears the memory, so no need for the additional memset here. Cc: Paolo Bonzini Cc: David Gibson Cc: Alexander Graf Signed-off-by: Thomas Huth --- hw/scsi/spapr_vscsi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/scsi/spapr_vscsi.c b/hw/scsi/spapr_vscsi.c index 891

[Qemu-devel] [PATCH 0/5] Small optimizations for code using g_malloc0 + memset/memcpy

2015-10-08 Thread Thomas Huth
There are a couple of spots in the QEMU code which use g_malloc0, directly followed by a memset or memcpy which fill the whole allocated buffer. In this case it either does not make sense to zero the buffer via g_malloc0 first (so g_malloc should be used instead), or if the second command is a mems

Re: [Qemu-devel] [PATCH v3 5/9] target-arm: Add ARMMMUFaultInfo

2015-10-08 Thread Edgar E. Iglesias
On Wed, Oct 07, 2015 at 05:24:27PM +0100, Alex Bennée wrote: > > Edgar E. Iglesias writes: > > > From: "Edgar E. Iglesias" > > > > Introduce ARMMMUFaultInfo to propagate MMU Fault information > > across the MMU translation code path. This is in preparation for > > adding State-2 translation. >

[Qemu-devel] [PULL 00/19] Trivial patches for 2015-10-08

2015-10-08 Thread Michael Tokarev
Another trivial-patches pull request. Everything's trivial in there this time. Please consider applying/pulling. Thank you! /mjt The following changes since commit 31c9bd164ddb653915b9029ba0edd40cd57530d9: Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20151007' into staging (2015-

Re: [Qemu-devel] Qemu - Install Windows 7 64bits hang on startup with Blue Screen of Death

2015-10-08 Thread Stefan Weil
Am 08.10.2015 um 19:57 schrieb Olga Krishtal: > On 07/10/15 14:41, Aaron Elkins wrote: >> Hi, >> >> With this command line: >> >> qemu-system-x86_64 -boot d -cdrom /dev/disk1 -hda windows7.img -m 1024 >> >> and I have my windows 7 cd in the cd rom, and the installation >> process hang on startup, h

Re: [Qemu-devel] [PATCH v3 4/5] qom: replace object property list with GHashTable

2015-10-08 Thread Eric Blake
On 10/08/2015 08:09 AM, Daniel P. Berrange wrote: > From: Pavel Fedin > > ARM GICv3 systems with large number of CPUs create lots of IRQ pins. Since > every pin is represented as a property, number of these properties becomes > very large. Every property add first makes sure there's no duplicates

Re: [Qemu-devel] [PATCH v3 3/5] vl: convert machine help to use object_property_foreach

2015-10-08 Thread Eric Blake
On 10/08/2015 08:09 AM, Daniel P. Berrange wrote: > Rather than directly traversing the object property list, > use the object_property_foreach iterator. This removes a > dependancy on the implementation approach for properties. s/dependancy/dependency/ > > Signed-off-by: Daniel P. Berrange > -

Re: [Qemu-devel] [PATCH 7/8] migration: new migration test mode

2015-10-08 Thread Dr. David Alan Gilbert
* Denis V. Lunev (d...@openvz.org) wrote: > On 10/07/2015 04:56 PM, Dr. David Alan Gilbert wrote: > >* Denis V. Lunev (d...@openvz.org) wrote: > >>From: Igor Redko > >> > >>In this patch the ability to start a migration with test-only > >>capability was added. It allows to gather the guest VM’s me

[Qemu-devel] [PULL 19/19] tests: Unique test path for /string-visitor/output

2015-10-08 Thread Michael Tokarev
From: "Dr. David Alan Gilbert" Newer GLib's want unique test paths, and thus moan at dupes. (Seen on Fedora 23 which has glib 2.46) Uniquify the paths. Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Eric Blake Signed-off-by: Michael Tokarev --- tests/test-string-output-visitor.c | 16 ++

[Qemu-devel] [PATCH] armv7-m: exit on external reset request

2015-10-08 Thread Michael Davidsaver
Signed-off-by: Michael Davidsaver --- hw/intc/armv7m_nvic.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/hw/intc/armv7m_nvic.c b/hw/intc/armv7m_nvic.c index 3ec8408..a671d84 100644 --- a/hw/intc/armv7m_nvic.c +++ b/hw/intc/armv7m_nvic.c @@ -15,6 +15,7 @@ #include "hw/

Re: [Qemu-devel] [PATCHv4] target-arm: Implement AArch64 OSLAR/OSLSR_EL1 sysregs

2015-10-08 Thread Peter Maydell
On 6 October 2015 at 20:47, Davorin Mista wrote: > Added oslar_write function to OSLAR_EL1 sysreg, using a status variable > in ARMCPUState.cp15 struct (oslsr_el1). This variable is also linked > to the newly added read-only OSLSR_EL1 register. > > Linux reads from this register during its suspend

Re: [Qemu-devel] [PATCH v2 2/2] target-arm: Fix CPU breakpoint handling

2015-10-08 Thread Peter Maydell
On 28 September 2015 at 11:07, Sergey Fedorov wrote: > A QEMU breakpoint match is not definitely an architectural breakpoint > match. If an exception is generated unconditionally during translation, > it is hardly possible to ignore it in the debug exception handler. > > Generate a call to a helpe

Re: [Qemu-devel] [PATCH v5 1/4] Add new block driver interface to add/delete a BDS's child

2015-10-08 Thread Dr. David Alan Gilbert
* Wen Congyang (we...@cn.fujitsu.com) wrote: > On 10/08/2015 03:00 AM, Dr. David Alan Gilbert wrote: > > * Wen Congyang (we...@cn.fujitsu.com) wrote: > >> In some cases, we want to take a quorum child offline, and take > >> another child online. > > > > Hi, > > Have you checked the output of 'in

Re: [Qemu-devel] [PATCH] hw/arm/virt: smbios: inform guest of kvm

2015-10-08 Thread Peter Maydell
On 28 September 2015 at 16:31, Andrew Jones wrote: > On Thu, Sep 24, 2015 at 12:13:08PM +0200, Andrew Jones wrote: >> On Wed, Sep 23, 2015 at 08:43:39AM -0700, Peter Maydell wrote: >> > On 23 September 2015 at 07:18, Andrew Jones wrote: >> > > ARM/AArch64 KVM guests don't have any way to identify

Re: [Qemu-devel] [PATCH 3/4] hw/i386: Introduce AMD IOMMU

2015-10-08 Thread Marcel Apfelbaum
On 10/09/2015 05:53 AM, David Kiarie wrote: From: David Introduce basic AMD IOMMU emulation in Qemu. IOMMU implements event logging and host translation which should allow nested PCI passthrough.It also implemented a very basic IOTLB implementation Signed-off-by: David Kiarie --- hw/i386/Ma

[Qemu-devel] [PATCH v2 2/2] virtio-input: Fix device introspection on non-Linux hosts

2015-10-08 Thread Markus Armbruster
When CONFIG_LINUX is off, devices "virtio-keyboard-device", "virtio-mouse-device", "virtio-tablet-device" and "virtio-input-host-device" aren't compiled in, yet "virtio-keyboard-pci", "virtio-mouse-pci", "virtio-tablet-pci" and "virtio-input-host-pci" still are. Attempts to introspect them crash,

[Qemu-devel] [PATCH] Exit on reset for armv7-m

2015-10-08 Thread Michael Davidsaver
Implement the SYSRESETREQ bit of the AIRCR register for armv7-m (ie. cortex-m3). A small patch to see if I have the submission process figured out. Michael Davidsaver (1): armv7-m: exit on external reset request hw/intc/armv7m_nvic.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-)

[Qemu-devel] how to debug PCI passthrough with PLX (Avago) bridge and TI endpoint

2015-10-08 Thread Jeff Brower
All- We're trying to get a PCIe card working with PCI passthrough and it's close but not yet. The card has a PLX (Avago) PEX 8624 bridge chip and multiple Texas Instruments CPU end-points, and there is a second card with PEX 8748 and same TI endpoints. Both cards are recognized, all TI devices

Re: [Qemu-devel] Qemu - Install Windows 7 64bits hang on startup with Blue Screen of Death

2015-10-08 Thread Olga Krishtal
On 07/10/15 14:41, Aaron Elkins wrote: Hi, With this command line: qemu-system-x86_64 -boot d -cdrom /dev/disk1 -hda windows7.img -m 1024 and I have my windows 7 cd in the cd rom, and the installation process hang on startup, here is the screen shot: https://www.dropbox.com/s/xzi28a5gibnp9xv

[Qemu-devel] [PULL 07/19] pc: check for underflow in load_linux

2015-10-08 Thread Michael Tokarev
From: Paolo Bonzini If (setup_size+1)*512 is small enough, kernel_size -= setup_size can allocate a huge amount of memory. Avoid that. Signed-off-by: Paolo Bonzini Signed-off-by: Michael Tokarev --- hw/i386/pc.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/i386/pc.c b/hw/i386/p

[Qemu-devel] [PULL 05/19] block/ssh: remove dead code

2015-10-08 Thread Michael Tokarev
From: Paolo Bonzini The "err" label cannot be reached with qp != NULL. Remove the free-ing of qp and avoid future regressions by removing the initializer. Signed-off-by: Paolo Bonzini ACKed-by: Richard W.M. Jones Reviewed-by: Fam Zheng Signed-off-by: Michael Tokarev --- block/ssh.c | 5 +--

Re: [Qemu-devel] [PATCH] trace: remove malloc tracing

2015-10-08 Thread Peter Maydell
On 8 October 2015 at 15:20, Stefan Hajnoczi wrote: > On Wed, Sep 16, 2015 at 05:38:44PM +0200, Paolo Bonzini wrote: >> The malloc vtable is not supported anymore in glib, because it broke >> when constructors called g_malloc. Remove tracing of g_malloc, >> g_realloc and g_free calls. >> >> Note t

  1   2   3   >