[Qemu-devel] [Bug 904308] Re: x86: BT/BTS/BTR/BTC: ZF flag is unaffected

2017-08-09 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/904308 Title: x86:

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Thomas Huth
On 09.08.2017 20:12, Programmingkid wrote: > >> On Aug 9, 2017, at 12:37 PM, Paolo Bonzini wrote: >> >> On 09/08/2017 16:56, Programmingkid wrote: >>> The default vga card not longer works with a Windows 2000 guest. All I see >>> is a black screen after a the Windows splash

Re: [Qemu-devel] [PATCH v3] 9pfs: fix dependencies

2017-08-09 Thread Thomas Huth
On 09.08.2017 18:30, Cornelia Huck wrote: > Nothing in fsdev/ or hw/9pfs/ depends on pci; it should rather depend > on CONFIG_VIRTFS and CONFIG_VIRTIO/CONFIG_XEN only. > > Signed-off-by: Cornelia Huck > --- > > v2->v3: switch dependencies to VIRTFS && (VIRTIO || XEN) >

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Programmingkid
> On Aug 9, 2017, at 6:42 PM, Paolo Bonzini wrote: > > >> I haven't test version QEMU 2.9.0 but I did test version 2.8.0 and it has the >> same problem. Starting up Windows 2000 in VGA mode allowed me to access the >> operating system. Found out QEMU's default video

[Qemu-devel] [PATCH V2 2/3] xen-pt: bind/unbind interrupt remapping format MSI

2017-08-09 Thread Lan Tianyu
From: Chao Gao If a vIOMMU is exposed to guest, guest will configure the msi to remapping format. The original code isn't suitable to the new format. A new pair bind/unbind interfaces are added for this usage. This patch recognizes this case and uses new interfaces to

[Qemu-devel] [PATCH V2 3/3] msi: Handle remappable format interrupt request

2017-08-09 Thread Lan Tianyu
From: Chao Gao According to VT-d spec Interrupt Remapping and Interrupt Posting -> Interrupt Remapping -> Interrupt Request Formats On Intel 64 Platforms, fields of MSI data register have changed. This patch avoids wrongly regarding a remappable format interrupt request as an

[Qemu-devel] [PATCH v2 0/3] Qemu: Add Xen vIOMMU interrupt remapping function support

2017-08-09 Thread Lan Tianyu
This patchset is to deal with MSI interrupt remapping request when guest updates MSI registers. Chao Gao (3): i386/msi: Correct mask of destination ID in MSI address xen-pt: bind/unbind interrupt remapping format MSI msi: Handle remappable format interrupt request configure

[Qemu-devel] [PATCH V2 1/3] i386/msi: Correct mask of destination ID in MSI address

2017-08-09 Thread Lan Tianyu
From: Chao Gao According to SDM 10.11.1, only [19:12] bits of MSI address are Destination ID, change the mask to avoid ambiguity for VT-d spec has used the bit 4 to indicate a remappable interrupt request. Signed-off-by: Chao Gao Signed-off-by: Lan

Re: [Qemu-devel] [PATCH] target/ppc: 'PVR != host PVR' in KVM_SET_SREGS workaround

2017-08-09 Thread David Gibson
On Wed, Aug 09, 2017 at 05:43:46PM -0300, Daniel Henrique Barboza wrote: > Commit d5fc133eed ("ppc: Rework CPU compatibility testing > across migration") changed the way cpu_post_load behaves with > the PVR setting, causing an unexpected bug in KVM-HV migrations > between hosts that are compatible

Re: [Qemu-devel] [PATCH v2 2/4] vhost-user-blk: introduce a new vhost-user-blk host device

2017-08-09 Thread Liu, Changpeng
> -Original Message- > From: Marc-André Lureau [mailto:marcandre.lur...@redhat.com] > Sent: Wednesday, August 9, 2017 11:39 PM > To: Liu, Changpeng > Cc: qemu-devel@nongnu.org; stefa...@gmail.com; pbonz...@redhat.com; > m...@redhat.com; fel...@nutanix.com;

[Qemu-devel] [PATCH] target/i386: fix packusdw in-place operation

2017-08-09 Thread Joseph Myers
The SSE4.1 packusdw instruction combines source and destination vectors of signed 32-bit integers into a single vector of unsigned 16-bit integers, with unsigned saturation. When the source and destination are the same register, this means each 32-bit element of that register is used twice as an

Re: [Qemu-devel] [PATCH 01/12] qemu-iotests: remove dead code

2017-08-09 Thread Paolo Bonzini
- Original Message - > From: "Eric Blake" > To: "Paolo Bonzini" , qemu-devel@nongnu.org > Cc: kw...@redhat.com, qemu-bl...@nongnu.org > Sent: Thursday, August 10, 2017 12:18:54 AM > Subject: Re: [Qemu-devel] [PATCH 01/12] qemu-iotests: remove dead

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Paolo Bonzini
> I haven't test version QEMU 2.9.0 but I did test version 2.8.0 and it has the > same problem. Starting up Windows 2000 in VGA mode allowed me to access the > operating system. Found out QEMU's default video controller is not > recognized. In the Device Manager I can see a yellow question mark

Re: [Qemu-devel] [PATCH 02/12] qemu-iotests: get rid of AWK_PROG

2017-08-09 Thread Philippe Mathieu-Daudé
On 08/09/2017 06:55 PM, Paolo Bonzini wrote: Signed-off-by: Paolo Bonzini Reviewed-by: Philippe Mathieu-Daudé --- tests/qemu-iotests/check | 4 ++-- tests/qemu-iotests/common| 2 +- tests/qemu-iotests/common.config | 3 --- 3 files

Re: [Qemu-devel] [PATCH 03/12] qemu-iotests: move "check" code out of common.rc

2017-08-09 Thread Philippe Mathieu-Daudé
On 08/09/2017 06:55 PM, Paolo Bonzini wrote: Some functions in common.rc are never used by the tests. Move them out of that file and into common, which is already included only by "check". Code that actually *is* common to "check" and tests can be placed in common.config. Signed-off-by: Paolo

Re: [Qemu-devel] [PATCH 05/12] qemu-iotests: do not include common.rc in "check"

2017-08-09 Thread Philippe Mathieu-Daudé
On 08/09/2017 06:55 PM, Paolo Bonzini wrote: It only provides functions used by the test programs. Signed-off-by: Paolo Bonzini Reviewed-by: Philippe Mathieu-Daudé --- tests/qemu-iotests/check | 6 -- tests/qemu-iotests/common.rc | 13

Re: [Qemu-devel] [PATCH 10/12] qemu-iotests: get rid of $iam

2017-08-09 Thread Philippe Mathieu-Daudé
On 08/09/2017 06:55 PM, Paolo Bonzini wrote: The variable is almost unused, and one of the two uses is actually uninitialized. Signed-off-by: Paolo Bonzini Reviewed-by: Philippe Mathieu-Daudé --- tests/qemu-iotests/check | 5 +

Re: [Qemu-devel] [PATCH 02/12] qemu-iotests: get rid of AWK_PROG

2017-08-09 Thread Eric Blake
On 08/09/2017 04:55 PM, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > --- > tests/qemu-iotests/check | 4 ++-- > tests/qemu-iotests/common| 2 +- > tests/qemu-iotests/common.config | 3 --- > 3 files changed, 3 insertions(+), 6 deletions(-) >

Re: [Qemu-devel] [PATCH 01/12] qemu-iotests: remove dead code

2017-08-09 Thread Eric Blake
On 08/09/2017 04:54 PM, Paolo Bonzini wrote: > This includes shell function, shell variables, command line options > (randomize.awk does not exist) and conditions that can never be true > (./qemu does not exist anymore). Can we point to a commit id where we stopped making ./qemu? Is it still

Re: [Qemu-devel] OvmfPkg (PlatformPei?): supporting Qemu's nvdimm device

2017-08-09 Thread Laszlo Ersek
On 08/09/17 22:47, Rebecca Cran wrote: > I've been investigating adding support for Qemu's nvdimm devices to > OVMF. I was thinking such support would go into PlatformPei, but it > looks like I can only read the ACPI NFIT in the DXE phase. So, should > Qemu be changed to add non-volatile memory

Re: [Qemu-devel] [PATCH v4 17/22] libqtest: Add qmp_args() helper

2017-08-09 Thread Eric Blake
On 08/09/2017 10:57 AM, Markus Armbruster wrote: > I don't like the qmp_args name. It's not about arguments, it's about > sending a command with arguments. > > I'm afraid I'm getting pretty thorougly confused by the evolving > interface. So I stop and think how it should look like when done.

[Qemu-devel] [PATCH 12/12] qemu-iotests: merge "check" and "common"

2017-08-09 Thread Paolo Bonzini
"check" is full of qemu-iotests--specific details. Separating it from "common" does not make much sense anymore. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 482 +++- tests/qemu-iotests/common | 501

[Qemu-devel] [PATCH 11/12] qemu-iotests: include common.env and common.config early

2017-08-09 Thread Paolo Bonzini
They are now very small and they do not have external dependencies. We can source them as soon as we have their path. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git

[Qemu-devel] [PATCH 04/12] qemu-iotests: limit non-_PROG-suffixed variables to common.rc

2017-08-09 Thread Paolo Bonzini
These are never used by "check", with one exception that does not need $QEMU_OPTIONS. Keep them in common.rc, which will be soon included only by the tests. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/039.out | 10 ++--- tests/qemu-iotests/061.out | 4

[Qemu-devel] [PATCH 01/12] qemu-iotests: remove dead code

2017-08-09 Thread Paolo Bonzini
This includes shell function, shell variables, command line options (randomize.awk does not exist) and conditions that can never be true (./qemu does not exist anymore). Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 36 +

[Qemu-devel] [PATCH 10/12] qemu-iotests: get rid of $iam

2017-08-09 Thread Paolo Bonzini
The variable is almost unused, and one of the two uses is actually uninitialized. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 5 + tests/qemu-iotests/common.rc | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git

[Qemu-devel] [PATCH 09/12] qemu-iotests: do not search for binaries in the current directory

2017-08-09 Thread Paolo Bonzini
Looking in the build root is enough. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/qemu-iotests/check b/tests/qemu-iotests/check index 5b9c0c8ade..e5d1ae3d92 100755 ---

[Qemu-devel] [PATCH 05/12] qemu-iotests: do not include common.rc in "check"

2017-08-09 Thread Paolo Bonzini
It only provides functions used by the test programs. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 6 -- tests/qemu-iotests/common.rc | 13 + 2 files changed, 5 insertions(+), 14 deletions(-) diff --git a/tests/qemu-iotests/check

[Qemu-devel] [PATCH 08/12] qemu-iotests: fix uninitialized variable

2017-08-09 Thread Paolo Bonzini
The function is used in "common" but defined only after the file is sourced. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 2 -- tests/qemu-iotests/common | 2 ++ 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/qemu-iotests/check

[Qemu-devel] [PATCH 02/12] qemu-iotests: get rid of AWK_PROG

2017-08-09 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/check | 4 ++-- tests/qemu-iotests/common| 2 +- tests/qemu-iotests/common.config | 3 --- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/tests/qemu-iotests/check b/tests/qemu-iotests/check

[Qemu-devel] [PATCH 07/12] qemu-iotests: disintegrate more parts of common.config

2017-08-09 Thread Paolo Bonzini
Split "check" parts from tests part. For the directory setup, the actual computation of directories goes in "check", while the sanity checks go in the tests. Signed-off-by: Paolo Bonzini --- tests/qemu-iotests/common| 24

[Qemu-devel] [PATCH 06/12] qemu-iotests: do not do useless search for QEMU_*_PROG

2017-08-09 Thread Paolo Bonzini
With the exception of qnio_server, all the variables are initialized by "check" prior to "common" being sourced. They cannot be empty. Only the "realpath" invocation is useful, and can be done just once in "check" rather than in the tests. For qnio_server, move the detection to "common",

[Qemu-devel] [PATCH 03/12] qemu-iotests: move "check" code out of common.rc

2017-08-09 Thread Paolo Bonzini
Some functions in common.rc are never used by the tests. Move them out of that file and into common, which is already included only by "check". Code that actually *is* common to "check" and tests can be placed in common.config. Signed-off-by: Paolo Bonzini ---

[Qemu-devel] [PATCH 00/12] cleanup qemu-iotests

2017-08-09 Thread Paolo Bonzini
The purpose of this series is to separate the "check" sources from the tests. After these patches, common.config is reduced to simple shell initialization, and common.rc is only included by the tests. Along the way, a lot of dead code is removed too. Paolo Paolo Bonzini (12): qemu-iotests:

[Qemu-devel] [PATCH v3 1/2] tests/vhost-user-bridge: disable debug output by default

2017-08-09 Thread Jens Freimann
From: Jens Freimann vhost-user-bridge prints out a lot of information, including dumps of all transmitted data. When called from a testcase this output clutters the actual test results, so let's make the default no debug output. Reviewed-by: Maxime Coquelin

[Qemu-devel] [PATCH v3 0/2] tests/pxe-test: add testcase using vhost-user-bridge

2017-08-09 Thread Jens Freimann
This implements a testcase for pxe-test using the vhost-user interface. Spawn a vhost-user-bridge process and connect it to the qemu process. changes v2->v3: - fixed build on aarch64 by removing a flag for g_spawn_async() that is not even needed. Patch builds on my system for target aarch64 now

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-09 Thread Philippe Mathieu-Daudé
[I added mails of other person who reply to this series, this mail is not directly addressed to Alex] On 08/09/2017 11:38 AM, Alex Bennée wrote: Philippe Mathieu-Daudé writes: [...] Do you think there should be another entry in "Block QAPI, monitor, command line"? or this

[Qemu-devel] [PATCH v2 4/5] qcow2: Drop debugging dump_refcounts()

2017-08-09 Thread Eric Blake
It's been #if 0'd since its introduction in 2006, commit 585f8587. We can revive dead code if we need it, but in the meantime, it has bit-rotted (for example, not checking for failure in bdrv_getlength()). Signed-off-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé

[Qemu-devel] [PATCH] target/ppc: 'PVR != host PVR' in KVM_SET_SREGS workaround

2017-08-09 Thread Daniel Henrique Barboza
Commit d5fc133eed ("ppc: Rework CPU compatibility testing across migration") changed the way cpu_post_load behaves with the PVR setting, causing an unexpected bug in KVM-HV migrations between hosts that are compatible (POWER8 and POWER8E, for example). Even with pvr_match() returning true, the

Re: [Qemu-devel] [PATCH 1/2] IDE: Do not flush empty CDROM drives

2017-08-09 Thread Eric Blake
On 08/09/2017 11:02 AM, Stefan Hajnoczi wrote: > The block backend changed in a way that flushing empty CDROM drives now > crashes. Amend IDE to avoid doing so until the root problem can be > addressed for 2.11. > > Original patch by John Snow . > > Reported-by: Kieron

[Qemu-devel] [PATCH for-2.10 v2 0/5] More bdrv_getlength() fixes

2017-08-09 Thread Eric Blake
We already have a lot of bdrv_getlength() fixes in -rc2; so I think this is still okay for -rc3. v1 was here (with a typo'd subject line): https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg01226.html Since v1: - patch 1: fix error message capitalization (Kevin, R-b kept) - fix locking bug

[Qemu-devel] [PATCH v2 2/5] qcow: Change signature of get_cluster_offset()

2017-08-09 Thread Eric Blake
The old signature has an ambiguous meaning for a return of 0: either no allocation was requested or necessary, or an error occurred (but any errno associated with the error is lost to the caller, which then has to assume EIO). Better is to follow the example of qcow2, by changing the signature to

[Qemu-devel] [PATCH v2 5/5] qcow2: Check failure of bdrv_getlength()

2017-08-09 Thread Eric Blake
qcow2_co_pwritev_compressed() should not call bdrv_truncate() if determining the size failed. Reported-by: Markus Armbruster Signed-off-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Jeff Cody

[Qemu-devel] [NOTFORMERGE PATCH for-2.11 v3 16/16] travis/osx: build using bleeding Xcode

2017-08-09 Thread Philippe Mathieu-Daudé
test 'unstable' Xcode (9beta4) ... OS X 10.13 (darwin16.7.0) Apple LLVM version 9.0.0 (clang-900.0.31) see https://docs.travis-ci.com/user/reference/osx/#OS-X-Version Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 5 + 1 file changed, 5 insertions(+) diff

[Qemu-devel] [PATCH v2 1/5] vpc: Check failure of bdrv_getlength()

2017-08-09 Thread Eric Blake
vpc_open() was checking for bdrv_getlength() failure in one, but not the other, location. Reported-by: Markus Armbruster Signed-off-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Jeff Cody Reviewed-by:

[Qemu-devel] [PATCH v2 3/5] qcow: Check failure of bdrv_getlength() and bdrv_truncate()

2017-08-09 Thread Eric Blake
Omitting the check for whether bdrv_getlength() and bdrv_truncate() failed meant that it was theoretically possible to return an incorrect offset to the caller. More likely, conditions for either of these functions to fail would also cause one of our other calls (such as bdrv_pread() or

[Qemu-devel] [RFC PATCH for-2.11 v3 15/16] travis/osx: build using more Xcode versions

2017-08-09 Thread Philippe Mathieu-Daudé
currently default builder is based on Xcode 7.3.1: OS X 10.11 (darwin14.5.0) Apple LLVM version 7.3.0 add the oldest available Xcode (6.4): OS X 10.10 (darwin14.5.0) Apple LLVM 6.1.0 (based on LLVM 3.6.0svn) and the newer available Xcode (8.3.3): OS X 10.12 (darwin16.6.0)

[Qemu-devel] [RFC PATCH for-2.11 v3 13/16] travis: use gcc-6 sanitizers

2017-08-09 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.travis.yml b/.travis.yml index 24f62fb7cf..1da542f99f 100644 --- a/.travis.yml +++ b/.travis.yml @@ -185,8 +185,8 @@ matrix: -

[Qemu-devel] [RFC PATCH for-2.11 v3 14/16] travis: allow_failures for gprof/gcov

2017-08-09 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 6 ++ 1 file changed, 6 insertions(+) diff --git a/.travis.yml b/.travis.yml index 1da542f99f..7c93a10c5f 100644 --- a/.travis.yml +++ b/.travis.yml @@ -87,6 +87,7 @@ before_script: script: - make ${MAKEFLAGS} &&

[Qemu-devel] [PATCH for-2.11 v3 05/16] travis: retry if llvm.org timeouts

2017-08-09 Thread Philippe Mathieu-Daudé
example of failure: https://travis-ci.org/qemu/qemu/jobs/243232857 $ sudo apt-get update -qq W: Failed to fetch http://llvm.org/apt/trusty/dists/llvm-toolchain-trusty-3.9/Release.gpg Connection failed E: Some index files failed to download. They have been ignored, or old ones used

[Qemu-devel] [PATCH for-2.11 v3 10/16] travis: check ./configure outcome, dump config.log on failure

2017-08-09 Thread Philippe Mathieu-Daudé
also de-duplicate 'before_script' Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/.travis.yml b/.travis.yml index 72aad5ca35..88ca78c775 100644 --- a/.travis.yml +++ b/.travis.yml @@ -79,7 +79,7

[Qemu-devel] [PATCH for-2.11 v3 11/16] travis/osx: use readable YAML

2017-08-09 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index 88ca78c775..4bf69b0116 100644 --- a/.travis.yml +++ b/.travis.yml @@ -74,8 +74,10 @@ git: # we want to do this

[Qemu-devel] [PATCH for-2.11 v3 12/16] travis/osx: silent texinfo warnings

2017-08-09 Thread Philippe Mathieu-Daudé
$ make info GEN qemu-doc.html qemu-doc.texi:8: warning: unrecognized encoding name `UTF-8'. GEN qemu-doc.txt qemu-doc.texi:8: warning: unrecognized encoding name `UTF-8'. Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 4 +++- 1 file changed, 3

[Qemu-devel] [PATCH for-2.11 v3 04/16] travis: increase S3 cache timeout

2017-08-09 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alex Bennée --- .travis.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.travis.yml b/.travis.yml index 311f25eb28..d622d8c433 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,6 +6,7 @@ compiler:

[Qemu-devel] [PATCH for-2.11 v3 07/16] travis: split the gprof/gcov job

2017-08-09 Thread Philippe Mathieu-Daudé
gcov generates lot of output and often trigger job limit timeout Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alex Bennée --- .travis.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index

[Qemu-devel] [PATCH for-2.11 v3 08/16] travis: reorder matrix to speedup failure

2017-08-09 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 36 +++- 1 file changed, 19 insertions(+), 17 deletions(-) diff --git a/.travis.yml b/.travis.yml index e56c928bef..57462b0471 100644 --- a/.travis.yml +++ b/.travis.yml @@ -64,11 +64,11 @@

[Qemu-devel] [PATCH for-2.11 v3 03/16] travis: enable multiple caching features

2017-08-09 Thread Philippe Mathieu-Daudé
(this eases git workflow) see https://docs.travis-ci.com/user/caching#enabling-multiple-caching-features Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Alex Bennée --- .travis.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH for-2.11 v3 09/16] travis: reduce git clone depth

2017-08-09 Thread Philippe Mathieu-Daudé
see https://docs.travis-ci.com/user/customizing-the-build#Git-Clone-Depth Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.travis.yml b/.travis.yml index 57462b0471..72aad5ca35 100644 --- a/.travis.yml +++ b/.travis.yml

[Qemu-devel] [PATCH for-2.11 v3 06/16] travis: retry when git submodules initialization fails

2017-08-09 Thread Philippe Mathieu-Daudé
example of failure: https://travis-ci.org/philmd/qemu/jobs/245612939 $ git submodule update --init --recursive [...] Submodule 'pixman' (git://anongit.freedesktop.org/pixman) registered for path 'pixman' Cloning into 'pixman'... fatal: unable to connect to anongit.freedesktop.org:

[Qemu-devel] [PATCH for-2.11 v3 00/16] travis: speedup to reduce failures

2017-08-09 Thread Philippe Mathieu-Daudé
These patches try to improve our Travis CI usage (few failures the last days): - lower false negative rate (mostly network issues), - improved code coverage, - speedup failure detection. dropped since v2 (will follow as a separate series) - multicore parallelism use - cache use Patch 1 is

[Qemu-devel] [PATCH for-2.11 v3 01/16] travis: update sudo-enabled Trusty images

2017-08-09 Thread Philippe Mathieu-Daudé
see https://blog.travis-ci.com/2017-06-19-trusty-updates-2017-Q2 and https://blog.travis-ci.com/2017-06-21-trusty-updates-2017-Q2-launch Signed-off-by: Philippe Mathieu-Daudé --- .travis.yml | 4 1 file changed, 4 insertions(+) diff --git a/.travis.yml b/.travis.yml index

[Qemu-devel] [PATCH for-2.11 v3 02/16] travis: install more library dependencies

2017-08-09 Thread Philippe Mathieu-Daudé
From: Peter Maydell Update the travis list of library packages to install so that our build tests cover more of our code base. Signed-off-by: Peter Maydell Reviewed-by: Alex Bennée [PMD: added more dependencies:

[Qemu-devel] [PATCH] QEMU Backup Tool

2017-08-09 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

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Michael S. Tsirkin
On Wed, Aug 09, 2017 at 09:05:54PM +0300, Michael S. Tsirkin wrote: > On Wed, Aug 09, 2017 at 01:54:23PM -0400, Programmingkid wrote: > > > > > On Aug 9, 2017, at 1:18 PM, Michael S. Tsirkin wrote: > > > > > > On Wed, Aug 09, 2017 at 06:37:12PM +0200, Paolo Bonzini wrote: > >

Re: [Qemu-devel] [PATCH v2 3/4] contrib/libvhost-user: enable virtio config space messages

2017-08-09 Thread Marc-André Lureau
Hi - Original Message - > Enable VHOST_USER_GET_CONFIG/VHOST_USER_SET_CONFIG/VHOST_USER_SET_CONFIG_FD > messages in libvhost-user library, users can implement their own I/O target > based on the library. This enable the virtio config space delivered between > Qemu host device and the I/O

Re: [Qemu-devel] [PATCH v2 4/4] contrib/vhost-user-blk: introduce a vhost-user-blk sample application

2017-08-09 Thread Marc-André Lureau
Hi - Original Message - > This commit introcudes a vhost-user-blk backend device, it uses UNIX > domain socket to communicate with Qemu. The vhost-user-blk sample > application should be used with Qemu vhost-user-blk-pci device. > > To use it, complie with: > make vhost-user-blk > > and

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Programmingkid
> On Aug 9, 2017, at 12:37 PM, Paolo Bonzini wrote: > > On 09/08/2017 16:56, Programmingkid wrote: >> The default vga card not longer works with a Windows 2000 guest. All I see >> is a black screen after a the Windows splash screen. >> >> This is the command-line I used:

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Michael S. Tsirkin
On Wed, Aug 09, 2017 at 01:54:23PM -0400, Programmingkid wrote: > > > On Aug 9, 2017, at 1:18 PM, Michael S. Tsirkin wrote: > > > > On Wed, Aug 09, 2017 at 06:37:12PM +0200, Paolo Bonzini wrote: > >> On 09/08/2017 16:56, Programmingkid wrote: > >>> The default vga card not

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Programmingkid
> On Aug 9, 2017, at 1:18 PM, Michael S. Tsirkin wrote: > > On Wed, Aug 09, 2017 at 06:37:12PM +0200, Paolo Bonzini wrote: >> On 09/08/2017 16:56, Programmingkid wrote: >>> The default vga card not longer works with a Windows 2000 guest. All I see >>> is a black screen after a

Re: [Qemu-devel] [PATCH v4 5/5] docs: update documentation considering PCIE-PCI bridge

2017-08-09 Thread Laszlo Ersek
On 08/09/17 18:52, Aleksandr Bezzubikov wrote: > 2017-08-09 13:18 GMT+03:00 Laszlo Ersek : >> On 08/08/17 21:21, Aleksandr Bezzubikov wrote: >>> 2017-08-08 18:11 GMT+03:00 Laszlo Ersek : one comment below On 08/05/17 22:27, Aleksandr Bezzubikov

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Michael S. Tsirkin
On Wed, Aug 09, 2017 at 06:37:12PM +0200, Paolo Bonzini wrote: > On 09/08/2017 16:56, Programmingkid wrote: > > The default vga card not longer works with a Windows 2000 guest. All I see > > is a black screen after a the Windows splash screen. > > > > This is the command-line I used: > > > >

Re: [Qemu-devel] Qemu and 32 PCIe devices

2017-08-09 Thread Michael S. Tsirkin
On Wed, Aug 09, 2017 at 09:26:11AM +0200, Paolo Bonzini wrote: > On 09/08/2017 03:06, Laszlo Ersek wrote: > >> 20.14% qemu-system-x86_64 [.] render_memory_region > >> 17.14% qemu-system-x86_64 [.] subpage_register > >> 10.31% qemu-system-x86_64

Re: [Qemu-devel] [PATCH v2 2/4] vhost-user-blk: introduce a new vhost-user-blk host device

2017-08-09 Thread Michael S. Tsirkin
I only had time for a quick look. More review when you repost after release. On Thu, Aug 10, 2017 at 06:12:29PM +0800, Changpeng Liu wrote: > This commit introduces a new vhost-user device for block, it uses a > chardev to connect with the backend, same with Qemu virito-blk device, > Guest OS

Re: [Qemu-devel] [PATCH v2 0/4] *** Introduce a new vhost-user-blk host device to Qemu ***

2017-08-09 Thread Michael S. Tsirkin
On Thu, Aug 10, 2017 at 06:12:27PM +0800, Changpeng Liu wrote: > Althrough virtio scsi specification was designed as a replacement for > virtio_blk, > there are still many users using virtio_blk. Qemu 2.9 introduced a new device > vhost user scsi which can process I/O in user space for

Re: [Qemu-devel] [PATCH v3] 9pfs: fix dependencies

2017-08-09 Thread Greg Kurz
On Wed, 9 Aug 2017 18:30:19 +0200 Cornelia Huck wrote: > Nothing in fsdev/ or hw/9pfs/ depends on pci; it should rather depend > on CONFIG_VIRTFS and CONFIG_VIRTIO/CONFIG_XEN only. > > Signed-off-by: Cornelia Huck > --- > I think we're ok now. Thanks

Re: [Qemu-devel] [PATCH v4 5/5] docs: update documentation considering PCIE-PCI bridge

2017-08-09 Thread Aleksandr Bezzubikov
2017-08-09 13:18 GMT+03:00 Laszlo Ersek : > On 08/08/17 21:21, Aleksandr Bezzubikov wrote: >> 2017-08-08 18:11 GMT+03:00 Laszlo Ersek : >>> one comment below >>> >>> On 08/05/17 22:27, Aleksandr Bezzubikov wrote: >>> +Capability layout (defined in

Re: [Qemu-devel] [PATCH RFC 0/6] QEMU: kvm: cleanup kvm_slot handling

2017-08-09 Thread Paolo Bonzini
On 09/08/2017 15:33, David Hildenbrand wrote: > If I am not missing something important here, we can heavily simplify > the kvm_slot code. Flatview will make sure that we don't have to deal > with overlapping slots. E.g. when a memory section is resized, we are > first notified about the removal

Re: [Qemu-devel] [PATCH v5 10/17] migration: Create ram_multifd_page

2017-08-09 Thread Paolo Bonzini
On 08/08/2017 21:14, Dr. David Alan Gilbert wrote: >> There is no barrier there that I can see. I know that it probably work >> on x86, but in others? I think that it *** HERE we need that >> memory barrier that we don't have. > Yes, I think that's smp_mb_release() - and you have to do an >

Re: [Qemu-devel] [PATCH v5 12/17] migration: Send the fd number which we are going to use for this page

2017-08-09 Thread Paolo Bonzini
On 17/07/2017 15:42, Juan Quintela wrote: > We are still sending the page through the main channel, that would > change later in the series > > Signed-off-by: Juan Quintela > --- > migration/ram.c | 13 +++-- > 1 file changed, 11 insertions(+), 2 deletions(-) > >

Re: [Qemu-devel] [PATCH v4 21/22] libqtest: Drop now-unused qmp()

2017-08-09 Thread Eric Blake
On 08/09/2017 11:01 AM, Markus Armbruster wrote: > Eric Blake writes: > >> All callers have been converted to a form of qmp_cmd() or >> qmp_args() that takes the command name with less boilerplate. >> Therefore, we also know that all commands are using >> interpolation, and

Re: [Qemu-devel] [PATCH v4 19/22] libqtest: Add qmp_args_dict() helper

2017-08-09 Thread Eric Blake
On 08/09/2017 10:59 AM, Markus Armbruster wrote: > Eric Blake writes: > >> Leaving interpolation into JSON to qobject_from_jsonf() is more >> robust than building QMP input manually; however, we have a few >> places where code is already creating a QDict to interpolate >>

[Qemu-devel] [for-2.10 PATCH v4] 9pfs: local: fix fchmodat_nofollow() limitations

2017-08-09 Thread Greg Kurz
This function has to ensure it doesn't follow a symlink that could be used to escape the virtfs directory. This could be easily achieved if fchmodat() on linux honored the AT_SYMLINK_NOFOLLOW flag as described in POSIX, but it doesn't. There was a tentative to implement a new fchmodat2() syscall

Re: [Qemu-devel] [PATCH v4 16/22] libqtest: Add qmp_cmd() helper

2017-08-09 Thread Eric Blake
On 08/09/2017 10:40 AM, Markus Armbruster wrote: > Eric Blake writes: > >> Now that we've asserted that all of our interpolated QMP commands >> include 'execute', we can reduce some of the caller boilerplate >> by providing a helpr function to wrap commands with no arguments >

Re: [Qemu-devel] No video for Windows 2000 guest

2017-08-09 Thread Paolo Bonzini
On 09/08/2017 16:56, Programmingkid wrote: > The default vga card not longer works with a Windows 2000 guest. All I see is > a black screen after a the Windows splash screen. > > This is the command-line I used: > > qemu-system-i386 -hda Windows2000HD.qcow2 -boot c -m 512 > > When using the

Re: [Qemu-devel] >256 Virtio-net-pci hotplug Devices

2017-08-09 Thread Kinsella, Ray
Marcel, The findings are pretty consistent with what I identified. Although it looks like SeaBIOS fairs better than UEFI. Thanks for the headsup, will reply on the thread itself. Ray K -Original Message- From: Marcel Apfelbaum [mailto:mar...@redhat.com] Sent: Wednesday, August 9, 2017

Re: [Qemu-devel] [PATCH v4 15/22] libqtest: Delete qtest_qmp() wrappers

2017-08-09 Thread Eric Blake
On 08/09/2017 10:34 AM, Markus Armbruster wrote: > Eric Blake writes: > >> None of our tests were directly using qtest_qmp() and friends; >> even tests like postcopy-test.c that manage multiple connections >> get along just fine changing global_qtest as needed (other than >>

Re: [Qemu-devel] [for-2.10 PATCH v3] 9pfs: local: fix fchmodat_nofollow() limitations

2017-08-09 Thread Greg Kurz
On Wed, 9 Aug 2017 11:19:42 -0500 Eric Blake wrote: > On 08/09/2017 11:00 AM, Greg Kurz wrote: > > This function has to ensure it doesn't follow a symlink that could be used > > to escape the virtfs directory. This could be easily achieved if fchmodat() > > on linux honored

[Qemu-devel] [PATCH v3] 9pfs: fix dependencies

2017-08-09 Thread Cornelia Huck
Nothing in fsdev/ or hw/9pfs/ depends on pci; it should rather depend on CONFIG_VIRTFS and CONFIG_VIRTIO/CONFIG_XEN only. Signed-off-by: Cornelia Huck --- v2->v3: switch dependencies to VIRTFS && (VIRTIO || XEN) add explict VIRTIO dependency for virtio-9p-device.o

Re: [Qemu-devel] [for-2.10 PATCH v3] 9pfs: local: fix fchmodat_nofollow() limitations

2017-08-09 Thread Eric Blake
On 08/09/2017 11:00 AM, Greg Kurz wrote: > This function has to ensure it doesn't follow a symlink that could be used > to escape the virtfs directory. This could be easily achieved if fchmodat() > on linux honored the AT_SYMLINK_NOFOLLOW flag as described in POSIX, but > it doesn't. There was a

Re: [Qemu-devel] [PATCH v2 for 2.10] iotests: fix 185

2017-08-09 Thread Eric Blake
On 08/09/2017 10:57 AM, Vladimir Sementsov-Ogievskiy wrote: This is because quite happens before first mirror request is done >>> s/quite/quit/ >>> (and, in specified case, even before block-job len field is set). To prevent it let's just add a sleep for 0.3 seconds before quite.

Re: [Qemu-devel] [PATCH 4/4] block-backend: test flush op on empty backend

2017-08-09 Thread Kevin Wolf
Am 08.08.2017 um 19:57 hat John Snow geschrieben: > From: Kevin Wolf > > Signed-off-by: Kevin Wolf > Signed-off-by: John Snow > --- > tests/Makefile.include | 2 ++ > tests/test-block-backend.c | 62 >

[Qemu-devel] [PATCH 2/2] IDE: test flush on empty CDROM

2017-08-09 Thread Stefan Hajnoczi
From: Kevin Wolf Signed-off-by: Kevin Wolf Signed-off-by: John Snow Reviewed-by: Eric Blake Signed-off-by: Stefan Hajnoczi --- tests/ide-test.c | 19 +++ 1 file changed, 19

[Qemu-devel] [PATCH 1/2] IDE: Do not flush empty CDROM drives

2017-08-09 Thread Stefan Hajnoczi
The block backend changed in a way that flushing empty CDROM drives now crashes. Amend IDE to avoid doing so until the root problem can be addressed for 2.11. Original patch by John Snow . Reported-by: Kieron Shorrock Signed-off-by: Stefan

[Qemu-devel] [PATCH 0/2] IDE: Do not flush empty drives

2017-08-09 Thread Stefan Hajnoczi
John Snow is offline until Monday, QEMU 2.10-rc3 is due to be tagged on Tuesday, and so I've taken two patches John sent for 2.10 and addressed review comments. Kevin Wolf (1): IDE: test flush on empty CDROM Stefan Hajnoczi (1): IDE: Do not flush empty CDROM drives hw/ide/core.c| 10

Re: [Qemu-devel] [PATCH v2] 9pfs: fix dependencies

2017-08-09 Thread Cornelia Huck
On Wed, 9 Aug 2017 14:10:01 +0200 Greg Kurz wrote: > On Wed, 9 Aug 2017 13:06:14 +0200 > Cornelia Huck wrote: > > Should the condition be VIRTFS && (VIRTIO || XEN), then? > > That's what I was beginning to think as well :) OK, here's what should work: -

Re: [Qemu-devel] [PATCH 3/4] block-backend: shift in-flight counter to BB from BDS

2017-08-09 Thread Kevin Wolf
Am 08.08.2017 um 19:57 hat John Snow geschrieben: > From: Kevin Wolf > > This allows us to detect errors in cache flushing (ENOMEDIUM) > without choking on a null dereference because we assume that > blk_bs(bb) is always defined. > > Signed-off-by: Kevin Wolf

Re: [Qemu-devel] [for-2.10 PATCH v2] 9pfs: local: fix fchmodat_nofollow() limitations

2017-08-09 Thread Greg Kurz
On Wed, 9 Aug 2017 10:59:46 -0500 Eric Blake wrote: > On 08/09/2017 10:22 AM, Greg Kurz wrote: > > >>> > >>> The solution is to use O_PATH: openat() now succeeds in both cases, and we > >>> can ensure the path isn't a symlink with fstat(). The associated entry in > >>>

Re: [Qemu-devel] [PATCH v4 21/22] libqtest: Drop now-unused qmp()

2017-08-09 Thread Markus Armbruster
Eric Blake writes: > All callers have been converted to a form of qmp_cmd() or > qmp_args() that takes the command name with less boilerplate. > Therefore, we also know that all commands are using > interpolation, and can remove an assertion. > > This also means that we have

[Qemu-devel] [for-2.10 PATCH v3] 9pfs: local: fix fchmodat_nofollow() limitations

2017-08-09 Thread Greg Kurz
This function has to ensure it doesn't follow a symlink that could be used to escape the virtfs directory. This could be easily achieved if fchmodat() on linux honored the AT_SYMLINK_NOFOLLOW flag as described in POSIX, but it doesn't. There was a tentative to implement a new fchmodat2() syscall

Re: [Qemu-devel] [for-2.10 PATCH v2] 9pfs: local: fix fchmodat_nofollow() limitations

2017-08-09 Thread Eric Blake
On 08/09/2017 10:22 AM, Greg Kurz wrote: >>> >>> The solution is to use O_PATH: openat() now succeeds in both cases, and we >>> can ensure the path isn't a symlink with fstat(). The associated entry in >>> "/proc/self/fd" can hence be safely passed to the regular chmod() syscall. >> >> Hey -

  1   2   3   >