Re: [Qemu-devel] [PATCH V3 1/2] arm64: Add an option to turn on/off vPMU support

2016-09-15 Thread Andrew Jones
On Thu, Sep 15, 2016 at 01:04:15AM -0400, Wei Huang wrote: > This patch adds a pmu=[on/off] option to enable/disable vPMU support > in guest vCPU. This option is only available for cortex-a57/cortex-53/ > host under both TCG and KVM modes, but unavailable on ARMv7 and other > processors. It allows

[Qemu-devel] [PATCH v2] pc: apic: introduce APIC macro

2016-09-15 Thread Wanpeng Li
From: Wanpeng Li Introduce a new APIC macro to replace APIC_COMMON macro in hw/intc/apic.c in order to capture access LAPIC in qemu even if LAPIC is emulated in kvm. Suggested-by: Paolo Bonzini Reviewed-by: Michael S. Tsirkin Cc: Paolo Bonzini Cc: Radim Krčmář Cc: Michael S. Tsirkin Cc: E

Re: [Qemu-devel] [PATCH V3 2/2] arm: virt: add PMU property to mach-virt machine type

2016-09-15 Thread Andrew Jones
On Thu, Sep 15, 2016 at 01:04:16AM -0400, Wei Huang wrote: > CPU vPMU is now turned off by default, but it was ON in virt-2.7 > machine type. To solve this problem, this patch adds a PMU option > in machine state, which is used to control CPU's vPMU status. This > PMU option is not exposed to comma

[Qemu-devel] [Bug 1536487] Re: Unable to migrate pc-i440fx-2.4 KVM guest from QEMU 2.5.0 to QEMU 2.4.1

2016-09-15 Thread ChristianEhrhardt
Serge, Paulo - thank you both! I already had the patch but I think it was good to discuss and list the expected behavior not only for me, but for whoever else that comes by this or a similar case. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscrib

Re: [Qemu-devel] [PATCH 5/5] spapr_vscsi: convert to trace framework instead of DPRINTF

2016-09-15 Thread Laurent Vivier
On 14/09/2016 22:09, Eric Blake wrote: > On 09/14/2016 01:48 PM, Laurent Vivier wrote: >> Signed-off-by: Laurent Vivier >> --- >> hw/scsi/spapr_vscsi.c | 89 >> +-- >> hw/scsi/trace-events | 27 >> 2 files changed, 63 insertions

Re: [Qemu-devel] [PATCH 0/5] ps2: fix keyboard set 3 and misc improvements

2016-09-15 Thread Gerd Hoffmann
Hi, > - Patches 1 and 2 are fixes to let Linux use set 3 if instructed so. > - Patch 3 makes scancodes untranslated by default and translates them if > required, > instead of receiving translated scancodes by default and untranslating them > if > required. > - Patch 4 switches to use qcode

Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p

2016-09-15 Thread Greg Kurz
Hi, Even if I am maintainer for 9P stuff, I'm not sure I can send a pull request for patches without at least an Acked-by... :-\ I plan to add true 9P functional tests in the future, but these are more about the virtio device actually. Any chances some virtio people can have a look and ack or na

[Qemu-devel] [PATCH v2 00/11] icount/replay additions

2016-09-15 Thread Pavel Dovgalyuk
This set of patches includes several fixes for replay and adds network record/replay for network devices. It also makes possible saving/restoring vmstate in replay mode. Record and replay for network interactions is performed with the network filter. Each backend must have its own instance of the

[Qemu-devel] [PATCH v2 01/11] target-ppc: exceptions handling in icount mode

2016-09-15 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch fixes exception handling in PowerPC. 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 instr

[Qemu-devel] [PATCH v2 02/11] record/replay: add network support

2016-09-15 Thread Pavel Dovgalyuk
This patch adds support of recording and replaying network packets in irount rr mode. Record and replay for network interactions is performed with the network filter. Each backend must have its own instance of the replay filter as follows: -netdev user,id=net1 -device rtl8139,netdev=net1 -object

Re: [Qemu-devel] [PATCH 04/10] block: Accept device model name for blockdev-open/close-tray

2016-09-15 Thread Kevin Wolf
Am 14.09.2016 um 22:49 hat Eric Blake geschrieben: > On 08/19/2016 11:50 AM, Kevin Wolf wrote: > > In order to remove the necessity to use BlockBackend names in the > > external API, we want to allow qdev device names in all device related > > commands. > > > > This converts blockdev-open/close-tr

Re: [Qemu-devel] [PATCH 05/10] block: Accept device model name for x-blockdev-insert-medium

2016-09-15 Thread Kevin Wolf
Am 14.09.2016 um 22:57 hat Eric Blake geschrieben: > On 08/19/2016 11:50 AM, Kevin Wolf wrote: > > In order to remove the necessity to use BlockBackend names in the > > external API, we want to allow qdev device names in all device related > > commands. > > > > This converts x-blockdev-insert-medi

Re: [Qemu-devel] [PATCH RFC] docs: add PCIe devices placement guidelines

2016-09-15 Thread Andrew Jones
On Wed, Sep 07, 2016 at 10:39:28PM +0300, Marcel Apfelbaum wrote: > On 09/07/2016 08:55 PM, Laine Stump wrote: > > On 09/07/2016 04:06 AM, Marcel Apfelbaum wrote: [snip] > > > Good point, maybe libvirt can avoid adding switches unless the user > > > explicitly > > > asked for them. I checked and it

Re: [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test

2016-09-15 Thread Cornelia Huck
On Fri, 09 Sep 2016 10:04:51 +0200 Greg Kurz wrote: > This adds PCI init code and a basic test that checks the device config > matches what is passed on the command line. > > Signed-off-by: Greg Kurz > --- > tests/Makefile.include |2 + > tests/virtio-9p-test.c | 82 > ++

Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p

2016-09-15 Thread Cornelia Huck
On Fri, 09 Sep 2016 10:04:37 +0200 Greg Kurz wrote: > As with other virtio-* qtests, PC platform is assumed. > > --- > > Greg Kurz (3): > tests: virtio-9p: introduce start/stop functions > tests: virtio-9p: add basic configuration test > tests: virtio-9p: add basic transaction

[Qemu-devel] [PATCH 0/2] target-mips: rework conditional stores for mttcg

2016-09-15 Thread Leon Alrae
This small series changes MIPS conditional stores implementation for mttcg. Specifically we compare virtual address of LL and SC (rather than physical) which allows us to have just a single inlined implementation for user and system emulation and to use new atomic helpers. This is done in 2 steps:

[Qemu-devel] [PATCH 1/2] target-mips: compare virtual addresses in LL/SC sequence

2016-09-15 Thread Leon Alrae
Until now we have been comparing physical addresses in LL/SC sequence. Unfortunately that means that on each SC we have to do the address translation which is a quite complex operation. If we could get rid of it then it would allow us to throw away SC helpers and benefit from having common implemen

[Qemu-devel] [PATCH 2/2] target-mips: reimplement SC instruction and use cmpxchg

2016-09-15 Thread Leon Alrae
This patch completely rewrites conditional stores. Now we use cmpxchg and no longer need separate implementations for user and system emulation. Signed-off-by: Leon Alrae --- linux-user/main.c | 58 - target-mips/cpu.h | 4 -- target-mips/helper.c| 6 +--

Re: [Qemu-devel] [PATCH 10/10] qemu-iotests/118: Test media change with qdev name

2016-09-15 Thread Kevin Wolf
Am 15.09.2016 um 00:13 hat Eric Blake geschrieben: > On 08/19/2016 11:50 AM, Kevin Wolf wrote: > > We just added the option to use qdev device names in all device related > > block QMP commands. This patch converts some of the test cases in 118 to > > use qdev device names instead of BlockBackend n

[Qemu-devel] [PATCH v2 01/10] record/replay: add network support

2016-09-15 Thread Pavel Dovgalyuk
This patch adds support of recording and replaying network packets in irount rr mode. Record and replay for network interactions is performed with the network filter. Each backend must have its own instance of the replay filter as follows: -netdev user,id=net1 -device rtl8139,netdev=net1 -object

[Qemu-devel] [PATCH v2 00/10] replay additions

2016-09-15 Thread Pavel Dovgalyuk
This set of patches includes several fixes for replay and adds network record/replay for network devices. It also makes possible saving/restoring vmstate in replay mode. Record and replay for network interactions is performed with the network filter. Each backend must have its own instance of the

[Qemu-devel] [PATCH v2 03/10] block: don't make snapshots for filters

2016-09-15 Thread Pavel Dovgalyuk
This patch disables snapshotting for block driver filters. It is needed, because snapshots should be created in underlying disk images, not in filters itself. Signed-off-by: Pavel Dovgalyuk --- block/snapshot.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/block/snapshot.c b/block/sn

[Qemu-devel] [PATCH v2 07/10] replay: allow replay stopping and restarting

2016-09-15 Thread Pavel Dovgalyuk
This patch fixes bug with stopping and restarting replay through monitor. Signed-off-by: Pavel Dovgalyuk --- block/blkreplay.c| 15 +-- cpus.c |1 + include/sysemu/replay.h |4 replay/replay-events.c |8 replay/replay-internal.

[Qemu-devel] [PATCH v2 02/10] block: set snapshot option for block devices in blkreplay module

2016-09-15 Thread Pavel Dovgalyuk
This patch adds overlay option for blkreplay filter. It allows creating persistent overlay file for saving and reloading VM snapshots in record/replay modes. Signed-off-by: Pavel Dovgalyuk --- block/blkreplay.c | 119 + docs/replay.txt |

[Qemu-devel] [PATCH v2 06/10] replay: vmstate for replay module

2016-09-15 Thread Pavel Dovgalyuk
This patch introduces vmstate for replay data structures. It allows saving and loading vmstate while replaying. Signed-off-by: Pavel Dovgalyuk --- include/sysemu/replay.h |4 replay/replay-internal.h |2 ++ replay/replay-snapshot.c | 40 v

[Qemu-devel] [PATCH v2 04/10] replay: save/load initial state

2016-09-15 Thread Pavel Dovgalyuk
This patch implements initial vmstate creation or loading at the start of record/replay. It is needed for rewinding the execution in the replay mode. Signed-off-by: Pavel Dovgalyuk --- include/sysemu/replay.h |6 ++ replay/Makefile.objs |1 + replay/replay-snapshot.c | 31

[Qemu-devel] [PATCH v2 09/10] pcspk: adding vmstate for save/restore

2016-09-15 Thread Pavel Dovgalyuk
VMState added by this patch preserves correct loading of the PC speaker device state. Signed-off-by: Pavel Dovgalyuk --- hw/audio/pcspk.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/hw/audio/pcspk.c b/hw/audio/pcspk.c index 42a6f48..984534b 100644 ---

[Qemu-devel] [PATCH v2 05/10] replay: move internal data to the structure

2016-09-15 Thread Pavel Dovgalyuk
This patch moves replay static variables into the structure to allow saving and loading them with savevm/loadvm. Signed-off-by: Pavel Dovgalyuk --- replay/replay-events.c |2 +- replay/replay-internal.c | 19 --- replay/replay-internal.h |8 +--- replay/replay-tim

[Qemu-devel] [PATCH v2 10/10] integratorcp: adding vmstate for save/restore

2016-09-15 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk VMState added by this patch preserves correct loading of the integratorcp device state. Signed-off-by: Pavel Dovgalyuk --- hw/arm/integratorcp.c | 62 + 1 file changed, 62 insertions(+) diff --git a/hw/arm/integratorcp.c

Re: [Qemu-devel] [PATCH v2 2/6] trace: convert code to use event iterators

2016-09-15 Thread Daniel P. Berrange
On Thu, Sep 15, 2016 at 12:16:52AM +0200, Lluís Vilanova wrote: > Daniel P Berrange writes: > > > This converts the HMP/QMP monitor API implementations > > and some internal trace control methods to use the new > > trace event iterator APIs. > > > Reviewed-by: Stefan Hajnoczi > > Signed-off-by:

Re: [Qemu-devel] [PATCH v2 1/6] trace: add trace event iterator APIs

2016-09-15 Thread Daniel P. Berrange
On Wed, Sep 14, 2016 at 11:53:10PM +0200, Lluís Vilanova wrote: > Daniel P Berrange writes: > > > Currently methods which want to iterate over trace events, > > do so using the trace_event_count() and trace_event_id() > > methods. This leaks the concept of a single ID enum to > > the callers. Ther

[Qemu-devel] [PATCH v2 08/10] kvmvapic: fix state change handler

2016-09-15 Thread Pavel Dovgalyuk
This patch fixes kvmvapic state change handler. It clears vmsentry field to allow recreating it at further vmstate loads. Signed-off-by: Pavel Dovgalyuk --- hw/i386/kvmvapic.c |1 + 1 file changed, 1 insertion(+) diff --git a/hw/i386/kvmvapic.c b/hw/i386/kvmvapic.c index 3bf1ddd..a1cd9b5 10

Re: [Qemu-devel] [Qemu-block] [PATCH 2/7] block: Set BDRV_O_ALLOW_RDWR and snapshot_options before storing the flags

2016-09-15 Thread Alberto Garcia
On Wed 14 Sep 2016 08:54:19 PM CEST, Jeff Cody wrote: >> If an image is opened with snapshot=on, its flags are modified by >> bdrv_backing_options() and then bs->open_flags is updated accordingly. >> This last step is unnecessary if we calculate the new flags before >> setting bs->open_flags. >> >

Re: [Qemu-devel] Adding a custom monitoring device to virt machine

2016-09-15 Thread Stefan Hajnoczi
On Tue, Aug 23, 2016 at 07:31:53AM +, Nassim Corteggiani wrote: > Hi all, > > > This is the first time I am playing with QEmu. > > > I made a custom qemu device in oder to monitor all memory accesses (sdram, > flash, peripherals, gpio, ...). > > > The device is parent of TYPE_SYS_BUS_DEV

Re: [Qemu-devel] how to understand proccessor codes in Qemu

2016-09-15 Thread Stefan Hajnoczi
On Tue, Aug 23, 2016 at 12:57:09PM +, Mahmoud Abd el-hameed wrote: > I am a new developer at Qemu and i want to understand how the processors code > works. I am get lost through google and didn't know where i can begin to > understand the code. > > my aim is to understand how can we model th

Re: [Qemu-devel] [PATCH v2 4/6] trace: remove global 'uint16 dstate[]' array

2016-09-15 Thread Daniel P. Berrange
On Thu, Sep 15, 2016 at 12:56:57AM +0200, Lluís Vilanova wrote: > Daniel P Berrange writes: > > > Instead of having a global dstate array, declare a single > > 'uint16 TRACE_${EVENT_NAME}_DSTATE' variable for each > > trace event. Record a pointer to this variable in the > > TraceEvent struct too.

Re: [Qemu-devel] [PATCH v7 0/2] qemu-qdisk: Implementation of grant copy operation.

2016-09-15 Thread Wei Liu
On Wed, Sep 14, 2016 at 06:33:00PM -0700, Stefano Stabellini wrote: > Hi Wei, > > I am happy to queue up this for QEMU, but I'll wait for the first patch > to be committed to Xen before sending a pull request. Is that OK? > Yes, that's fine. I will get around to this next week. One of the prereq

Re: [Qemu-devel] [PATCH v2 00/10] replay additions

2016-09-15 Thread Paolo Bonzini
On 15/09/2016 11:00, Pavel Dovgalyuk wrote: > This set of patches includes several fixes for replay and > adds network record/replay for network devices. It also makes possible > saving/restoring vmstate in replay mode. > > Record and replay for network interactions is performed with the network

Re: [Qemu-devel] [PATCH v2 04/10] replay: save/load initial state

2016-09-15 Thread Paolo Bonzini
On 15/09/2016 11:01, Pavel Dovgalyuk wrote: > +{ > +if (replay_mode == REPLAY_MODE_RECORD) { > +QDict *opts = qdict_new(); > +qdict_put(opts, "name", qstring_from_str("replay_init")); > +hmp_savevm(cur_mon, opts); > +QDECREF(opts); > +} else if (replay_mode

Re: [Qemu-devel] [PATCH v2 5/6] trace: remove use of event ID enums from APIs

2016-09-15 Thread Daniel P. Berrange
On Thu, Sep 15, 2016 at 01:26:06AM +0200, Lluís Vilanova wrote: > Daniel P Berrange writes: > > > Since there will shortly be multiple event groups allowed, > > we can no longer use the TraceEventID and TraceEventVCPUID > > enums in the trace control APIs. There will in fact be > > multiple distin

Re: [Qemu-devel] [PATCH v2 02/10] block: set snapshot option for block devices in blkreplay module

2016-09-15 Thread Paolo Bonzini
On 15/09/2016 11:00, Pavel Dovgalyuk wrote: > diff --git a/docs/replay.txt b/docs/replay.txt > index 347b2ff..5be8f25 100644 > --- a/docs/replay.txt > +++ b/docs/replay.txt > @@ -196,6 +196,14 @@ is recorded to the log. In replay phase the queue is > matched with > events read from the log. The

Re: [Qemu-devel] [PATCH v2 05/10] replay: move internal data to the structure

2016-09-15 Thread Paolo Bonzini
On 15/09/2016 11:01, Pavel Dovgalyuk wrote: > This patch moves replay static variables into the structure > to allow saving and loading them with savevm/loadvm. > > Signed-off-by: Pavel Dovgalyuk > --- > replay/replay-events.c |2 +- > replay/replay-internal.c | 19 ---

Re: [Qemu-devel] [PATCH v2 02/10] block: set snapshot option for block devices in blkreplay module

2016-09-15 Thread Paolo Bonzini
On 15/09/2016 11:25, Paolo Bonzini wrote: > So in this case the image is actually overlay.qcow2, and it is created > with img-direct as the backing file? Since you have to create > overlay.qcow2 outside QEMU anyway, overlay.qcow2 might as well be the > "image". That is, you could choose betw

Re: [Qemu-devel] [PATCH v3 30/34] linux-user: remove handling of ARM's EXCP_STREX

2016-09-15 Thread Alex Bennée
Richard Henderson writes: > From: "Emilio G. Cota" > > The exception is not emitted anymore. > > Signed-off-by: Emilio G. Cota > Signed-off-by: Richard Henderson > Message-Id: <1467054136-10430-29-git-send-email-c...@braap.org> Reviewed-by: Alex Bennée > --- > linux-user/main.c | 93 > --

Re: [Qemu-devel] [PATCH v2 06/10] replay: vmstate for replay module

2016-09-15 Thread Paolo Bonzini
On 15/09/2016 11:01, Pavel Dovgalyuk wrote: > This patch introduces vmstate for replay data structures. > It allows saving and loading vmstate while replaying. > > Signed-off-by: Pavel Dovgalyuk > --- > include/sysemu/replay.h |4 > replay/replay-internal.h |2 ++ > replay/replay

Re: [Qemu-devel] [PATCH v3 31/34] linux-user: remove handling of aarch64's EXCP_STREX

2016-09-15 Thread Alex Bennée
Richard Henderson writes: > From: "Emilio G. Cota" > > The exception is not emitted anymore. > > Signed-off-by: Emilio G. Cota > Signed-off-by: Richard Henderson > Message-Id: <1467054136-10430-30-git-send-email-c...@braap.org> Reviewed-by: Alex Bennée > --- > linux-user/main.c | 125 > -

Re: [Qemu-devel] [virtio-dev][RFC v3] virtio-sdm: new device specification

2016-09-15 Thread Edgar E. Iglesias
On Mon, Sep 12, 2016 at 05:42:18PM +0200, Christian Pinto wrote: > Hello Edgar, > > > sorry for the delay. No worries! > On 08/09/2016 17:38, Edgar E. Iglesias wrote: > >On Thu, Sep 08, 2016 at 09:06:00AM +0200, Christian Pinto wrote: > >>Hello Edgar, > >> > >> > >>On 07/09/2016 18:02, Edgar E

Re: [Qemu-devel] [PATCH v2 07/10] replay: allow replay stopping and restarting

2016-09-15 Thread Paolo Bonzini
On 15/09/2016 11:01, Pavel Dovgalyuk wrote: > This patch fixes bug with stopping and restarting replay > through monitor. > > Signed-off-by: Pavel Dovgalyuk > --- > block/blkreplay.c| 15 +-- > cpus.c |1 + > include/sysemu/replay.h |4 > r

Re: [Qemu-devel] [PATCH v3 32/34] target-arm: remove EXCP_STREX + cpu_exclusive_{test, info}

2016-09-15 Thread Alex Bennée
Richard Henderson writes: > From: "Emilio G. Cota" > > The exception is not emitted anymore; remove it and the associated > TCG variables. > > Signed-off-by: Emilio G. Cota > Signed-off-by: Richard Henderson > Message-Id: <1467054136-10430-31-git-send-email-c...@braap.org> > --- > target-arm

Re: [Qemu-devel] [PATCH v5 3/5] linux-user: Fix structure target_flock definition for Mips

2016-09-15 Thread Leon Alrae
On Mon, Sep 12, 2016 at 09:40:02PM +0200, Aleksandar Markovic wrote: > From: Aleksandar Markovic > > Structure flock is defined for Mips in a way different from any > other platform. For reference, see Linux kernel source code files: > > arch/mips/include/uapi/asm/fcntl.h#L63 (for Mips) > includ

Re: [Qemu-devel] [PATCH] hmp: Improve 'info mtree' with optional parm for mapinfo

2016-09-15 Thread Paolo Bonzini
On 07/09/2016 02:48, Thorsten Kohfeldt wrote: > From: Thorsten Kohfeldt > Date: Wed, 31 Aug 2016 22:43:14 +0200 > Subject: [PATCH] hmp: Improve 'info mtree' with optional parm for mapinfo > > Motivation > When 'tuning' 'quirks' for VFIO imported devices, it is not easy to > directly grasp the i

Re: [Qemu-devel] [PATCH RESEND v2 14/17] target-ppc: improve lxvw4x implementation

2016-09-15 Thread Nikunj A Dadhania
David Gibson writes: > [ Unknown signature status ] > On Mon, Sep 12, 2016 at 12:11:43PM +0530, Nikunj A Dadhania wrote: >> Load 8byte at a time and manipulate. >> >> Signed-off-by: Nikunj A Dadhania >> --- >> target-ppc/helper.h | 1 + >> target-ppc/mem_helper.c |

Re: [Qemu-devel] Adding new source files

2016-09-15 Thread Stefan Hajnoczi
On Wed, Aug 24, 2016 at 03:11:16PM +0200, Pradeep Kiruvale wrote: > I have added two new files (.h/.c) file in 9pfs for some new feature. > When I compile there is a lot of compilation time errors and also I do not > see a dependency > file getting created for this source file. > > Please let me k

Re: [Qemu-devel] [PATCH v7 1/2] libs/gnttab: introduce grant copy interface

2016-09-15 Thread Wei Liu
On Wed, Sep 14, 2016 at 09:10:02PM +0200, Paulina Szubarczyk wrote: > In a linux part an ioctl(gntdev, IOCTL_GNTDEV_GRANT_COPY, ..) > system call is invoked. In mini-os the operation is yet not > implemented. For the OSs that does not implement gnttab the > call of the grant copy operation causes a

Re: [Qemu-devel] [PATCH v3 33/34] target-alpha: Introduce MMU_PHYS_IDX

2016-09-15 Thread Alex Bennée
Richard Henderson writes: > Rather than using helpers for physical accesses, use a mmu index. > The primary cleanup is with store-conditional on physical addresses. > > Signed-off-by: Richard Henderson > --- > target-alpha/cpu.h| 18 +--- > target-alpha/helper.c | 10 ++

Re: [Qemu-devel] Make file problem?

2016-09-15 Thread Stefan Hajnoczi
On Wed, Aug 31, 2016 at 09:43:43AM -0400, Programmingkid wrote: > When I tried to make a change to the file hw/misc/macio/cuda.c, the make > command would not detect the changes. I would have to delete the cuda.o file > to make the changes actually compile. Try "make -d hw/misc/macio/cuda.o" to

Re: [Qemu-devel] how to monitor vm exit

2016-09-15 Thread Stefan Hajnoczi
On Fri, Sep 02, 2016 at 12:15:57AM +0800, Yuxin Ren wrote: > 1. How can I track how many vm exit happens during vm execution using > KVM under Ubuntu? $ sudo perf record -a -e kvm:kvm_exit ...^C $ sudo perf report Samples: 37K of event 'kvm:kvm_exit', Event count (approx.): 37736 Overhead Trace o

Re: [Qemu-devel] [PULL v2 00/58] First round of misc patches for QEMU 2.8

2016-09-15 Thread Peter Maydell
On 13 September 2016 at 18:19, Paolo Bonzini wrote: > The following changes since commit fa9701240951093907076db0943f96972a396ef5: > > Merge remote-tracking branch 'remotes/kraxel/tags/pull-vga-20160913-1' into > staging (2016-09-13 13:56:35 +0100) > > are available in the git repository at: >

Re: [Qemu-devel] [PATCH 4/7] block: Add "read-only" to the options QDict

2016-09-15 Thread Kevin Wolf
Am 14.09.2016 um 17:52 hat Alberto Garcia geschrieben: > This adds the "read-only" option to the QDict. One important effect of > this change is that when a child inherits options from its parent, the > existing "read-only" mode can be preserved if it was explicitly set > previously. > > This addr

Re: [Qemu-devel] [PATCH v4] ui/cocoa.m: Make a better about dialog

2016-09-15 Thread Peter Maydell
On 16 August 2016 at 09:23, Peter Maydell wrote: > On 16 August 2016 at 03:11, Programmingkid wrote: >> The about dialog in QEMU on Mac OS X is very plain and unhelpful. This patch >> makes the about dialog look a lot better and have some descriptive >> information >> on what version of QEMU the

Re: [Qemu-devel] Adding a custom monitoring device to virt machine

2016-09-15 Thread Nassim Corteggiani
On Tue, Aug 23, 2016 at 07:31:53AM +, Nassim Corteggiani wrote: > Hi all, > > > This is the first time I am playing with QEmu. > > > I made a custom qemu device in oder to monitor all memory accesses (sdram, > flash, peripherals, gpio, ...). > > > The device is parent of TYPE_SYS_BUS_DEVICE a

Re: [Qemu-devel] [PATCH 2/7] block: Set BDRV_O_ALLOW_RDWR and snapshot_options before storing the flags

2016-09-15 Thread Alberto Garcia
On Wed 14 Sep 2016 06:40:29 PM CEST, Kevin Wolf wrote: >> +if (flags & BDRV_O_RDWR) { >> +flags |= BDRV_O_ALLOW_RDWR; >> +} >> + >> +if (flags & BDRV_O_SNAPSHOT) { >> +snapshot_options = qdict_new(); >> +bdrv_temp_snapshot_options(&snapshot_flags, snapshot_option

Re: [Qemu-devel] [PATCH v11 1/6] qdict: implement a qdict_crumple method for un-flattening a dict

2016-09-15 Thread Daniel P. Berrange
On Wed, Sep 14, 2016 at 04:18:42PM +0200, Kevin Wolf wrote: > Am 05.09.2016 um 17:16 hat Daniel P. Berrange geschrieben: > > The qdict_flatten() method will take a dict whose elements are > > further nested dicts/lists and flatten them by concatenating > > keys. > > > > The qdict_crumple() method

Re: [Qemu-devel] [PATCH v11 5/6] qapi: add a QmpInputVisitor that does string conversion

2016-09-15 Thread Daniel P. Berrange
On Wed, Sep 14, 2016 at 04:59:50PM +0200, Kevin Wolf wrote: > Am 05.09.2016 um 17:16 hat Daniel P. Berrange geschrieben: > > Currently the QmpInputVisitor assumes that all scalar > > values are directly represented as their final types. > > ie it assumes an 'int' is using QInt, and a 'bool' is > >

[Qemu-devel] [PATCH] virtio: add check for descriptor's mapped address

2016-09-15 Thread P J P
From: Prasad J Pandit virtio back end uses set of buffers to facilitate I/O operations. If its size is too large, 'cpu_physical_memory_map' could return a null address. This would result in a null dereference while un-mapping descriptors. Add check to avoid it. Reported-by: Qinghao Tang Signed-

Re: [Qemu-devel] [PATCH v11 0/6] QAPI/QOM work for non-scalar object properties

2016-09-15 Thread Daniel P. Berrange
On Wed, Sep 14, 2016 at 05:04:05PM +0200, Kevin Wolf wrote: > Am 05.09.2016 um 17:16 hat Daniel P. Berrange geschrieben: > > This patch series contains only the QAPI/QOM bits of my previous > > access control patch series: > > > > v1: https://lists.gnu.org/archive/html/qemu-devel/2016-02/msg04618

[Qemu-devel] [Bug 1536487] Re: Unable to migrate pc-i440fx-2.4 KVM guest from QEMU 2.5.0 to QEMU 2.4.1

2016-09-15 Thread ChristianEhrhardt
I backported this and tried my tests again, but this alone isn't sufficient to get the T->X->T working (which is effectively 2.0->2.5->2.0). Wily (2.4) is already out of service, so setting this to won't fix. Thanks for your guidance, but that now properly known I'll set the Xenial task to won't

Re: [Qemu-devel] [PATCH 4/7] block: Add "read-only" to the options QDict

2016-09-15 Thread Alberto Garcia
On Thu 15 Sep 2016 12:51:27 PM CEST, Kevin Wolf wrote: >> @@ -707,6 +707,9 @@ static void bdrv_inherited_options(int *child_flags, >> QDict *child_options, >> qdict_copy_default(child_options, parent_options, >> BDRV_OPT_CACHE_DIRECT); >> qdict_copy_default(child_options, parent_options

Re: [Qemu-devel] [PATCH v4 0/3] block: Fix libbz2 library dependency regresssion

2016-09-15 Thread Stefan Hajnoczi
On Wed, Sep 14, 2016 at 04:48:17PM +0200, Max Reitz wrote: > On 2016-09-14 at 16:35, Stefan Hajnoczi wrote: > > On Mon, Sep 05, 2016 at 10:50:42AM +0800, Fam Zheng wrote: > > > v4: Remove unused variable in patch 1 and unwanted warning in patch 2. > > > [Max] > > > > > > v3: Fix typo in copyright

Re: [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test

2016-09-15 Thread Greg Kurz
On Thu, 15 Sep 2016 10:40:45 +0200 Cornelia Huck wrote: > On Fri, 09 Sep 2016 10:04:51 +0200 > Greg Kurz wrote: > > > This adds PCI init code and a basic test that checks the device config > > matches what is passed on the command line. > > > > Signed-off-by: Greg Kurz > > --- > > tests/Make

Re: [Qemu-devel] [PATCH 1/2] trace: Add "cpu_init" event

2016-09-15 Thread Stefan Hajnoczi
On Wed, Sep 14, 2016 at 06:01:17PM +0200, Lluís Vilanova wrote: > Stefan Hajnoczi writes: > > > On Tue, Sep 06, 2016 at 04:25:53PM +0200, Lluís Vilanova wrote: > >> +## vCPU > >> + > >> +# Create a new virtual (guest) CPU > >> +# > >> +# Targets: all > >> +guest_cpu_init(void *cpu) "cpu=%p" > > >

Re: [Qemu-devel] [PATCH 2/7] block: Set BDRV_O_ALLOW_RDWR and snapshot_options before storing the flags

2016-09-15 Thread Kevin Wolf
Am 15.09.2016 um 13:24 hat Alberto Garcia geschrieben: > On Wed 14 Sep 2016 06:40:29 PM CEST, Kevin Wolf wrote: > >> +if (flags & BDRV_O_RDWR) { > >> +flags |= BDRV_O_ALLOW_RDWR; > >> +} > >> + > >> +if (flags & BDRV_O_SNAPSHOT) { > >> +snapshot_options = qdict_new(); >

Re: [Qemu-devel] [PATCH v2 5/6] trace: remove use of event ID enums from APIs

2016-09-15 Thread Lluís Vilanova
Daniel P Berrange writes: > On Thu, Sep 15, 2016 at 01:26:06AM +0200, Lluís Vilanova wrote: >> Daniel P Berrange writes: >> >> > Since there will shortly be multiple event groups allowed, >> > we can no longer use the TraceEventID and TraceEventVCPUID >> > enums in the trace control APIs. There w

Re: [Qemu-devel] [Xen-devel] [RFC] e1000: Don't save writes to ICS/ICR masked by IMS

2016-09-15 Thread Ed Swierk
On Thu, Sep 15, 2016 at 2:15 AM, Denis V. Lunev wrote: > On 09/13/2016 11:59 PM, Konrad Rzeszutek Wilk wrote: > > On Thu, Sep 01, 2016 at 10:57:48AM -0700, Ed Swierk wrote: > >> Windows 8, 10 and Server 2012 guests hang intermittently while booting > >> on Xen 4.5.3 with 1 vCPU and 4 e1000 vNICs,

Re: [Qemu-devel] [PATCH v2 5/6] trace: remove use of event ID enums from APIs

2016-09-15 Thread Daniel P. Berrange
On Thu, Sep 15, 2016 at 02:20:05PM +0200, Lluís Vilanova wrote: > Daniel P Berrange writes: > > > On Thu, Sep 15, 2016 at 01:26:06AM +0200, Lluís Vilanova wrote: > >> Daniel P Berrange writes: > >> > >> > Since there will shortly be multiple event groups allowed, > >> > we can no longer use the T

Re: [Qemu-devel] [PATCH 2/7] block: Set BDRV_O_ALLOW_RDWR and snapshot_options before storing the flags

2016-09-15 Thread Alberto Garcia
On Thu 15 Sep 2016 02:19:36 PM CEST, Kevin Wolf wrote: >> >> +if (flags & BDRV_O_RDWR) { >> >> +flags |= BDRV_O_ALLOW_RDWR; >> >> +} >> >> + >> >> +if (flags & BDRV_O_SNAPSHOT) { >> >> +snapshot_options = qdict_new(); >> >> +bdrv_temp_snapshot_options(&snapshot_

Re: [Qemu-devel] [PATCH 0/4] Move some trace events to the correct files

2016-09-15 Thread Stefan Hajnoczi
On Wed, Sep 14, 2016 at 06:03:40PM +0100, Daniel P. Berrange wrote: > When I split up the monolithic trace-events file, a few > events ended up in incorrect files, partly due to incorrect > file name annotations against the events, partly due to my > mistakes. > > These patches were previously pos

[Qemu-devel] [PATCH v12 0/6] QAPI/QOM work for non-scalar object properties

2016-09-15 Thread Daniel P. Berrange
This patch series contains only the QAPI/QOM bits of my previous access control patch series: v1: https://lists.gnu.org/archive/html/qemu-devel/2016-02/msg04618.html v2: https://lists.gnu.org/archive/html/qemu-devel/2016-03/msg01454.html v3: https://lists.gnu.org/archive/html/qemu-devel/2016-03

[Qemu-devel] [PATCH v12 3/6] qapi: rename QmpInputVisitor to QObjectInputVisitor

2016-09-15 Thread Daniel P. Berrange
The QmpInputVisitor has no direct dependency on QMP. It is valid to use it anywhere that one has a QObject. Rename it to better reflect its functionality as a generic QObject to QAPI converter. Reviewed-by: Eric Blake Signed-off-by: Daniel P. Berrange --- docs/qapi-code-gen.txt

[Qemu-devel] [PATCH v3 05/10] ppc/pnv: add a PnvCore object

2016-09-15 Thread Cédric Le Goater
This is largy inspired by sPAPRCPUCore with some simplification, no hotplug for instance. But the differences are small and the objects could possibly be merged. A set of PnvCore objects is added to the PnvChip and the device tree is populated looping on these cores. Real HW cpu ids are now gener

Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p

2016-09-15 Thread Greg Kurz
On Thu, 15 Sep 2016 10:41:38 +0200 Cornelia Huck wrote: > On Fri, 09 Sep 2016 10:04:37 +0200 > Greg Kurz wrote: > > > As with other virtio-* qtests, PC platform is assumed. > > > > --- > > > > Greg Kurz (3): > > tests: virtio-9p: introduce start/stop functions > > tests: virtio-9p

[Qemu-devel] [PATCH v12 6/6] qom: support arbitrary non-scalar properties with -object

2016-09-15 Thread Daniel P. Berrange
The current -object command line syntax only allows for creation of objects with scalar properties, or a list with a fixed scalar element type. Objects which have properties that are represented as structs in the QAPI schema cannot be created using -object. This is a design limitation of the way t

[Qemu-devel] [PATCH v12 4/6] qapi: rename QmpOutputVisitor to QObjectOutputVisitor

2016-09-15 Thread Daniel P. Berrange
The QmpOutputVisitor has no direct dependency on QMP. It is valid to use it anywhere that one wants a QObject. Rename it to better reflect its functionality as a generic QAPI to QObject converter. Reviewed-by: Eric Blake Signed-off-by: Daniel P. Berrange --- block/qapi.c

[Qemu-devel] [PATCH v3 08/10] ppc/pnv: add a XScomDevice to PnvCore

2016-09-15 Thread Cédric Le Goater
Now that we are using real HW ids for the cores in PowerNV chips, we can route the XSCOM accesses to them. We just need to attach a specific XSCOM memory region to each core in the appropriate window for the core number. To start with, let's install the DTS (Digital Thermal Sensor) handlers which

[Qemu-devel] [PATCH v12 2/6] option: make parse_option_bool/number non-static

2016-09-15 Thread Daniel P. Berrange
The opts-visitor.c opts_type_bool() method has code for parsing a string to set a bool value, as does the qemu-option.c parse_option_bool() method, except it handles fewer cases. To enable consistency across the codebase, extend parse_option_bool() to handle "yes", "no", "y" and "n", and make it n

[Qemu-devel] [PATCH v3 00/10] ppc/pnv: loading skiboot and booting the kernel

2016-09-15 Thread Cédric Le Goater
Hello, Here is a new version addressing all comments from v2 plus a couple of important changes : - QOM'ification of all the models. should be clean. - some P9 basic support - PnvChip has its own routine to populate the device tree - rework of the XSCOM model to use an address space. see belo

[Qemu-devel] [PATCH v3 03/10] ppc/pnv: add a core mask to PnvChip

2016-09-15 Thread Cédric Le Goater
This will be used to build real HW ids for the cores and enforce some limits on the available cores per chip. Signed-off-by: Cédric Le Goater --- Changes since v2 : - added POWER9 support - removed cores_max - introduces a pnv_chip_core_sanitize() helper to check the core ids_mask and t

[Qemu-devel] [PATCH v12 1/6] qdict: implement a qdict_crumple method for un-flattening a dict

2016-09-15 Thread Daniel P. Berrange
The qdict_flatten() method will take a dict whose elements are further nested dicts/lists and flatten them by concatenating keys. The qdict_crumple() method aims to do the reverse, taking a flat qdict, and turning it into a set of nested dicts/lists. It will apply nesting based on the key name, wi

[Qemu-devel] [PATCH v3 01/10] ppc/pnv: add skeleton PowerNV platform

2016-09-15 Thread Cédric Le Goater
From: Benjamin Herrenschmidt The goal is to emulate a PowerNV system at the level of the skiboot firmware, which loads the OS and provides some runtime services. Power Systems have a lower firmware (HostBoot) that does low level system initialization, like DRAM training. This is beyond the scope

[Qemu-devel] [PATCH v12 5/6] qapi: add a QObjectInputVisitor that does string conversion

2016-09-15 Thread Daniel P. Berrange
Currently the QObjectInputVisitor assumes that all scalar values are directly represented as their final types. ie it assumes an 'int' is using QInt, and a 'bool' is using QBool. This adds an alternative constructor for QObjectInputVisitor that will set it up such that it expects a QString for all

Re: [Qemu-devel] [Xen-devel] [RFC] e1000: Don't save writes to ICS/ICR masked by IMS

2016-09-15 Thread Denis V. Lunev
On 09/15/2016 03:22 PM, Ed Swierk wrote: > On Thu, Sep 15, 2016 at 2:15 AM, Denis V. Lunev wrote: >> On 09/13/2016 11:59 PM, Konrad Rzeszutek Wilk wrote: >>> On Thu, Sep 01, 2016 at 10:57:48AM -0700, Ed Swierk wrote: Windows 8, 10 and Server 2012 guests hang intermittently while booting

Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p

2016-09-15 Thread Cornelia Huck
On Thu, 15 Sep 2016 14:45:06 +0200 Greg Kurz wrote: > On Thu, 15 Sep 2016 10:41:38 +0200 > Cornelia Huck wrote: > > > On Fri, 09 Sep 2016 10:04:37 +0200 > > Greg Kurz wrote: > > > > > As with other virtio-* qtests, PC platform is assumed. > > > > > > --- > > > > > > Greg Kurz (3): > > >

[Qemu-devel] [PATCH v3 06/10] monitor: fix crash for platforms without a CPU 0

2016-09-15 Thread Cédric Le Goater
On PowerNV, CPU ids start at 0x8 or 0x20, we don't have a CPU 0 anymore. So let's use the first_cpu index to initialize the monitor. Signed-off-by: Cédric Le Goater --- monitor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/monitor.c b/monitor.c index 5c003731e288..0192209

Re: [Qemu-devel] [PATCH 1/2] trace: Add "cpu_init" event

2016-09-15 Thread Lluís Vilanova
Stefan Hajnoczi writes: > On Wed, Sep 14, 2016 at 06:01:17PM +0200, Lluís Vilanova wrote: >> Stefan Hajnoczi writes: >> >> > On Tue, Sep 06, 2016 at 04:25:53PM +0200, Lluís Vilanova wrote: >> >> +## vCPU >> >> + >> >> +# Create a new virtual (guest) CPU >> >> +# >> >> +# Targets: all >> >> +guest

[Qemu-devel] [PATCH v3 10/10] ppc/pnv: add a ISA bus

2016-09-15 Thread Cédric Le Goater
As Qemu only supports a single instance of the ISA bus, we use the LPC controller of chip 0 to create one and plug in a couple of useful devices, like an UART and RTC. An IPMI BT device, which is also an ISA device, can be defined on the command line to connect an external BMC. That is for later.

[Qemu-devel] [PATCH v3 07/10] ppc/pnv: add XSCOM infrastructure

2016-09-15 Thread Cédric Le Goater
On a real POWER8 system, the Pervasive Interconnect Bus (PIB) serves as a backbone to connect different units of the system. The host firmware connects to the PIB through a bridge unit, the Alter-Display-Unit (ADU), which gives him access to all the chiplets on the PCB network (Pervasive Connect Bu

Re: [Qemu-devel] [PATCH 2/7] block: Set BDRV_O_ALLOW_RDWR and snapshot_options before storing the flags

2016-09-15 Thread Alberto Garcia
On Thu 15 Sep 2016 02:50:53 PM CEST, Kevin Wolf wrote: >> Moving this chunk of code saves us from having to update >> bs->open_flags and bs->options and makes things more readable. > > I fully understand understand what the patch is for I see, I thought it wasn't clear, I'll update the message th

[Qemu-devel] [PATCH v3 04/10] ppc/pnv: add a PIR handler to PnvChip

2016-09-15 Thread Cédric Le Goater
P9 and P8 have some differences in the CPU PIR encoding. Signed-off-by: Cédric Le Goater --- hw/ppc/pnv.c | 14 ++ include/hw/ppc/pnv.h | 1 + 2 files changed, 15 insertions(+) diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c index ec7dd6ac5ea1..f4c125503249 100644 --- a/hw/ppc/pnv

[Qemu-devel] [PATCH v3 02/10] ppc/pnv: add a PnvChip object

2016-09-15 Thread Cédric Le Goater
This is is an abstraction of a POWER8 chip which is a set of cores plus other 'units', like the pervasive unit, the interrupt controller, the memory controller, the on-chip microcontroller, etc. The whole can be seen as a socket. It depends on a cpu model and its characteristics: max cores, specifi

Re: [Qemu-devel] [PATCH v5] fpu: add mechanism to check for invalid long double formats

2016-09-15 Thread Peter Maydell
On 18 August 2016 at 16:35, Peter Maydell wrote: > On 17 August 2016 at 01:14, Andrew Dutcher wrote: >> All operations that take a floatx80 as an operand need to have their >> inputs checked for malformed encodings. In all of these cases, use the >> function floatx80_invalid_encoding to perform t

  1   2   3   >