[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [for-2.7 PATCH v3 15/15] spapr: implement query-hotplugga
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [for-2.7 PATCH v3 15/15] spapr: implement query-hotpluggable-cpus callback |
Date: |
Mon, 6 Jun 2016 15:37:29 +1000 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Thu, May 12, 2016 at 09:18:25AM +0530, Bharata B Rao wrote:
> From: Igor Mammedov <address@hidden>
>
> It returns a list of present/possible to hotplug CPU
> objects with a list of properties to use with
> device_add.
>
> in spapr case returned list would looks like:
> -> { "execute": "query-hotpluggable-cpus" }
> <- {"return": [
> { "props": { "core": 8 }, "type": "POWER8-spapr-cpu-core",
> "vcpus-count": 2 },
> { "props": { "core": 0 }, "type": "POWER8-spapr-cpu-core",
> "vcpus-count": 2,
> "qom-path": "/machine/unattached/device[0]"}
> ]}'
>
> TODO:
> add 'node' property for core <-> numa node mapping
>
> Signed-off-by: Igor Mammedov <address@hidden>
> Signed-off-by: Bharata B Rao <address@hidden>
> ---
> hw/ppc/spapr.c | 34 ++++++++++++++++++++++++++++++++++
> 1 file changed, 34 insertions(+)
>
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index a8cd74d..ae95580 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -66,6 +66,7 @@
> #include "hw/compat.h"
> #include "qemu/cutils.h"
> #include "hw/ppc/spapr_cpu_core.h"
> +#include "qmp-commands.h"
>
> #include <libfdt.h>
>
> @@ -2391,6 +2392,38 @@ static unsigned spapr_cpu_index_to_socket_id(unsigned
> cpu_index)
> return cpu_index / smp_threads / smp_cores;
> }
>
> +static HotpluggableCPUList *spapr_query_hotpluggable_cpus(MachineState
> *machine)
> +{
> + int i;
> + HotpluggableCPUList *head = NULL;
> + sPAPRMachineState *spapr = SPAPR_MACHINE(machine);
> + int spapr_max_cores = max_cpus / smp_threads;
> + int smt = kvmppc_smt_threads();
> +
> + for (i = 0; i < spapr_max_cores; i++) {
> + HotpluggableCPUList *list_item = g_new0(typeof(*list_item), 1);
> + HotpluggableCPU *cpu_item = g_new0(typeof(*cpu_item), 1);
> + CpuInstanceProperties *cpu_props = g_new0(typeof(*cpu_props), 1);
> +
> + cpu_item->type = spapr_get_cpu_core_type(machine->cpu_model);
> + cpu_item->vcpus_count = smp_threads;
> + cpu_props->has_core = true;
> + cpu_props->core = i * smt;
I'm thinking we should probably use the existing vcpu_dt_id logic
here, to make sure we don't change things if we ever alter that.
> + /* TODO: add 'has_node/node' here to describe
> + to which node core belongs */
> +
> + cpu_item->props = cpu_props;
> + if (spapr->cores[i]) {
> + cpu_item->has_qom_path = true;
> + cpu_item->qom_path = object_get_canonical_path(spapr->cores[i]);
> + }
> + list_item->value = cpu_item;
> + list_item->next = head;
> + head = list_item;
> + }
> + return head;
> +}
> +
> static void spapr_machine_class_init(ObjectClass *oc, void *data)
> {
> MachineClass *mc = MACHINE_CLASS(oc);
> @@ -2421,6 +2454,7 @@ static void spapr_machine_class_init(ObjectClass *oc,
> void *data)
> hc->plug = spapr_machine_device_plug;
> hc->unplug = spapr_machine_device_unplug;
> mc->cpu_index_to_socket_id = spapr_cpu_index_to_socket_id;
> + mc->query_hotpluggable_cpus = spapr_query_hotpluggable_cpus;
>
> smc->dr_lmb_enabled = true;
> smc->dr_cpu_enabled = true;
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-ppc] [for-2.7 PATCH v3 15/15] spapr: implement query-hotpluggable-cpus callback,
David Gibson <=