Am 29.08.2013 06:33, schrieb Alexey Kardashevskiy:
> On 08/16/2013 08:35 AM, Andreas Färber wrote:
>> Whenever DeviceClass::fw_name is not available, derive it from the CPU's
>> type name, resorting to the parent's type in case of -cpu host, and fill
>> it in for that class in a PAPR-compliant way with "PowerPC," prefix.
>>
>> Reported-by: Prerna Saxena <pre...@linux.vnet.ibm.com>
>> Signed-off-by: Andreas Färber <afaer...@suse.de>
>> ---
>> hw/ppc/spapr.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
>> index 6d984dc..0e9be32 100644
>> --- a/hw/ppc/spapr.c
>> +++ b/hw/ppc/spapr.c
>> @@ -344,6 +344,10 @@ static void *spapr_create_fdt_skel(hwaddr initrd_base,
>> const char *typename;
>>
>> typename = object_class_get_name(oc);
>> + if (kvm_enabled() &&
>> + strcmp(typename, "host-" TYPE_POWERPC_CPU) == 0) {
>> + typename =
>> object_class_get_name(object_class_get_parent(oc));
>> + }
>> nodename = g_strndup(typename,
>> strlen(typename) - strlen("-"
>> TYPE_POWERPC_CPU));
>> dc->fw_name = g_strdup_printf("PowerPC,%s", nodename);
>>
>
> You could use dc->desc of a parent, would be simpler, no?
Answered on 2/4.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
