[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v4 3/8] target/arm/helper: Move M profile routines
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH v4 3/8] target/arm/helper: Move M profile routines to m_helper.c |
Date: |
Tue, 2 Jul 2019 15:01:08 +0100 |
On Mon, 1 Jul 2019 at 20:50, Philippe Mathieu-Daudé <address@hidden> wrote:
>
> In preparation for supporting TCG disablement on ARM, we move most
> of TCG related v7m/v8m helpers and APIs into their own file.
>
> Note: It is easier to review this commit using the 'histogram'
> diff algorithm:
>
> $ git diff --diff-algorithm=histogram ...
> or
> $ git diff --histogram ...
>
> Suggested-by: Samuel Ortiz <address@hidden>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> patch generated with git diff --histogram.
>
> v4: rebased
> ---
> target/arm/Makefile.objs | 1 +
> target/arm/helper.c | 2661 +------------------------------------
> target/arm/m_helper.c | 2676 ++++++++++++++++++++++++++++++++++++++
> 3 files changed, 2681 insertions(+), 2657 deletions(-)
> create mode 100644 target/arm/m_helper.c
This patch doesn't compile:
>
> -hwaddr arm_cpu_get_phys_page_attrs_debug(CPUState *cs, vaddr addr,
> - MemTxAttrs *attrs)
> -{
> - ARMCPU *cpu = ARM_CPU(cs);
> - CPUARMState *env = &cpu->env;
> - hwaddr phys_addr;
> - target_ulong page_size;
> - int prot;
> - bool ret;
> - ARMMMUFaultInfo fi = {};
> - ARMMMUIdx mmu_idx = arm_mmu_idx(env);
> -
> - *attrs = (MemTxAttrs) {};
> -
> - ret = get_phys_addr(env, addr, 0, mmu_idx, &phys_addr,
> - attrs, &prot, &page_size, &fi, NULL);
> -
> - if (ret) {
> - return -1;
> - }
> - return phys_addr;
> -}
> -
...it deletes the definition of arm_cpu_get_phys_page_attrs_debug(),
which should stay where it is.
thanks
-- PMM
- [Qemu-arm] [PATCH v4 0/8] Support disabling TCG on ARM, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-arm] [PATCH v4 1/8] target/arm: Move debug routines to debug_helper.c, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-arm] [PATCH v4 8/8] target/arm: Do not build TCG objects when TCG is off, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-arm] [PATCH v4 4/8] RFC target/arm: Restrict pre-ARMv7 cpus to TCG, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-arm] [PATCH v4 7/8] RFC target/arm: Do not build A/M-profile cpus when using KVM, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-arm] [PATCH v4 3/8] target/arm/helper: Move M profile routines to m_helper.c, Philippe Mathieu-Daudé, 2019/07/01
- Re: [Qemu-arm] [PATCH v4 3/8] target/arm/helper: Move M profile routines to m_helper.c,
Peter Maydell <=
- [Qemu-arm] [PATCH v4 6/8] RFC target/arm: Restrict R and M profiles to TCG, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-arm] [PATCH v4 2/8] target/arm: Restrict semi-hosting to TCG, Philippe Mathieu-Daudé, 2019/07/01
- [Qemu-arm] [PATCH v4 5/8] RFC target/arm: Do not build pre-ARMv7 cpus when using KVM, Philippe Mathieu-Daudé, 2019/07/01
- Re: [Qemu-arm] [PATCH v4 0/8] Support disabling TCG on ARM, Peter Maydell, 2019/07/02
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/8] Support disabling TCG on ARM, no-reply, 2019/07/02
- Re: [Qemu-arm] [Qemu-devel] [PATCH v4 0/8] Support disabling TCG on ARM, no-reply, 2019/07/03