Re: [Qemu-devel] [RFC PATCH 5/6] module: load modules at start

2013-09-05 Thread Lluís Vilanova
Fam Zheng writes: Add module_load_all to load all DSO modules under: /usr/lib/qemu/block/ /usr/lib/qemu/net/ /usr/lib/qemu/ui/ when starting process. This should probably be based on a define with the prefix set at configure time. Adding directories from command-line arguments

Re: [Qemu-devel] [RFC PATCH v2 1/6] make.rule: fix $(obj) to a real relative path

2013-09-06 Thread Lluís Vilanova
Fam Zheng writes: [...] Because $(obj) here is './block', instead of '../block'. This doesn't hurt compiling because we basically build all .o from top Makefile, before entering Makefile.target, but it will affact arriving per-object libs support. I'm curious. What's the reason to not use

Re: [Qemu-devel] outlined TLB lookup on x86

2013-11-27 Thread Lluís Vilanova
Xin Tong writes: I am trying to implement a out-of-line TLB lookup for QEMU softmmu-x86-64 on x86-64 machine, potentially for better instruction cache performance, I have a few questions. 1. I see that tcg_out_qemu_ld_slow_path/tcg_out_qemu_st_slow_path are generated when

Re: [Qemu-devel] outlined TLB lookup on x86

2013-11-28 Thread Lluís Vilanova
Xin Tong writes: Hi LIuis we can probably generate vector intrinsics using the tcg, e.g. add support to tcg to emit vector instructions directly in code cache There was some discussion long ago about adding vector instructions to TCG, but I don't remember what was the conclusion. Also

Re: [Qemu-devel] QEMU Memory subsystem

2013-06-21 Thread Lluís Vilanova
Basim Baig writes: Hello, I am currently working on a project where I aim to log every memory access made by a virtual machine running inside of QEMU (for analyzing kernel behavior). My initial approach is to possibly hook into the QEMU mmu implementation and find the place where the

Re: [Qemu-devel] outlined TLB lookup on x86

2013-12-09 Thread Lluís Vilanova
Xin Tong writes: On Thu, Nov 28, 2013 at 8:12 AM, Lluís Vilanova vilan...@ac.upc.edu wrote: Xin Tong writes: Hi LIuis we can probably generate vector intrinsics using the tcg, e.g. add support to tcg to emit vector instructions directly in code cache

Re: [Qemu-devel] interrupt handling in qemu

2011-12-28 Thread Lluís Vilanova
Xin Tong writes: My main concern here is not how timely the interrupts can be handled, i am more interested in reducing the number of TB enters/exits due to interrupt. Returning to qemu mainloop requires saving and restoring register contexts which are expensive, what i am thinking is that

Re: [Qemu-devel] [RFC PATCH 0/4] simpletrace : support var num/size of args, strings.

2012-01-02 Thread Lluís Vilanova
Harsh Prateek Bora writes: Converting tracetool.sh to tracetool.py Excellent! I was thinking about switching the script language to python, but wasn't sure if python was an acceptable build dependency for the project. I was modifying the bash tracetool to add a tracing layer more amenable to

Re: [Qemu-devel] [RFC PATCH v3 1/3] Converting tracetool.sh to tracetool.py

2012-01-11 Thread Lluís Vilanova
Harsh Prateek Bora writes: Signed-off-by: Harsh Prateek Bora ha...@linux.vnet.ibm.com --- Makefile.objs|6 +- Makefile.target | 10 +- configure|7 +- scripts/tracetool| 643 -- scripts/tracetool.py

[Qemu-devel] [PATCH 1/6] trace: [tracetool] Do not rebuild event list in backend code

2012-01-11 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool.py | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index 6874f66..80e5684 100755 --- a/scripts/tracetool.py +++ b/scripts/tracetool.py

[Qemu-devel] [PATCH 2/6] trace: [tracetool] Simplify event line parsing

2012-01-11 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool.py | 46 ++ 1 files changed, 14 insertions(+), 32 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index 80e5684..7a877dc 100755 --- a/scripts/tracetool.py

[Qemu-devel] [PATCH 3/6] trace: [ŧracetool] Do not precompute the event number

2012-01-11 Thread Lluís Vilanova
This would otherwise break event numbering when actually using the disable property. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool.py | 21 + 1 files changed, 9 insertions(+), 12 deletions(-) diff --git a/scripts/tracetool.py b/scripts

[Qemu-devel] [PATCH 4/6] trace: [tracetool] Add support for event properties

2012-01-11 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index b7401a3..e3e665d 100755 --- a/scripts/tracetool.py +++ b/scripts/tracetool.py @@ -472,7 +472,7

[Qemu-devel] [PATCH 5/6] trace: [tracetool] Process the disable event property

2012-01-11 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool.py | 13 +++-- trace-events |2 +- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index e3e665d..5432f61 100755 --- a/scripts/tracetool.py

[Qemu-devel] [PATCH 6/6] trace: [tracetool] Rewrite event argument parsing

2012-01-11 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool.py | 186 -- 1 files changed, 90 insertions(+), 96 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index 5432f61..a9020de 100755 --- a/scripts

Re: [Qemu-devel] [RFC PATCH v3 1/3] Converting tracetool.sh to tracetool.py

2012-01-11 Thread Lluís Vilanova
Harsh Prateek Bora writes: Signed-off-by: Harsh Prateek Bora ha...@linux.vnet.ibm.com --- Makefile.objs|6 +- Makefile.target | 10 +- configure|7 +- scripts/tracetool| 643 -- scripts/tracetool.py

[Qemu-devel] [PATH 7/6] trace: [tracetool] Make format-specific code optional and with access to event information

2012-01-11 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool.py | 35 ++- 1 files changed, 10 insertions(+), 25 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index a9020de..ddf4c31 100755 --- a/scripts/tracetool.py +++ b

Re: [Qemu-devel] [PATCH 1/6] trace: [tracetool] Do not rebuild event list in backend code

2012-01-18 Thread Lluís Vilanova
Harsh Bora writes: On 01/11/2012 11:35 PM, Lluís Vilanova wrote: Signed-off-by: Lluís Vilanovavilan...@ac.upc.edu --- scripts/tracetool.py | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index 6874f66

Re: [Qemu-devel] [PATCH 1/6] trace: [tracetool] Do not rebuild event list in backend code

2012-01-18 Thread Lluís Vilanova
Lluís Vilanova writes: Harsh Bora writes: Hi Lluis, This looks really nice. I can include your fixes while addressing other review comments. Shall I fold your patches with mine or do you want to keep them separate? Whatever works best for you. BTW, I did some more changes, but didn't

Re: [Qemu-devel] Removing indeterminism in qemu execution

2012-01-23 Thread Lluís Vilanova
batuzovk writes: I'm debugging an operating system with QEMU and I have a race condition in the OS. The problem is that each time I run QEMU I get this error in a different place, so it makes impossible for gdb to debug it. My plan is to remove this indeterminism and be able to reproduce the

[Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-05 Thread Lluís Vilanova
Provides the ability for the guest to communicate with user-provided code inside QEMU itself, using a lightweight mechanism. See first commit for a full description. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Changes in v2: * Rebase on 1c8a881daaca6fe0646a425b0970fb3ad25f6732 from

[Qemu-devel] [PATCH v2 1/5] backdoor: Add documentation

2011-12-05 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/backdoor.txt | 167 + 1 files changed, 167 insertions(+), 0 deletions(-) create mode 100644 docs/backdoor.txt diff --git a/docs/backdoor.txt b/docs/backdoor.txt new file mode 100644

[Qemu-devel] [PATCH v2 2/5] backdoor: Add build infrastructure

2011-12-05 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile |4 +++- Makefile.objs | 18 ++ Makefile.target |4 +++- backdoor/qemu/qemu-backdoor.h | 31 +++ configure

[Qemu-devel] [PATCH v2 3/5] backdoor: [*-user] Add QEMU-side proxy to libbackdoor.a

2011-12-05 Thread Lluís Vilanova
QEMU detects when the guest uses 'mmap' on the control channel file, and then uses 'mprotect' to detect accesses to it, which are redirected to the user-provided libbackdoor.a. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs|2 backdoor/qemu/user.c | 202

[Qemu-devel] [PATCH v2 4/5] backdoor: [softmmu] Add QEMU-side proxy to libbackdoor.a

2011-12-05 Thread Lluís Vilanova
Uses a virtual device to proxy uses of the backdoor communication channel to the user-provided code. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |1 backdoor/qemu/softmmu.c | 186 +++ hw/pci.h

[Qemu-devel] [PATCH v2 5/5] backdoor: Add guest-side library

2011-12-05 Thread Lluís Vilanova
Guest applications can link against libqemu-backdoor-guest.a to use the backdoor communication channel. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- backdoor/guest/Makefile| 18 backdoor/guest/common.c| 205 backdoor/guest

Re: [Qemu-devel] [PATCH v3 1/3] Make 'qemu_init_vcpu' a function (instead of a macro)

2011-12-06 Thread Lluís Vilanova
Lluís Vilanova writes: Implementation with 'CONFIG_USER_ONLY' is moved into new file 'cpus-user.c'. Ping Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |1 + cpus-user.c | 15 +++ qemu-common.h |4 3 files changed, 16 insertions(+), 4

[Qemu-devel] [PATCH v2] trace: Provide a per-event status define for conditional compilation

2011-12-06 Thread Lluís Vilanova
will probably have the disable property by default, in order to avoid such costs on regular builds. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt | 46 -- scripts/tracetool |9 - 2 files changed, 48 insertions

Re: [Qemu-devel] [PATCH v2 4/5] backdoor: [softmmu] Add QEMU-side proxy to libbackdoor.a

2011-12-06 Thread Lluís Vilanova
Anthony Liguori writes: I really worry about us introducing so many of these one-off paravirtual devices. I would much prefer that you look at doing this as an extension to the ivshmem device as it already has this sort of scope. You should be able to do this by just extending the size of

Re: [Qemu-devel] [PATCH v2 4/5] backdoor: [softmmu] Add QEMU-side proxy to libbackdoor.a

2011-12-06 Thread Lluís Vilanova
Lluís Vilanova writes: Anthony Liguori writes: I really worry about us introducing so many of these one-off paravirtual devices. I would much prefer that you look at doing this as an extension to the ivshmem device as it already has this sort of scope. You should be able to do

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-07 Thread Lluís Vilanova
Anthony Liguori writes: On 12/05/2011 04:22 PM, Lluís Vilanova wrote: Provides the ability for the guest to communicate with user-provided code inside QEMU itself, using a lightweight mechanism. See first commit for a full description. Signed-off-by: Lluís Vilanovavilan...@ac.upc.edu

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-07 Thread Lluís Vilanova
Anthony Liguori writes: [...] If you want to extend QEMU, then send proper patches. Adding random C files to the build is unacceptable. What do you mean by proper patches? If you want to add custom functionality to QEMU, send a patch to upstream QEMU. See below. The whole point of

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-07 Thread Lluís Vilanova
Anthony Liguori writes: Well, both backdoor and trace instrumentation are implemented using the same approach (a static library selected at compile-time). The user sets which events to instrument in the trace-events file. This has the effect that the tracetool script will not generate the

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-07 Thread Lluís Vilanova
Anthony Liguori writes: [...] Why should this analyzer live outside of QEMU in the first place? I fail to see the rationale for that other than not wanting to do the work of making it suitable for upstream consumption. For the same reason that SystemTap lets you add user-provided code into

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-07 Thread Lluís Vilanova
Anthony Liguori writes: On 12/07/2011 12:51 PM, Peter Maydell wrote: 2011/12/7 Lluís Vilanovavilan...@ac.upc.edu: Anthony Liguori writes: [...] Why should this analyzer live outside of QEMU in the first place? I fail to see the rationale for that other than not wanting to do the work of

[Qemu-devel] [RFC][PATCH RFC 0/4] trace-instrument: Let the user wrap/override specific event tracing routines

2011-12-07 Thread Lluís Vilanova
tracing functions are expected to be in the static library libtrace-instrument.a, identified by the --with-trace-instrument configuration parameter and compiled by each of QEMU's target/frontend pairs. See the documentation added in the second patch for more information. Signed-off-by: Lluís Vilanova

[Qemu-devel] [PATCH RFC 1/4] trace: [tracetool] Add 'get_api_name' to construct the name of tracing routines

2011-12-07 Thread Lluís Vilanova
future tracetool options to change the naming of the tracing API on a per-event basis without having to modify backend-specific code. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool | 41 + 1 files changed, 29 insertions(+), 12

[Qemu-devel] [PATCH RFC 2/4] trace-instrument: Add documentation

2011-12-07 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/instrumentation.txt | 142 ++ docs/tracing.txt |9 +++ 2 files changed, 151 insertions(+), 0 deletions(-) create mode 100644 docs/instrumentation.txt diff --git a/docs

[Qemu-devel] [PATCH RFC 3/4] trace-instrument: Add support for user-provided code on a per-event basis

2011-12-07 Thread Lluís Vilanova
Adds support for the 'instrument' propery into the trace-events file. Events with this property will generate tracing routines named '${api_name}_backend' (instead of '${api_name}'), and expect the user to provide its own implementation for the '${api_name}' routines. Signed-off-by: Lluís

[Qemu-devel] [PATCH RFC 4/4] trace-instrument: Add a per-vCPU opaque pointer for the instrumentation backend

2011-12-07 Thread Lluís Vilanova
The 'instrument' opaque pointer can be used by the user-provided trace instrumentation backend to hold arbitrary data. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- cpu-defs.h |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/cpu-defs.h b/cpu-defs.h index

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-07 Thread Lluís Vilanova
Lluís Vilanova writes: [...] From my experience, these are the basics I need: * Decide what to do when an event is translated (by default - no instrumentation -, it just generates a TCG call to an execution-time tracing routine). This includes: * Deciding whether to generate

Re: [Qemu-devel] [PATCH v2] trace: Provide a per-event status define for conditional compilation

2011-12-08 Thread Lluís Vilanova
Stefan Hajnoczi writes: 2011/12/6 Lluís Vilanova vilan...@ac.upc.edu: Adds a 'TRACE_${NAME}_ENABLED' preprocessor define for each tracing event in trace.h. I think we should take it a step further: support TRACE_${NAME}_ENABLED at run-time. This means skipping the trace event code

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-08 Thread Lluís Vilanova
Stefan Hajnoczi writes: [...] * Support for tracing guest code through TCG. I'm not clear on whether a backdoor mechanism is needed or not. A backdoor mechanism allows a modified guest to participate in tracing. Depending on the type of analysis you are doing it might be possible to

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-08 Thread Lluís Vilanova
Blue Swirl writes: 2011/12/8 Lluís Vilanova vilan...@ac.upc.edu: Stefan Hajnoczi writes: [...] * Support for tracing guest code through TCG. I'm not clear on whether a backdoor mechanism is needed or not.  A backdoor mechanism allows a modified guest to participate in tracing

[Qemu-devel] [RFC][PATCH 00/10] trace-tcg: Allow tracing guest events in TCG-generated code

2011-12-08 Thread Lluís Vilanova
casts to match the signature of the callee. NOTE: This patch series has been stripped out of all trace instrumentation features, and is thus applicable on top of QEMU 1.0 Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Lluís Vilanova (10): trace: [doc] Document event properties

[Qemu-devel] [PATCH 01/10] trace: [doc] Document event properties on a separate section

2011-12-08 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt | 22 -- 1 files changed, 16 insertions(+), 6 deletions(-) diff --git a/docs/tracing.txt b/docs/tracing.txt index ea29f2c..853cbf8 100644 --- a/docs/tracing.txt +++ b/docs/tracing.txt @@ -98,12 +98,6

[Qemu-devel] [PATCH 02/10] trace-tcg: Add documentation

2011-12-08 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt | 40 1 files changed, 40 insertions(+), 0 deletions(-) diff --git a/docs/tracing.txt b/docs/tracing.txt index 853cbf8..44c5dba 100644 --- a/docs/tracing.txt +++ b/docs/tracing.txt

[Qemu-devel] [PATCH 03/10] Trivial changes to eliminate auto-generated files

2011-12-08 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile | 11 --- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 301c75e..506db1b 100644 --- a/Makefile +++ b/Makefile @@ -3,10 +3,14 @@ # Always point to the root of the build tree (needs

[Qemu-devel] [PATCH 04/10] [m68k, s390, xtensa] Move helpers.h to helper.h

2011-12-08 Thread Lluís Vilanova
This provides a consistent file naming scheme across all targets. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-m68k/helper.c |2 - target-m68k/helper.h | 54 target-m68k/helpers.h | 54 target-m68k/op_helper.c |2

[Qemu-devel] [PATCH 05/10] trace: [tracetool] Common functions to manage event arguments

2011-12-08 Thread Lluís Vilanova
transformation capabilities of 'get_argtypes'. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool | 117 + 1 files changed, 116 insertions(+), 1 deletions(-) diff --git a/scripts/tracetool b/scripts/tracetool index 4c9951d

[Qemu-devel] [PATCH 06/10] trace: [tracetool] Add 'get_api_name' to construct the name of tracing routines

2011-12-08 Thread Lluís Vilanova
tracing backends. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- scripts/tracetool | 40 1 files changed, 28 insertions(+), 12 deletions(-) diff --git a/scripts/tracetool b/scripts/tracetool index f2b6680..853f1bd 100755 --- a/scripts/tracetool

[Qemu-devel] [PATCH 07/10] trace-tcg: [tracetool] Allow TCG types in trace event declarations

2011-12-08 Thread Lluís Vilanova
When found, TCG types are translated into the host native types when declaring and defining the tracing routines in trace.h and trace.c. --- scripts/tracetool | 31 +++ 1 files changed, 23 insertions(+), 8 deletions(-) diff --git a/scripts/tracetool

[Qemu-devel] [PATCH 08/10] trace-tcg: [tracetool] Declare TCG tracing helper routines

2011-12-08 Thread Lluís Vilanova
Generate the necessary TCG helper declarations for tracing events in guest code. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- .gitignore |1 Makefile |2 Makefile.objs |9 ++ def-helper.h |9 ++ scripts/simpletrace.py |2

[Qemu-devel] [PATCH 10/10] trace-tcg: [all] Include TCG-tracing helpers

2011-12-08 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-alpha/helper.h |2 ++ target-arm/helper.h|2 ++ target-cris/helper.h |2 ++ target-i386/helper.h |2 ++ target-lm32/helper.h |2 ++ target-m68k/helper.h |2 ++ target

[Qemu-devel] [PATCH 09/10] trace-tcg: [tracetool] Define TCG tracing helper routines

2011-12-08 Thread Lluís Vilanova
Generate the necessary TCG helper routines for tracing events in guest code. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- .gitignore|1 + Makefile |2 +- Makefile.objs |8 +++ Makefile.target |2 ++ scripts/tracetool | 65

Re: [Qemu-devel] [PATCH 01/10] trace: [doc] Document event properties on a separate section

2011-12-09 Thread Lluís Vilanova
This series must be updated to apply on top of the patch recently accepted by Stefan into the tracing tree [1]. [1] http://lists.gnu.org/archive/html/qemu-devel/2011-12/msg00763.html Lluis Lluís Vilanova writes: Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt | 22

[Qemu-devel] [PATCH 0/7] trace: Add some simple TCG tracing events

2011-12-09 Thread Lluís Vilanova
in the list). Depends on the trace-tcg series. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Lluís Vilanova (7): Make 'qemu_init_vcpu' a function (instead of a macro) trace: Add vcpu_init event trace: Add vcpu_reset event trace: [all] Add vbbl TCG tracing event

[Qemu-devel] [PATCH 1/7] Make 'qemu_init_vcpu' a function (instead of a macro)

2011-12-09 Thread Lluís Vilanova
Implementation with 'CONFIG_USER_ONLY' is moved into new file 'cpus-user.c'. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |1 + cpus-user.c | 15 +++ qemu-common.h |4 3 files changed, 16 insertions(+), 4 deletions(-) create mode 100644 cpus

[Qemu-devel] [PATCH 2/7] trace: Add vcpu_init event

2011-12-09 Thread Lluís Vilanova
Signals the creation of a new vCPU (CPUState structure). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- cpus-user.c |2 ++ cpus.c |2 ++ trace-events |7 +++ 3 files changed, 11 insertions(+), 0 deletions(-) diff --git a/cpus-user.c b/cpus-user.c index 50e45ce

[Qemu-devel] [PATCH 3/7] trace: Add vcpu_reset event

2011-12-09 Thread Lluís Vilanova
Signals the reset of the state a vCPU (CPUState structure). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-arm/helper.c |3 +++ target-cris/translate.c |3 +++ target-i386/helper.c |3 +++ target-lm32/helper.c |3 +++ target-m68k

[Qemu-devel] [PATCH 4/7] trace: [all] Add vbbl TCG tracing event

2011-12-09 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-alpha/translate.c |1 + target-arm/translate.c|1 + target-cris/translate.c |1 + target-i386/translate.c |1 + target-lm32/translate.c |1 + target-m68k/translate.c |1

[Qemu-devel] [PATCH 5/7] [all] Trivial 'tcg_gen_debug_insn_start' unification in 'gen_intermediate_code_internal'

2011-12-09 Thread Lluís Vilanova
Make sure 'tcg_gen_debug_insn_start' is always called in 'gen_intermediate_code_internal' and after calls to 'gen_io_start' (before the instruction decoding). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-alpha/translate.c | 11 ++- target-cris/translate.c

[Qemu-devel] [PATCH 6/7] trace: [all] Add vfetch TCG tracing event

2011-12-09 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-alpha/translate.c |2 ++ target-arm/translate.c|2 ++ target-cris/translate.c |2 ++ target-i386/translate.c |2 ++ target-lm32/translate.c |2 ++ target-m68k/translate.c |2

[Qemu-devel] [PATCH 7/7] trace: [all] Add vmem TCG tracing event

2011-12-09 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- cpu-all.h | 58 +++-- exec-all.h |3 ++ softmmu_header.h| 15 ++ tcg/tcg-op.h|6 trace-events|9 ++ trace/tcg-op

Re: [Qemu-devel] [PATCH v2 0/5] backdoor: lightweight guest-to-QEMU backdoor channel

2011-12-09 Thread Lluís Vilanova
Stefan Hajnoczi writes: 2011/12/8 Lluís Vilanova vilan...@ac.upc.edu: Stefan Hajnoczi writes: [...] * Support for tracing guest code through TCG. I'm not clear on whether a backdoor mechanism is needed or not.  A backdoor mechanism allows a modified guest to participate in tracing

Re: [Qemu-devel] Slow kernel/initrd loading via fw_cfg; Was Re: Hack integrating SeaBios / LinuxBoot option rom with QEMU trace backends

2011-10-16 Thread Lluís Vilanova
Richard W M Jones writes: On Sat, Oct 15, 2011 at 10:00:02AM +, Blue Swirl wrote: I don't understand why PC can't use the same way of loading initrd by QEMU to guest memory before boot as Sparc32 uses. It should even be possible to deduplicate the kernel and initrd images: improve the

Re: [Qemu-devel] GSoC mentor summit QEMU users session

2011-11-04 Thread Lluís Vilanova
Stefan Hajnoczi writes: On Thu, Nov 03, 2011 at 10:35:28AM +0100, Fabien Chouteau wrote: On 03/11/2011 08:44, Stefan Hajnoczi wrote: On Wed, Nov 2, 2011 at 5:39 PM, Fabien Chouteau chout...@adacore.com wrote: On 29/10/2011 15:52, Alexander Graf wrote: I took a quick peak at the

Re: [Qemu-devel] GSoC mentor summit QEMU users session

2011-11-07 Thread Lluís Vilanova
Fabien Chouteau writes: On 04/11/2011 19:45, Lluís Vilanova wrote: I've only had a brief look into the changes, but I think the mechanism I implemented has a cleaner fit into QEMU, thanks to previous feedback from this list. I don't know about your implementation, can you give more

Re: [Qemu-devel] GSoC mentor summit QEMU users session

2011-11-07 Thread Lluís Vilanova
Fabien Chouteau writes: The process is basically: * Add trace events that can work during TCG code generation (e.g., start TB, start instruction fetch, memory access, etc.) * Let the user select which trace events to instrument, including both regular trace events and TCG trace events

[Qemu-devel] [PATCH] build: [ŧrace] Create trace/ directory for out-of-tree builds

2011-09-01 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- configure |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 4d22470..9f3a43d 100755 --- a/configure +++ b/configure @@ -3618,6 +3618,7 @@ DIRS=$DIRS roms/seabios roms/vgabios DIRS=$DIRS fsdev

Re: [Qemu-devel] [PATCH] build: sort objects to remove duplicates for link

2011-09-01 Thread Lluís Vilanova
Anthony Liguori writes: On 08/25/2011 03:18 AM, Stefan Hajnoczi wrote: Avoid duplicate object files during the link. There are legitimate cases where a link command-line would include duplicate object files because two independent subsystems both depend on common infrastructure. Use GNU

Re: [Qemu-devel] [PULL 00/15] Tracing patches

2011-09-02 Thread Lluís Vilanova
Anthony Liguori writes: On 09/02/2011 04:39 AM, Stefan Hajnoczi wrote: On Thu, Sep 1, 2011 at 8:08 PM, Anthony Liguorialigu...@us.ibm.com wrote: On 09/01/2011 03:06 AM, Stefan Hajnoczi wrote: The following changes since commit f0fb8b7180fdcf536ea635a0720e1496110ecb3b: Merge branch

Re: [Qemu-devel] [PATCH 4/9] runstate_set(): Check for valid transitions

2011-09-06 Thread Lluís Vilanova
Luiz Capitulino writes: On Tue, 06 Sep 2011 17:55:12 +0200 Jan Kiszka jan.kis...@siemens.com wrote: On 2011-09-06 15:14, Luiz Capitulino wrote: This commit could have been folded with the previous one, however doing it separately will allow for easy bisect and revert if needed.

Re: [Qemu-devel] trace build breakage with --enable-user-pie

2011-09-14 Thread Lluís Vilanova
Richard Henderson writes: The build with --enable-user-pie has recently broken: LINK i386-linux-user/qemu-i386 /usr/bin/ld: ../trace/control.o: relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC ../trace/control.o: could

Re: [Qemu-devel] trace build breakage with --enable-user-pie

2011-09-14 Thread Lluís Vilanova
Richard Henderson writes: On 09/14/2011 02:00 PM, Lluís Vilanova wrote: Strange, I tried with vanilla 44520db1 and it's compiling fine... $ ./configure --target-list=x86_64-linux-user i386-linux-user See $SUBJECT and add that configure option. Ugh... I'm sorry. I'll look at it tomorrow

[Qemu-devel] [PATCH] build: Move tracing objects into libuser on usermode emulation targets

2011-09-15 Thread Lluís Vilanova
This will apply libuser-specific compilation flags (like the ones added by --enable-user-pie), but keep softmmu emulation targets as-is. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |1 + Makefile.target |2 +- Makefile.user |4 +++- configure |1

Re: [Qemu-devel] [PATCH] build: Move tracing objects into libuser on usermode emulation targets

2011-09-15 Thread Lluís Vilanova
Richard Henderson writes: On 09/15/2011 08:28 AM, Lluís Vilanova wrote: This will apply libuser-specific compilation flags (like the ones added by --enable-user-pie), but keep softmmu emulation targets as-is. This is missing some dependencies. make[1]: *** No rule to make target `config

Re: [Qemu-devel] [PATCH] build: Move tracing objects into libuser on usermode emulation targets

2011-09-15 Thread Lluís Vilanova
Lluís Vilanova writes: Richard Henderson writes: On 09/15/2011 08:28 AM, Lluís Vilanova wrote: This will apply libuser-specific compilation flags (like the ones added by --enable-user-pie), but keep softmmu emulation targets as-is. This is missing some dependencies. make[1]: *** No rule

[Qemu-devel] [PATCH] build: Move tracing objects into libuser on usermode emulation targets

2011-09-15 Thread Lluís Vilanova
This will apply libuser-specific compilation flags (like the ones added by --enable-user-pie), but keep softmmu emulation targets as-is. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |7 --- Makefile.target |2 +- Makefile.user |4 +++- configure

[Qemu-devel] [PATCH 1/2] build: Always have 'BUILD_DIR' variable pointing to the root of the build tree

2011-09-15 Thread Lluís Vilanova
--- Makefile | 11 +++ 1 files changed, 7 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index 7e9382f..97b26c6 100644 --- a/Makefile +++ b/Makefile @@ -1,5 +1,8 @@ # Makefile for QEMU. +# Always point to the root of the build tree +BUILD_DIR=$(PWD) +

[Qemu-devel] [PATCH 2/2] build: Move tracing objects into libuser on usermode emulation targets

2011-09-15 Thread Lluís Vilanova
This will apply libuser-specific compilation flags (like the ones added by --enable-user-pie), but keep softmmu emulation targets as-is. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |7 --- Makefile.target |2 +- Makefile.user |4 +++- configure

Re: [Qemu-devel] [PATCH] build: Move tracing objects into libuser on usermode emulation targets

2011-09-15 Thread Lluís Vilanova
Please ignore this one; sorry. -- And it's much the same thing with knowledge, for whenever you learn something new, the whole world becomes that much richer. -- The Princess of Pure Reason, as told by Norton Juster in The Phantom Tollbooth

[Qemu-devel] [PATCH 1/2] build: Always have 'BUILD_DIR' variable pointing to the root of the build tree

2011-09-15 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/Makefile b/Makefile index 7e9382f..57cc399 100644 --- a/Makefile +++ b/Makefile @@ -1,5 +1,8 @@ # Makefile for QEMU. +# Always point to the root

[Qemu-devel] [PATCH 2/2] build: Move tracing objects into libuser on usermode emulation targets

2011-09-15 Thread Lluís Vilanova
This will apply libuser-specific compilation flags (like the ones added by --enable-user-pie), but keep softmmu emulation targets as-is. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- Makefile.objs |7 --- Makefile.target |2 +- Makefile.user |4 +++- configure

[Qemu-devel] [PATCH 1/3] trace: Fix print format for mipsnet_write

2011-09-16 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- trace-events |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/trace-events b/trace-events index c09399f..9d1fbbb 100644 --- a/trace-events +++ b/trace-events @@ -454,7 +454,7 @@ milkymist_vgafb_memory_write(uint32_t

[Qemu-devel] [PATCH 2/3] trace: [ust] Do not use 'm' in event argument names (used by ust macros)

2011-09-16 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- trace-events |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/trace-events b/trace-events index 9d1fbbb..b653d70 100644 --- a/trace-events +++ b/trace-events @@ -418,8 +418,8 @@ milkymist_pfpu_pulse_irq(void) Pulse

[Qemu-devel] [PATCH 3/3] trace: [ust] Do not use zero-length format strings (avoid compiler warning)

2011-09-16 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- trace-events |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/trace-events b/trace-events index b653d70..7b2e619 100644 --- a/trace-events +++ b/trace-events @@ -480,7 +480,7 @@ qemu_coroutine_yield(void *from, void

[Qemu-devel] [PATCH 1/3] Add qemu_reset_vcpu

2011-09-16 Thread Lluís Vilanova
Mimics the already-existing qemu_init_vcpu. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- qemu-common.h |2 ++ target-arm/helper.c |2 ++ target-cris/translate.c |2 ++ target-i386/helper.c |2 ++ target-lm32/helper.c

[Qemu-devel] [PATCH 2/3] trace: Add qemu_init_vcpu event

2011-09-16 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- cpus.c|1 + qemu-common.h |4 +++- trace-events |3 +++ 3 files changed, 7 insertions(+), 1 deletions(-) diff --git a/cpus.c b/cpus.c index 54c188c..8f7ce15 100644 --- a/cpus.c +++ b/cpus.c @@ -861,6 +861,7 @@ void

[Qemu-devel] [PATCH 3/3] trace: Add qemu_reset_vcpu event

2011-09-16 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- qemu-common.h |2 +- trace-events |1 + 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/qemu-common.h b/qemu-common.h index cdb6b3b..2915ccf 100644 --- a/qemu-common.h +++ b/qemu-common.h @@ -304,7 +304,7 @@ struct

Re: [Qemu-devel] [PATCH 1/3] Add qemu_reset_vcpu

2011-09-16 Thread Lluís Vilanova
Jan Kiszka writes: On 2011-09-16 19:01, Lluís Vilanova wrote: Mimics the already-existing qemu_init_vcpu. Bad name as it implies something important happens inside. Call it trace_reset_vcpu because that is what it will do. I just thought it is much cleaner to call a generic qemu_reset_vcpu

Re: [Qemu-devel] [PATCH 2/3] trace: [ust] Do not use 'm' in event argument names (used by ust macros)

2011-09-20 Thread Lluís Vilanova
Stefan Hajnoczi writes: On Fri, Sep 16, 2011 at 06:59:38PM +0200, Lluís Vilanova wrote: Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- trace-events |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/trace-events b/trace-events index 9d1fbbb..b653d70 100644

Re: [Qemu-devel] [ltt-dev] [PATCH 2/3] trace: [ust] Do not use 'm' in event argument names (used by ust macros)

2011-09-20 Thread Lluís Vilanova
Stefan Hajnoczi writes: AFAICT the only problem with libust is the __tp_cb_data set but not used warning that gcc 4.6 emits, see my test program: Here's a prettified version of the failing (pre-processed) code: static inline void trace_ust_g_malloc(size_t size, void* ptr) { do {

[Qemu-devel] [PATCH] trace: Update docs to use example events that exist

2011-09-20 Thread Lluís Vilanova
The events 'qemu_malloc' and 'qemu_free' used in the examples no longer exist, so use 'qemu_vmalloc' and 'qemu_vfree' instead. Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- docs/tracing.txt | 16 +--- 1 files changed, 9 insertions(+), 7 deletions(-) diff --git a/docs

[Qemu-devel] [PATCH 1/2] trace: Add vcpu_init event

2011-09-20 Thread Lluís Vilanova
Signals the creation of a new vCPU (CPUState structure). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- cpus.c|1 + qemu-common.h |4 +++- trace-events |7 +++ 3 files changed, 11 insertions(+), 1 deletions(-) diff --git a/cpus.c b/cpus.c index 54c188c..ab699bb

[Qemu-devel] [PATCH 2/2] trace: Add vcpu_reset event

2011-09-20 Thread Lluís Vilanova
Signals the reset of the state a vCPU (CPUState structure). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-arm/helper.c |3 +++ target-cris/translate.c |3 +++ target-i386/helper.c |3 +++ target-lm32/helper.c |3 +++ target-m68k

[Qemu-devel] [PATCH 2/3] trace: Add vcpu_init event

2011-09-21 Thread Lluís Vilanova
Signals the creation of a new vCPU (CPUState structure). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- cpus.c|1 + qemu-common.h |3 +++ trace-events |7 +++ 3 files changed, 11 insertions(+), 0 deletions(-) diff --git a/cpus.c b/cpus.c index 8978779..2b4f9b8

[Qemu-devel] [PATCH 3/3] trace: Add vcpu_reset event

2011-09-21 Thread Lluís Vilanova
Signals the reset of the state a vCPU (CPUState structure). Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- target-arm/helper.c |3 +++ target-cris/translate.c |3 +++ target-i386/helper.c |3 +++ target-lm32/helper.c |3 +++ target-m68k

[Qemu-devel] [PATCH 1/3] Make 'qemu_init_vcpu' a function (instead of a macro)

2011-09-21 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova vilan...@ac.upc.edu --- qemu-common.h |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/qemu-common.h b/qemu-common.h index 404c421..fca101e 100644 --- a/qemu-common.h +++ b/qemu-common.h @@ -297,7 +297,9 @@ struct qemu_work_item

Re: [Qemu-devel] [PATCH 1/3] Make 'qemu_init_vcpu' a function (instead of a macro)

2011-09-21 Thread Lluís Vilanova
Please ignore these patches. Implementation of 'qemu_init_vcpu' in CONFIG_USER_ONLY must be moved into a C file to avoid an include loop between trace.h and qemu-common.h Ideas on which file this is supposed to go? Lluis Lluís Vilanova writes: Signed-off-by: Lluís Vilanova vilan

  1   2   3   4   5   6   7   8   9   10   >