Signed-off-by: Fam Zheng
---
hw/net/xilinx_axienet.c | 31 +--
1 file changed, 13 insertions(+), 18 deletions(-)
diff --git a/hw/net/xilinx_axienet.c b/hw/net/xilinx_axienet.c
index b670184..e92762b 100644
--- a/hw/net/xilinx_axienet.c
+++ b/hw/net/xilinx_axienet.c
@@
"Daniel P. Berrange" writes:
> On Mon, Jul 03, 2017 at 04:07:21PM +0200, Markus Armbruster wrote:
>> "Daniel P. Berrange" writes:
>>
>> > On Thu, Jun 29, 2017 at 12:42:38PM -0700, Alistair Francis wrote:
>> >> This patch removes the exisinting error_vreport() function and replaces it
>> >> with
Signed-off-by: Fam Zheng
---
hw/dma/xilinx_axidma.c | 26 +++---
1 file changed, 11 insertions(+), 15 deletions(-)
diff --git a/hw/dma/xilinx_axidma.c b/hw/dma/xilinx_axidma.c
index 6065689..df293d3 100644
--- a/hw/dma/xilinx_axidma.c
+++ b/hw/dma/xilinx_axidma.c
@@ -124,8 +1
Signed-off-by: Fam Zheng
---
hw/arm/xlnx-zynqmp.c | 12
include/hw/arm/xlnx-zynqmp.h | 3 ++-
2 files changed, 6 insertions(+), 9 deletions(-)
diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c
index 64f52f8..2ce9cad 100644
--- a/hw/arm/xlnx-zynqmp.c
+++ b/hw/arm/xlnx
Unlike the usual object_property_add_link() invocations in other
devices, dimm checks the "is mapped" state of the backend in addition to
qdev_prop_allow_set_link_before_realize. To convert it without
specializing DEFINE_PROP_LINK which always uses the qdev general check
callback, move the extra ch
We add the vnet_hdr_support option for filter-mirror, default is disabled.
If you use virtio-net-pci or other driver needs vnet_hdr, please enable it.
You can use it for example:
-object filter-mirror,id=m0,netdev=hn0,queue=tx,outdev=mirror0,vnet_hdr_support
If it has vnet_hdr_support flag, we wil
Signed-off-by: Fam Zheng
---
hw/intc/arm_gicv3_its_kvm.c| 18 ++
include/hw/intc/arm_gicv3_its_common.h | 3 ++-
2 files changed, 8 insertions(+), 13 deletions(-)
diff --git a/hw/intc/arm_gicv3_its_kvm.c b/hw/intc/arm_gicv3_its_kvm.c
index 1f8991b..5520c28 100644
---
Signed-off-by: Zhang Chen
---
docs/colo-proxy.txt | 26 ++
1 file changed, 26 insertions(+)
diff --git a/docs/colo-proxy.txt b/docs/colo-proxy.txt
index c4941de..f6a624f 100644
--- a/docs/colo-proxy.txt
+++ b/docs/colo-proxy.txt
@@ -182,6 +182,32 @@ Secondary(ip:3.3.3.8):
Unlike other object_property_add_link() occurrences in virtio devices,
virtio-crypto checks the "in use" state of the linked backend object in
addition to qdev_prop_allow_set_link_before_realize. To convert it
without needing to specialize DEFINE_PROP_LINK which always uses the
qdev callback, move
We can use this property flush and send packet with vnet_hdr_len.
Signed-off-by: Zhang Chen
---
net/colo-compare.c| 8 ++--
net/colo.c| 3 ++-
net/colo.h| 4 +++-
net/filter-rewriter.c | 2 +-
4 files changed, 12 insertions(+), 5 deletions(-)
diff --git a/net/col
Signed-off-by: Fam Zheng
---
hw/s390x/virtio-ccw.c | 2 --
hw/virtio/virtio-pci.c | 2 --
hw/virtio/virtio-rng.c | 14 ++
include/hw/virtio/virtio-rng.h | 3 ++-
4 files changed, 4 insertions(+), 17 deletions(-)
diff --git a/hw/s390x/virtio-ccw.c b/hw/s390
Signed-off-by: Fam Zheng
---
hw/arm/armv7m.c | 10 +++---
include/hw/arm/armv7m.h | 2 +-
2 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/hw/arm/armv7m.c b/hw/arm/armv7m.c
index 3a13d82..3b0ada0 100644
--- a/hw/arm/armv7m.c
+++ b/hw/arm/armv7m.c
@@ -132,12 +132,6 @@ st
This property can be used to replace the object_property_add_link in
device code, to add a link to other objects, which is a common pattern.
Signed-off-by: Fam Zheng
---
hw/core/qdev-properties.c| 18 ++
include/hw/qdev-core.h | 1 +
include/hw/qdev-properties.h | 9 +
Signed-off-by: Fam Zheng
---
hw/block/dataplane/virtio-blk.c | 2 +-
hw/block/virtio-blk.c | 5 +
hw/s390x/virtio-ccw.c | 2 --
hw/virtio/virtio-pci.c | 2 --
include/hw/virtio/virtio-blk.h | 3 ++-
5 files changed, 4 insertions(+), 10 deletions(-)
diff --git a/
Signed-off-by: Fam Zheng
---
hw/s390x/virtio-ccw.c | 2 --
hw/scsi/virtio-scsi-dataplane.c | 2 +-
hw/scsi/virtio-scsi.c | 13 ++---
hw/virtio/virtio-pci.c | 2 --
include/hw/virtio/virtio-scsi.h | 3 ++-
5 files changed, 5 insertions(+), 17 deletions(-)
This allows property implementation to provide a specialized property
creation method.
Update conditions guarding property types accordingly.
Signed-off-by: Fam Zheng
---
hw/core/qdev.c | 31 +++
include/hw/qdev-core.h | 1 +
qmp.c | 2 +-
Unlike the usual object_property_add_link() invocations in other
devices, ivshmem checks the "is mapped" state of the backend in addition
to qdev_prop_allow_set_link_before_realize. To convert it without
specializing DEFINE_PROP_LINK which always uses the qdev callback, move
the extra check to devi
v3: Include Igor's patch, and fix virtio-crypto too.
Always use qdev_prop_allow_set_link_before_realize and
OBJ_PROP_LINK_UNREF_ON_RELEASE.
Include as many applicable devices as possible and some more clean-ups.
There are still more left but they use uncommon check and flags paramet
The remaining non-const ones are in e1000e which modifies description at
runtime. They can be addressed separatedly.
Signed-off-by: Fam Zheng
---
hw/core/qdev-properties-system.c | 8 +++
hw/core/qdev-properties.c| 44 +-
hw/s390x/css.c
Signed-off-by: Fam Zheng
---
hw/arm/armv7m.c | 10 +++---
include/hw/arm/armv7m.h | 3 ++-
2 files changed, 5 insertions(+), 8 deletions(-)
diff --git a/hw/arm/armv7m.c b/hw/arm/armv7m.c
index c8a11f2..3a13d82 100644
--- a/hw/arm/armv7m.c
+++ b/hw/arm/armv7m.c
@@ -97,12 +97,6 @@ sta
From: Igor Mammedov
link's check callback is supposed to verify/permit setting it,
however currently nothing restricts it from misusing it
and modifying target object from within.
Make sure that readonly semantics are checked by compiler
to prevent callback's misuse.
Signed-off-by: Igor Mammedov
The dynamic value is more informative in the case of link property,
otherwise it is the same.
Signed-off-by: Fam Zheng
---
qmp.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/qmp.c b/qmp.c
index 133fdfc..9f8a167 100644
--- a/qmp.c
+++ b/qmp.c
@@ -496,7 +496,8 @@ static De
Halil Pasic writes:
> On 07/03/2017 03:52 PM, Markus Armbruster wrote:
>> Halil Pasic writes:
>>
>>> On 06/30/2017 04:54 PM, Eric Blake wrote:
On 06/30/2017 09:41 AM, Halil Pasic wrote:
>>> 'This' basically boils down to the question and
>>> 'Why aren't hints reported in QMP contex
On (Mon) 03 Jul 2017 [14:50:07], Anton Nefedov wrote:
> On 06/29/2017 01:02 PM, Marc-André Lureau wrote:
> > Hi
> >
> > Looks good, but please write something in the commit message about what
> > needs to be done for be-change (what this patch does).
> >
> > thanks
> >
>
> Hi,
>
> thank you!
On Mon, Jul 3, 2017 at 12:13 PM, Peter Maydell wrote:
> Add documentation comments describing the public API of the
> ptimer countdown timer.
>
> Signed-off-by: Peter Maydell
Reviewed-by: Philippe Mathieu-Daudé
> ---
> I was trying to write a timer device and discovered that the ptimer
> API w
On Tue, Jul 04, 2017 at 09:01:43AM +0530, Bharata B Rao wrote:
> On Mon, Jul 03, 2017 at 02:21:31PM +0200, Greg Kurz wrote:
> > QEMU shouldn't abort if spapr_add_lmbs()->spapr_drc_attach() fails.
> > Let's propagate the error instead, like it is done everywhere else
> > where spapr_drc_attach() is
在 2017/7/3 19:48, Cornelia Huck 写道:
On Mon, 3 Jul 2017 09:38:36 +0200
QingFeng Hao wrote:
Do not check kvm_eventfds_enabled() when KVM is disabled since it
always returns 0. Since commit
8c56c1a592b5092d91da8d8943c1d6462a6f ("memory: emulate
ioeventfd") it has been possible to use ioev
在 2017/7/3 18:20, Christian Borntraeger 写道:
On 07/03/2017 10:51 AM, QingFeng Hao wrote:
This patch is based on a similar patch from Stefan Hajnoczi -
commit c324fd0a39c (" virtio-pci: use ioeventfd even when KVM is disabled)
Do not check kvm_eventfds_enabled() when KVM is disabled since it
al
On Mon, Jul 03, 2017 at 02:21:31PM +0200, Greg Kurz wrote:
> QEMU shouldn't abort if spapr_add_lmbs()->spapr_drc_attach() fails.
> Let's propagate the error instead, like it is done everywhere else
> where spapr_drc_attach() is called.
>
> Signed-off-by: Greg Kurz
> ---
> hw/ppc/spapr.c | 10 +
On 07/03/2017 11:47 PM, Paolo Bonzini wrote:
On 03/07/2017 16:39, Xiao Guangrong wrote:
On 06/20/2017 05:15 PM, guangrong.x...@gmail.com wrote:
From: Xiao Guangrong
Changelog in v2:
thanks to Paolo's review, this version disables write-protect-all if
PML is supported
Hi Paolo,
Do you
On Mon, Jul 03, 2017 at 04:28:00PM +0200, Paolo Bonzini wrote:
>
>
> On 03/07/2017 12:12, Yang Zhong wrote:
> > Since Paolo's below patch has fixed A20 issue
> > commit bbfa326fc8028e275eddf8c9965c2a1b59405b2e
> > target/i386: enable A20 automatically in system management mod
>
> How is this pat
On Thu, Jun 29, 2017 at 01:49:52AM -0400, Lan Tianyu wrote:
> 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: C
On 07/03/17 17:45 +0200, Paolo Bonzini wrote:
>
>
> On 03/07/2017 17:23, Haozhong Zhang wrote:
> > Otherwise, QEMU on a host with MPX support will try to set guest
> > MSR_IA32_BNDCFGS although guest MPX is not enabled, and result in
> > abort.
> >
> > For example,
> >qemu-system-x86_64 -ena
On Mon, Jul 03, 2017 at 02:07:12PM -0300, Eduardo Habkost wrote:
> On Mon, Jul 03, 2017 at 10:44:07AM +0800, Peter Xu wrote:
> > It's never documented, and now we have one more parameter for it (which
> > means this one can be obsolete in the future). Document it properly.
> >
> > Although now whe
Hi Anthony:
On 2017年06月30日 23:48, Anthony PERARD wrote:
> On Thu, Jun 29, 2017 at 01:49:53AM -0400, Lan Tianyu wrote:
>> 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
>> bin
Hi, Michael
On 07/04/2017 01:41 AM, Michael S. Tsirkin wrote:
On Tue, Jun 27, 2017 at 02:16:46PM +0800, Mao Zhongyi wrote:
This series mainly implements the conversions of pci-bridge devices
i82801b11, io3130_upstream/downstream and so on to realize(). Naturally
part of error messages need to b
On Mon, Jul 03, 2017 at 11:59:03AM -0300, Eduardo Habkost wrote:
> On Mon, Jul 03, 2017 at 10:44:06AM +0800, Peter Xu wrote:
> > Currently drive_init_func() may call migrate_get_current() while the
> > migrate object is still not ready yet at that time. Move the migration
> > object init earlier, a
On 07/04/2017 03:44 AM, Michael S. Tsirkin wrote:
From: Wei Wang
This patch enables the virtio-net tx queue size to be configurable
between 256 (the default queue size) and 1024 by the user when the
vhost-user backend is used.
Currently, the maximum tx queue size for other backends is 512 due
That is why I think we can consider a possibility of forgetting about ACPI
hot plug in pcie-pci bridge and use only SHPC (with some correcting work).
Especially since q35 is used only for 'modern' Windows guests and there're
no big problems with SHPC on Linux guests.
вт, 4 июля 2017 г. в 1:06, Ale
Le 03/07/2017 à 22:31, Richard Henderson a écrit :
> On 07/03/2017 12:50 PM, Laurent Vivier wrote:
>> fmove.x #0x0ABCDEF12345,%fp0
>> fgetman.x %fp0,%fp6
>> fp02.1518178707571747286191852003521627e-4938(raw
>> 0x0abcdef12345)
>> fp61.3
Previously, the alloc command required that input parameters be
sector-aligned and clamped to 32 bits, because the underlying
bdrv_is_allocated used a 32-bit parameter and asserted aligned
inputs. But now that we have fixed block status to report a
64-bit bytes value, and to properly round request
We are gradually converting to byte-based interfaces, as they are
easier to reason about than sector-based. Convert another internal
function (no semantic change), and as with its public counterpart,
rename to bdrv_co_block_status() to make the compiler enforce that
we catch all uses. For now, we
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file
We are gradually converting to byte-based interfaces, as they are
easier to reason about than sector-based. Continue by converting
an internal function (no semantic change), and simplifying its
caller accordingly.
Signed-off-by: Eric Blake
---
v2: no change
---
qemu-img.c | 24 +++-
We are gradually moving away from sector-based interfaces, towards
byte-based. In the common case, allocation is unlikely to ever use
values that are not naturally sector-aligned, but it is possible
that byte-based values will let us be more precise about allocation
at the end of an unaligned file
We are gradually converting to byte-based interfaces, as they are
easier to reason about than sector-based. Change the internal
loop iteration of zeroing a device to track by bytes instead of
sectors (although we are still guaranteed that we iterate by steps
that are sector-aligned).
Signed-off-b
On 07/03/2017 01:31 PM, Eric Blake wrote:
>> The throttle driver I'm working on passes bdrv_co_get_block_status() to
>> bs->file. If there is a problem with an unused default function (it's
>> not static so will compile, but it might not be up to standard), you can
>> just remove it and I will rein
We are gradually moving away from sector-based interfaces, towards
byte-based. Now that the block layer exposes byte-based allocation,
it's time to tackle the drivers. Add a new callback that operates
on as small as byte boundaries. Subsequent patches will then update
individual drivers, then fina
We are gradually converting to byte-based interfaces, as they are
easier to reason about than sector-based. Convert another internal
function (no semantic change), and rename it to is_zero() in the
process.
Signed-off-by: Eric Blake
---
v2: rename function, rebase to upstream changes
---
block
In the process of converting sector-based interfaces to bytes,
I'm finding it easier to represent a byte count as a 64-bit
integer at the block layer (even if we are internally capped
by SIZE_MAX or even INT_MAX for individual transactions, it's
still nicer to not have to worry about truncation/ove
We are gradually converting to byte-based interfaces, as they are
easier to reason about than sector-based. Convert another internal
type (no semantic change), and rename it to match the corresponding
public function rename.
Signed-off-by: Eric Blake
---
v2: rebase to earlier changes
---
block
Any device that has request_alignment greater than 512 should be
unable to report status at a finer granularity; it may also be
simpler for such devices to be guaranteed that the block layer
has rounded things out to the granularity boundary (the way the
block layer already rounds all other I/O out
Not all callers care about which BDS owns the mapping for a given
range of the file. This patch merely simplifies the callers by
consolidating the logic in the common call point, while guaranteeing
a non-NULL file to all the driver callbacks, for no semantic change.
The only caller that does not c
From: Manos Pitsidianakis
bdrv_co_get_block_status_from_file() and
bdrv_co_get_block_status_from_backing() set *file to bs->file and
bs->backing respectively, so that bdrv_co_get_block_status() can recurse
to them. Future block drivers won't have to duplicate code to implement
this.
Reviewed-by:
We are gradually converting to byte-based interfaces, as they are
easier to reason about than sector-based. Convert another internal
type (no semantic change), and rename it to match the corresponding
public function rename.
Signed-off-by: Eric Blake
---
v2: rebase to earlier changes
---
block
We are gradually converting to byte-based interfaces, as they are
easier to reason about than sector-based. Convert another internal
function (no semantic change).
Signed-off-by: Eric Blake
---
v2: new patch
---
block/io.c | 42 +-
1 file changed, 21 ins
Not all callers care about which BDS owns the mapping for a given
range of the file. In particular, bdrv_is_allocated() cares more
about finding the largest run of allocated data from the guest
perspective, whether or not that data is consecutive from the
host perspective. Therefore, doing subseq
There are patches floating around to add NBD_CMD_BLOCK_STATUS,
but NBD wants to report status on byte granularity (even if the
reporting will probably be naturally aligned to sectors or even
much higher levels). I've therefore started the task of
converting our block status code to report at a byt
Tried it on Win7 Enterprise SP1 - SHPC works well, _OSC patches aren't
necessary (since pci-bridge has its own controller, I suppose).
On Linux guests it works when adding device from CLI with -device, but OS
seems to fail detecting the device when I add it with device_add from
monitor.
Also there
Am Mon, 3 Jul 2017 23:34:14 +0200
schrieb Halil Pasic :
> Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for
> flexibility (extending using subsections) and for fun.
>
> To achieve this we need to hack the config_vector, which is
> VirtIODevice (that is common virtio) state, i
On 07/02/2017 01:28 PM, Aurelien Jarno wrote:
This patchset should fix the bug#1701821 reported by Bruno Haible,
which makes the gnulib testsuite to fail for single precision libm
tests or for tests relying on unordered comparisons.
It also fixes an inversion of cause and flag bits in the FPSCR
Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for
flexibility (extending using subsections) and for fun.
To achieve this we need to hack the config_vector, which is VirtIODevice
(that is common virtio) state, in the middle of the VirtioCcwDevice state
representation. This is s
On 07/02/2017 09:32 AM, Aurelien Jarno wrote:
This patchset should fix the bug #1701821 reported by Bruno Haible,
which makes the gnulib testsuite to fail for single precision libm
tests.
Aurelien Jarno (2):
target/sh4: do not check for PR bit for fabs instruction
target/sh4: do not use a
On 07/02/2017 06:18 AM, Aurelien Jarno wrote:
+if (!s390_has_feat(s->insn->fac == S390_FEAT_EXTENDED_TRANSLATION_3
+ ? S390_FEAT_ETF3_ENH : S390_FEAT_ETF2_ENH)) {
+m3 = 0;
+}
This doesn't look correct to me. The well-formedness checking is part of
ETF3_ENH
On 07/01/2017 03:44 PM, Peter Maydell wrote:
On 1 July 2017 at 23:35, Richard Henderson wrote:
Perhaps it ought to be telling me to use %x to force Xn in spite of the
type?
You always get Xn anyway, regardless of the type.
For us, I think the right thing to do is make 'ctr' be a uint64_t,
be
On 06/30/2017 07:36 AM, Pranith Kumar wrote:
This patch enables the indirect jump path using an LDR (literal)
instruction. It will be interesting to test and see which performs
better among the two paths.
CC: Alex Bennée
Reviewed-by: Richard Henderson
Signed-off-by: Pranith Kumar
Applied al
On Mon, 3 Jul 2017, Igor Druzhinin wrote:
> On 01/07/17 01:06, Stefano Stabellini wrote:
> > On Fri, 30 Jun 2017, Igor Druzhinin wrote:
> >> Dummys are simple anonymous mappings that are placed instead
> >> of regular foreign mappings in certain situations when we need
> >> to postpone the actual m
On 01/07/17 01:08, Stefano Stabellini wrote:
> On Fri, 30 Jun 2017, Igor Druzhinin wrote:
>> This new call is trying to update a requested map cache entry
>> according to the changes in the physmap. The call is searching
>> for the entry, unmaps it, tries to translate the address and
>> maps again
On 07/03/2017 12:50 PM, Laurent Vivier wrote:
fmove.x #0x0ABCDEF12345,%fp0
fgetman.x %fp0,%fp6
fp02.1518178707571747286191852003521627e-4938 (raw
0x0abcdef12345)
fp61.34103012886691431049257516861 (raw
0x3f
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
--disable-tcg will almost certainly fail to compile on non-x86 platforms,
so issue a warning.
Signed-off-by: Paolo Bonzini
---
configure | 7 +++
1 file changed, 7 insertions(+)
Reviewed-by: Richard Henderson
r~
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
Add the CONFIG_TCG for frontend and backend's files in the related
Makefiles.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
target/i386/Makefile.objs | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
di
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
@@ -215,10 +215,12 @@ void breakpoint_handler(CPUState *cs)
if (cs->watchpoint_hit) {
if (cs->watchpoint_hit->flags & BP_CPU) {
cs->watchpoint_hit = NULL;
-if (check_hw_breakpoints(env, false)) {
-
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
+static inline void cpu_set_mxcsr(CPUX86State *env, uint32_t mxcsr)
+{
+env->mxcsr = mxcsr;
+if (tcg_enabled()) {
+tcg_update_mxcsr(env);
I'd prefer update_mxcsr_status for this new name.
+}
+}
+
+static inline void cpu_set_fpuc
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
This function calls tlb_set_page_with_attrs, which is not available
when TCG is disabled. Move it to excp_helper.c.
Signed-off-by: Paolo Bonzini
---
target/i386/excp_helper.c | 343 +
target/i386/helper.
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
Move cpu_get_fp80()/cpu_set_fp80() from fpu_helper.c to
machine.c because fpu_helper.c will be disabled if tcg is
disabled in the build.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
target/i386/cpu.h| 2 --
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
@@ -1302,10 +1332,12 @@ void cpu_report_tpr_access(CPUX86State *env, TPRAccess
access)
env->tpr_access_type = access;
cpu_interrupt(cs, CPU_INTERRUPT_TPR);
-} else {
+} else if (tcg_enabled()) {
cpu_restore_st
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
Add the CONFIG_TCG for frontend and backend's files in the related
Makefiles.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
Makefile.target | 4 ++--
accel/Makefile.objs | 2 +-
2 files changed, 3 insertions(+),
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
Adding assertions fixes link errors.
Signed-off-by: Paolo Bonzini
---
exec.c | 3 +++
1 file changed, 3 insertions(+)
Reviewed-by: Richard Henderson
r~
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
--- a/include/exec/helper-proto.h
+++ b/include/exec/helper-proto.h
@@ -28,7 +28,9 @@ dh_ctype(ret) HELPER(name) (dh_ctype(t1), dh_ctype(t2),
dh_ctype(t3), \
#include "helper.h"
#include "trace/generated-helpers.h"
+#ifdef CONFIG_TCG
#includ
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini
---
bsd-user/main.c | 1 -
include/exec/exec-all.h | 4
tcg/tcg.h | 4
3 files changed, 4 insertions(+), 5 deletions(-)
Reviewed-by: Richard Henderson
r~
On Mon, Jul 03, 2017 at 03:50:52PM -0300, Eduardo Habkost wrote:
> On Mon, Jul 03, 2017 at 09:38:52PM +0300, Michael S. Tsirkin wrote:
> > On Thu, Jun 29, 2017 at 03:23:04PM +0200, Marc-André Lureau wrote:
> > > This compat property sole function is to prevent the device from being
> > > instantiat
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini
---
hw/i386/kvmvapic.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
Reviewed-by: Richard Henderson
r~
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini
---
accel/tcg/translate-all.c | 5 +
hmp-commands-info.hx | 4
monitor.c | 2 ++
3 files changed, 11 insertions(+)
Reviewed-by: Richard Henderson
r~
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
If tcg is disabled, the functions in tcg-stub.c file will be called.
This file is target-independent file, do not include any platform
related stub functions into this file.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
We dropped some dead code, update extected table binaries.
Fixes: 4d7e7f2702912 ("hw/acpi: remove dead acpi code")
Signed-off-by: Michael S. Tsirkin
---
tests/acpi-test-data/q35/DSDT| Bin 7824 -> 7782 bytes
tests/acpi-test-data/q35/DSDT.bridge | Bin 7841 -> 7799 bytes
tests/acpi-test-d
On 01/07/17 01:06, Stefano Stabellini wrote:
> On Fri, 30 Jun 2017, Igor Druzhinin wrote:
>> Dummys are simple anonymous mappings that are placed instead
>> of regular foreign mappings in certain situations when we need
>> to postpone the actual mapping but still have to give a
>> memory region to
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
translate-all.c will be disabled if tcg is disabled in the build,
so page_size_init() function and related variables will be moved
to exec.c file.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
accel/tcg/translate-all.c
From: Maxime Coquelin
If the backend sends a request just before closing the socket,
the aio dispatcher might schedule its reading after the vhost
device has been cleaned, leading to a NULL pointer dereference
in slave_read();
vhost_user_cleanup() already closes the socket but it is not
enough,
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
Change the tcg_enabled() and make sure user build still enable tcg
even x86 softmmu disable tcg.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
accel/tcg/cpu-exec-common.c | 2 ++
accel/tcg/tcg-all.c | 1 -
ac
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
Move tcg_handle_interrupt() from translate-common.c to
accel/tcg/tcg-all.c.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
accel/tcg/Makefile.objs | 2 +-
accel/tcg/tcg-all.c | 32 +++
On 07/03/2017 09:34 AM, Paolo Bonzini wrote:
From: Yang Zhong
Need to disable the tcg related code in the vl.c if the
disable-tcg option is added into ./configure command.
Signed-off-by: Yang Zhong
Signed-off-by: Paolo Bonzini
---
vl.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletio
From: Peter Xu
Migration is broken after the vfio integration work:
qemu-kvm: AHCI: Failed to start FIS receive engine: bad FIS receive buffer
address
qemu-kvm: Failed to load ich9_ahci:ahci
qemu-kvm: error while loading state for instance 0x0 of device
':00:1f.2/ich9_ahci'
qemu-kvm: load
Le 03/07/2017 à 21:26, Richard Henderson a écrit :
> On 07/03/2017 09:23 AM, Laurent Vivier wrote:
>> +void HELPER(fgetman)(CPUM68KState *env, FPReg *res, FPReg *val)
>> +{
>> +if (floatx80_is_infinity(val->d)) {
>> +res->d = floatx80_default_nan(NULL);
>> +/* FIXME: set the OPE
From: Mark Cave-Ayland
As indicated by Laszlo it is a QOM bug for the realize() method to actually
map the device. Set up the IO regions within fw_cfg_io_realize() and defer
the mapping with sysbus_add_io() to the caller, as already done in
fw_cfg_init_mem_wide().
This makes the iobase and dma_i
Current code segfaults when no nic peer is specified.
Fix it up - fall back to default queue size.
Fixes: 9b02e1618cf26a ("virtio-net: enable configurable tx queue size")
Cc: Wei Wang
Signed-off-by: Michael S. Tsirkin
---
hw/net/virtio-net.c | 28
1 file changed, 20
From: Ben Warren
The following tests are implemented:
* test that a GUID passed in by command line is propagated to the guest.
Read the GUID from guest memory
* test that the "auto" argument to the GUID generates a valid GUID, as
seen by the guest.
* test that a GUID passed in can be queried
From: Mark Cave-Ayland
The setting of the FW_CFG_VERSION_DMA bit is the same across both the
TYPE_FW_CFG_MEM and TYPE_FW_CFG_IO devices, so unify the logic in
fw_cfg_init1().
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laszlo Ersek
Reviewed-by: Eduardo Habkost
Reviewed-by: Philippe Mathieu-D
Am 27.06.2017 um 17:04 schrieb Eric Blake:
> On 06/27/2017 09:49 AM, Peter Lieven wrote:
>
>> Before I continue, can you please give feedback on the following spec
>> change:
>>
>> diff --git a/docs/interop/qcow2.txt b/docs/interop/qcow2.txt
>> index 80cdfd0..f1428e9 100644
>> --- a/docs/interop/qc
From: Maxime Coquelin
This patch fixes a crash that happens when vhost-user iommu
support is enabled and vhost-user socket is closed.
When it happens, if an IOTLB invalidation notification is sent
by the IOMMU, vhost_ops's NULL pointer is dereferenced.
Signed-off-by: Maxime Coquelin
Reviewed-b
From: Mao Zhongyi
After the patch 'Make errp the last parameter of pci_add_capability()',
pci_add_capability() and pci_add_capability2() now do exactly the same.
So drop the wrapper pci_add_capability() of pci_add_capability2(), then
replace the pci_add_capability2() with pci_add_capability() eve
1 - 100 of 351 matches
Mail list logo
