Re: [Qemu-devel] [PATCH 3/7] trace: Add event "guest_inst_info_before"

2017-09-14 Thread Richard Henderson
On 09/14/2017 07:59 AM, Lluís Vilanova wrote: > Since physical address information is not gonna be added in this series (if at > all), what do you say about hoisting instruction length info into > guest_insn_before/after and dropping the *_info variants? That should be fine. r~

Re: [Qemu-devel] [PATCH 3/7] trace: Add event "guest_inst_info_before"

2017-09-14 Thread Lluís Vilanova
Richard Henderson writes: > On 09/10/2017 09:23 AM, Lluís Vilanova wrote: >> Signed-off-by: Lluís Vilanova >> --- >> accel/tcg/translator.c | 18 ++ >> trace-events |9 + >> 2 files changed, 27 insertions(+) >> >> diff --git a/accel/tcg/translator.c b/accel/

Re: [Qemu-devel] [PATCH 3/7] trace: Add event "guest_inst_info_before"

2017-09-13 Thread Richard Henderson
On 09/10/2017 09:23 AM, Lluís Vilanova wrote: > Signed-off-by: Lluís Vilanova > --- > accel/tcg/translator.c | 18 ++ > trace-events |9 + > 2 files changed, 27 insertions(+) > > diff --git a/accel/tcg/translator.c b/accel/tcg/translator.c > index 287d27b4

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

2017-09-10 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- accel/tcg/translator.c | 18 ++ trace-events |9 + 2 files changed, 27 insertions(+) diff --git a/accel/tcg/translator.c b/accel/tcg/translator.c index 287d27b4f7..6598931171 100644 --- a/accel/tcg/translator.c +++ b/accel