[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 7/7] ppc: move the cpu_dt_id logic to machine
From: |
Bharata B Rao |
Subject: |
Re: [Qemu-devel] [PATCH v2 7/7] ppc: move the cpu_dt_id logic to machine code |
Date: |
Sat, 2 Jul 2016 13:45:44 +0530 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Sat, Jul 02, 2016 at 12:42:12AM +0200, Greg Kurz wrote:
> Now that every supported machine type is able to provide a cpu_index, we
> can safely move all the cpu_dt_id bits to the machine code.
>
> TODO: the cpu_dt_id logic remains the same wannabe generic one as before
> because of its target code background: machine types should provide their
> own cpu_dt_id logic (it is required by the future powernv machine type for
> example).
>
> Signed-off-by: Greg Kurz <address@hidden>
> ---
> hw/ppc/ppc.c | 28 +++++++++++++++++++++++++++-
> target-ppc/translate_init.c | 30 ------------------------------
> 2 files changed, 27 insertions(+), 31 deletions(-)
>
> diff --git a/hw/ppc/ppc.c b/hw/ppc/ppc.c
> index 12de255fb211..506b493bf43b 100644
> --- a/hw/ppc/ppc.c
> +++ b/hw/ppc/ppc.c
> @@ -1354,7 +1354,33 @@ PowerPCCPU *ppc_get_vcpu_by_dt_id(int cpu_dt_id)
>
> void ppc_set_vcpu_dt_id(PowerPCCPU *cpu, int cpu_index, Error **errp)
> {
> - ;
> + int max_smt = kvmppc_smt_threads();
> + int vcpu_dt_id;
> +
> + if (smp_threads > max_smt) {
> + error_setg(errp, "Cannot support more than %d threads on PPC with
> %s",
> + max_smt, kvm_enabled() ? "KVM" : "TCG");
> + return;
> + }
> + if (!is_power_of_2(smp_threads)) {
> + error_setg(errp, "Cannot support %d threads on PPC with %s, "
> + "threads count must be a power of 2.",
> + smp_threads, kvm_enabled() ? "KVM" : "TCG");
> + return;
> + }
Not sure if the above two checks belong here in the routine which
set the cpu_dt_id.
Regards,
Bharata.
[Qemu-devel] [PATCH v2 5/7] ppc: introduce ppc_set_vcpu_dt_id(), Greg Kurz, 2016/07/01
[Qemu-devel] [PATCH v2 6/7] spapr: use ppc_set_vcpu_dt_id() in CPU hotplug code, Greg Kurz, 2016/07/01
[Qemu-devel] [PATCH v2 7/7] ppc: move the cpu_dt_id logic to machine code, Greg Kurz, 2016/07/01
- Re: [Qemu-devel] [PATCH v2 7/7] ppc: move the cpu_dt_id logic to machine code,
Bharata B Rao <=
Re: [Qemu-devel] [PATCH v2 0/7] ppc: compute cpu_dt_id in the machine code, Bharata B Rao, 2016/07/02