On 07/28/10 12:51, Avi Kivity wrote:
>  On 07/28/2010 01:05 PM, jes.soren...@redhat.com wrote:
>> From: Jes Sorensen<jes.soren...@redhat.com>
>>
>> KVM has a minimum CPU requirement in order to run, so there is no
>> reason to default to the very basic family 6, model 2 (or model 3 for
>> qemu32) CPU since the additional features are going to be available on
>> the host CPU.
>>
>>
>> @@ -866,11 +867,19 @@ void pc_cpus_init(const char *cpu_model)
>>
>>       /* init CPUs */
>>       if (cpu_model == NULL) {
>> +        if (kvm_enabled()) {
>>   #ifdef TARGET_X86_64
>> -        cpu_model = "qemu64";
>> +            cpu_model = "kvm64";
>>   #else
>> -        cpu_model = "qemu32";
>> +            cpu_model = "kvm32";
>>   #endif
>> +        } else {
>> +#ifdef TARGET_X86_64
>> +            cpu_model = "qemu64";
>> +#else
>> +            cpu_model = "qemu32";
>> +#endif
>> +        }
>>       }
> 
> What about -M 0.12?  It needs to retain the old values.
> 

Urgh, good point. I'll come up with a fix for that. So much for a simple
patch :)

Jes


Reply via email to