On 6/7/2023 8:24 PM, Fei Wu wrote: > v15 > --- > This is a large change: > * remove all time related stuffs, including cmd 'info profile' > * remove the per-TB flag, use global flag instead > * remove tb_stats pause/filter, but add status > * remove qemu_log changes, and use monitor_printf > * use array instead of list for sorting > * remove async_safe_run_on_cpu for cmd info tb-list & tb > * use monitor_disas instead of regenerate TB, but **doesn't work yet** > * other cleanups > Hi Richard,
Could you please take a look at this series? I hope most of the comments on v14 have been addressed. Next revision I will change: * add async_safe_run_on_cpu back in case of any concurrency issue * add tbs->gpa_pc for monitor_disas, which requires get_page_addr_code_hostp() return both ram_addr_t and gpa * finalize the commit logs and documents Thanks, Fei. > > Alex Bennée (1): > tb-stats: reset the tracked TBs on a tb_flush > > Fei Wu (5): > accel/tcg: remove CONFIG_PROFILER > accel/tcg: add jit stats to TBStatistics > debug: add -d tb_stats to control TBStatistics > tb-stats: dump hot TBs at the end of the execution > docs: add tb-stats how to > > Vanderson M. do Rosario (4): > accel/tcg: introduce TBStatistics structure > accel: collecting TB execution count > monitor: adding tb_stats hmp command > tb-stats: Adding info [tb-list|tb] commands to HMP (WIP) > > MAINTAINERS | 1 + > accel/tcg/cpu-exec.c | 6 + > accel/tcg/meson.build | 1 + > accel/tcg/monitor.c | 184 +++++++++++++++-- > accel/tcg/tb-context.h | 1 + > accel/tcg/tb-hash.h | 7 + > accel/tcg/tb-maint.c | 20 ++ > accel/tcg/tb-stats.c | 365 ++++++++++++++++++++++++++++++++++ > accel/tcg/tcg-accel-ops.c | 10 - > accel/tcg/tcg-runtime.c | 1 + > accel/tcg/translate-all.c | 110 ++++++---- > accel/tcg/translator.c | 30 +++ > disas/disas.c | 2 + > docs/devel/tcg-tbstats.rst | 97 +++++++++ > hmp-commands-info.hx | 31 +-- > hmp-commands.hx | 16 ++ > include/exec/exec-all.h | 3 + > include/exec/gen-icount.h | 1 + > include/exec/tb-stats-dump.h | 21 ++ > include/exec/tb-stats-flags.h | 29 +++ > include/exec/tb-stats.h | 130 ++++++++++++ > include/monitor/hmp.h | 3 + > include/qemu/log.h | 1 + > include/qemu/timer.h | 9 - > include/tcg/tcg.h | 26 +-- > linux-user/exit.c | 2 + > meson.build | 2 - > meson_options.txt | 2 - > qapi/machine.json | 18 -- > scripts/meson-buildoptions.sh | 3 - > softmmu/runstate.c | 11 +- > stubs/meson.build | 1 + > stubs/tb-stats.c | 36 ++++ > tcg/tcg.c | 237 +++------------------- > tests/qtest/qmp-cmd-test.c | 3 - > util/log.c | 26 +++ > 36 files changed, 1093 insertions(+), 353 deletions(-) > create mode 100644 accel/tcg/tb-stats.c > create mode 100644 docs/devel/tcg-tbstats.rst > create mode 100644 include/exec/tb-stats-dump.h > create mode 100644 include/exec/tb-stats-flags.h > create mode 100644 include/exec/tb-stats.h > create mode 100644 stubs/tb-stats.c >
