Alexey Kardashevskiy <a...@ozlabs.ru> writes:
> This adds handling of the RESOURCE_ADDR_TRANS_MODE resource from
> the H_SET_MODE, for POWER8 (PowerISA 2.07) only.
>
> Signed-off-by: Alexey Kardashevskiy <a...@ozlabs.ru>
Reviewed-by: Mike Day <ncm...@ncultra.org>
> ---
> hw/ppc/spapr_hcall.c | 26 ++++++++++++++++++++++++++
> target-ppc/cpu.h | 2 ++
> 2 files changed, 28 insertions(+)
>
> diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c
> index fc5211b..fb23730 100644
> --- a/hw/ppc/spapr_hcall.c
> +++ b/hw/ppc/spapr_hcall.c
> @@ -747,6 +747,32 @@ static target_ulong h_set_mode(PowerPCCPU *cpu,
> sPAPREnvironment *spapr,
> default:
> ret = H_UNSUPPORTED_FLAG;
> }
> + } else if (resource == H_SET_MODE_RESOURCE_ADDR_TRANS_MODE) {
> + PowerPCCPUClass *pcc = POWERPC_CPU_GET_CLASS(cpu);
> +
> + if (!(pcc->insns_flags2 & PPC2_ISA207S)) {
> + return H_P2;
ret = H_P2;
goto out;
Just a nit to make for easier review. The above would be more
consistent. (Even though ret is already initialized to H_P2.)
> + }
> + if (value1) {
> + ret = H_P3;
> + goto out;
> + }
> + if (value2) {
> + ret = H_P4;
> + goto out;
> + }
> + switch (mflags) {
> + case 0:
> + case 2:
> + case 3:
> + CPU_FOREACH(cs) {
> + set_spr(cs, SPR_LPCR, mflags << LPCR_AIL_SH, LPCR_AIL);
> + }
> + return H_SUCCESS;
> +
> + default:
> + return H_UNSUPPORTED_FLAG;
> + }
> }
>
> out:
> diff --git a/target-ppc/cpu.h b/target-ppc/cpu.h
> index 72cb546..577193a 100644
> --- a/target-ppc/cpu.h
> +++ b/target-ppc/cpu.h
> @@ -462,6 +462,8 @@ struct ppc_slb_t {
> #define MSR_LE 0 /* Little-endian mode 1 hflags
> */
>
> #define LPCR_ILE (1 << (63-38))
> +#define LPCR_AIL 0x01800000 /* Alternate interrupt location */
> +#define LPCR_AIL_SH (63-40)
>
> #define msr_sf ((env->msr >> MSR_SF) & 1)
> #define msr_isf ((env->msr >> MSR_ISF) & 1)
> --
> 1.8.4.rc4
>
>
--
Mike Day | "Endurance is a Virtue"
