[PATCH for-6.1 v4 00/15] tcg: breakpoint reorg

From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, mark.cave-ayland@ilande.co.uk,
	alex.bennee@linaro.org, f4bug@amsat.org
Subject: [PATCH for-6.1 v4 00/15] tcg: breakpoint reorg
Date: Mon, 19 Jul 2021 11:22:24 -1000	[thread overview]
Message-ID: <20210719212239.428740-1-richard.henderson@linaro.org> (raw)

This is fixing #404 ("windows xp boot takes much longer...")
and several other similar reports.

Changes for v4:
  * Issue breakpoints directly from cflags_for_breakpoints.
    Do not generate code for a TB beginning with a BP at all.
  * Drop the problematic TranslatorOps.breakpoint_check hook entirely.

Changes for v3:
  * Map CF_COUNT_MASK == 0 -> TCG_MAX_INSNS.
  * Split out *_breakpoint_check fixes for avr, mips, riscv.

Changes for v2:
  * All prerequisites and 7 of the patches from v1 with are merged.

Patches lacking review are all new:
  03-target-alpha-Drop-goto_tb-path-in-gen_call_pal.patch
  09-hw-core-Introduce-TCGCPUOps.debug_check_breakpoin.patch
  10-target-arm-Implement-debug_check_breakpoint.patch
  11-target-i386-Implement-debug_check_breakpoint.patch
  12-accel-tcg-Move-breakpoint-recognition-outside-tra.patch
  13-accel-tcg-Remove-TranslatorOps.breakpoint_check.patch
  15-accel-tcg-Record-singlestep_enabled-in-tb-cflags.patch

r~

Richard Henderson (15):
  accel/tcg: Reduce CF_COUNT_MASK to match TCG_MAX_INSNS
  accel/tcg: Move curr_cflags into cpu-exec.c
  target/alpha: Drop goto_tb path in gen_call_pal
  accel/tcg: Add CF_NO_GOTO_TB and CF_NO_GOTO_PTR
  accel/tcg: Drop CF_NO_GOTO_PTR from -d nochain
  accel/tcg: Handle -singlestep in curr_cflags
  accel/tcg: Use CF_NO_GOTO_{TB, PTR} in cpu_exec_step_atomic
  accel/tcg: Move cflags lookup into tb_find
  hw/core: Introduce TCGCPUOps.debug_check_breakpoint
  target/arm: Implement debug_check_breakpoint
  target/i386: Implement debug_check_breakpoint
  accel/tcg: Move breakpoint recognition outside translation
  accel/tcg: Remove TranslatorOps.breakpoint_check
  accel/tcg: Hoist tb_cflags to a local in translator_loop
  accel/tcg: Record singlestep_enabled in tb->cflags

 include/exec/exec-all.h       |  24 +++---
 include/exec/translator.h     |  11 ---
 include/hw/core/tcg-cpu-ops.h |   6 ++
 target/arm/internals.h        |   3 +
 accel/tcg/cpu-exec.c          | 143 ++++++++++++++++++++++++++++------
 accel/tcg/translate-all.c     |   7 +-
 accel/tcg/translator.c        |  39 ++--------
 cpu.c                         |  24 ------
 target/alpha/translate.c      |  31 +-------
 target/arm/cpu.c              |   1 +
 target/arm/cpu_tcg.c          |   1 +
 target/arm/debug_helper.c     |   7 +-
 target/arm/translate-a64.c    |  25 ------
 target/arm/translate.c        |  29 -------
 target/avr/translate.c        |  10 ---
 target/cris/translate.c       |  20 -----
 target/hexagon/translate.c    |  17 ----
 target/hppa/translate.c       |  11 ---
 target/i386/tcg/tcg-cpu.c     |  12 +++
 target/i386/tcg/translate.c   |  28 -------
 target/m68k/translate.c       |  18 -----
 target/microblaze/translate.c |  18 -----
 target/mips/tcg/translate.c   |  19 -----
 target/nios2/translate.c      |  27 -------
 target/openrisc/translate.c   |  17 ----
 target/ppc/translate.c        |  18 -----
 target/riscv/translate.c      |  17 ----
 target/rx/translate.c         |  14 ----
 target/s390x/tcg/translate.c  |  24 ------
 target/sh4/translate.c        |  18 -----
 target/sparc/translate.c      |  17 ----
 target/tricore/translate.c    |  16 ----
 target/xtensa/translate.c     |  17 ----
 tcg/tcg-op.c                  |  28 +++----
 34 files changed, 185 insertions(+), 532 deletions(-)

-- 
2.25.1