On 9/14/21 4:20 PM, Daniel P. Berrangé wrote: > This will allow us to reduce duplication between the different targets > implementing the 'info tlb' command. > > Signed-off-by: Daniel P. Berrangé <berra...@redhat.com> > --- > hw/core/cpu-common.c | 9 +++++++++ > include/hw/core/cpu.h | 11 +++++++++++ > 2 files changed, 20 insertions(+)
> diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h > index 4c47e1df18..64fc57c8d9 100644 > --- a/include/hw/core/cpu.h > +++ b/include/hw/core/cpu.h > * @has_work: Callback for checking if there is work to do. > * @memory_rw_debug: Callback for GDB memory access. > * @format_state: Callback for formatting state. > + * @format_tlb: Callback for formatting memory mappings > * @get_arch_id: Callback for getting architecture-dependent CPU ID. > * @set_pc: Callback for setting the Program Counter register. This > * should have the semantics used by the target architecture when > @@ -136,6 +137,7 @@ struct CPUClass { > int (*memory_rw_debug)(CPUState *cpu, vaddr addr, > uint8_t *buf, int len, bool is_write); > void (*format_state)(CPUState *cpu, GString *buf, int flags); > + void (*format_tlb)(CPUState *cpu, GString *buf); Doesn't this belong to SysemuCPUOps?