Re: [Qemu-ppc] [PATCH v4 03/20] ppc/pnv: add a core mask to PnvChip

[Cédric Le Goater]

>> @@ -227,11 +227,44 @@ static void ppc_powernv_init(MachineState *machine)
>>          snprintf(chip_name, sizeof(chip_name), "chip[%d]", CHIP_HWID(i));
>>          object_property_add_child(OBJECT(pnv), chip_name, chip, 
>> &error_fatal);
>>          object_property_set_int(chip, CHIP_HWID(i), "chip-id", 
>> &error_fatal);
>> +        object_property_set_int(chip, smp_cores, "nr-cores", &error_fatal);
>> +        /*
>> +         * We could customize cores_mask for the chip here. May be
>> +         * using a powernv machine property, like 'num-chips'. Let the
>> +         * chip choose the default for now.
> 
> I don't think you need any special mechanism for this.  If you just
> remove this explicit assignment the chip default will apply, but the
> user can alter it using -global.

Using a command line with :

        -global powernv-chip-POWER8.cores-mask=0x7070

would work for one chip but not for more. Let's start with that, I will 
remove the comment for now. multiple chip is for later.

Thanks,

C.