Re: [PATCH] qemu: Don't use -enable-nesting with qemu 1.2.0+

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

 



On 10/22/2012 05:06 AM, Jiri Denemark wrote:
> On Sun, Oct 21, 2012 at 13:16:35 -0400, Cole Robinson wrote:
>> On 10/20/2012 03:49 PM, Eric Blake wrote:
>>> On 10/20/2012 01:28 PM, Cole Robinson wrote:
>>>> Since the option doesn't exist. Fixes booting with
>>>> cpu mode='host-model' and qemu 1.2.0
>>>> ---
>>>>  src/qemu/qemu_capabilities.c | 1 -
>>>>  1 file changed, 1 deletion(-)
>>>>
>>>> diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
>>>> index c9a4ab7..d691d51 100644
>>>> --- a/src/qemu/qemu_capabilities.c
>>>> +++ b/src/qemu/qemu_capabilities.c
>>>> @@ -2162,7 +2162,6 @@ qemuCapsInitQMPBasic(qemuCapsPtr caps)
>>>>      qemuCapsSet(caps, QEMU_CAPS_NODEFCONFIG);
>>>>      qemuCapsSet(caps, QEMU_CAPS_BOOT_MENU);
>>>>      qemuCapsSet(caps, QEMU_CAPS_FSDEV);
>>>> -    qemuCapsSet(caps, QEMU_CAPS_NESTING);
>>
>> In qemu-kvm.git
>>
>> commit 02885019915d7df85b0e8200aa305dfbe2942443
>> Author: Jan Kiszka <jan.kiszka@xxxxxxxxxxx>
>> Date:   Mon Jun 27 20:19:46 2011 +0200
>>
>>     qemu-kvm: Drop -enable-nesting command line switch
>>
>>     As in upstream, this feature can be controlled via -cpu ...,+/-svm.
>>
>>
>> So the command line switch doesn't actually exist in qemu-1.2.0+ which is what
>> qemuCapsInitQMPBasic is supposed to representing.
> 
> OK, the QEMU commit message helps a lot. We used -enable-nesting iff svm
> feature was requested but now when enabling svm is enough, it makes sense that
> upstream dropped the command line switch completely.
> 
> ACK
> 
> Jirka
> 

Thanks Jirka, pushed now.

- Cole

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]