[Qemu-devel] [PULL v2 16/37] Makefile: Add rules to run vm tests

2017-09-08 Thread Fam Zheng
Signed-off-by: Fam Zheng Message-Id: <20170905021201.25684-10-f...@redhat.com> --- Makefile | 2 ++ configure | 2 +- tests/vm/Makefile.include | 42 ++ 3 files changed, 45 insertions(+), 1 deletion(-)

[Qemu-devel] [PULL v2 23/37] docker: Add nettle-devel to fedora image

2017-09-08 Thread Fam Zheng
The LUKS cases in qemu-iotests requires this. Reviewed-by: Kashyap Chamarthy Signed-off-by: Fam Zheng Message-Id: <20170905025614.579-5-f...@redhat.com> Reviewed-by: Stefan Hajnoczi Based-on: 20170905021201.25684-1-f...@redhat.com ---

[Qemu-devel] [PULL v2 17/37] MAINTAINERS: Add tests/vm entry

2017-09-08 Thread Fam Zheng
Signed-off-by: Fam Zheng Reviewed-by: Stefan Hajnoczi Message-Id: <20170905021201.25684-11-f...@redhat.com> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 36eeb42d19..42f5454311 100644 --- a/MAINTAINERS

[Qemu-devel] [PULL v2 18/37] tests: Add README for vm tests

2017-09-08 Thread Fam Zheng
Signed-off-by: Fam Zheng Message-Id: <20170905021201.25684-12-f...@redhat.com> --- tests/vm/README | 63 + 1 file changed, 63 insertions(+) create mode 100644 tests/vm/README diff --git a/tests/vm/README b/tests/vm/README

[Qemu-devel] [PULL v2 15/37] tests: Add OpenBSD image

2017-09-08 Thread Fam Zheng
The image is prepared following instructions as in: https://wiki.qemu.org/Hosts/BSD Signed-off-by: Fam Zheng Message-Id: <20170905021201.25684-9-f...@redhat.com> --- tests/vm/openbsd | 43 +++ 1 file changed, 43 insertions(+) create

[Qemu-devel] [PULL v2 14/37] tests: Add NetBSD image

2017-09-08 Thread Fam Zheng
The image is prepared following instructions as in: https://wiki.qemu.org/Hosts/BSD Signed-off-by: Fam Zheng Reviewed-by: Kamil Rytarowski Message-Id: <20170905021201.25684-8-f...@redhat.com> --- tests/vm/netbsd | 42 ++ 1

[Qemu-devel] [PULL v2 12/37] tests: Add ubuntu.i386 image

2017-09-08 Thread Fam Zheng
This adds a 32bit guest. The official LTS cloud image is downloaded and initialized with cloud-init. Signed-off-by: Fam Zheng Message-Id: <20170905021201.25684-6-f...@redhat.com> --- tests/vm/ubuntu.i386 | 88 1 file

[Qemu-devel] [PULL v2 13/37] tests: Add FreeBSD image

2017-09-08 Thread Fam Zheng
The image is prepared following instructions as in: https://wiki.qemu.org/Hosts/BSD Signed-off-by: Fam Zheng Message-Id: <20170905021201.25684-7-f...@redhat.com> --- tests/vm/freebsd | 42 ++ 1 file changed, 42 insertions(+) create mode

[Qemu-devel] [PULL v2 05/37] docker: Update ubuntu image

2017-09-08 Thread Fam Zheng
Base on the newer ubuntu-lts (16.06) and include more packages for better build coverage. Signed-off-by: Fam Zheng Message-Id: <20170907141245.31946-2-f...@redhat.com> Signed-off-by: Fam Zheng --- tests/docker/dockerfiles/ubuntu.docker | 11 --- 1 file

[Qemu-devel] [PULL v2 11/37] tests: Add vm test lib

2017-09-08 Thread Fam Zheng
This is the common code to implement a "VM test" to 1) Download and initialize a pre-defined VM that has necessary dependencies to build QEMU and SSH access. 2) Archive $SRC_PATH to a .tar file. 3) Boot the VM, and pass the source tar file to the guest. 4) SSH into the VM, untar the

[Qemu-devel] [PULL v2 10/37] scripts: Add archive-source.sh

2017-09-08 Thread Fam Zheng
Signed-off-by: Fam Zheng Message-Id: <20170905021201.25684-4-f...@redhat.com> --- scripts/archive-source.sh | 31 +++ 1 file changed, 31 insertions(+) create mode 100755 scripts/archive-source.sh diff --git a/scripts/archive-source.sh

[Qemu-devel] [PULL v2 07/37] tests/docker: Clean up paths

2017-09-08 Thread Fam Zheng
The 'run' script already creats src, build and install directories under $TEST_DIR, use it in common.rc. Also the tests always run from $QEMU_SRC/tests/docker, so use a relative $CMD string. Message-Id: <20170817035721.11064-1-f...@redhat.com> Signed-off-by: Fam Zheng ---

[Qemu-devel] [PULL v2 06/37] docker: Enable features explicitly in test-full

2017-09-08 Thread Fam Zheng
Also avoid "set -e". Signed-off-by: Fam Zheng Message-Id: <20170907141245.31946-3-f...@redhat.com> Reviewed-by: Eric Blake Signed-off-by: Fam Zheng --- tests/docker/test-full | 82 ++ 1 file

[Qemu-devel] [PULL v2 09/37] qemu.py: Add "wait()" method

2017-09-08 Thread Fam Zheng
Signed-off-by: Fam Zheng Reviewed-by: Stefan Hajnoczi Message-Id: <20170905021201.25684-3-f...@redhat.com> --- scripts/qemu.py | 7 +++ 1 file changed, 7 insertions(+) diff --git a/scripts/qemu.py b/scripts/qemu.py index 4d8ee10943..99963053a5 100644

[Qemu-devel] [PULL v2 08/37] gitignore: Ignore vm test images

2017-09-08 Thread Fam Zheng
Signed-off-by: Fam Zheng Message-Id: <20170905021201.25684-2-f...@redhat.com> --- .gitignore | 1 + tests/.gitignore | 1 + 2 files changed, 2 insertions(+) diff --git a/.gitignore b/.gitignore index cf65316863..40acfcb9e2 100644 --- a/.gitignore +++ b/.gitignore @@ -52,6

[Qemu-devel] [PULL v2 04/37] docker: reduce noise when building travis.docker

2017-09-08 Thread Fam Zheng
From: Alex Bennée Set the DEBIAN_FRONTEND and locale env vars to stop apt complaining so much as we build the image. Suggested-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée Message-Id:

[Qemu-devel] [PULL v2 01/37] docker: ensure NOUSER for travis images

2017-09-08 Thread Fam Zheng
From: Alex Bennée While adding the current user is a useful default behaviour for creating new images it is not appropriate for Travis which already has a default user. Signed-off-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé

[Qemu-devel] [PULL v2 00/37] Staging patches

2017-09-08 Thread Fam Zheng
The following changes since commit fcea73709b966a7ded9efa7b106ea50c7fe9025c: Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2017-09-08 16:04:42 +0100) are available in the git repository at: git://github.com/famz/qemu.git tags/staging-pull-request for you to

[Qemu-devel] [PULL v2 03/37] docker: don't install device-tree-compiler build-deps in travis.docker

2017-09-08 Thread Fam Zheng
From: Alex Bennée Installing the device-tree-compiler build-deps is a little extreme. We only actually need the binary so include it with the other packages. Suggested-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée

[Qemu-devel] [PULL v2 02/37] docker: docker.py make --no-cache skip checksum test

2017-09-08 Thread Fam Zheng
From: Alex Bennée If you invoke with NOCACHE=1 we pass --no-cache in the argv to docker.py but may still not force a rebuild if the dockerfile checksum hasn't changed. By testing for its presence we can force builds without having to manually remove the docker image.

Re: [Qemu-devel] [PATCH v6 04/12] tests: Add vm test lib

2017-09-08 Thread Fam Zheng
On Fri, 09/08 22:25, Philippe Mathieu-Daudé wrote: > Hi Fam, Alex, > > > > > > > If this fails it's fairly cryptic: > > > > > > /home/alex/lsrc/qemu/qemu.git/tests/vm/openbsd --debug --image > > > "tests/vm/openbsd.img" --build-qemu /home/alex/lsrc/qemu/qemu.git > > > DEBUG:root:Creating

Re: [Qemu-devel] [PATCH v6 04/12] tests: Add vm test lib

2017-09-08 Thread Philippe Mathieu-Daudé
Hi Fam, Alex, If this fails it's fairly cryptic: /home/alex/lsrc/qemu/qemu.git/tests/vm/openbsd --debug --image "tests/vm/openbsd.img" --build-qemu /home/alex/lsrc/qemu/qemu.git DEBUG:root:Creating archive ./vm-test-fxejnB.tmp/data-2de24.tar for src_dir dir: /home/alex/lsrc/qemu/qemu.git

Re: [Qemu-devel] [PATCH] .dir-locals.el: Explicitly set indentation level

2017-09-08 Thread Thiago Jung Bauermann
Markus Armbruster writes: > Thiago Jung Bauermann writes: > >> At least in some configurations, setting c-file-style is not enough to >> conform to the QEMU coding style, so explicitly set c-basic-offset as well. >> >> Signed-off-by: Thiago Jung

Re: [Qemu-devel] [PULL 00/38] Test and build system patches

2017-09-08 Thread Fam Zheng
On Fri, 09/08 14:44, Peter Maydell wrote: > On 8 September 2017 at 10:54, Fam Zheng wrote: > > Hi Peter, > > > > The following changes since commit cda4a338c4243fa3bff4498b935340ac7121cc76: > > > > tcg/tci: Add TCG_TARGET_DEFAULT_MO (2017-09-07 18:57:34 +0100) > > > > are

Re: [Qemu-devel] [PATCH v6 04/12] tests: Add vm test lib

2017-09-08 Thread Fam Zheng
On Fri, 09/08 16:22, Alex Bennée wrote: > > Fam Zheng writes: > > > This is the common code to implement a "VM test" to > > > > 1) Download and initialize a pre-defined VM that has necessary > > dependencies to build QEMU and SSH access. > > > > 2) Archive $SRC_PATH to a

Re: [Qemu-devel] [PATCH v6 03/12] scripts: Add archive-source.sh

2017-09-08 Thread Fam Zheng
On Fri, 09/08 15:42, Alex Bennée wrote: > > Fam Zheng writes: > > > Signed-off-by: Fam Zheng > > --- > > scripts/archive-source.sh | 31 +++ > > 1 file changed, 31 insertions(+) > > create mode 100755 scripts/archive-source.sh > >

Re: [Qemu-devel] [PATCH v6 03/12] scripts: Add archive-source.sh

2017-09-08 Thread Fam Zheng
On Fri, 09/08 15:56, Peter Maydell wrote: > On 5 September 2017 at 03:11, Fam Zheng wrote: > > Signed-off-by: Fam Zheng > > --- > > scripts/archive-source.sh | 31 +++ > > 1 file changed, 31 insertions(+) > > create mode 100755

Re: [Qemu-devel] [PATCH v6 01/12] gitignore: Ignore vm test images

2017-09-08 Thread Fam Zheng
On Fri, 09/08 15:12, Alex Bennée wrote: > > Fam Zheng writes: > > > Signed-off-by: Fam Zheng > > --- > > .gitignore | 1 + > > tests/.gitignore | 1 + > > 2 files changed, 2 insertions(+) > > > > diff --git a/.gitignore b/.gitignore > > index

Re: [Qemu-devel] [PATCH v2 00/13] add support for Hypervisor.framework in QEMU

2017-09-08 Thread Sergio Andrés Gómez del Real
Guys, I'm almost done with the new version of the patchset (hopefully the definite one). What I am missing essentially are presenting a few tests as suggested by Stefan. Paolo and I only ran the eventinj test from the kvm suite. If I modify this boot-serial-test to include hvf, should I send these

[Qemu-devel] irqfd for QEMU NVMe

2017-09-08 Thread Huaicheng Li
Hi all, I'm writing to ask if it's possible to use irqfd mechanism in QEMU's NVMe virtual controller implementation. My search results show that back in 2015, there is a discussion on improving QEMU NVMe performance by utilizing eventfd for guest-to-host notification, thus i guess irqfd should

[Qemu-devel] [Bug 1516446] Re: Migration always causes guest freeze in one direction.

2017-09-08 Thread Dr. David Alan Gilbert
Hi Magnus, I think there's a fair possibility you're hitting a kernel bug that we found in https://bugzilla.redhat.com/show_bug.cgi?id=1401767 which was also a failure migrating from a newer AMD to an older AMD processor. That was fixed by upstream kernel fix: commit

Re: [Qemu-devel] [PATCH] slirp: Add explanation for hostfwd parsing failure

2017-09-08 Thread Philippe Mathieu-Daudé
On 09/08/2017 03:36 PM, Dr. David Alan Gilbert wrote: * Philippe Mathieu-Daudé (f4...@amsat.org) wrote: On 09/08/2017 01:22 PM, Dr. David Alan Gilbert wrote: * Philippe Mathieu-Daudé (f4...@amsat.org) wrote: Hi David, On 09/08/2017 12:53 PM, Dr. David Alan Gilbert (git) wrote: From: "Dr.

Re: [Qemu-devel] [PATCH] slirp: Add explanation for hostfwd parsing failure

2017-09-08 Thread Dr. David Alan Gilbert
* Philippe Mathieu-Daudé (f4...@amsat.org) wrote: > On 09/08/2017 01:22 PM, Dr. David Alan Gilbert wrote: > > * Philippe Mathieu-Daudé (f4...@amsat.org) wrote: > > > Hi David, > > > > > > On 09/08/2017 12:53 PM, Dr. David Alan Gilbert (git) wrote: > > > > From: "Dr. David Alan Gilbert"

Re: [Qemu-devel] [PATCH v2] Add myself as maintainer for TPM code

2017-09-08 Thread Markus Armbruster
Stefan Berger writes: > Also add backends/tpm.c to the list of files. > > Signed-off-by: Stefan Berger > --- > MAINTAINERS | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index

Re: [Qemu-devel] [PATCH v2 0/6] Update websocket code to more fully support the RFC

2017-09-08 Thread Eric Blake
On 09/08/2017 01:11 PM, Brandon Carpenter wrote: > My apologies. I read that document before submitting the patch, but > there was a lot to take in. Would you like me to resend the patch > without the references? Not a problem - we all had to submit our first patch at some point in time in the

Re: [Qemu-devel] [PATCH v2 0/6] Update websocket code to more fully support the RFC

2017-09-08 Thread Brandon Carpenter
My apologies. I read that document before submitting the patch, but there was a lot to take in. Would you like me to resend the patch without the references? -- Brandon Carpenter | Software Engineer Cypherpath, Inc. 400 Columbia Point Drive Ste 101 | Richland, Washington USA Office: (650)

Re: [Qemu-devel] [PATCH] slirp: Add explanation for hostfwd parsing failure

2017-09-08 Thread Philippe Mathieu-Daudé
On 09/08/2017 01:22 PM, Dr. David Alan Gilbert wrote: * Philippe Mathieu-Daudé (f4...@amsat.org) wrote: Hi David, On 09/08/2017 12:53 PM, Dr. David Alan Gilbert (git) wrote: From: "Dr. David Alan Gilbert" e.g. ./x86_64-softmmu/qemu-system-x86_64 -nographic -netdev

Re: [Qemu-devel] [PATCH] target/arm: Avoid an extra temporary for store_exclusive

2017-09-08 Thread Alistair Francis
On Fri, Sep 8, 2017 at 9:38 AM, Richard Henderson wrote: > Instead of copying addr to a local temp, reuse the value (which we > have just compared as equal) already saved in cpu_exclusive_addr. > > Signed-off-by: Richard Henderson

[Qemu-devel] [PATCH v2 0/6] Update websocket code to more fully support the RFC

2017-09-08 Thread Brandon Carpenter
We've been experiencing issues where the qemu websocket server closes connections from noVNC clients for no apparent reason. Debugging shows that certain web browsers are injecting ping and pong frames when the connection becomes idle. Some browsers send those frames without a payload, which also

Re: [Qemu-devel] [PATCH v3 4/7] block: remove legacy I/O throttling

2017-09-08 Thread Manos Pitsidianakis
On Fri, Sep 08, 2017 at 06:00:11PM +0200, Kevin Wolf wrote: Am 08.09.2017 um 17:44 hat Manos Pitsidianakis geschrieben: On Thu, Sep 07, 2017 at 03:26:11PM +0200, Kevin Wolf wrote: > We shouldn't really need any throttling code in > blk_root_drained_begin/end any more now because the throttle

Re: [Qemu-devel] [PATCH v2 0/6] Update websocket code to more fully support the RFC

2017-09-08 Thread Eric Blake
On 09/08/2017 12:37 PM, Brandon Carpenter wrote: > We've been experiencing issues where the qemu websocket server closes > connections from noVNC clients for no apparent reason. Debugging shows > that certain web browsers are injecting ping and pong frames when the > connection becomes idle. Some

[Qemu-devel] [PATCH v2 5/6] io: Ignore websocket PING and PONG frames

2017-09-08 Thread Brandon Carpenter
Keep pings and gratuitous pongs generated by web browsers from killing websocket connections. Signed-off-by: Brandon Carpenter --- io/channel-websock.c | 21 + 1 file changed, 17 insertions(+), 4 deletions(-) diff --git

Re: [Qemu-devel] [RFC v2 4/8] QAPI: new QMP command option "without-bql"

2017-09-08 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > On Fri, Aug 25, 2017 at 10:06:27AM +0100, Dr. David Alan Gilbert wrote: > > * Peter Xu (pet...@redhat.com) wrote: > > > On Wed, Aug 23, 2017 at 06:44:12PM +0100, Dr. David Alan Gilbert wrote: > > > > > > [...] > > > > > > > > +Most of the commands require

[Qemu-devel] [PATCH v2 2/6] io: Small updates in preparation for websocket changes

2017-09-08 Thread Brandon Carpenter
Gets rid of unnecessary bit shifting and performs proper EOF checking to avoid a large number of repeated calls to recvmsg() when a client abruptly terminates a connection (bug fix). Signed-off-by: Brandon Carpenter --- io/channel-websock.c | 62

[Qemu-devel] [PATCH v2 6/6] io: Reply to ping frames

2017-09-08 Thread Brandon Carpenter
Add an immediate ping reply (pong) to the outgoing stream when a ping is received. Unsolicited pongs are ignored. Signed-off-by: Brandon Carpenter --- io/channel-websock.c | 50 -- 1 file changed, 32

Re: [Qemu-devel] [RFC v2 4/8] QAPI: new QMP command option "without-bql"

2017-09-08 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > On Fri, Aug 25, 2017 at 10:14:12AM +0100, Dr. David Alan Gilbert wrote: > > * Peter Xu (pet...@redhat.com) wrote: > > > On Thu, Aug 24, 2017 at 07:37:32AM +0800, Fam Zheng wrote: > > > > On Wed, 08/23 18:44, Dr. David Alan Gilbert wrote: > > > > > * Peter Xu

[Qemu-devel] [PATCH v2 3/6] io: Add support for fragmented websocket binary frames

2017-09-08 Thread Brandon Carpenter
Allows fragmented binary frames by saving the previous opcode. Handles the case where an intermediary (i.e., web proxy) fragments frames originally sent unfragmented by the client. Signed-off-by: Brandon Carpenter --- include/io/channel-websock.h | 1 +

[Qemu-devel] [PATCH v3 15/15] MAINTAINERS: update docs/interop/ entries

2017-09-08 Thread Philippe Mathieu-Daudé
moved in commit 7746cf8aab68 Signed-off-by: Philippe Mathieu-Daudé Acked-by: Fam Zheng Acked-by: John Snow --- MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index

[Qemu-devel] [PATCH v2 4/6] io: Allow empty websocket payload

2017-09-08 Thread Brandon Carpenter
Some browsers send pings/pongs with no payload, so allow empty payloads instead of closing the connection. Signed-off-by: Brandon Carpenter --- io/channel-websock.c | 62 +--- 1 file changed, 30 insertions(+), 32

[Qemu-devel] [PATCH v3 13/15] MAINTAINERS: add missing Cryptography entry

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Acked-by: Daniel P. Berrange --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 8024a3880b..53891bd36e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1540,6 +1540,7 @@ S:

[Qemu-devel] [PATCH v2 1/6] io: Always remove an old channel watch before adding a new one

2017-09-08 Thread Brandon Carpenter
Also set saved handle to zero when removing without adding a new watch. Signed-off-by: Brandon Carpenter Reviewed-by: Paolo Bonzini Reviewed-by: Daniel P. Berrange --- ui/vnc-auth-vencrypt.c | 3 +++ ui/vnc-ws.c

[Qemu-devel] [PATCH v3 14/15] MAINTAINERS: update docs/devel/ entries

2017-09-08 Thread Philippe Mathieu-Daudé
moved in commit ac06724a7158 Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Eric Blake --- MAINTAINERS | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 53891bd36e..f8630561de 100644 ---

[Qemu-devel] [PATCH v3 09/15] MAINTAINERS: add missing SSI entries

2017-09-08 Thread Philippe Mathieu-Daudé
Alistair Francis volunteered :) Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis --- MAINTAINERS | 3 +++ 1 file changed, 3 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 69ad3aac29..517c8f95d7 100644 ---

[Qemu-devel] [PATCH v3 11/15] MAINTAINERS: add missing AIO entry

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Acked-by: Fam Zheng Reviewed-by: Stefan Hajnoczi --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 4e05831bda..98b7edc8b3 100644 --- a/MAINTAINERS +++

[Qemu-devel] [PATCH v3 12/15] MAINTAINERS: add missing entry for Generic Loader

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alistair Francis --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 98b7edc8b3..8024a3880b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1163,6

[Qemu-devel] [PATCH v3 07/15] MAINTAINERS: add missing USB entry

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 39ac5d132a..04cc35cc3a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1004,6 +1004,7 @@ F:

[Qemu-devel] [PATCH v3 10/15] MAINTAINERS: add missing entries for throttling infra

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Stefan Hajnoczi Reviewed-by: Alberto Garcia --- MAINTAINERS | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 517c8f95d7..4e05831bda

[Qemu-devel] [PATCH v3 06/15] MAINTAINERS: add missing qcow2 entry

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Acked-by: Kevin Wolf Reviewed-by: Stefan Hajnoczi --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index fa1620d8a8..39ac5d132a 100644 --- a/MAINTAINERS

[Qemu-devel] [PATCH v3 04/15] MAINTAINERS: add missing VMWare entry

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth Reviewed-by: Dmitry Fleytman --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index afc0072a17..e0e5a8dc3f 100644 --- a/MAINTAINERS

[Qemu-devel] [PATCH v3 08/15] MAINTAINERS: add missing PCI entries

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Marcel Apfelbaum Acked-by: Michael S. Tsirkin --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 04cc35cc3a..69ad3aac29 100644 ---

Re: [Qemu-devel] [PATCH 3/3] Unified Datagram Socket Transport - raw support

2017-09-08 Thread Anton Ivanov
On 24/07/17 05:03, Jason Wang wrote: > > > On 2017年07月22日 02:50, Anton Ivanov wrote: >> >> [snip] >> +"-netdev raw,id=str,ifname=ifname\n" +"configure a network backend with ID 'str' connected to\n" +"an Ethernet interface named

[Qemu-devel] [PATCH v3 05/15] MAINTAINERS: add missing Guest Agent entries

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Acked-by: Michael Roth --- MAINTAINERS | 4 1 file changed, 4 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index e0e5a8dc3f..fa1620d8a8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1440,6 +1440,10

Re: [Qemu-devel] [PATCH v7 03/22] migration: Teach it about G_SOURCE_REMOVE

2017-09-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > As this is defined on glib 2.32, add compatibility macros for older glibs. > > Signed-off-by: Juan Quintela > Reviewed-by: Daniel P. Berrange > Reviewed-by: Peter Xu I think 03 and 04

Re: [Qemu-devel] [PATCH v7 13/22] migration: Split migration_fd_process_incoming

2017-09-08 Thread Dr. David Alan Gilbert
* Juan Quintela (quint...@redhat.com) wrote: > We need that on later patches. > > Signed-off-by: Juan Quintela > Reviewed-by: Dr. David Alan Gilbert > Reviewed-by: Peter Xu > Reviewed-by: Daniel P. Berrange I

Re: [Qemu-devel] [PATCH v2 12/17] MAINTAINERS: add missing megasas test entry

2017-09-08 Thread Philippe Mathieu-Daudé
Let's try another SUSE domain On 09/08/2017 02:12 PM, Philippe Mathieu-Daudé wrote: I wonder if "megasas" is still maintained, I got: To: Hannes Reinecke Final-Recipient: rfc822; h...@suse.de Action: failed Status: 5.1.1 Remote-MTA: dns; mx2.suse.de. (195.135.220.15, the server

[Qemu-devel] [PATCH v3 00/15] add missing entries in MAINTAINERS

2017-09-08 Thread Philippe Mathieu-Daudé
Hi, I tried to have a more helpful ./scripts/get_maintainer.pl output, filling missing entries in MAINTAINERS. Regards, Phil. v3: - rebased, add R-b & A-b - squashed ARM patches (Thomas Huth) - remove 'megasas' patch, maintainer email not working v2: - add R-b & A-b - clean ARM entries

Re: [Qemu-devel] [RFC v2 2/8] monitor: allow monitor to create thread to poll

2017-09-08 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > On Fri, Aug 25, 2017 at 10:30:42AM +0100, Dr. David Alan Gilbert wrote: > > [...] > > > > > c) As mentioned on irc there's fun to be had with cur_mon and error > > > > handling - in my local world I have cur_mon declared as __thread > > > > but

[Qemu-devel] [PATCH v3 03/15] MAINTAINERS: add missing entry for vhost

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Acked-by: Michael S. Tsirkin --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index fb93e4b554..afc0072a17 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1029,6 +1029,7 @@ vhost

[Qemu-devel] [PATCH v3 02/15] MAINTAINERS: add missing STM32 entry

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth Reviewed-by: Alistair Francis --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 35c8c9c752..fb93e4b554 100644 ---

[Qemu-devel] [PATCH v3 01/15] MAINTAINERS: add missing ARM entries

2017-09-08 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth --- MAINTAINERS | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 36eeb42d19..35c8c9c752 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@

Re: [Qemu-devel] [PATCH v2 12/17] MAINTAINERS: add missing megasas test entry

2017-09-08 Thread Philippe Mathieu-Daudé
I wonder if "megasas" is still maintained, I got: To: Hannes Reinecke Final-Recipient: rfc822; h...@suse.de Action: failed Status: 5.1.1 Remote-MTA: dns; mx2.suse.de. (195.135.220.15, the server for the domain suse.de.) Diagnostic-Code: smtp; 550 5.1.1 : Recipient

Re: [Qemu-devel] [PATCH 1/1] AArch64: Fix single stepping of ERET instruction

2017-09-08 Thread Richard Henderson
On 09/08/2017 04:02 AM, Jaroslaw Pelczar wrote: > Previously when single stepping through ERET instruction via GDB > would result in debugger entering the "next" PC after ERET instruction. > When debugging in kernel mode, this will also cause unintended behavior, > because debugger will try to

[Qemu-devel] [Bug 1178101] Re: Could not enable gtk UI on build for Windows target

2017-09-08 Thread Thomas Huth
Triaging old bug tickets... can you still reproduce this issue with the latest version of QEMU? Or could we close this ticket nowadays? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Qemu-devel] [Bug 1060928] Re: Error in launch virtual server port

2017-09-08 Thread Thomas Huth
Triaging old bug tickets... can you anyhow still reproduce this issue with the latest version of QEMU? Or could we close this ticket nowadays? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is

[Qemu-devel] [Bug 1196426] Re: Setup virtual serial connection for Windows VMs FAILS

2017-09-08 Thread Thomas Huth
Triaging old bug tickets... can you still reproduce this issue with the latest version of QEMU? Or could we close this ticket nowadays? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Qemu-devel] [Bug 1135567] Re: QXL crashes a Windows 7 guest if host goes into screensaver

2017-09-08 Thread Thomas Huth
Triaging old bug tickets... can you still reproduce this issue with the latest version of QEMU? Or could we close this ticket nowadays? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Qemu-devel] [Bug 1061778] Re: signal mask not reset on exec

2017-09-08 Thread Thomas Huth
Triaging old bug tickets... can you still reproduce this issue with the latest version of QEMU? Or could we close this ticket nowadays? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Qemu-devel] [Bug 1119861] Re: Poor console performance in Windows 7

2017-09-08 Thread Thomas Huth
Triaging old bug tickets... can you still reproduce this issue with the latest version of QEMU? Or could we close this ticket nowadays? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Qemu-devel] [PATCH v4 3/3] Test for full Backup

2017-09-08 Thread Ishani Chugh
This patch is the test for full backup implementation in Backup tool. The test employs two basic substests: 1) Backing up an empty guest and comparing it with base image. 2) Writing a pattern to the guest, creating backup and comparing with the base image. Signed-off-by: Ishani Chugh

[Qemu-devel] [PATCH v4 1/3] Add manpage for QEMU Backup Tool

2017-09-08 Thread Ishani Chugh
qemu-backup will be a command-line tool for performing full and incremental disk backups on running VMs. It is intended as a reference implementation for management stack and backup developers to see QEMU's backup features in action. This commit is an initial implementation of manpage listing the

[Qemu-devel] [PATCH v4 2/3] backup: Adds Backup Tool

2017-09-08 Thread Ishani Chugh
qemu-backup will be a command-line tool for performing full and incremental disk backups on running VMs. It is intended as a reference implementation for management stack and backup developers to see QEMU's backup features in action. The tool writes details of guest in a configuration file and the

[Qemu-devel] [PATCH v4 0/3] QEMU Backup Tool

2017-09-08 Thread Ishani Chugh
This patch series is intended to introduce QEMU Backup tool. qemu-backup will be a command-line tool for performing full and incremental disk backups on running VMs. It is intended as a reference implementation for management stack and backup developers to see QEMU's backup features in action.

[Qemu-devel] 答复: [PATCH v11 5/6] target-arm: kvm64: handle SIGBUS signal for synchronous External Abort

2017-09-08 Thread gengdongjiu
[...] > > > > /* > > * xx > > */ > > void kvm_hwpoison_page_add(ram_addr_t ram_addr); > > It should be in the doc-comment format, which begins "/**" and has some > stylization of how you list parameters and so on. Lots of > examples in the existing headers.

Re: [Qemu-devel] [PATCH v3 7/7] Convert remaining single line fprintf() to warn_report()

2017-09-08 Thread Alistair Francis
On Fri, Sep 8, 2017 at 2:22 AM, Markus Armbruster wrote: > Alistair Francis writes: > >> Convert any remaining uses of fprintf(stderr, "warning:"... >> to use warn_report() instead. This helps standardise on a single >> method of printing warnings

[Qemu-devel] [PATCH] target/arm: Avoid an extra temporary for store_exclusive

2017-09-08 Thread Richard Henderson
Instead of copying addr to a local temp, reuse the value (which we have just compared as equal) already saved in cpu_exclusive_addr. Signed-off-by: Richard Henderson --- target/arm/translate-a64.c | 26 +- 1 file changed, 9 insertions(+), 17

Re: [Qemu-devel] [PATCH v2 06/17] MAINTAINERS: add missing Guest Agent entries

2017-09-08 Thread Michael Roth
Quoting Philippe Mathieu-Daudé (2017-09-08 08:04:45) > ping? > > Michael, you told me this looks OK on IRC, can I respin with your Acked-by? Sure: Acked-by: Michael Roth > > On 08/30/2017 06:55 PM, Philippe Mathieu-Daudé wrote: > > Signed-off-by: Philippe

Re: [Qemu-devel] [Qemu-ppc] [PATCH 1/3] ppc/xive: fix OV5_XIVE_EXPLOIT bits

2017-09-08 Thread Greg Kurz
Shouldn't the patch title mention spapr instead of ppc/xive ? On Fri, 8 Sep 2017 16:33:42 +0200 Cédric Le Goater wrote: > On POWER9, the Client Architecture Support (CAS) negotiation process > determines whether the guest operates in XIVE Legacy compatibility or > in XIVE

Re: [Qemu-devel] [PATCH v11 5/6] target-arm: kvm64: handle SIGBUS signal for synchronous External Abort

2017-09-08 Thread Peter Maydell
On 8 September 2017 at 17:17, gengdongjiu wrote: >> >> This code has all just been copied-and-pasted from target/i386/kvm.c. >> Please instead abstract it out properly into a cpu-independent source file. > > > Yes, it copied from x86. > Do you mean abstracting this code to

Re: [Qemu-devel] [PATCH] io: Always remove an old channel watch before adding a new one.

2017-09-08 Thread Daniel P. Berrange
On Fri, Sep 08, 2017 at 09:18:04AM -0700, Brandon Carpenter wrote: > I haven't seen this patch hit master yet and am about to submit a patch set > that is dependent on this one because it triggers the bug fixed by this > patch, causing a segmentation fault. Is it preferred that I include this >

Re: [Qemu-devel] [PATCH] slirp: Add explanation for hostfwd parsing failure

2017-09-08 Thread Dr. David Alan Gilbert
* Philippe Mathieu-Daudé (f4...@amsat.org) wrote: > Hi David, > > On 09/08/2017 12:53 PM, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > e.g. > > ./x86_64-softmmu/qemu-system-x86_64 -nographic -netdev > >

Re: [Qemu-devel] [PATCH v6 00/12] tests: Add VM based build tests (for non-x86_64 and/or non-Linux)

2017-09-08 Thread Alex Bennée
Fam Zheng writes: > v6: Add license to new file. [Philippe] > Change tests/.gitignore. [Philippe] > > v5: Generate source tar file with a script. > Fix tmpdir, use pwd. > Reduce default -j to half cores. > > v4: Drop unused imports and parameters. [Cleber] > Use

Re: [Qemu-devel] [PATCH] slirp: Add explanation for hostfwd parsing failure

2017-09-08 Thread Philippe Mathieu-Daudé
Hi David, On 09/08/2017 12:53 PM, Dr. David Alan Gilbert (git) wrote: From: "Dr. David Alan Gilbert" e.g. ./x86_64-softmmu/qemu-system-x86_64 -nographic -netdev 'user,id=vnet,hostfwd=:555.0.0.0:0-:22' qemu-system-x86_64: -netdev user,id=vnet,hostfwd=:555.0.0.0:0-:22:

Re: [Qemu-devel] [PATCH] io: Always remove an old channel watch before adding a new one.

2017-09-08 Thread Brandon Carpenter
I haven't seen this patch hit master yet and am about to submit a patch set that is dependent on this one because it triggers the bug fixed by this patch, causing a segmentation fault. Is it preferred that I include this patch in that series with the Reviewed-by: tags or to just reference this

Re: [Qemu-devel] [PATCH v11 5/6] target-arm: kvm64: handle SIGBUS signal for synchronous External Abort

2017-09-08 Thread gengdongjiu
Hi peter, Sorry for the late response. > > On 18 August 2017 at 15:23, Dongjiu Geng wrote: > > Add SIGBUS signal handler. In this handler, it checks the exception > > type, translates the host VA which is delivered by host or KVM to > > guest PA, then fills this PA to

Re: [Qemu-devel] [PATCH v3 1/3] io: send proper HTTP response for websocket errors

2017-09-08 Thread Philippe Mathieu-Daudé
On 09/08/2017 10:58 AM, Daniel P. Berrange wrote: When any error occurs while processing the websockets handshake, QEMU just terminates the connection abruptly. This is in violation of the HTTP specs and does not help the client understand what they did wrong. This is particularly bad when the

Re: [Qemu-devel] libvirt/QEMU/SEV interaction

2017-09-08 Thread Laszlo Ersek
On 09/08/17 17:51, Daniel P. Berrange wrote: > On Fri, Sep 08, 2017 at 10:48:10AM -0500, Brijesh Singh wrote: >> I am looking at [1] to get the feel for how do we model it in the XML. >> As you can see I am using ad-hoc to create the sev-guest >> object. Currently, sev-guest object accepts the

Re: [Qemu-devel] libvirt/QEMU/SEV interaction

2017-09-08 Thread Brijesh Singh
On 09/08/2017 10:51 AM, Daniel P. Berrange wrote: On Fri, Sep 08, 2017 at 10:48:10AM -0500, Brijesh Singh wrote: So I could see a flow like the following: The flow looks good 1. mgmt tool calls virConnectGetCapabilities. This returns an XML document that includes the

Re: [Qemu-devel] [PATCH v3 4/7] block: remove legacy I/O throttling

2017-09-08 Thread Kevin Wolf
Am 08.09.2017 um 17:44 hat Manos Pitsidianakis geschrieben: > On Thu, Sep 07, 2017 at 03:26:11PM +0200, Kevin Wolf wrote: > > We shouldn't really need any throttling code in > > blk_root_drained_begin/end any more now because the throttle node will > > be drained. If this code is necessary, a

Re: [Qemu-devel] [PATCH 0/8] TCG vectorization and example conversion

2017-09-08 Thread Richard Henderson
On 09/08/2017 06:49 AM, Alex Bennée wrote: > > Richard Henderson writes: > >> When Alex and I started talking about this topic, this is the direction >> I was thinking. The primary difference from Alex's version is that the >> interface on the target/cpu/ side

Re: [Qemu-devel] [PATCH 2/3] iotests: use -ccw on s390x for 051

2017-09-08 Thread Thomas Huth
On 08.09.2017 13:54, Kevin Wolf wrote: > Am 08.09.2017 um 13:24 hat Cornelia Huck geschrieben: >> On Fri, 8 Sep 2017 13:04:25 +0200 >> Kevin Wolf wrote: >> >>> Am 05.09.2017 um 17:16 hat Cornelia Huck geschrieben: The default cpu model on s390x does not provide zPCI, which

  1   2   3   4   5   >