[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 11/13] numa: use new machine.cpu property with -nu
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [RFC 11/13] numa: use new machine.cpu property with -numa cpus=... CLI |
Date: |
Thu, 19 Jan 2017 15:36:52 +0100 |
On Wed, 18 Jan 2017 16:46:37 -0200
Eduardo Habkost <address@hidden> wrote:
> On Wed, Jan 18, 2017 at 06:13:27PM +0100, Igor Mammedov wrote:
> > add compat layer to legacy cpu_index based '-numa cpus=...'
> > CLI option, which will use new machine.cpu property to set
> > numa mapping if that propety is available.
> >
> > This way machine that supports machine.cpu property will
> > not break legacy CLI but will be able to move away from
> > using numa_info[node_id].node_cpu bitmaps and use only
> > possible_cpus instead.
> >
> > Signed-off-by: Igor Mammedov <address@hidden>
> > ---
> > numa.c | 63 ++++++++++++++++++++++++++++++++++++++++++++++++---------------
> > 1 file changed, 48 insertions(+), 15 deletions(-)
> >
> > diff --git a/numa.c b/numa.c
> > index 887ee58..0c34130 100644
> > --- a/numa.c
> > +++ b/numa.c
> > @@ -141,10 +141,35 @@ uint32_t numa_get_node(ram_addr_t addr, Error **errp)
> > return -1;
> > }
> >
> > -static void numa_node_parse(NumaNodeOptions *node, QemuOpts *opts, Error
> > **errp)
> > +static void numa_cpu_set_nid(MachineState *ms, CpuInstanceProperties
> > *props,
> > + Error **errp)
> > +{
> > + Visitor *v;
> > + QObject *qobj;
> > + Error *local_error = NULL;
> > +
> > + v = qobject_output_visitor_new(&qobj);
> > + visit_type_CpuInstanceProperties(v, "cpu", &props, &local_error);
> > + visit_complete(v, &qobj);
> > + visit_free(v);
> > + if (local_error) {
> > + goto end;
> > + }
> > + v = qobject_input_visitor_new(qobj, true);
> > + object_property_set(OBJECT(ms), v, "cpu", &local_error);
> > + visit_free(v);
> > + qobject_decref(qobj);
> > +end:
> > + error_propagate(errp, local_error);
> > +}
> > +
> > +static void numa_node_parse(MachineState *ms, NumaNodeOptions *node,
> > + QemuOpts *opts, Error **errp)
> > {
> > uint16_t nodenr;
> > uint16List *cpus = NULL;
> > + MachineClass *mc = MACHINE_GET_CLASS(ms);
> > + bool has_cpu_prop = object_property_find(OBJECT(ms), "cpu", NULL);
> >
> > if (node->has_nodeid) {
> > nodenr = node->nodeid;
> > @@ -172,6 +197,19 @@ static void numa_node_parse(NumaNodeOptions *node,
> > QemuOpts *opts, Error **errp)
> > return;
> > }
> > bitmap_set(numa_info[nodenr].node_cpu, cpus->value, 1);
> > + if (mc->cpu_index_to_instance_props && has_cpu_prop) {
> > + CpuInstanceProperties props;
> > + Error *local_error = NULL;
> > +
> > + props = mc->cpu_index_to_instance_props(ms, cpus->value);
> > + props.has_node_id = true;
> > + props.node_id = nodenr;
> > + numa_cpu_set_nid(ms, &props, &local_error);
> > + if (local_error) {
> > + error_propagate(errp, local_error);
> > + return;
> > + }
> > + }
> > }
>
> With this, now we have two completely different ways to store the
> results of "-numa node,cpus...", and both are likely to stay
> forever if we take too long to update the other machines.
>
> Do you plan to make ppc/spapr and arm/virt use the new
> machine.cpu system so we could remove numa_info[].node_cpu
> completely in the same series?
this RFC is just to demonstrate an approach.
it would be better to do conversion within the same merge
window (however note ARM is not even close to -device based cpus)
so I would prefer do it incrementally per target
(under promise of not abandoning project midways)
above said doing conversion for x86 and spapr within one merge
windows seems feasible.
- Re: [Qemu-devel] [RFC 08/13] pc: add writeonly 'cpu' property to PCMachine, (continued)
- [Qemu-devel] [RFC 07/13] pc: pass apic_id to pc_find_cpu_slot() directly so lookup could be done without CPU object, Igor Mammedov, 2017/01/18
- [Qemu-devel] [RFC 09/13] numa: introduce '-numa cpu' cpu option, Igor Mammedov, 2017/01/18
- [Qemu-devel] [RFC 10/13] numa: replace cpu_index_to_socket_id() with cpu_index_to_instance_props(), Igor Mammedov, 2017/01/18
- [Qemu-devel] [RFC 11/13] numa: use new machine.cpu property with -numa cpus=... CLI, Igor Mammedov, 2017/01/18
- [Qemu-devel] [RFC 13/13] pc: cpu: make sure that cpu.node-id matches -numa mapping, Igor Mammedov, 2017/01/18
- [Qemu-devel] [RFC 12/13] pc: drop usage of legacy numa_get_node_for_cpu(), Igor Mammedov, 2017/01/18
- Re: [Qemu-devel] [RFC 00/13] numa: add '-numa cpu' option, Daniel P. Berrange, 2017/01/19
- Re: [Qemu-devel] [RFC 00/13] numa: add '-numa cpu' option, Igor Mammedov, 2017/01/20