[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 13/27] target/arm: Move debug routines to deb
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v3 13/27] target/arm: Move debug routines to debug_helper.c |
Date: |
Mon, 1 Jul 2019 16:19:42 +0100 |
On Mon, 1 Jul 2019 at 14:26, Philippe Mathieu-Daudé <address@hidden> wrote:
>
> These routines are TCG specific.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> target/arm/Makefile.objs | 2 +-
> target/arm/cpu.c | 11 +-
> target/arm/debug_helper.c | 334 ++++++++++++++++++++++++++++++++++++++
> target/arm/helper.c | 23 ---
> target/arm/op_helper.c | 295 ---------------------------------
> 5 files changed, 339 insertions(+), 326 deletions(-)
> create mode 100644 target/arm/debug_helper.c
> --- a/target/arm/cpu.c
> +++ b/target/arm/cpu.c
> @@ -2566,7 +2566,6 @@ static void arm_cpu_class_init(ObjectClass *oc, void
> *data)
> cc->gdb_write_register = arm_cpu_gdb_write_register;
> #ifndef CONFIG_USER_ONLY
> cc->do_interrupt = arm_cpu_do_interrupt;
> - cc->get_phys_page_attrs_debug = arm_cpu_get_phys_page_attrs_debug;
> cc->asidx_from_attrs = arm_asidx_from_attrs;
> cc->vmsd = &vmstate_arm_cpu;
> cc->virtio_is_big_endian = arm_cpu_virtio_is_big_endian;
> @@ -2578,19 +2577,17 @@ static void arm_cpu_class_init(ObjectClass *oc, void
> *data)
> cc->gdb_arch_name = arm_gdb_arch_name;
> cc->gdb_get_dynamic_xml = arm_gdb_get_dynamic_xml;
> cc->gdb_stop_before_watchpoint = true;
> - cc->debug_excp_handler = arm_debug_excp_handler;
> - cc->debug_check_watchpoint = arm_debug_check_watchpoint;
> -#if !defined(CONFIG_USER_ONLY)
> - cc->adjust_watchpoint_address = arm_adjust_watchpoint_address;
> -#endif
> -
> cc->disas_set_info = arm_disas_set_info;
> #ifdef CONFIG_TCG
> cc->tcg_initialize = arm_translate_init;
> cc->tlb_fill = arm_cpu_tlb_fill;
> + cc->debug_excp_handler = arm_debug_excp_handler;
> + cc->debug_check_watchpoint = arm_debug_check_watchpoint;
> #if !defined(CONFIG_USER_ONLY)
> cc->do_unaligned_access = arm_cpu_do_unaligned_access;
> cc->do_transaction_failed = arm_cpu_do_transaction_failed;
> + cc->adjust_watchpoint_address = arm_adjust_watchpoint_address;
> + cc->get_phys_page_attrs_debug = arm_cpu_get_phys_page_attrs_debug;
> #endif /* CONFIG_TCG && !CONFIG_USER_ONLY */
I don't think get_phys_page_attrs_debug is TCG specific -- anything
that wants to do a virt-to-phys translation will use it, including
for instance cpu_memory_rw_debug() or the monitor gva2gpa command.
target/i386/cpu.c doesn't put its cc->get_phys_page_debug assignment
inside a CONFIG_TCG ifdef.
thanks
-- PMM
- [Qemu-devel] [PATCH v3 04/27] target/arm: Makefile cleanup (softmmu), (continued)
- [Qemu-devel] [PATCH v3 04/27] target/arm: Makefile cleanup (softmmu), Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 05/27] target/arm: Add copyright boilerplate, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 06/27] target/arm/helper: Remove unused include, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 08/27] target/arm: Fix coding style issues, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 07/27] target/arm: Fix multiline comment syntax, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 11/27] target/arm: Declare get_phys_addr() function publicly, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 10/27] target/arm: Move CPU state dumping routines to cpu.c, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 09/27] target/arm: Move the DC ZVA helper into op_helper, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 14/27] target/arm/vfp_helper: Move code around, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 13/27] target/arm: Move debug routines to debug_helper.c, Philippe Mathieu-Daudé, 2019/07/01
- Re: [Qemu-devel] [PATCH v3 13/27] target/arm: Move debug routines to debug_helper.c,
Peter Maydell <=
- [Qemu-devel] [PATCH v3 12/27] target/arm: Move TLB related routines to tlb_helper.c, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 15/27] target/arm/vfp_helper: Extract vfp_set_fpscr_to_host(), Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 16/27] target/arm/vfp_helper: Extract vfp_set_fpscr_from_host(), Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 20/27] target/arm: Declare arm_log_exception() function publicly, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 17/27] target/arm/vfp_helper: Restrict the SoftFloat use to TCG, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 21/27] target/arm: Declare some M-profile functions publicly, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 19/27] target/arm: Restrict PSCI to TCG, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [RFC PATCH v3 23/27] target/arm: Restrict pre-ARMv7 cpus to TCG, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-devel] [PATCH v3 18/27] target/arm: Restrict semi-hosting to TCG, Philippe Mathieu-Daudé, 2019/07/01