qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] 答复: [PATCH qom-cpu for-1. 4] target-unicore32: Rename


From: Andreas Färber
Subject: Re: [Qemu-devel] 答复: [PATCH qom-cpu for-1. 4] target-unicore32: Rename CPU subtypes
Date: Tue, 29 Jan 2013 11:12:04 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130105 Thunderbird/17.0.2

嗨,

Am 29.01.2013 10:25, schrieb Guan Xuetao:
>> -----邮件原件-----
>> 发件人: Andreas Färber [mailto:address@hidden
>> 发送时间: Monday, January 28, 2013 23:47
>> 收件人: address@hidden
>> 抄送: Guan Xuetao
>> 主题: Re: [Qemu-devel] [PATCH qom-cpu for-1.4] target-unicore32: Rename
>> CPU subtypes
>>
>> Am 27.01.2013 23:50, schrieb Andreas Färber:
>>> In the initial conversion of CPU models to QOM types, model names were
>>> mapped 1:1 to type names. As a side effect this gained us a type
>>> "any", which is now a device.
>>>
>>> To avoid "-device any" silliness and to pave the way for compiling
>>> multiple targets into one executable, adopt a <name>-<arch>-cpu scheme.
>>>
>>> No functional changes for -cpu arguments.
>>>
>>> Signed-off-by: Andreas Färber <address@hidden>
>>
>> Patch breaks linux-user due to forgetting to update cpu_model_str.
>> I'll double-check the other targets.
>>
> After this patch was applied, I saw the following error when running:
>       qemu: hardware error: Unable to find CPU definition

As indicated above, this patch has a bug, sorry. I will send a v2.
Forking in linux-user will be broken (cpu_copy()), everything else
should work though, no?

Can you please test qemu.git master and confirm that it is not broken
without this patch?

I do not have any unicore32 image for testing. I therefore tested
"./unicore32-softmmu/qemu-system-unicore32 -cpu ?" (which did not seem
implemented) and "./unicore32-softmmu/qemu-system-unicore32" without
parameters, which ran into a kernel_filename assertion IIRC.
Can you upload a test image and static linux-user binaries somewhere? Or
provide me with instructions how to build a toolchain and kernel?

Thanks,
Andreas

P.S. FYI a hw/uc32/ directory has been created in the meantime. If you
still wish to move your puv3 files out of hw/, please feel invited to
send a patch.

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg



reply via email to

[Prev in Thread] Current Thread [Next in Thread]