Re: [Qemu-devel] [PATCH 3/3] target-arm: A64: fix use 12 bit page tables for aarch64

[Peter Maydell]

On 24 July 2014 16:52, Alex Bennée <address@hidden> wrote:
> The aarch64 architecture only support 4k+ pages so using a smaller value
> for QEMU's internal page table handling only makes us less efficient.
>
> Signed-off-by: Alex Bennée <address@hidden>
>
> diff --git a/target-arm/cpu.h b/target-arm/cpu.h
> index c83f249..33359b9 100644
> --- a/target-arm/cpu.h
> +++ b/target-arm/cpu.h
> @@ -1051,11 +1051,18 @@ bool write_cpustate_to_list(ARMCPU *cpu);
>  #if defined(CONFIG_USER_ONLY)
>  #define TARGET_PAGE_BITS 12
>  #else
> -/* The ARM MMU allows 1k pages.  */
> -/* ??? Linux doesn't actually use these, and they're deprecated in recent
> -   architecture revisions.  Maybe a configure option to disable them.  */
> +#if defined(TARGET_AARCH64)
> +/* You can't configure 1k pages on aarch64 hardware */

"AArch64" (here and in commit messages).

Also, qemu-system-aarch64 still supports all the 32 bit
CPUs, including the ARMv5 ones (v5 is the last revision
that supported 1K pages). So we'd have to at least remove
those CPUs from the TARGET_AARCH64 system emulator
compilation, if we can't come up with anything cleverer.

thanks
-- PMM