Commit 063cac53 authored by Alexey Kardashevskiy's avatar Alexey Kardashevskiy Committed by Alexander Graf
Browse files

target-ppc: Fix number of threads per core limit 



The number of threads per core is different for POWER6/7/8 CPUs.
Guest systems do not expect to see more threads per core than
a specific CPU supports so we need to limit this number.
This limit is implemented by ppc_get_compat_smt_threads().

However it has a problem as it checks for PCR (Processor Compatibility
Register) mask, 2.05 means 2 threads per core, 2.06 - 4 threads.
For POWER8 one would expect PCR_COMPAT_2_07 bit set and
ppc_get_compat_smt_threads() checking for it to return 8 threads
per core. But the latest PowerISA spec now is 2.07 and there is
no 2.07 compatibility mode defined, QEMU does not define it either
(will be in PowerISA 2.08).

Instead of relying on a PCR mask, this uses kvmppc_smt_threads()
which returns the maximum supported threads number for KVM or
1 for TCG.

Signed-off-by: default avatarAlexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: default avatarAlexander Graf <agraf@suse.de>
parent 0e162974

target-ppc/translate_init.c

+5 −13
Original line number Original line Diff line number Diff line
@@ -9098,29 +9098,21 @@ static void ppc_cpu_unrealizefn(DeviceState *dev, Error **errp) 




int ppc_get_compat_smt_threads(PowerPCCPU *cpu)

int ppc_get_compat_smt_threads(PowerPCCPU *cpu)

{

{

    int ret = smp_threads;

    int ret = MIN(smp_threads, kvmppc_smt_threads());

    PowerPCCPUClass *pcc = POWERPC_CPU_GET_CLASS(cpu);





    switch (cpu->cpu_version) {

    switch (cpu->cpu_version) {

    case CPU_POWERPC_LOGICAL_2_05:

    case CPU_POWERPC_LOGICAL_2_05:

        ret = 2;

        ret = MIN(ret, 2);

        break;

        break;

    case CPU_POWERPC_LOGICAL_2_06:

    case CPU_POWERPC_LOGICAL_2_06:

        ret = 4;

        ret = MIN(ret, 4);

        break;

        break;

    case CPU_POWERPC_LOGICAL_2_07:

    case CPU_POWERPC_LOGICAL_2_07:

        ret = 8;

        ret = MIN(ret, 8);

        break;

    default:

        if (pcc->pcr_mask & PCR_COMPAT_2_06) {

            ret = 4;

        } else if (pcc->pcr_mask & PCR_COMPAT_2_05) {

            ret = 2;

        }

        break;

        break;

    }

    }





    return MIN(ret, smp_threads);

    return ret;

}

}





int ppc_set_compat(PowerPCCPU *cpu, uint32_t cpu_version)

int ppc_set_compat(PowerPCCPU *cpu, uint32_t cpu_version)