hw/arm/virt: use machine->possible_cpus for storing possible topology info
for now precalculate and store mp_afinity in possible_cpus as ARM cpus don't have socket/core/thread-id properties yet. In follow patches possible_cpus will be used for storing and setting NUMA node mapping and replace legacy bitmap based numa_info[node_id].node_cpu/numa_get_node_for_cpu() For the lack of better idea, this patch cannibalizes possible_cpus.cpus[x].props.thread_id so that *_cpu_index_to_props() callback could return addressable by props CPU which will be used by machine_set_cpu_numa_node() in follow up patches to assign a CPU to node. But cannibalizing is fine for now as that thread_id isn't exposed to users (no hotpluggable_cpus callback support for ARM yet) and it will be used only internally until 'device_add cpu' is supported where we can decide on which properties to use. Signed-off-by: Igor Mammedov <imammedo@redhat.com> Message-Id: <1493816238-33120-4-git-send-email-imammedo@redhat.com> Reviewed-by: Andrew Jones <drjones@redhat.com> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
This commit is contained in:
		
							parent
							
								
									46de5913b6
								
							
						
					
					
						commit
						17d3d0e2d9
					
				| 
						 | 
				
			
			@ -1221,6 +1221,8 @@ static void machvirt_init(MachineState *machine)
 | 
			
		|||
{
 | 
			
		||||
    VirtMachineState *vms = VIRT_MACHINE(machine);
 | 
			
		||||
    VirtMachineClass *vmc = VIRT_MACHINE_GET_CLASS(machine);
 | 
			
		||||
    MachineClass *mc = MACHINE_GET_CLASS(machine);
 | 
			
		||||
    const CPUArchIdList *possible_cpus;
 | 
			
		||||
    qemu_irq pic[NUM_IRQS];
 | 
			
		||||
    MemoryRegion *sysmem = get_system_memory();
 | 
			
		||||
    MemoryRegion *secure_sysmem = NULL;
 | 
			
		||||
| 
						 | 
				
			
			@ -1344,10 +1346,16 @@ static void machvirt_init(MachineState *machine)
 | 
			
		|||
        exit(1);
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    for (n = 0; n < smp_cpus; n++) {
 | 
			
		||||
        Object *cpuobj = object_new(typename);
 | 
			
		||||
    possible_cpus = mc->possible_cpu_arch_ids(machine);
 | 
			
		||||
    for (n = 0; n < possible_cpus->len; n++) {
 | 
			
		||||
        Object *cpuobj;
 | 
			
		||||
 | 
			
		||||
        object_property_set_int(cpuobj, virt_cpu_mp_affinity(vms, n),
 | 
			
		||||
        if (n >= smp_cpus) {
 | 
			
		||||
            break;
 | 
			
		||||
        }
 | 
			
		||||
 | 
			
		||||
        cpuobj = object_new(typename);
 | 
			
		||||
        object_property_set_int(cpuobj, possible_cpus->cpus[n].arch_id,
 | 
			
		||||
                                "mp-affinity", NULL);
 | 
			
		||||
 | 
			
		||||
        if (!vms->secure) {
 | 
			
		||||
| 
						 | 
				
			
			@ -1527,6 +1535,31 @@ static void virt_set_gic_version(Object *obj, const char *value, Error **errp)
 | 
			
		|||
    }
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
static const CPUArchIdList *virt_possible_cpu_arch_ids(MachineState *ms)
 | 
			
		||||
{
 | 
			
		||||
    int n;
 | 
			
		||||
    VirtMachineState *vms = VIRT_MACHINE(ms);
 | 
			
		||||
 | 
			
		||||
    if (ms->possible_cpus) {
 | 
			
		||||
        assert(ms->possible_cpus->len == max_cpus);
 | 
			
		||||
        return ms->possible_cpus;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    ms->possible_cpus = g_malloc0(sizeof(CPUArchIdList) +
 | 
			
		||||
                                  sizeof(CPUArchId) * max_cpus);
 | 
			
		||||
    ms->possible_cpus->len = max_cpus;
 | 
			
		||||
    for (n = 0; n < ms->possible_cpus->len; n++) {
 | 
			
		||||
        ms->possible_cpus->cpus[n].arch_id =
 | 
			
		||||
            virt_cpu_mp_affinity(vms, n);
 | 
			
		||||
        ms->possible_cpus->cpus[n].props.has_thread_id = true;
 | 
			
		||||
        ms->possible_cpus->cpus[n].props.thread_id = n;
 | 
			
		||||
 | 
			
		||||
        /* TODO: add 'has_node/node' here to describe
 | 
			
		||||
           to which node core belongs */
 | 
			
		||||
    }
 | 
			
		||||
    return ms->possible_cpus;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
static void virt_machine_class_init(ObjectClass *oc, void *data)
 | 
			
		||||
{
 | 
			
		||||
    MachineClass *mc = MACHINE_CLASS(oc);
 | 
			
		||||
| 
						 | 
				
			
			@ -1543,6 +1576,7 @@ static void virt_machine_class_init(ObjectClass *oc, void *data)
 | 
			
		|||
    mc->pci_allow_0_address = true;
 | 
			
		||||
    /* We know we will never create a pre-ARMv7 CPU which needs 1K pages */
 | 
			
		||||
    mc->minimum_page_bits = 12;
 | 
			
		||||
    mc->possible_cpu_arch_ids = virt_possible_cpu_arch_ids;
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
static const TypeInfo virt_machine_info = {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue