[v3] sysemu/replay: Restrict icount to TCG system emulation

[PATCH v3 0/6] sysemu/replay: Restrict icount to TCG system emulation

Philippe Mathieu-Daudé posted 6 patches 11 months, 3 weeks ago

Diff against v1 v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231208113529.74067-1-philmd@linaro.org

Maintainers: Richard Henderson <richard.henderson@linaro.org>, Paolo Bonzini <pbonzini@redhat.com>, Pavel Dovgalyuk <pavel.dovgaluk@ispras.ru>, Peter Maydell <peter.maydell@linaro.org>, Stefan Hajnoczi <stefanha@redhat.com>, Fam Zheng <fam@euphon.net>

include/sysemu/cpu-timers.h | 32 ++++++++++++++++++++++----------
include/sysemu/replay.h     | 11 ++++++++---
accel/tcg/icount-common.c   | 36 +++++++++++++++++++-----------------
stubs/icount.c              | 29 ++---------------------------
system/cpu-timers.c         |  2 +-
system/vl.c                 | 19 ++++++++++---------
target/arm/helper.c         |  5 ++++-
util/async.c                | 16 +++++++++-------
8 files changed, 75 insertions(+), 75 deletions(-)

Expand all Fold all

[PATCH v3 0/6] sysemu/replay: Restrict icount to TCG system emulation

Posted by Philippe Mathieu-Daudé 11 months, 3 weeks ago

Slightly simplify non-TCG and user emulation code.

This series still adds assertions in ARM INST_RETIRED
PMU events, in order to bypass a linking failure. Better
would be to restrict ARM PMU events to TCG. Left for
another series.

Since v2:
- Have icount_configure() return bool
- Addressed rth's review comments

Since v1:
- Introduce enum of icount modes
- Fix ARM INST_RETIRED event

Philippe Mathieu-Daudé (6):
  sysemu/cpu-timers: Have icount_configure() return a boolean
  system/vl: Evaluate icount after accelerator options are parsed
  sysemu/cpu-timers: Introduce ICountMode enumerator
  target/arm: Ensure icount is enabled when emulating INST_RETIRED
  util/async: Only call icount_notify_exit() if icount is enabled
  sysemu/replay: Restrict icount to system emulation

 include/sysemu/cpu-timers.h | 32 ++++++++++++++++++++++----------
 include/sysemu/replay.h     | 11 ++++++++---
 accel/tcg/icount-common.c   | 36 +++++++++++++++++++-----------------
 stubs/icount.c              | 29 ++---------------------------
 system/cpu-timers.c         |  2 +-
 system/vl.c                 | 19 ++++++++++---------
 target/arm/helper.c         |  5 ++++-
 util/async.c                | 16 +++++++++-------
 8 files changed, 75 insertions(+), 75 deletions(-)

-- 
2.41.0

Re: [PATCH v3 0/6] sysemu/replay: Restrict icount to TCG system emulation

Posted by Philippe Mathieu-Daudé 10 months, 2 weeks ago

On 8/12/23 12:35, Philippe Mathieu-Daudé wrote:

> Philippe Mathieu-Daudé (6):
>    sysemu/cpu-timers: Have icount_configure() return a boolean
>    system/vl: Evaluate icount after accelerator options are parsed
>    sysemu/cpu-timers: Introduce ICountMode enumerator
>    target/arm: Ensure icount is enabled when emulating INST_RETIRED
>    util/async: Only call icount_notify_exit() if icount is enabled
>    sysemu/replay: Restrict icount to system emulation

Thanks, series queued.