Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
bsd-user/main.c |1 +
include/qom/cpu.h|6 +
linux-user/main.c|1 +
qom/cpu.c
Homogenizes the command capabilities with QMP.
Signed-off-by: Lluís Vilanova
Reviewed-by: Eric Blake
Reviewed-by: Markus Armbruster
---
hmp-commands-info.hx |8 +---
hmp.h|1 +
monitor.c| 31 ++-
3 files changed, 36
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index 5e0a5ab..c5f5f3d 100644
--- a/trace/control-internal.h
+++ b/trace
t;trace: Per-vCPU tracing states".
* Simplify event state initialization.
* Simplify logic deciding which events are treated by this patch (previously,
execution-time events with 'tcg' and 'vcpu' properties; now it's simply events
with the 'vcpu' pro
Eric Blake writes:
> On 07/04/2016 03:41 AM, Lluís Vilanova wrote:
>> Signed-off-by: Lluís Vilanova
>> Reviewed-by: Stefan Hajnoczi
>> ---
>> hmp-commands-info.hx |6 +-
>> hmp-commands.hx |7 +-
>> monitor.c| 17 +-
>> q
Eliminates a future compilation error when UI code includes the tracing
headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
GLib's i18n '_' macro.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
disas/alpha.c |6
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
hmp-commands-info.hx |6 +-
hmp-commands.hx |7 +-
monitor.c| 17 +-
qapi/trace.json | 32 +--
qmp-commands.hx | 35 +++-
trace/qmp.c | 148
Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
bsd-user/main.c |1 +
include/qom/cpu.h|6 +
linux-user/main.c|1 +
qom/cpu.c
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index 5e0a5ab..c5f5f3d 100644
--- a/trace/control-internal.h
+++ b/trace
Events with the 'vcpu' property are conditionally emitted according to
their per-vCPU state. Other events are emitted normally based on their
global tracing state.
Note that the per-vCPU condition check applies to all tracing backends.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefa
Homogenizes the command capabilities with QMP.
Signed-off-by: Lluís Vilanova
Reviewed-by: Eric Blake
Reviewed-by: Markus Armbruster
---
hmp-commands-info.hx |8 +---
hmp.h|1 +
monitor.c| 31 ++-
3 files changed, 36
A new event attribute 'cpu_id' is added to have a separate ID
space ('TRACE_VCPU_*') for all events with the 'vcpu' property.
These are later used to identify which events are enabled on each vCPU.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
this patch (previously,
execution-time events with 'tcg' and 'vcpu' properties; now it's simply events
with the 'vcpu' property).
* Make tracing backends comply with the per-vCPU tracing state.
Signed-off-by: Lluís Vilanova
---
Lluís Vilanova (7):
trace:
Markus Armbruster writes:
> Lluís Vilanova writes:
>> Signed-off-by: Lluís Vilanova
>> Reviewed-by: Stefan Hajnoczi
>> ---
>> hmp-commands-info.hx |6 +-
>> hmp-commands.hx |7 +-
>> monitor.c| 17 +-
>> qapi/trace.js
Stefan Hajnoczi writes:
> On Wed, Jun 22, 2016 at 12:04:35PM +0200, Lluís Vilanova wrote:
>> @@ -4047,6 +4064,12 @@ static const struct qemu_argument arg_table[] = {
>> "", "log system calls"},
>> {"seed", "QEMU_RAND_SEED&qu
Ayaz Akram writes:
> Hi All,
> I want to count number of host instructions (only for guest code) executed
> when qemu emulates an application. I wonder if helper functions are
> supported with tcg back end as well, which can be a possible solution for
> the mentioned problem ? If not, is there an
Emilio G Cota writes:
[...]
> - What to do when atomic ops are used on something other than RAM?
> Should we have a "slow path" that is not atomic for these cases, or
> it's OK to assume code is bogus? For now, I just wrote XXX.
[...]
You mean, for example, on I/O space? In these cases, it dep
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
hmp-commands-info.hx |6 +-
hmp-commands.hx |7 +-
monitor.c| 17 +-
qapi/trace.json | 32 +--
qmp-commands.hx | 35 +++-
trace/qmp.c | 148
Homogenizes the command capabilities with QMP.
Signed-off-by: Lluís Vilanova
Reviewed-by: Eric Blake
---
hmp-commands-info.hx |8 +---
hmp.h|1 +
monitor.c| 31 ++-
3 files changed, 36 insertions(+), 4 deletions(-)
diff
Eliminates a future compilation error when UI code includes the tracing
headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
GLib's i18n '_' macro.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
disas/alpha.c |6
A new event attribute 'cpu_id' is added to have a separate ID
space ('TRACE_VCPU_*') for all events with the 'vcpu' property.
These are later used to identify which events are enabled on each vCPU.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
bsd-user/main.c |1 +
include/qom/cpu.h|6 +
linux-user/main.c|1 +
qom/cpu.c
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index 5e0a5ab..c5f5f3d 100644
--- a/trace/control-internal.h
+++ b/trace
Events with the 'vcpu' property are conditionally emitted according to
their per-vCPU state. Other events are emitted normally based on their
global tracing state.
Note that the per-vCPU condition check applies to all tracing backends.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefa
anges in v1
=
* Rebase on 1b16240.
* Split from v4 of "trace: Per-vCPU tracing states".
* Simplify event state initialization.
* Simplify logic deciding which events are treated by this patch (previously,
execution-time events with 'tcg' and 'vcpu' properties; now
Eric Blake writes:
> On 06/23/2016 09:21 AM, Lluís Vilanova wrote:
>> Homogeinizes the command capabilities with QMP.
> s/Homogeinizes/homogenizes/
>>
>> Signed-off-by: Lluís Vilanova
>> ---
>> hmp-commands-info.hx |8 +---
>>
Events with the 'vcpu' property are conditionally emitted according to
their per-vCPU state. Other events are emitted normally based on their
global tracing state.
Note that the per-vCPU condition check applies to all tracing backends.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefa
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
hmp-commands-info.hx |6 +-
hmp-commands.hx |7 +-
monitor.c| 17 +-
qapi/trace.json | 32 +--
qmp-commands.hx | 35 +++-
trace/qmp.c | 148
"trace: Per-vCPU tracing states".
* Simplify event state initialization.
* Simplify logic deciding which events are treated by this patch (previously,
execution-time events with 'tcg' and 'vcpu' properties; now it's simply events
with the 'vcpu' pro
Eliminates a future compilation error when UI code includes the tracing
headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
GLib's i18n '_' macro.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
disas/alpha.c |6
Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
bsd-user/main.c |1 +
include/qom/cpu.h|6 +
linux-user/main.c|1 +
qom/cpu.c
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index 5e0a5ab..c5f5f3d 100644
--- a/trace/control-internal.h
+++ b/trace
Homogeinizes the command capabilities with QMP.
Signed-off-by: Lluís Vilanova
---
hmp-commands-info.hx |8 +---
hmp.h|1 +
monitor.c| 31 ++-
3 files changed, 36 insertions(+), 4 deletions(-)
diff --git a/hmp-commands
A new event attribute 'cpu_id' is added to have a separate ID
space ('TRACE_VCPU_*') for all events with the 'vcpu' property.
These are later used to identify which events are enabled on each vCPU.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
Eric Blake writes:
> On 06/21/2016 07:23 AM, Lluís Vilanova wrote:
>> Signed-off-by: Lluís Vilanova
>> ---
>> linux-user/main.c | 28
>> 1 file changed, 28 insertions(+)
>>
>> +{"trace-enable", "QEMU_TRACE_
Signed-off-by: Lluís Vilanova
---
bsd-user/main.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/bsd-user/main.c b/bsd-user/main.c
index 9f592be..542e50e 100644
--- a/bsd-user/main.c
+++ b/bsd-user/main.c
@@ -30,6 +30,7 @@
#include "qemu/timer.h"
#inc
Signed-off-by: Lluís Vilanova
---
linux-user/main.c | 28
1 file changed, 28 insertions(+)
diff --git a/linux-user/main.c b/linux-user/main.c
index f8a8764..6d70821 100644
--- a/linux-user/main.c
+++ b/linux-user/main.c
@@ -33,6 +33,7 @@
#include "qemu/envl
t docs for bsd-user [Eric Blake].
Signed-off-by: Lluís Vilanova
---
Lluís Vilanova (2):
trace: [linux-user] Commandline arguments to control tracing
trace: [bsd-user] Commandline arguments to control tracing
bsd-user/main.c | 19 +++
Eric Blake writes:
> On 06/21/2016 07:23 AM, Lluís Vilanova wrote:
>> Signed-off-by: Lluís Vilanova
>> ---
>> bsd-user/main.c | 21 +
>> 1 file changed, 21 insertions(+)
>>
>> diff --git a/bsd-user/main.c b/bsd-user/main.c
>> in
Homogeinizes the command capabilities with QMP.
Signed-off-by: Lluís Vilanova
---
hmp-commands-info.hx |8 +---
hmp.h|1 +
monitor.c| 31 ++-
3 files changed, 36 insertions(+), 4 deletions(-)
diff --git a/hmp-commands
.
Signed-off-by: Lluís Vilanova
---
bsd-user/syscall.c |9 +
linux-user/syscall.c |2 ++
trace-events | 16
3 files changed, 27 insertions(+)
diff --git a/bsd-user/syscall.c b/bsd-user/syscall.c
index a9fe869..66492aa 100644
--- a/bsd-user/syscall.c
+++ b/bsd
s are treated by this patch (previously,
execution-time events with 'tcg' and 'vcpu' properties; now it's simply events
with the 'vcpu' property).
* Make tracing backends comply with the per-vCPU tracing state.
Signed-off-by: Lluís Vilanova
---
Lluís Vilanova
Eliminates a future compilation error when UI code includes the tracing
headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
GLib's i18n '_' macro.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
disas/alpha.c |6
Events with the 'vcpu' property are conditionally emitted according to
their per-vCPU state. Other events are emitted normally based on their
global tracing state.
Note that the per-vCPU condition check applies to all tracing backends.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefa
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index 5e0a5ab..c5f5f3d 100644
--- a/trace/control-internal.h
+++ b/trace
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
hmp-commands-info.hx |6 +-
hmp-commands.hx |7 +-
monitor.c| 17 +-
qapi/trace.json | 32 +--
qmp-commands.hx | 35 +++-
trace/qmp.c | 148
Adds three commandline arguments to the main *-user programs, following what's
already available in softmmu:
* -trace-enable
* -trace-events
* -trace-file
Signed-off-by: Lluís Vilanova
---
Lluís Vilanova (2):
trace: [linux-user] Commandline arguments to control tracing
trace:
A new event attribute 'cpu_id' is added to have a separate ID
space ('TRACE_VCPU_*') for all events with the 'vcpu' property.
These are later used to identify which events are enabled on each vCPU.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
bsd-user/main.c |1 +
include/qom/cpu.h|6 +
linux-user/main.c|1 +
qom/cpu.c
Signed-off-by: Lluís Vilanova
---
linux-user/main.c | 28
1 file changed, 28 insertions(+)
diff --git a/linux-user/main.c b/linux-user/main.c
index f8a8764..b0d7d2b 100644
--- a/linux-user/main.c
+++ b/linux-user/main.c
@@ -33,6 +33,7 @@
#include "qemu/envl
Signed-off-by: Lluís Vilanova
---
bsd-user/main.c | 21 +
1 file changed, 21 insertions(+)
diff --git a/bsd-user/main.c b/bsd-user/main.c
index 9f592be..ce2cc17 100644
--- a/bsd-user/main.c
+++ b/bsd-user/main.c
@@ -30,6 +30,7 @@
#include "qemu/timer.h"
#inc
Markus Armbruster writes:
> Lluís Vilanova writes:
>> Markus Armbruster writes:
>>
>>> Lluís Vilanova writes:
[...]
>>>> diff --git a/qapi/trace.json b/qapi/trace.json
>>>> index 01b0a52..25d8095 100644
>>>> --- a/qapi/trace.json
>
Stefan Hajnoczi writes:
> On Fri, Jun 17, 2016 at 10:16:25PM +0200, Lluís Vilanova wrote:
>> Stefan Hajnoczi writes:
>>
>> > On Tue, Jun 14, 2016 at 03:11:12PM +0200, Lluís Vilanova wrote:
>> >> @@ -1116,6 +1117,7 @@ int main(int argc, char **argv)
Markus Armbruster writes:
> Lluís Vilanova writes:
>> Markus Armbruster writes:
>>
>>> Lluís Vilanova writes:
>>>> Signed-off-by: Lluís Vilanova
>>>> Reviewed-by: Stefan Hajnoczi
>>>> ---
>>>> monitor.c |
Markus Armbruster writes:
> Lluís Vilanova writes:
>> Signed-off-by: Lluís Vilanova
>> Reviewed-by: Stefan Hajnoczi
>> ---
>> monitor.c |4 +-
>> qapi/trace.json | 20 ++--
>> qmp-commands.hx |
Markus Armbruster writes:
> Lluís Vilanova writes:
>> Signed-off-by: Lluís Vilanova
>> Reviewed-by: Stefan Hajnoczi
>> ---
>> monitor.c |4 +-
>> qapi/trace.json | 20 ++--
>> qmp-commands.hx |
Stefan Hajnoczi writes:
> On Tue, Jun 14, 2016 at 03:11:12PM +0200, Lluís Vilanova wrote:
>> @@ -1116,6 +1117,7 @@ int main(int argc, char **argv)
>> gdbserver_start (gdbstub_port);
>> gdb_handlesig(cpu, 0);
>> }
>> +trace_init_vcpu_events();
> Do vcpu ev
Events with the 'vcpu' property are conditionally emitted according to
their per-vCPU state. Other events are emitted normally based on their
global tracing state.
Note that the per-vCPU condition check applies to all tracing backends.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefa
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index 0ba10fe..3f368fd 100644
--- a/trace/control-internal.h
+++ b/trace
Eliminates a future compilation error when UI code includes the tracing
headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
GLib's i18n '_' macro.
Signed-off-by: Lluís Vilanova
---
disas/alpha.c |6 +++---
disas/arm.c |
A new event attribute 'cpu_id' is added to have a separate ID
space ('TRACE_VCPU_*') for all events with the 'vcpu' property.
These are later used to identify which events are enabled on each vCPU.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
ace: Per-vCPU tracing states".
* Simplify event state initialization.
* Simplify logic deciding which events are treated by this patch (previously,
execution-time events with 'tcg' and 'vcpu' properties; now it's simply events
with the 'vcpu' property).
* Make t
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
monitor.c |4 +-
qapi/trace.json | 20 ++--
qmp-commands.hx | 17 ++-
trace/qmp.c | 143 ---
4 files changed, 147 insertions(+), 37 deletions(-)
diff
Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
Makefile.objs|1 +
bsd-user/main.c |2 ++
include/qom/cpu.h|6
Paolo Bonzini writes:
> On 14/06/2016 10:39, Stefan Hajnoczi wrote:
>> On Mon, Jun 13, 2016 at 06:39:46PM +0200, Lluís Vilanova wrote:
>>> Paolo Bonzini writes:
>>>
>>>> On 13/06/2016 14:15, Lluís Vilanova wrote:
>>>>>> That said, I
Paolo Bonzini writes:
> On 13/06/2016 14:15, Lluís Vilanova wrote:
>> > That said, I am skeptical about the benefit of the interfaces you are
>> > adding. They add a lot of complication and overhead (especially
>> > regarding the memory/cache overhead of the
Lluís Vilanova writes:
> Paolo Bonzini writes:
>> First of all, a generic problem I see with your patches is that the
>> newly-introduced APIs are not providing a good abstraction.
>> If something is only used internally, as is the case for
>> trace_event_get_cpu_i
Paolo Bonzini writes:
> On 25/02/2016 16:03, Lluís Vilanova wrote:
>> static inline bool trace_event_get_state_dynamic(TraceEvent *ev)
>> {
>> -int id = trace_event_get_id(ev);
>> +TraceEventID id;
>> +assert(ev != NULL);
> Please don't add
"cpu" in the C file. My suggestion is to
> prefix functions with vcpu_trace_event if they refer to per-VCPU trace
> events, and only use the VCPU ids in those functions.
I'll fix these two.
> On 25/02/2016 16:03, Lluís Vilanova wrote:
>
Paolo Bonzini writes:
> On 10/06/2016 19:52, Lluís Vilanova wrote:
>> Fair point. But now I feel tempted to change both trace_events_dstate and
>> trace_events_enabled_count into unsigned int... it burns me when I see signed
>> types used only on their positives by design.
&
Stefan Hajnoczi writes:
> On Thu, Jun 09, 2016 at 04:17:11PM +0200, Lluís Vilanova wrote:
>> >> @@ -61,7 +69,7 @@ static inline bool
>> >> trace_event_get_state_static(TraceEvent *ev)
>> >> static inline bool trace_event_get_state_dynamic_by_id(TraceEventID
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
monitor.c |4 +-
qapi/trace.json | 20 ++--
qmp-commands.hx | 17 ++-
trace/qmp.c | 143 ---
4 files changed, 147 insertions(+), 37 deletions(-)
diff
Eliminates a future compilation error when UI code includes the tracing
headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
GLib's i18n '_' macro.
Signed-off-by: Lluís Vilanova
---
disas/alpha.c |6 +++---
disas/arm.c |
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index c78a45a..d1f99e3 100644
--- a/trace/control-internal.h
+++ b/trace
Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
Makefile.objs|1 +
bsd-user/main.c |2 +
include/qom/cpu.h|6
linux-user/
A new event attribute 'cpu_id' is added to have a separate ID
space ('TRACE_VCPU_*') for all events with the 'vcpu' property.
These are later used to identify which events are enabled on each vCPU.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
Events with the 'vcpu' property are conditionally emitted according to
their per-vCPU state. Other events are emitted normally based on their
global tracing state.
Note that the per-vCPU condition check applies to all tracing backends.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefa
ic deciding which events are treated by this patch (previously,
execution-time events with 'tcg' and 'vcpu' properties; now it's simply events
with the 'vcpu' property).
* Make tracing backends comply with the per-vCPU tracing state.
Signed-off-by: Lluís Vil
Eric Blake writes:
> On 06/09/2016 11:34 AM, Lluís Vilanova wrote:
>> Signed-off-by: Lluís Vilanova
>> Reviewed-by: Stefan Hajnoczi
>> ---
>> monitor.c |4 +-
>> qapi/trace.json | 20 ++--
>> qmp-commands.hx |
A new event attribute 'cpu_id' is added to have a separate ID
space ('TRACE_VCPU_*') for all events with the 'vcpu' property.
These are later used to identify which events are enabled on each vCPU.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
Eliminates a future compilation error when UI code includes the tracing
headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
GLib's i18n '_' macro.
Signed-off-by: Lluís Vilanova
---
disas/alpha.c |6 +++---
disas/arm.c |
Daniel P Berrange writes:
> To quote Peter today:
> pm215: "trace-events: causing merge conflicts since 2010"
[...]
Silly note. You can skip the documentation header on all trace-events file and
just reference a single global one (is /trace-events gonna survive for some
events?). Otherwise it's
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
monitor.c |4 +-
qapi/trace.json | 20 ++--
qmp-commands.hx | 18 ++-
trace/qmp.c | 143 ---
4 files changed, 147 insertions(+), 38 deletions(-)
diff
The event is described in "trace-events". Note that the "MO_AMASK" flag
is not traced, since it does not seem to affect the visible semantics of
instructions.
Signed-off-by: Lluís Vilanova
---
include/exec/cpu_ldst_template.h | 25 ++
with 'tcg' and 'vcpu' properties; now it's simply events
with the 'vcpu' property).
* Make tracing backends comply with the per-vCPU tracing state.
Signed-off-by: Lluís Vilanova
---
Lluís Vilanova (6):
trace: Identify events with the 'vcpu'
This series adds an event to track information related to memory accesses
performed by the guest CPUs ("guest_mem_before").
A future series might extend this to contain the physical address and memory
value (e.g., "guest_mem_after").
Signed-off-by: Lluís Vilanova
Events with the 'vcpu' property are conditionally emitted according to
their per-vCPU state. Other events are emitted normally based on their
global tracing state.
Note that the per-vCPU condition check applies to all tracing backends.
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefa
Richard Henderson writes:
> On 05/27/2016 09:01 AM, Lluís Vilanova wrote:
>> -void tcg_gen_qemu_ld_i32(TCGv_i32 val, TCGv addr, TCGArg idx, TCGMemOp
>> memop)
>> +static inline void do_tcg_gen_qemu_ld_i32(TCGv_i32 val, TCGv
Each vCPU gets a 'trace_dstate' bitmap to control the per-vCPU dynamic
tracing state of events with the 'vcpu' property.
Signed-off-by: Lluís Vilanova
---
Makefile.objs|1 +
bsd-user/main.c |2 +
include/qom/cpu.h|6
linux-user/
Signed-off-by: Lluís Vilanova
Reviewed-by: Stefan Hajnoczi
---
trace/control-internal.h |8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/trace/control-internal.h b/trace/control-internal.h
index c78a45a..d1f99e3 100644
--- a/trace/control-internal.h
+++ b/trace
t to
execution-time events ("*_exec").
Signed-off-by: Lluís Vilanova
Reviewed-by: Peter Maydell
Reviewed-by: Richard Henderson
---
target-alpha/translate.c |1 +
target-arm/translate.c|1 +
target-cris/translate.c |1 +
target-cris/translate_v10.c |
Stefan Hajnoczi writes:
> On Thu, Feb 25, 2016 at 04:03:06PM +0100, Lluís Vilanova wrote:
> @@ -332,6 +334,16 @@ struct CPUState {
>> struct KVMState *kvm_state;
>> struct kvm_run *kvm_run;
>>
>> +#define TRACE_VCPU_DSTATE_TYPE uint32_t
>> +TRACE_VCPU_DST
Stefan Hajnoczi writes:
> On Thu, Feb 25, 2016 at 04:02:55PM +0100, Lluís Vilanova wrote:
>> Eliminates a future compilation error when UI code includes the tracing
>> headers (indirectly pulling "disas/bfd.h" through "qom/cpu.h") and
>> GLib's i18n &
Lluís Vilanova writes:
> There was not much discussion about this before the soft-freeze, but I'm
> pinging
> just in case (feel free to ignore until next window opens).
[...]
Ping.
Thanks,
Lluis
t to
execution-time events ("*_exec").
Signed-off-by: Lluís Vilanova
Reviewed-by: Peter Maydell
Reviewed-by: Richard Henderson
---
target-alpha/translate.c |1 +
target-arm/translate.c|1 +
target-cris/translate.c |1 +
target-cris/translate_v10.c |
The event is described in "trace-events". Note that the "MO_AMASK" flag
is not traced, since it does not seem to affect the visible semantics of
instructions.
Signed-off-by: Lluís Vilanova
---
include/exec/cpu_ldst_template.h | 25 ++
This series adds an event to track information related to memory accesses
performed by the guest CPUs ("guest_mem_before").
A future series might extend this to contain the physical address and memory
value (e.g., "guest_mem_after").
Signed-off-by: Lluís Vilanova
Richard Henderson writes:
> This commit introduces the TCGOpcode for fence instruction.
[...]
I think this patch sould also document the opcpde in "tcg/README".
Cheers,
Lluis
Richard Henderson writes:
> On 05/18/2016 03:47 AM, Lluís Vilanova wrote:
>> Signed-off-by: Lluís Vilanova
>> ---
>> include/exec/cpu_ldst_template.h | 25
>> include/exec/cpu_ldst_useronly_template.h | 22 +++
Signed-off-by: Lluís Vilanova
---
include/exec/cpu_ldst_template.h | 25
include/exec/cpu_ldst_useronly_template.h | 22 ++
tcg/tcg-op.c | 32 ++--
trace-events | 22
701 - 800 of 1848 matches
Mail list logo
