On 04/11/2014 13:47, Shannon Zhao wrote:
> Add host/guest notifiers support for virtio-mmio, so that qemu can
> enable vhost-net for kvm-arm.
>
> Refer to the patches from Ying-Shiuan Pan
> https://lists.gnu.org/archive/html/qemu-devel/2014-02/msg00715.html
>
> As vhost-net can improve the net
On Thu, 22 Jan 2015 14:16:22 +1100
David Gibson wrote:
> On Wed, Jan 21, 2015 at 08:56:46AM +0100, Cornelia Huck wrote:
> > David: I saw you tested Stefan's endianness patch.
> >
> > Could you also give my alternative implementation a try? Thanks!
>
> Yes, sorry, I got a bit confused about whic
On Mi, 2015-01-21 at 23:48 +0100, dval...@suse.de wrote:
> From: Dinar Valeev
>
> When ever USB keyboard is used, e.g. '-usbdevice keyboard' pressing
> caps lock key send 0x32 hid code, which is treated as backslash.
> Instead it should be 0x39 code. This affects sending uppercase keys,
> as they
On Wed, Jan 21, 2015 at 08:56:46AM +0100, Cornelia Huck wrote:
> David: I saw you tested Stefan's endianness patch.
>
> Could you also give my alternative implementation a try? Thanks!
Yes, sorry, I got a bit confused about which series was which there.
I gave this a whirl on our P8 and with it
-Original Message-
From: Jeff Cody [mailto:jc...@redhat.com]
Sent: Wednesday, January 21, 2015 2:41 AM
To: Lokesha, Amulya
Cc: Bug 1399191; Geoffroy, Daniel
Subject: Re: [Qemu-devel] [Bug 1399191] [NEW] Large VHDX image size
On Mon, Jan 19, 2015 at 03:19:05PM +, Lokesha, Amulya wrote:
At Tue, 20 Jan 2015 16:14:28 +0900,
Teruaki Ishizaki wrote:
>
> Previously, qemu block driver of sheepdog used hard-coded VDI object size.
> This patch enables users to handle "block_size_shift" value for
> calculating VDI object size.
>
> When you start qemu, you don't need to specify additional
From: Dinar Valeev
When ever USB keyboard is used, e.g. '-usbdevice keyboard' pressing
caps lock key send 0x32 hid code, which is treated as backslash.
Instead it should be 0x39 code. This affects sending uppercase keys,
as they typed whith caps lock active.
While on x86 this can be workarounded
> -Original Message-
> From: Chen, Tiejun
> Sent: Tuesday, January 20, 2015 7:17 PM
> To: Michael S. Tsirkin; Kay, Allen M
> Cc: pbonz...@redhat.com; aligu...@amazon.com; r...@twiddle.net; Zhang,
> Yang Z; qemu-devel@nongnu.org
> Subject: Re: [v6][PATCH 08/10] xen, gfx passthrough: suppor
Signed-off-by: Chen Fan
---
hw/vfio/common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/vfio/common.c b/hw/vfio/common.c
index cf483ff..e71385e 100644
--- a/hw/vfio/common.c
+++ b/hw/vfio/common.c
@@ -32,7 +32,7 @@
#include "trace.h"
struct vfio_group_head vfio_gro
For memory block command, we only support for linux with sysfs.
Signed-off-by: zhanghailiang
Reviewed-by: Michael Roth
---
qga/commands-posix.c | 4 +++-
qga/commands-win32.c | 2 ++
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/qga/commands-posix.c b/qga/commands-posix.c
index
The size of a memory block is architecture dependent, it represents the logical
unit upon which memory online/offline operations are to be performed.
Signed-off-by: zhanghailiang
---
qga/commands-posix.c | 29 +++--
1 file changed, 27 insertions(+), 2 deletions(-)
diff -
Introduce three new guest commands:
guest-get-memory-blocks, guest-set-memory-blocks, guest-get-memory-block-size.
With these three commands, we can support online/offline guest's memory block
(logical memory hotplug/unplug) as required from host.
Signed-off-by: zhanghailiang
---
qga/commands-p
We can change guest's online/offline state of memory blocks, by using
command 'guest-set-memory-blocks'.
Signed-off-by: zhanghailiang
---
qga/commands-posix.c | 30 +-
1 file changed, 29 insertions(+), 1 deletion(-)
diff --git a/qga/commands-posix.c b/qga/commands-po
This patch series add three guest commands about memory block:
guest-get-memory-blocks, guest-set-memory-blocks, guest-get-memory-block-size.
With these three commands, we can get information about guest's memory block
online/offline status and memory block size (unit of memory online/offline
ope
We can get guest's memory block information by using command
"guest-get-memory-blocks", the returned value contains a list of memory block
info, such as phys-index, online state, can-offline info.
Signed-off-by: zhanghailiang
---
qga/commands-posix.c | 231 +++
This patch adds the Cortex-A9 ARM CPU to the A9MPCore.
The CPU is only created if the num-cpu property is set.
This patch relies on Stefan Hajnoczi's v3 'virtio-blk:
use alias properties in transport devices' patch. This is
used to pass the CPU properties through to the mpcore.
This patch allows
This patch removes the initialisation of the ARM Cortex-A9
in Zynq and instead allows the a9mpcore device to init the
CPU. This also updates components that rely on the CPU
and GIC, as they are now initialised in a slightly different
way
Signed-off-by: Alistair Francis
---
Changes since RFC:
- R
On Thu, Dec 11, 2014 at 02:25:04PM +0100, Cornelia Huck wrote:
> The only user of this function was virtio-ccw, and it should use
> virtio_set_features() like everybody else: We need to make sure
> that bad features are masked out properly, which this function did
> not do.
>
> Reviewed-by: Thomas
On Thu, Dec 11, 2014 at 02:25:10PM +0100, Cornelia Huck wrote:
> Handle endianness conversion for virtio-1 virtqueues correctly.
>
> Note that dataplane now needs to be built per-target.
>
> Signed-off-by: Cornelia Huck
Reviewed-by: David Gibson
--
David Gibson| I'll have
On Thu, Dec 11, 2014 at 02:25:07PM +0100, Cornelia Huck wrote:
> With virtio-1, we support more than 32 feature bits. Let's extend both
> host and guest features to 64, which should suffice for a while.
>
> vhost and migration have been ignored for now.
[snip]
> diff --git a/include/hw/virtio/vi
On Thu, Dec 11, 2014 at 02:25:14PM +0100, Cornelia Huck wrote:
> For virtio-1 devices, the driver must not attempt to set feature bits
> after it set FEATURES_OK in the device status. Simply reject it in
> that case.
>
> Signed-off-by: Cornelia Huck
Reviewed-by: David Gibson
--
David Gibson
On Thu, Dec 11, 2014 at 02:25:05PM +0100, Cornelia Huck wrote:
> Add virtio_{add,clear}_feature helper functions for manipulating a
> feature bits variable. This has some benefits over open coding:
> - add check that the bit is in a sane range
> - make it obvious at a glance what is going on
> - ha
On Thu, Dec 11, 2014 at 02:25:09PM +0100, Cornelia Huck wrote:
> For virtio-1 devices, we allow a more complex queue layout that doesn't
> require descriptor table and rings on a physically-contigous memory area:
> add virtio_queue_set_rings() to allow transports to set this up.
>
> Signed-off-by:
On Thu, Dec 11, 2014 at 02:25:06PM +0100, Cornelia Huck wrote:
> Add a helper function for checking whether a bit is set in the guest
> features for a vdev as well as one that works on a feature bit set.
>
> Convert code that open-coded this: It cleans up the code and makes it
> easier to extend t
On Fri, Dec 12, 2014 at 11:17:47AM +0100, Cornelia Huck wrote:
> On Fri, 12 Dec 2014 11:06:40 +0100
> Thomas Huth wrote:
>
> > On Thu, 11 Dec 2014 14:25:07 +0100
> > Cornelia Huck wrote:
[snip]
> > > vdev->queue_sel = 0;
> > > vdev->status = 0;
> > > vdev->isr = 0;
> > > @@ -924,7
On Thu, Dec 11, 2014 at 02:25:08PM +0100, Cornelia Huck wrote:
> Add code that checks for the VERSION_1 feature bit in order to make
> decisions about the device's endianness. This allows us to support
> transitional devices.
>
> Signed-off-by: Cornelia Huck
> ---
> hw/virtio/virtio.c
On Thu, Jan 15, 2015 at 4:54 PM, Kevin Wolf wrote:
> Hm, I guess there might be uses for holding references after the
> coroutine has exited if there are external references to the coroutine,
> but is it useful not to drop one reference after the coroutine exits? I
> guess in most cases, and in qe
On 2015/1/21 18:52, Paolo Bonzini wrote:
>
>
> On 21/01/2015 03:14, Gonglei wrote:
>> On 2015/1/21 0:10, Paolo Bonzini wrote:
>>
>>>
>>>
>>> On 19/01/2015 14:23, arei.gong...@huawei.com wrote:
@@ -780,6 +788,12 @@ static int qdev_get_fw_dev_path_helper(DeviceState
*dev, char *p, int s
On 2015/1/21 19:56, Ian Campbell wrote:
On Wed, 2015-01-21 at 11:37 +, Ian Jackson wrote:
Tiejun Chen writes ("[RFC][PATCH 1/1] libxl: add one machine property to support IGD
GFX passthrough"):
When we're working to support IGD GFX passthrough with qemu
upstream, instead of "-gfx_passthru"
On 2015/1/21 21:48, Gerd Hoffmann wrote:
On Mi, 2015-01-21 at 11:37 +, Ian Jackson wrote:
Tiejun Chen writes ("[RFC][PATCH 1/1] libxl: add one machine property to support IGD
GFX passthrough"):
When we're working to support IGD GFX passthrough with qemu
upstream, instead of "-gfx_passthru"
On 01/21/2015 09:48 AM, Jiri Slaby wrote:
> I am using qemu for teaching the Linux kernel at our university. I
> wrote a simple PCI device that can answer to writes/reads, generate
> interrupts and perform DMA. As I am dragging it locally over 2 years,
> I am sending it to you now.
>
> Signed-off-
On 01/21/2015 09:48 AM, Jiri Slaby wrote:
> I am using qemu for teaching the Linux kernel at our university. I
> wrote a simple PCI device that can answer to writes/reads, generate
> interrupts and perform DMA. As I am dragging it locally over 2 years,
> I am sending it to you now.
>
> Signed-off-
On 01/21/2015 04:09 AM, Michal Privoznik wrote:
> The command is not implemented correctly yet. The documentation allows
> to not pass any value to set, in which case the time is re-read from
> RTC. However, reading CMOS on Windows is not trivial to implement. So
> instead of pretending we've set t
On 2015-01-21 at 11:53, Stefan Hajnoczi wrote:
On Mon, Nov 24, 2014 at 04:56:49PM +0100, Max Reitz wrote:
+static void build_window_bitmap(Qcow2MetadataList *mdl,
+Qcow2MetadataWindow *window)
+{
+int cache_i, oldest_cache_i = -1, i;
+unsigned oldest_cache
I also confirm the problem on Windows 7 host.
I tried the following over the past few nights:
Downloaded the OpenVPN tap 32 adapter. Installed it and gave it a different
network address (192.168.200.10) to my normal LAN (192.168.1.10). Bridged the 2
connections, which gets me a new IP via DHCP a
This patch updates the exit reasons for x86_vmx, x86_svm, and userspace
to the latest definition.
Signed-off-by: Wei Huang
---
scripts/kvm/kvm_stat | 4
1 file changed, 4 insertions(+)
diff --git a/scripts/kvm/kvm_stat b/scripts/kvm/kvm_stat
index 470ca08..e8b3295 100755
--- a/scripts/kvm/
kvm_stat uses syscall() to call perf_event_open(). If this function
call fails, the returned value is -1, which doesn't tell the details
of such failure (i.e. ENOSYS or EINVAL). This patch retrieves errno
and prints it when syscall() fails. The error message will look like
"Exception: perf_event_op
This patch set updates kvm_stat with the following new features:
* support for ARM aarch64
* the latest exit reasons (vmx, svm and userspace)
* print errno when syscall fails
Thanks,
-Wei
*** BLURB HERE ***
Wei Huang (3):
kvm_stat: Add aarch64 support
kvm_stat: Update exit reasons to the l
This patch enables aarch64 support for kvm_stat. The platform detection
is based on OS uname.
Signed-off-by: Wei Huang
---
scripts/kvm/kvm_stat | 8
1 file changed, 8 insertions(+)
diff --git a/scripts/kvm/kvm_stat b/scripts/kvm/kvm_stat
index 7b1437c..470ca08 100755
--- a/scripts/kvm/
Also add an alias of 'nssemul' for the default card
of hw=yes hw_type=cac, and an alias of 'passthru' for
the new card type of hw=yes hw_type=passthru.
This allows the spice-gtk client to take a more human
friendly set of arguments and relay them through to this code.
Signed-off-by: Jeremy White
This patch is based on a discussion on the spice-devel list:
http://lists.freedesktop.org/archives/spice-devel/2015-January/018639.html
and on a proposed spice-gtk patch:
http://lists.freedesktop.org/archives/spice-devel/2015-January/018648.html
Essentially, it tidies up the vcard_emul_option
On 01/21/2015 10:04 AM, Bastian Koppelmann wrote:
> Add microcode generator function gen_cond_sub.
>
> Add helper functions:
> * ixmax/ixmin: search for the max/min value and its related index in a
>vector of 16-bit values.
> * pack: dack two data registers into an IEEE
On 01/21/2015 10:12 AM, Bastian Koppelmann wrote:
> Richard, before I put too much work into this. Are you happy with the general
> direction of this documentation?
Yes.
r~
This patch makes several changes:
- Minimizes distorted full screen display by respecting aspect
ratios.
- Makes full screen mode available on Mac OS 10.7 and higher.
- Allows user to decide if video should be stretched to fill the
screen, using a menu item called "Zoom To Fit".
- Hides the normalW
On 01/21/2015 10:04 AM, Bastian Koppelmann wrote:
> Signed-off-by: Bastian Koppelmann
> ---
> target-tricore/translate.c | 63
> ++
> 1 file changed, 63 insertions(+)
>
> diff --git a/target-tricore/translate.c b/target-tricore/translate.c
> index 4af
Added support for running an AArch32 guest on a AArch64 KVM host. Support has
only been added to the QEMU machvirt machine. The addition of CPU properties
specifiable from the command line were added to allow disablement of AArch64
execution state hereby forcing EL1 to be AArch32. The new CPU co
Adds registration and get/set functions for enabling/disabling the AArch64
execution state on AArch64 CPUs. By default AArch64 execution state is enabled
on AArch64 CPUs, setting the property to off, will disable the execution state.
The below QEMU invocation would have AArch64 execution state dis
Added machvirt parsing of feature keywords added to the -cpu command line
option. Parsing occurs during machine initialization.
Signed-off-by: Greg Bellows
---
v1 -> v2
- Fix multiple property handling
---
hw/arm/virt.c | 20 +---
1 file changed, 17 insertions(+), 3 deletions(
Add AArch32 to AArch64 register sychronization functions.
Replace manual register synchronization with new functions in
aarch64_cpu_do_interrupt() and HELPER(exception_return)().
Signed-off-by: Greg Bellows
---
target-arm/helper-a64.c | 5 +--
target-arm/internals.h | 89 ++
Add 32-bit to/from 64-bit register synchronization on register gets and puts.
Set EL1_32BIT feature flag passed to KVM
Signed-off-by: Greg Bellows
---
target-arm/kvm64.c | 21 +
1 file changed, 17 insertions(+), 4 deletions(-)
diff --git a/target-arm/kvm64.c b/target-arm/kvm
Hi,
just to be clear, the backport.patch which you uploaded actually
increased jitter for you, making the situation worse, right?
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1297218
Title:
guest
On 01/21/2015 10:04 AM, Bastian Koppelmann wrote:
> +case OPC2_32_RR2_MUL_32:
> +gen_mul_i32s(cpu_gpr_d[r3], cpu_gpr_d[r1], cpu_gpr_d[r2]);
> +break;
> +case OPC2_32_RR2_MUL_64:
> +gen_mul_i64s(cpu_gpr_d[r3], cpu_gpr_d[r3+1], cpu_gpr_d[r1],
> + cp
On 01/21/2015 10:04 AM, Bastian Koppelmann wrote:
> Signed-off-by: Bastian Koppelmann
> ---
> target-tricore/translate.c | 276
> +
> 1 file changed, 276 insertions(+)
>
> diff --git a/target-tricore/translate.c b/target-tricore/translate.c
> index de
** Changed in: qemu (Ubuntu)
Status: New => Invalid
** Changed in: qemu
Status: New => Invalid
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1318830
Title:
High CPU usage on windows
Paolo Bonzini writes:
> On 21/01/2015 17:05, Markus Armbruster wrote:
>> What do you mean by "a GLib model"? scripts/coverity-model.c?
>>> >
>>> > Yes. It models g_malloc0 in a way that avoids a lot of false positives,
>>> > but still is able to flag leaks.
>> Understood. It's hugely bett
On Tue, Jan 20, 2015 at 03:40:38PM +, Stefan Hajnoczi wrote:
> Jason Wang will be co-maintaining the QEMU net subsystem with me. He
> has contributed improvements and reviewed patches over the past years as
> part of working on virtio-net and virtualized networking.
>
> Jason has already been
Signed-off-by: Bastian Koppelmann
---
target-tricore/translate.c | 63 ++
1 file changed, 63 insertions(+)
diff --git a/target-tricore/translate.c b/target-tricore/translate.c
index 4af31c2..73d8d7d 100644
--- a/target-tricore/translate.c
+++ b/target-
On 01/14/2015 03:51 PM, Bastian Koppelmann wrote:
There is no overview, that shows all the frontend operation one can use, as
found on the wiki. Unfortunatly the wiki is out of date, so let's try to move
this documentation to the source files, which has the benefit, that it is easy
to update the
Add microcode generator function gen_cond_sub.
Add helper functions:
* ixmax/ixmin: search for the max/min value and its related index in a
vector of 16-bit values.
* pack: dack two data registers into an IEEE-754 single precision floating
point format number
Hi,
this is a rather short patchset, that only implements instructions of four
formats. There will be another patchset, which has a few bugfixes.
Cheers,
Bastian
Bastian Koppelmann (4):
target-tricore: target-tricore: Add instructions of RR1 opcode format,
that have 0x93 as first opcode
Signed-off-by: Bastian Koppelmann
---
target-tricore/translate.c | 276 +
1 file changed, 276 insertions(+)
diff --git a/target-tricore/translate.c b/target-tricore/translate.c
index def7f4a..da8ecbc 100644
--- a/target-tricore/translate.c
+++ b/target
Signed-off-by: Bastian Koppelmann
---
target-tricore/translate.c | 37 +
1 file changed, 37 insertions(+)
diff --git a/target-tricore/translate.c b/target-tricore/translate.c
index da8ecbc..4af31c2 100644
--- a/target-tricore/translate.c
+++ b/target-tricore/t
Fix tc1796 cpu model using wrong ISA version.
Fix cond_add sometimes writing back wrong result.
Fix RCR_SEL and RCR_SELN using wrong registers for result and cond.
Signed-off-by: Bastian Koppelmann
---
target-tricore/cpu.c | 2 +-
target-tricore/op_helper.c | 1 +
target-tricore/translate.
This fixes one bug and one false positive found by coverity. The bug is,
that gen_mtcr was missing a mask to check the flag, which resulted in dead code.
The false positive is a intentional missing break for a jump and link address
insn followed by a jump and link insn. This adds a fall through co
suov checks unsigned for an overflow and an underflow, after some arithmetic
operations and saturates the result to either max_uint32 or 0. So far we
handled this by expanding to the next bigger data type and compare whether
the result is > max_uint32 or < 0.
However this approach can fail for an
64 bit mac instructions calculated the av bits after the saturation, which
resulted in a wrong PSW. This moves the av bit calculation before the
saturation.
Signed-off-by: Bastian Koppelmann
---
target-tricore/op_helper.c | 28
1 file changed, 16 insertions(+), 12 de
Hi,
this patchset only contains bugfixes, like the ones found by coverity, and some
minor corner cases in some instructions.
Cheers,
Bastian
Bastian Koppelmann (4):
target-tricore: Several translator and cpu model fixes
target-tricore: calculate av bits before saturation
target-tricore: Fi
Hi Peter,
This is mainly to apply your outstanding clang warning patches. Please pull.
ATB,
Mark.
The following changes since commit 699eae17b841e6784dc3864bf357e26bff1e9dfe:
Merge remote-tracking branch 'remotes/pmaydell/tags/pull-misc-20150120' into
staging (2015-01-20 16:19:58 +)
On 21/01/2015 17:48, Jiri Slaby wrote:
> I am using qemu for teaching the Linux kernel at our university. I
> wrote a simple PCI device that can answer to writes/reads, generate
> interrupts and perform DMA. As I am dragging it locally over 2 years,
> I am sending it to you now.
>
> Signed-off-b
I am using qemu for teaching the Linux kernel at our university. I
wrote a simple PCI device that can answer to writes/reads, generate
interrupts and perform DMA. As I am dragging it locally over 2 years,
I am sending it to you now.
Signed-off-by: Jiri Slaby
---
MAINTAINERS | 5 +
On Mon, Nov 24, 2014 at 04:56:49PM +0100, Max Reitz wrote:
> +static void build_window_bitmap(Qcow2MetadataList *mdl,
> +Qcow2MetadataWindow *window)
> +{
> +int cache_i, oldest_cache_i = -1, i;
> +unsigned oldest_cache_age = 0;
> +
> +for (cache_i = 0; c
Paolo Bonzini writes:
> On 21/01/2015 14:58, Markus Armbruster wrote:
>> Defect density by component, from
>> https://scan.coverity.com/projects/378?tab=overview
>
> Updated:
>
> bt46101.748
> slirp 69681.4410
> 9pfs 94931.3713
> user
On 01/21/2015, 05:40 PM, Paolo Bonzini wrote:
>
>
> On 21/01/2015 17:28, Jiri Slaby wrote:
>> +if (atomic_fetch_or(&edu->status, EDU_STATUS_COMPUTING) &
>> EDU_STATUS_COMPUTING) {
>
> Theoretically the other thread could see EDU_STATUS_COMPUTING here and
> not enter the condvar wait. S
On 21/01/2015 17:28, Jiri Slaby wrote:
> +if (atomic_fetch_or(&edu->status, EDU_STATUS_COMPUTING) &
> EDU_STATUS_COMPUTING) {
Theoretically the other thread could see EDU_STATUS_COMPUTING here and
not enter the condvar wait. So...
> +break;
> +}
> +qemu_mut
On 01/21/2015 11:56 AM, Chen Fan wrote:
On 01/16/2015 03:56 PM, Chen Fan wrote:
On 01/12/2015 09:56 PM, Marcel Apfelbaum wrote:
On 01/12/2015 05:04 AM, Chen Fan wrote:
in spec "PCI Express 3.0" section 6.2.6 Figure 6-3 virtual bridge part,
the flowchart showing tell us SERR# enable at Bridge
I am using qemu for teaching the Linux kernel at our university. I
wrote a simple PCI device that can answer to writes/reads, generate
interrupts and perform DMA. As I am dragging it locally over 2 years,
I am sending it to you now.
Signed-off-by: Jiri Slaby
---
MAINTAINERS | 5 +
On 21/01/2015 17:05, Markus Armbruster wrote:
>>> >> What do you mean by "a GLib model"? scripts/coverity-model.c?
>> >
>> > Yes. It models g_malloc0 in a way that avoids a lot of false positives,
>> > but still is able to flag leaks.
> Understood. It's hugely better than nothing,
Yes, I thin
Linux implements a nice binding to describe a "generic" PCI Express host bridge
using only device tree.
This patch set adds enough emulation logic to expose the parts that are
"generic" as a simple sysbus device and maps it into ARM's virt machine.
With this patch set, we can finally spawn PCI de
Now that we have a working "generic" PCIe host bridge driver, we can plug
it into ARMs virt machine to always have PCIe available to normal ARM VMs.
I've successfully managed to expose a Bochs VGA device, XHCI and an e1000
into an AArch64 VM with this and they all lived happily ever after.
Signed
With simple exposure of MMFG, ioport window, mmio window and an IRQ line we
can successfully create a workable PCIe host bridge that can be mapped anywhere
and only needs to get described to the OS using whatever means it likes.
This patch implements such a "generic" host bridge. It only supports
Every platform that supports PCI can also spawn the Bochs VGA PCI adapter. Move
it to pci.mak to enable it for everyone.
Signed-off-by: Alexander Graf
---
default-configs/alpha-softmmu.mak| 2 --
default-configs/i386-softmmu.mak | 2 --
default-configs/mips-softmmu.mak | 2 --
defaul
The mmcfg space is a memory region that allows access to PCI config space
in the PCIe world. To maintain abstraction layers, I would like to expose
the mmcfg space as a sysbus mmio region rather than have it mapped straight
into the system's memory address space though.
So this patch splits the in
On 18/01/15 21:15, Mark Cave-Ayland wrote:
> On 02/01/15 17:30, Peter Maydell wrote:
>
>> On 2 January 2015 at 13:57, Mark Cave-Ayland
>> wrote:
>>> On 23/12/14 22:11, Peter Maydell wrote:
These patches fix warnings generated by clang. Patches 1-3
have been onlist before (and reviewed
On 21/01/2015 16:59, Peter Maydell wrote:
>> > I wasn't bold enough to suggest "daily", let alone "release blocker".
> I think the Coverity FAQ says we can't do more than 2
> scans a week for a project of QEMU's size anyway...
That's just a default. You can ask them to be an exception, and QEMU
Paolo Bonzini writes:
> On 21/01/2015 15:57, Markus Armbruster wrote:
>>> QEMU is also using a GLib model on Coverity Scan, as well as a
>>> QEMU-specific model, which suggests one of the following:
>>
>> What do you mean by "a GLib model"? scripts/coverity-model.c?
>
> Yes. It models g_malloc
On 2015-01-21 at 10:23, Stefan Hajnoczi wrote:
On Mon, Nov 24, 2014 at 04:56:59PM +0100, Max Reitz wrote:
@@ -136,6 +138,34 @@ int qcow2_read_snapshots(BlockDriverState *bs)
size_to_clusters(s, sn->l1_size *
Issuing loadvm under -M mac99 would fail for two reasons: firstly an incorrect
version number for openpic would cause openpic_load() to abort, and secondly
a cut/paste error when restoring the IVPR and IDR registers caused subsequent
vmstate sections to become misaligned and abort early.
Signed-of
On Tue, 20 Jan 2015 13:18:14 +
Stefan Hajnoczi wrote:
> On Thu, Dec 11, 2014 at 02:25:18PM +0100, Cornelia Huck wrote:
> > diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> > index ebbea60..7ee2bd6 100644
> > --- a/hw/net/virtio-net.c
> > +++ b/hw/net/virtio-net.c
> > @@ -373,15 +373,2
A simple copy/paste error causes savevm on -M mac99 to segfault.
Signed-off-by: Mark Cave-Ayland
---
hw/intc/openpic.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/intc/openpic.c b/hw/intc/openpic.c
index 7d1f3b9..8699a4a 100644
--- a/hw/intc/openpic.c
+++ b/hw/intc/
Make sure that we include the adb_poll_timer when saving the VM state for
client OSs that use it, e.g. Darwin.
Signed-off-by: Mark Cave-Ayland
---
hw/misc/macio/cuda.c |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/misc/macio/cuda.c b/hw/misc/macio/cuda.c
index b4
On 21/01/2015 14:58, Markus Armbruster wrote:
> Defect density by component, from
> https://scan.coverity.com/projects/378?tab=overview
Updated:
bt 46101.748
slirp 69681.4410
9pfs94931.3713
user32263 0.6822
fpu
Since env->msr has already been restored by the time cpu_post_load is called,
make sure that ppc_store_msr() is explicitly called with all msr bits except
MSR_TGPR marked as invalid.
This solves the issue where MSR flags aren't set correctly when restoring a VM
snapshot, in particular the internal
The parent ADBDevice contains the device id on the ADB bus. Make sure that
this state is included in both its subclasses since some clients (such as
OpenBIOS) reprogram each device id after enumeration.
Signed-off-by: Mark Cave-Ayland
---
hw/input/adb.c | 22 ++
1 file chan
This patchset fixes up various bugs in loadvm/savevm for -M g3beige and
-M mac99 so that it is becomes possible to save and restore image snapshots.
The focus of this patchset is on -M g3beige since this matches the majority
of my test images, but there were some easy fixes to be made to -M mac99
This ensures that the macio PCI device is correctly configured when restoring
from a VM snapshot.
Signed-off-by: Mark Cave-Ayland
---
hw/misc/macio/macio.c | 24
1 file changed, 24 insertions(+)
diff --git a/hw/misc/macio/macio.c b/hw/misc/macio/macio.c
index e0f1e88.
Otherwise when cpu_post_load calls ppc_store_sdr1() when restoring a VM
snapshot the value is deemed unchanged and so the internal env->htab*
variables aren't set correctly.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Paolo Bonzini
---
target-ppc/misc_helper.c |7 ++-
target-ppc/mmu_he
On 21 January 2015 at 15:55, Markus Armbruster wrote:
> I wasn't bold enough to suggest "daily", let alone "release blocker".
I think the Coverity FAQ says we can't do more than 2
scans a week for a project of QEMU's size anyway...
-- PMM
"Daniel P. Berrange" writes:
> On Wed, Jan 21, 2015 at 01:47:22PM +0100, Markus Armbruster wrote:
>> We're using the Coverity Scan service[*]. We've put in some effort, and
>> we've gotten some mileage out of it, but I feel we could get more.
>>
>> Judging from the report e-mail I have lying ab
On 01/21/2015 02:34 AM, Markus Armbruster wrote:
> Opinions?
>
I'm still thinking about my reply (it's a big enough question that I
want to make sure I consider ramifications), so this is just a
meta-reply to let you know I'm tracking this conversation.
CC'ing Eric Blake, as well, for comme
this patch avoids sign extension and fixes a data conversion
bug in stpcifc. Both issues where found by Coverity.
Signed-off-by: Frank Blaschka
---
hw/s390x/s390-pci-inst.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/s390x/s390-pci-inst.c b/hw/s390x/s390-pci-in
1 - 100 of 193 matches
Mail list logo
