[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] drop -enable-nesting
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] drop -enable-nesting |
Date: |
Mon, 30 May 2011 17:15:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2011-05-30 17:10, Roedel, Joerg wrote:
> On Mon, May 30, 2011 at 11:04:02AM -0400, Jan Kiszka wrote:
>> On 2011-05-30 16:38, Nadav Har'El wrote:
>>> On Mon, May 30, 2011, Jan Kiszka wrote about "drop -enable-nesting (was:
>>> [PATCH 3/7] cpu model bug fixes and definition corrections...)":
>>>> On 2011-05-30 10:18, Roedel, Joerg wrote:
>>>>> On Sat, May 28, 2011 at 04:39:13AM -0400, Jan Kiszka wrote:
>>>>>
>>>>>> J�rg, how to deal with -enable-nesting in qemu-kvm to align behavior
>>>>>> with upstream?
>>>>>
>>>>> My personal preference is to just remove it. In upstream-qemu it is
>>>>> enabled/disabled by +/-svm. -enable-nesting is just a historic thing
>>>>> which can be wiped out.
>>>
>>> "-enable-nesting" could remain as a synonym for enabling either VMX or SVM
>>> in the guest, depending on what was available in the host (because KVM now
>>> supports both nested SVM and nested VMX, but not SVM-on-VMX or vice versa).
>>
>> Why? Once nesting is stable (I think SVM already is), there is no reason
>> for an explicit enable. And you can always mask it out via -cpu.
>>
>> BTW, what are the defaults for SVM right now in qemu-kvm and upstream?
>> Enable if the modeled CPU supports it?
>
> qemu-kvm still needs -enable-nesting, otherwise it is disabled. Upstream
> qemu should enable it unconditionally (can be disabled with -cpu ,-svm).
Then let's start with aligning qemu-kvm defaults to upstream? I guess
that's what the diff I was citing yesterday is responsible for.
In the same run, -enable-nesting could dump a warning on the console
that this switch is obsolete and will be removed from future versions.
For VMX, I would suggest to keep it off by default until it matured,
asking the user to issue -cpu ...,+vmx.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] [PATCH 3/7] cpu model bug fixes and definition corrections: Add kvm emulated x2apic flag to config defined cpu models, john cooper, 2011/05/23
- Re: [Qemu-devel] [PATCH 3/7] cpu model bug fixes and definition corrections: Add kvm emulated x2apic flag to config defined cpu models, Jan Kiszka, 2011/05/28
- Re: [Qemu-devel] [PATCH 3/7] cpu model bug fixes and definition corrections: Add kvm emulated x2apic flag to config defined cpu models, Roedel, Joerg, 2011/05/30
- [Qemu-devel] drop -enable-nesting (was: [PATCH 3/7] cpu model bug fixes and definition corrections...), Jan Kiszka, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting (was: [PATCH 3/7] cpu model bug fixes and definition corrections...), Nadav Har'El, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting, Jan Kiszka, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting, Roedel, Joerg, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting,
Jan Kiszka <=
- Re: [Qemu-devel] drop -enable-nesting, Avi Kivity, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting, Jan Kiszka, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting, Jan Kiszka, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting, Daniel P. Berrange, 2011/05/31
- Re: [Qemu-devel] drop -enable-nesting, Avi Kivity, 2011/05/31
- Re: [Qemu-devel] drop -enable-nesting, Roedel, Joerg, 2011/05/31
- Re: [Qemu-devel] drop -enable-nesting, Paolo Bonzini, 2011/05/31
- Re: [Qemu-devel] drop -enable-nesting, Avi Kivity, 2011/05/31
- Re: [Qemu-devel] drop -enable-nesting, Nadav Har'El, 2011/05/30
- Re: [Qemu-devel] drop -enable-nesting, Jan Kiszka, 2011/05/30