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é <[email protected]>
> ---
> 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?