To have expression semantic analysis in one place rather than two.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi.py | 142 ++--
1 file changed, 66 insertions(+), 76 deletions(-)
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/scripts/qapi.py b/scripts/qapi.py
index f96a777..683669e 100644
--- a/scripts/qapi.py
+++ b/scripts/qapi.py
@@
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi.py | 2 +-
tests/qapi-schema/flat-union-array-branch.err | 11 +--
2 files changed, 2 insertions(+), 11 deletions(-)
diff --git a/scripts/qapi.py
On 18 June 2015 at 10:28, Markus Armbruster arm...@redhat.com wrote:
However, I can't see how I could define a new C style there without
pushing the local variables feature well beyond its intended use, and
triggering the confirmation prompts.
We wouldn't want to define a new C style, but in
On Thu, Jun 18, 2015 at 04:49:11PM +0800, Wen Congyang wrote:
Signed-off-by: Wen Congyang we...@cn.fujitsu.com
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
block.c | 9 +
1 file changed, 9 insertions(+)
diff --git
Hi,
did you have the time to take a look at this? Is there anything
pending from my side?
Thanks,
dimara
* Dimitris Aragiorgis dim...@arrikto.com [2015-05-29 08:15:34 +0300]:
Hi,
* Dimitris Aragiorgis dim...@arrikto.com [2015-05-20 12:57:34 +0300]:
Hi all,
These four patches make
On Thu 18 Jun 2015 12:45:35 PM CEST, Kevin Wolf wrote:
I believe our conclusion from an earlier version of the series was
that we need QAPI introspection so that libvirt can detect the
presence of the feature.
The initial version of this series had an extra 'top' parameter to
decide what
sPAPRTCETable has a vfio_accel flag which is passed to
kvmppc_create_spapr_tce() and controls whether to create a guest view
table in KVM as this depends on the host kernel ability to accelerate
H_PUT_TCE for VFIO devices. We would set this flag at the moment
when sPAPRTCETable is created in
On 06/18/2015 04:53 AM, Dr. David Alan Gilbert wrote:
* Juan Quintela (quint...@redhat.com) wrote:
We reuse the migration events from the source side, sending them on the
appropiate place.
s/appropiate/appropriate/
Signed-off-by: Juan Quintela quint...@redhat.com
Reviewed-by: Eric Blake
On 16 June 2015 at 15:20, Leon Alrae leon.al...@imgtec.com wrote:
On 16/06/2015 15:03, Peter Maydell wrote:
On 16 June 2015 at 13:32, Liviu Ionescu i...@livius.net wrote:
would it be possible to have all the semihosting patches ready for 2.4?
Yes, I agree that would be good. Is it just this 2
Screwed up in commit e53188a.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi.py | 26 --
1 file changed, 12 insertions(+), 14 deletions(-)
diff --git a/scripts/qapi.py b/scripts/qapi.py
index a24a7e2..6faa897
On Wed, Jun 17, 2015 at 04:00:06PM +0200, Paolo Bonzini wrote:
On 12/06/2015 16:08, Paul Donohue wrote:
+static inline int64_t get_max_clock_jump(void)
+{
+// This should be small enough to prevent excessive interrupts from
being
+// generated by the RTC on clock jumps, but
The number of slots per page being 511 (i.e. not a power of two) means
that the (32-bit) read and write indexes going beyond 2^32 will likely
disturb operation. The hypervisor side gets I/O req server creation
extended so we can indicate that we're using suitable atomic accesses
where needed (not
QEMU targets ISAs contain instruction that can break the execution
flow with exceptions. When exception breaks the execution of the translation
block it may corrupt PC and icount values.
This set of patches fixes exception handling for MIPS and i386 targets.
Incorrect execution for i386 is
On Wed, May 27, 2015 at 05:16:47PM +0100, Stefan Hajnoczi wrote:
This series cleans up net_client_init1() and reports an error when the -net
type isn't compiled into the QEMU binary.
It is based on my net branch with Markus' error cleanups:
https://github.com/stefanha/qemu/tree/net
Stefan
On Fri, Jun 12, 2015 at 03:05:10PM +0100, Frediano Ziglio wrote:
This patch allow to limit number of heads using qxl driver. By default
qxl driver is not limited on any kind on head use so can decide to use
as much heads.
libvirt has this as a video card parameter (actually set to 1 but not
* Juan Quintela (quint...@redhat.com) wrote:
This section would be sent:
a- for all new machine types
b- for old achine types if section state is different form {running,paused}
that were the only giving us troubles.
So, in new qemus: it is alwasy there. In old qemus: they are only
On 06/18/2015 06:50 PM, Stefan Hajnoczi wrote:
On Thu, Jun 18, 2015 at 04:49:09PM +0800, Wen Congyang wrote:
Usage:
-drive file=xxx,id=Y, \
-drive
file=,id=X,backing_reference.drive_id=Y,backing_reference.hidden-disk.*
It will create such backing chain:
{virtio-blk dev
This makes use of the new memory registering feature. The idea is
to provide the userspace ability to notify the host kernel about pages
which are going to be used for DMA. Having this information, the host
kernel can pin them all once per user process, do locked pages
accounting (once) and not
This adds support for Dynamic DMA Windows (DDW) option defined by
the SPAPR specification which allows to have additional DMA window(s)
This implements DDW for emulated and VFIO devices. As all TCE root regions
are mapped at 0 and 64bit long (and actual tables are child regions),
this replaces
This reworks finish_realize() which used to finalize DMA setup with
an assumption that it will not change later.
New callbacks supports various window parameters such as page and
windows sizes. The new callback return error code rather than Error**.
This is a mechanical change so no change in
Currently TCE tables are created once at start and their size never
changes. We are going to change that by introducing a Dynamic DMA windows
support where DMA configuration may change during the guest execution.
This changes spapr_tce_new_table() to create an empty stub object. Only
LIOBN is
On a system reset, DMA configuration has to reset too. At the moment
it clears the table content. This is enough for the single table case
but with DDW, we will also have to disable all DMA windows except
the default one. Furthermore according to sPAPR, if the guest removed
the default window and
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi-types.py | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/scripts/qapi-types.py b/scripts/qapi-types.py
index 12fb2ef..d28a6b0 100644
---
Insert comments to separate sections dealing with parsing, semantic
analysis, code generation, and so forth.
Move helpers to their proper section.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi.py | 128
Useless, because it can only occur in commands, and we're not dealing
with commands here.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi-types.py | 1 -
1 file changed, 1 deletion(-)
diff --git a/scripts/qapi-types.py
On 06/17/15 23:49, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 09:35:20PM +0200, Laszlo Ersek wrote:
On 06/17/15 21:21, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 02:45:05PM +0200, Laszlo Ersek wrote:
SeaBIOS expects OpenFirmware device paths in the bootorder fw_cfg file
to
This moves SPAPR bits to a separate file to avoid pollution of x86 code.
This enables spapr-vfio on CONFIG_SOFTMMU (not CONFIG_PSERIES) as
the config options are only visible in makefiles and not in the source code
so there is no an obvious way of implementing stubs if hw/vfio/spapr.c is
not
(cut-n-paste from kernel patchset)
Each Partitionable Endpoint (IOMMU group) has an address range on a PCI bus
where devices are allowed to do DMA. These ranges are called DMA windows.
By default, there is a single DMA window, 1 or 2GB big, mapped at zero
on a PCI bus.
PAPR defines a DDW RTAS
Since the changes are not in upstream yet, no tag or branch is specified here.
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
---
linux-headers/linux/vfio.h | 88 --
1 file changed, 85 insertions(+), 3 deletions(-)
diff --git
On 2015/6/18 19:03, Andrew Jones wrote:
On Thu, Jun 18, 2015 at 06:28:36PM +0800, Shannon Zhao wrote:
On 2015/6/17 17:42, Andrew Jones wrote:
On Wed, Jun 17, 2015 at 09:06:47AM +0800, Shannon Zhao wrote:
On 2015/6/16 22:19, Michael S. Tsirkin wrote:
On Tue, Jun 16, 2015 at 09:33:19AM
Falling back to qemu-trivial, hope that's okay.
Markus Armbruster arm...@redhat.com writes:
Ping?
Markus Armbruster arm...@redhat.com writes:
Since we now require GLib 2.22+ (commit f40685c), we don't have to
work around lack of g_strcmp0() anymore.
This reverts commit
The following changes since commit 1b58f5a7f6fbe811cc486cd5786483bad5d51bbf:
Merge remote-tracking branch 'remotes/mcayland/tags/qemu-openbios-signed'
into staging (2015-06-18 11:36:42 +0100)
are available in the git repository at:
git://repo.or.cz/qemu/armbru.git
old name new name
input_filefname
input_relname fname
input_fname abs_fname
include_path incl_abs_fname
parent_info incl_info
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi.py | 45
This patch introduces several helpers to pass return address
which points to the TB. Correct return address allows correct
restoring of the guest PC and icount. These functions should be used when
helpers embedded into TB invoke memory operations.
Signed-off-by: Pavel Dovgalyuk
This enables multiple IOMMU groups in one VFIO container which means
that multiple devices from different groups can share the same IOMMU
table (or tables if DDW).
This removes a group id from vfio_container_ioctl(). The kernel support
is required for this; if the host kernel does not have the
sPAPRTCETable is handling 2 TCE tables already:
1) guest view of the TCE table - emulated devices use only this table;
2) hardware IOMMU table - VFIO PCI devices use it for actual work but
it does not replace 1) and it is not visible to the guest.
The initialization of this table is driven by
We are going to have multiple DMA windows at different offsets on
a PCI bus. For the sake of migration, we will have as many TCE table
objects pre-created as many windows supported.
So we need a way to map windows dynamically onto a PCI bus
when migration of a table is completed but at this stage
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi-types.py | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/scripts/qapi-types.py b/scripts/qapi-types.py
index 86e5ddc..c408542 100644
---
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
MAINTAINERS | 4
1 file changed, 4 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 32c7ca4..0f801e0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -952,7 +952,10 @@ M: Markus
The new test demonstrates another generator crash.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
tests/Makefile | 3 ++-
tests/qapi-schema/flat-union-array-branch.err | 10 ++
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi-types.py | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/scripts/qapi-types.py b/scripts/qapi-types.py
index c408542..12fb2ef 100644
---
On Thu, Jun 18, 2015 at 04:49:12PM +0800, Wen Congyang wrote:
+void bdrv_connect(BlockDriverState *bs, Error **errp)
+{
+BlockDriver *drv = bs-drv;
+
+if (drv drv-bdrv_connect) {
+drv-bdrv_connect(bs, errp);
+} else if (bs-file) {
+bdrv_connect(bs-file, errp);
[adding libvirt, to make sure I capture a design idea]
On 06/18/2015 06:36 AM, Eric Blake wrote:
On 06/18/2015 06:07 AM, Alberto Garcia wrote:
On Thu 18 Jun 2015 01:47:20 PM CEST, Kevin Wolf wrote:
I believe our conclusion from an earlier version of the series was
that we need QAPI
On Thu, Jun 18, 2015 at 03:22:59PM +0200, Laszlo Ersek wrote:
On 06/17/15 23:50, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 09:44:07PM +0200, Laszlo Ersek wrote:
On 06/17/15 21:32, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 03:28:44PM -0400, Kevin O'Connor wrote:
On Wed, Jun
All the PCI enumeration and device node creation was off-loaded to
SLOF. With PCI hotplug support, code needed to be added to add device
node. This creates multiple copy of the code one in SLOF and other in
hotplug code. To unify this, the patch adds the pci device node
creation in Qemu. For
The patch series creates PCI device tree(DT) nodes in QEMU. The new
hotplug code needs the device node creation in QEMU. While during
boot, nodes were created in SLOF. It makes more sense to consolidate
the code to one place for better maintainability.
New slof.bin is already there in spapr-next
Each hardware instance has a platform unique location code. The OF
device tree that describes a part of a hardware entity must include
the “ibm,loc-code” property with a value that represents the location
code for that hardware entity.
Populate ibm,loc-code.
1) PCI passthru devices need to
* phb_index is not being used and if required can be obtained from sphb
* use helper to get drc_index in spapr_populate_pci_child_dt()
* Check if drc_index is zero
Suggested-by: Alexey Kardashevskiy a...@ozlabs.ru
Signed-off-by: Nikunj A Dadhania nik...@linux.vnet.ibm.com
---
hw/ppc/spapr_pci.c
27.05.2015 20:53, Markus Armbruster wrote:
Since we now require GLib 2.22+ (commit f40685c), we don't have to
work around lack of g_strcmp0() anymore.
qga/main.c | 2 +-
-if (domain strcmp(domain, syslog) == 0) {
+if (g_strcmp0(domain, syslog) == 0) {
Why bother? To me the
On 05/05/2015 10:49 PM, David Gibson wrote:
On Sat, Apr 25, 2015 at 10:24:43PM +1000, Alexey Kardashevskiy wrote:
This adds support for Dynamic DMA Windows (DDW) option defined by
the SPAPR specification which allows to have additional DMA window(s)
This implements DDW for emulated and VFIO
At the moment presence of vfio-pci devices on a bus affect the way
the guest view table is allocated. If there is no vfio-pci on a PHB
and the host kernel supports KVM acceleration of H_PUT_TCE, a table
is allocated in KVM. However, if there is vfio-pci and we do yet not
KVM acceleration for
sPAPR IOMMU is managing two copies of an TCE table:
1) a guest view of the table - this is what emulated devices use and
this is where H_GET_TCE reads from;
2) a hardware TCE table - only present if there is at least one vfio-pci
device on a PHB; it is updated via a memory listener on a PHB
Am 18.06.2015 um 13:41 hat Alberto Garcia geschrieben:
On Thu 18 Jun 2015 12:45:35 PM CEST, Kevin Wolf wrote:
I believe our conclusion from an earlier version of the series was
that we need QAPI introspection so that libvirt can detect the
presence of the feature.
The initial version
Commit 87a560c4 added it in the wrong place. Commit 59a2c4ce added it
in the right place, but didn't remove it from the wrong place. Do
that now.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
docs/qapi-code-gen.txt | 2 --
1 file changed, 2
We print the name as it appears in the include expression. Tools
processing error messages want it relative to the working directory.
Make it so.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake ebl...@redhat.com
---
scripts/qapi.py | 7 +++
We maintain a stack of filenames in include_hist for convenient cycle
detection.
As error_path() demonstrates, the same information is readily
available in the expr_info, so just use that, and drop include_hist.
Signed-off-by: Markus Armbruster arm...@redhat.com
Reviewed-by: Eric Blake
On 06/17/15 23:50, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 09:44:07PM +0200, Laszlo Ersek wrote:
On 06/17/15 21:32, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 03:28:44PM -0400, Kevin O'Connor wrote:
On Wed, Jun 17, 2015 at 09:15:24PM +0200, Laszlo Ersek wrote:
On 06/17/15
This patch passes TB return address into softmmu functions that are
invoked from target helpers. This allows correct PC and icount recovering
while handling MMU faults.
Signed-off-by: Pavel Dovgalyuk pavel.dovga...@ispras.ru
---
target-i386/cc_helper.c |2
target-i386/cpu.h |5
This allows dynamic allocation for migrating arrays.
Already existing VMSTATE_VARRAY_UINT32 requires an array to be
pre-allocated, however there are cases when the size is not known in
advance and there is no real need to enforce it.
This defines another variant of VMSTATE_VARRAY_UINT32 with
On Thu 18 Jun 2015 01:47:20 PM CEST, Kevin Wolf wrote:
I believe our conclusion from an earlier version of the series was
that we need QAPI introspection so that libvirt can detect the
presence of the feature.
The initial version of this series had an extra 'top' parameter to
decide
On 06/18/2015 06:07 AM, Alberto Garcia wrote:
On Thu 18 Jun 2015 01:47:20 PM CEST, Kevin Wolf wrote:
I believe our conclusion from an earlier version of the series was
that we need QAPI introspection so that libvirt can detect the
presence of the feature.
Detecting the presence of a feature
On Thu, Jun 18, 2015 at 04:49:20PM +0800, Wen Congyang wrote:
CCing Alberto Garcia for the quorum block driver.
If the child is not ready, read/write/getlength/flush will
return -errno. It is not critical error, and can be ignored:
1. read/write:
Just not report the error event.
2.
This patch improves exception handling in MIPS.
Instructions generate several types of exceptions.
When exception is generated, it breaks the execution of the current translation
block. Implementation of the exceptions handling does not correctly
restore icount for the instruction which caused the
On 17/06/2015 16:02, Michael S. Tsirkin wrote:
Are you okay with the flag's name, PCI_BRIDGE_DEV_F_HOTPLUG? If not,
what would be your preference?
PCI_BRIDGE_DEV_F_SHPC_REQ
Do not abbrev unless necessary. :) What about PCI_BRIDGE_DEV_F_HAS_SHPC
or even just PCI_BRIDGE_DEV_F_SHPC?
Paolo
On Thu, Jun 18, 2015 at 12:11:47PM +0100, Peter Maydell wrote:
On 18 June 2015 at 10:39, Michael S. Tsirkin m...@redhat.com wrote:
On Thu, Jun 18, 2015 at 11:33:55AM +0200, Gerd Hoffmann wrote:
+static Property virtio_input_host_pci_properties[] = {
+
On Thu, Jun 18, 2015 at 11:36:26AM +0100, Peter Maydell wrote:
On 17 June 2015 at 20:38, Michael S. Tsirkin m...@redhat.com wrote:
OK things seem to be calming down nicely.
Here's to an uneventful 2.4!
The following changes since commit 4ebc736e9938a7e88ecc785734b17145bf802a56:
On 18 June 2015 at 10:39, Michael S. Tsirkin m...@redhat.com wrote:
On Thu, Jun 18, 2015 at 11:33:55AM +0200, Gerd Hoffmann wrote:
+static Property virtio_input_host_pci_properties[] = {
+DEFINE_VIRTIO_INPUT_PROPERTIES(VirtIOInputPCI, vdev.input),
+DEFINE_PROP_STRING(evdev,
* Eric Blake (ebl...@redhat.com) wrote:
On 06/18/2015 04:53 AM, Dr. David Alan Gilbert wrote:
* Juan Quintela (quint...@redhat.com) wrote:
We reuse the migration events from the source side, sending them on the
appropiate place.
s/appropiate/appropriate/
Signed-off-by: Juan
On Thu, Jun 18, 2015 at 04:49:18PM +0800, Wen Congyang wrote:
CCing Alberto Garcia for the quorum block driver.
Signed-off-by: Wen Congyang we...@cn.fujitsu.com
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
block/quorum.c |
On 18/06/2015 14:58, Paul Donohue wrote:
On Wed, Jun 17, 2015 at 04:00:06PM +0200, Paolo Bonzini wrote:
On 12/06/2015 16:08, Paul Donohue wrote:
+static inline int64_t get_max_clock_jump(void)
+{
+// This should be small enough to prevent excessive interrupts from
being
+//
On Tue, Jun 16, 2015 at 10:05 AM, Jason Wang jasow...@redhat.com wrote:
On 06/16/2015 03:24 PM, Thibaut Collet wrote:
If my understanding is correct, on a resume operation, we have the
following callback trace:
1. virtio_pci_restore function that calls all restore call back of
virtio
Peter Maydell peter.mayd...@linaro.org writes:
On 18 June 2015 at 10:28, Markus Armbruster arm...@redhat.com wrote:
However, I can't see how I could define a new C style there without
pushing the local variables feature well beyond its intended use, and
triggering the confirmation prompts.
On Thu, Jun 18, 2015 at 08:42:40 +0100, Peter Maydell wrote:
What data structures are you referring to? Are they ppc-specific?
None of the code generation data structures are locked at all --
if two threads try to generate code at the same time they'll
tend to clobber each other.
AFAICT
At 2015/6/18 20:55, Stefan Hajnoczi Wrote:
On Thu, Jun 18, 2015 at 04:49:12PM +0800, Wen Congyang wrote:
+void bdrv_connect(BlockDriverState *bs, Error **errp)
+{
+BlockDriver *drv = bs-drv;
+
+if (drv drv-bdrv_connect) {
+drv-bdrv_connect(bs, errp);
+} else if (bs-file) {
On 18 June 2015 at 15:30, Liviu Ionescu i...@livius.net wrote:
On 18 Jun 2015, at 13:21, Peter Maydell peter.mayd...@linaro.org wrote:
... do_unassigned_access is the QOM CPUClass hook for this,
this hook seems associated with the Trying to execute code outside
RAM or ROM at 0x error, which
Hello,
commit ea96bc629cbd52be98b2967a4b4f72e91dfc3ee4
i386: drop FDC in pc-q35-2.4+ if neither it nor floppy drives are wanted
dropped the controller for older machine types too, despite the commit
message.
It seems the logic in the merged commit does not match the original patch:
Eric Blake ebl...@redhat.com writes:
On 06/17/2015 01:24 AM, Michael S. Tsirkin wrote:
makes it possible to copy error_abort pointers,
not just pass them on directly.
@@ -168,7 +175,7 @@ void error_free(Error *err)
void error_propagate(Error **dst_errp, Error *local_err)
{
-if
It is common for MMIO registers to overlap, for example a 4 byte register
at 0xcf8 (totally random choice... :)) and a 1 byte register at 0xcf9.
If these registers are implemented via separate MemoryRegions, it is
wrong to clamp the accesses as the value written would be truncated.
Hence for
The first patch fixes the remaining problems with Peter Crosthwaite's
access clamping patch (which broke kvmvapic and hence Windows XP/2003).
The second patch actually puts the clamping to good use: by fixing
address_space_translate_internal, the MIPS rc4030 emulation does not need
anymore the
On 18/06/2015 17:40, Radim Krčmář wrote:
This unfortunately has to be done only for new machine types. Old types
will remain buggy forever.
Ah, ok, which machine type should I target, 2.4?
Yes.
And is patch 2 is only supposed to work with new machine types?
I'm a bit undecided there,
On 18/06/2015 10:58, zhanghailiang wrote:
+void qemu_system_shutdown_request(void)
+{
+trace_qemu_system_shutdown_request();
+/*
+* if in colo mode, we need do some significant work before respond to the
+* shutdown request.
+*/
+if (loadvm_in_colo_state()) {
+
At 2015/6/18 20:47, Stefan Hajnoczi Wrote:
On Thu, Jun 18, 2015 at 04:49:11PM +0800, Wen Congyang wrote:
Signed-off-by: Wen Congyang we...@cn.fujitsu.com
Signed-off-by: zhanghailiang zhang.zhanghaili...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
block.c | 9 +
1
Because the clamping was done against the MemoryRegion,
address_space_rw was effectively broken if a write spanned
multiple sections that are not linear in underlying memory
(with the memory not being under an IOMMU).
This is visible with the MIPS rc4030 IOMMU, which is implemented
as a series of
On Wed, Jun 17, 2015 at 04:01:33PM +0200, Paolo Bonzini wrote:
On 12/06/2015 16:10, Paul Donohue wrote:
To correct that problem, this commit resets the periodic timer after
loading from a snapshot or migration if the clock has either jumped
backward or has jumped forward by more than the
Am 16.06.2015 um 23:23 schrieb Aurelien Jarno:
On 2015-06-16 23:08, Alexander Graf wrote:
We now finally have TCG support for the basic set of instructions necessary
to run the s390-ccw machine. That means in any aspect possible that machine
type is now superior to the legacy s390-virtio
On 18 June 2015 at 13:25, Markus Armbruster arm...@redhat.com wrote:
The following changes since commit 1b58f5a7f6fbe811cc486cd5786483bad5d51bbf:
Merge remote-tracking branch 'remotes/mcayland/tags/qemu-openbios-signed'
into staging (2015-06-18 11:36:42 +0100)
are available in the git
On Thu, Jun 18, 2015 at 03:47:41PM +0200, Paolo Bonzini wrote:
On 17/06/2015 16:02, Michael S. Tsirkin wrote:
Are you okay with the flag's name, PCI_BRIDGE_DEV_F_HOTPLUG? If not,
what would be your preference?
PCI_BRIDGE_DEV_F_SHPC_REQ
Do not abbrev unless necessary. :) What
On 18 June 2015 at 15:23, Emilio G. Cota c...@braap.org wrote:
On Thu, Jun 18, 2015 at 08:42:40 +0100, Peter Maydell wrote:
What data structures are you referring to? Are they ppc-specific?
None of the code generation data structures are locked at all --
if two threads try to generate code
On 18/06/2015 17:24, Radim Krčmář wrote:
W10 insider has a bug where it ignores CPUID level and interprets
CPUID.(EAX=07H, ECX=0H) incorrectly, because CPUID in fact returned
CPUID.(EAX=04H, ECX=0H); this resulted in execution of unsupported
instructions.
While it's a Windows bug, there
2015-06-18 17:29+0200, Paolo Bonzini:
On 18/06/2015 17:24, Radim Krčmář wrote:
W10 insider has a bug where it ignores CPUID level and interprets
CPUID.(EAX=07H, ECX=0H) incorrectly, because CPUID in fact returned
CPUID.(EAX=04H, ECX=0H); this resulted in execution of unsupported
W10 insider has a bug where it ignores CPUID level and interprets
CPUID.(EAX=07H, ECX=0H) incorrectly, because CPUID in fact returned
CPUID.(EAX=04H, ECX=0H); this resulted in execution of unsupported
instructions.
While it's a Windows bug, there is no reason to emulate incorrect level;
and
We already bump to level 7 if features there are requested, so do the
same for 0xD.
Signed-off-by: Radim Krčmář rkrc...@redhat.com
---
If we want this behavior, we should not do it by writing a case for
every level.
target-i386/cpu.c | 4
1 file changed, 4 insertions(+)
diff --git
The first patch uses CPUID levels from the internet, which has a nice
side effect of making W10 insider preview bootable.
The second patch is a traditional buggy bug fix.
Radim Krčmář (2):
target-i386: emulate CPUID level of real hardware
target-i386: automatically raise cpuid level to 0xd
On 18 Jun 2015, at 13:21, Peter Maydell peter.mayd...@linaro.org wrote:
... do_unassigned_access is the QOM CPUClass hook for this,
this hook seems associated with the Trying to execute code outside RAM or ROM
at 0x error, which is a bit different, I want to catch read/write accesses,
not
On Do, 2015-06-18 at 06:45 -0400, Frediano Ziglio wrote:
On Do, 2015-06-18 at 05:58 -0400, Frediano Ziglio wrote:
For the same reason there is the v = l test.
The v = l test state that the value can be out of range so it not always
a
constant in the range.
Adding the v 0 check
On 06/18/15 15:40, Michael S. Tsirkin wrote:
On Thu, Jun 18, 2015 at 03:22:59PM +0200, Laszlo Ersek wrote:
On 06/17/15 23:50, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 09:44:07PM +0200, Laszlo Ersek wrote:
On 06/17/15 21:32, Michael S. Tsirkin wrote:
On Wed, Jun 17, 2015 at 03:28:44PM
Hi,
+static Property virtio_input_host_pci_properties[] = {
+DEFINE_VIRTIO_INPUT_PROPERTIES(VirtIOInputPCI, vdev.input),
+DEFINE_PROP_STRING(evdev, VirtIOInputHostPCI, vdev.evdev),
+DEFINE_PROP_UINT32(vectors, VirtIOPCIProxy, nvectors, 2),
+DEFINE_PROP_END_OF_LIST(),
This function will be used to avoid recursive locking of the iothread lock
whenever address_space_rw/ld*/st* are called with the BQL held, which is
almost always the case.
Tracking whether the iothread is owned is very cheap (just use a TLS
variable) but requires some care because now the lock
From: KONRAD Frederic fred.kon...@greensocs.com
This mechanism replaces the existing load/store exclusive mechanism which seems
to be broken for multithread.
It follows the intention of the existing mechanism and stores the target address
and data values during a load operation and checks that
1 - 100 of 388 matches
Mail list logo