On 1/7/20 5:48 AM, David Gibson wrote:
> The only effect of the PPC_EMULATE_32BITS_HYPV define is to change how
> MSR_HVB is defined. This appears to be something to handle hypervisor mode
> for 32-bit CPUs, but there's really no other code to handle this. The
> MSR_THV bit, which it uses is implemented for no cpus we model.
>
> It's unlikely anyone is going to implement this any time soon, so just drop
> it.
>
> Signed-off-by: David Gibson <da...@gibson.dropbear.id.au>
Reviewed-by: Cédric Le Goater <c...@kaod.org>
> ---
> target/ppc/cpu.h | 7 -------
> 1 file changed, 7 deletions(-)
>
> diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
> index 103bfe9dc2..2de9e2fa2b 100644
> --- a/target/ppc/cpu.h
> +++ b/target/ppc/cpu.h
> @@ -26,8 +26,6 @@
> #include "exec/cpu-defs.h"
> #include "cpu-qom.h"
>
> -/* #define PPC_EMULATE_32BITS_HYPV */
> -
> #define TCG_GUEST_DEFAULT_MO 0
>
> #define TARGET_PAGE_BITS_64K 16
btw, this define ^ is unused.
> @@ -450,14 +448,9 @@ typedef struct ppc_v3_pate_t {
> #define MSR_HVB (1ULL << MSR_SHV)
> #define msr_hv msr_shv
> #else
> -#if defined(PPC_EMULATE_32BITS_HYPV)
> -#define MSR_HVB (1ULL << MSR_THV)
> -#define msr_hv msr_thv
> -#else
> #define MSR_HVB (0ULL)
> #define msr_hv (0)
> #endif
> -#endif
>
> /* DSISR */
> #define DSISR_NOPTE 0x40000000
>
