[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH 03/14] target-ppc: use separate indices for variou
From: |
Richard Henderson |
Subject: |
Re: [Qemu-ppc] [PATCH 03/14] target-ppc: use separate indices for various translation modes |
Date: |
Wed, 17 Sep 2014 08:33:52 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 |
On 09/17/2014 01:53 AM, Paolo Bonzini wrote:
> +/* All the TLBs together must be smaller than 64k on RISC machines */
> +#if !defined(__i386__) && !defined(__x86_64__) && !defined(__aarch64__) \
> + && !defined(__sparc__) && !defined(CONFIG_TCG_INTERPRETER)
> +#define CPU_TLB_BITS (NB_MMU_MODES < 8 ? 8 : 12 - CPU_TLB_ENTRY_BITS)
> +#else
> +#define CPU_TLB_BITS 8
> +#endif
Hum. Well, it's not that all the tlbs together that must be less than 64k,
it's the addend of the first entry of the last tlb that must be within 64k of
the start of env. Nit picking, but perhaps we can word the comment better.
And if we choose to do something like this, this is where I'd prefer a define
in the relevant tcg-target.h. Because you've missed ia64 and s390 that have
positive offsets larger than 64k (21 and 19 bits, respectively).
But otherwise I'm ok with this as a solution.
r~