Ilya Leoshkevich <i...@linux.ibm.com> writes:
> Preparation for moving perf.c to tcg/.
>
> This affects only profiling guest code, which has code in a non-0 based
> segment, e.g., 16-bit code, which is not particularly important.
I have no objection to removing the wart. Is it worth adding a note:: to
tcg.rst to say that profiles of 16-bit x64 code will be junk?
Anyway:
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
>
> Suggested-by: Richard Henderson <richard.hender...@linaro.org>
> Signed-off-by: Ilya Leoshkevich <i...@linux.ibm.com>
> ---
> accel/tcg/perf.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/accel/tcg/perf.c b/accel/tcg/perf.c
> index ba75c1bbe45..68a46b1b524 100644
> --- a/accel/tcg/perf.c
> +++ b/accel/tcg/perf.c
> @@ -337,10 +337,6 @@ void perf_report_code(uint64_t guest_pc,
> TranslationBlock *tb,
> q[insn].address = gen_insn_data[insn * start_words + 0];
> if (tb_cflags(tb) & CF_PCREL) {
> q[insn].address |= (guest_pc & qemu_target_page_mask());
> - } else {
> -#if defined(TARGET_I386)
> - q[insn].address -= tb->cs_base;
> -#endif
> }
> q[insn].flags = DEBUGINFO_SYMBOL | (jitdump ? DEBUGINFO_LINE : 0);
> }
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
