On Tue, Aug 08, 2017 at 01:40:58PM +0200, Juan Quintela wrote:
> Peter Xu wrote:
> > On Mon, Jul 17, 2017 at 03:42:38PM +0200, Juan Quintela wrote:
> >> Each time that we sync the bitmap, it is a possiblity that we receive
> >> a page that is being processed by a different thread. We fix this
> >
[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: BT/BTS/
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 screen.
>>>
>>> This
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)
> add explict VIRTIO
> 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 controller is not
>> recogni
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 bind/unbind msi.
Signed-off
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 interrupt binded wit
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
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 Tianyu
Reviewed-by: Anthony PERARD
Reviewed-
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
> -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; Harris, James R
>
> Subject: R
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 i
- 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 code
>
> On 08/09/2017 04:54 PM, Paolo
> 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 fo
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 changed, 3 insertions(+), 6 deletions(
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
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 +
2 files changed, 5 insertio
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 +
tests/qemu-iotests/common.rc | 2 +-
2 files chang
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(-)
>
Reviewed-by: 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
wort
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 to
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.
A
"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 ---
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 a/tests/qemu-iotests/check
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 +-
tests/qemu-iotests
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 +
tests/qemu-iotests/common
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 a/tests/qemu-iotests/check b/tests/qemu-iot
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
--- a/tests/qemu-iotests/check
+++ b/te
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 b/tests/qemu-iotests/check
in
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 b/tests/qemu-iotests/check
ind
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
index 01fd5a26e5..50750
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
tests/qemu-iotests/common.config | 49 --
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", simplify
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
---
tests/qemu-iotests/common
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: re
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
Signed-off-by: Jens Freimann
---
te
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
[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 file (and related i
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é
Reviewed-by: Jeff Cody
Revi
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 gues
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 Shorrock
> Signed-off-by:
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
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
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
Reviewed-by: Kevin Wolf
Reviewed-by: John Snow
---
block/qcow2.c | 5 -
1 fil
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 --git a/.travis.y
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: John Snow
---
v2: error message tweak [Kevin]
---
block/vpc.c | 9 ++
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 bdrv_pwrit
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)
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:
- ubuntu-toolchain-r-test
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} && ${TEST_CMD}
matrix
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
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 @@ before_install:
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 ourselves
submodules
$ 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 insertions(+), 1 dele
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:
- gcc
cache:
ccache: true
+ timeo
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 196412f5be..e56c928bef 100644
--- a/.travis.yml
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 @@ env:
- MAKEFL
(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 a/.travis.yml b/.travis.yml
index c8fecb58
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
@@ -70,6 +70,7 @@
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:
ano
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 expecte
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 f583839755..b9caf
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:
glusterfs-common, libbz2-dev, libncursesw5-dev, libnfs-dev]
Signed-off-by: Philip
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
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:
> > >> On 09/08/2017 16
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 t
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
> 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:
>>
>> qemu-system-i3
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 longer works with a Wi
> 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 the Windows splas
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 wrote:
> +Capability layout (d
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:
> >
> > qe
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
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 sti
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 virtio_scsi
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 for fixing that!
IIUC, this patch will b
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 include/hw/pci/pci_bridge.h):
+
+
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 an
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
>
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(-)
>
> diff --git a/migration/
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 can remove an assertio
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
>> individual arguments, wh
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
wi
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
>
> helper
>
> I do
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 -v
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
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
>> one callsite where it
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 the AT_SYMLINK_NOFOLLO
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
---
fsdev/Makefile.ob
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 te
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.
>>
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
> ++
> 2 files changed, 64 insertions(+)
>
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 insertions(+)
diff --git a/tests/ide-test.c b/tests/ide-test.c
index bfd79ddbdc..aa9de065fc 100644
---
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 Hajnoczi
---
hw/ide/core.c | 10 +-
1 file ch
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 +++
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:
- fsdev/Makefile.objs needs to check f
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
> Signed-off-by: John Snow
> ---
>
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
> >>> "/proc/self/fd" can hence
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 fixed the testsuite t
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
wi
1 - 100 of 248 matches
Mail list logo
