[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.10 05/23] numa: move source of
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.10 05/23] numa: move source of default CPUs to NUMA node mapping into boards |
Date: |
Thu, 23 Mar 2017 09:48:34 +0100 |
On Thu, 23 Mar 2017 11:40:29 +0530
Bharata B Rao <address@hidden> wrote:
> On Wed, Mar 22, 2017 at 7:02 PM, Igor Mammedov <address@hidden> wrote:
>
> > diff --git a/numa.c b/numa.c
> > index e01cb54..b6e71bc 100644
> > --- a/numa.c
> > +++ b/numa.c
> > @@ -294,9 +294,10 @@ static void validate_numa_cpus(void)
> > g_free(seen_cpus);
> > }
> >
> > -void parse_numa_opts(MachineClass *mc)
> > +void parse_numa_opts(MachineState *ms)
> > {
> > int i;
> > + MachineClass *mc = MACHINE_GET_CLASS(ms);
> >
> > for (i = 0; i < MAX_NODES; i++) {
> > numa_info[i].node_cpu = bitmap_new(max_cpus);
> > @@ -378,14 +379,16 @@ void parse_numa_opts(MachineClass *mc)
> > * rule grouping VCPUs by socket so that VCPUs from the same
> > socket
> > * would be on the same node.
> > */
> > + if (!mc->cpu_index_to_instance_props) {
> > + error_report("default CPUs to NUMA node mapping isn't
> > supported");
> > + exit(1);
> > + }
> >
>
> Just trying to understand the impact of the above enforcement. So targets
> and machine types that don't define ->cpu_index_to_instance_props() are
> expected not to boot ? Shouldn't they have a default to fall back upon ?
Currently there are 3 boards that support numa and with this series
they all implement cpu_index_to_instance_props callback,
so boards that has supported numa shouldn't be affected.
But if someone used '-numa' with a board that doesn't support numa,
it would stop booting with error message instead of silently parsing
not supported option or falling back bogus defaults (which aren't
used anyway).
> Regards,
> Bharata.
- [Qemu-ppc] [PATCH for-2.10 00/23] numa: add '-numa cpu' option, Igor Mammedov, 2017/03/22
- [Qemu-ppc] [PATCH for-2.10 02/23] hw/arm/virt: extract mp-affinity calculation in separate function, Igor Mammedov, 2017/03/22
- [Qemu-ppc] [PATCH for-2.10 01/23] tests: add CPUs to numa node mapping test, Igor Mammedov, 2017/03/22
- [Qemu-ppc] [PATCH for-2.10 03/23] hw/arm/virt: use machine->possible_cpus for storing possible topology info, Igor Mammedov, 2017/03/22
- [Qemu-ppc] [PATCH for-2.10 04/23] hw/arm/virt: explicitly allocate cpu_index for cpus, Igor Mammedov, 2017/03/22
- [Qemu-ppc] [PATCH for-2.10 05/23] numa: move source of default CPUs to NUMA node mapping into boards, Igor Mammedov, 2017/03/22
[Qemu-ppc] [PATCH for-2.10 06/23] spapr: add node-id property to sPAPR core, Igor Mammedov, 2017/03/22
[Qemu-ppc] [PATCH for-2.10 07/23] pc: add node-id property to CPU, Igor Mammedov, 2017/03/22
[Qemu-ppc] [PATCH for-2.10 08/23] virt-arm: add node-id property to CPU, Igor Mammedov, 2017/03/22
[Qemu-ppc] [PATCH for-2.10 09/23] numa: add check that board supports cpu_index to node mapping, Igor Mammedov, 2017/03/22
[Qemu-ppc] [PATCH for-2.10 10/23] numa: mirror cpu to node mapping in MachineState::possible_cpus, Igor Mammedov, 2017/03/22