[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semant
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics |
Date: |
Tue, 21 Dec 2010 17:05:06 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Avi Kivity <address@hidden> writes:
> On 12/21/2010 05:41 PM, Markus Armbruster wrote:
>> Avi Kivity<address@hidden> writes:
>>
>> > On 12/15/2010 07:57 PM, Markus Armbruster wrote:
>> >> > In the short term, it would be a good idea to modify qemu-kvm to
>> >> > switch the -enable-kvm semantics to match upstream (fail if KVM isn't
>> >> > available).
>> >>
>> >> That's what my patch does.
>> >>
>> >> Additionally, it changes the default to match upstream: KVM disabled.
>> >>
>> >> What do you want changed in my patch?
>> >
>> > The 'Additionally' bit. qemu-kvm users rely on the default enabling
>> > kvm. Likely they don't rely on -enable-kvm failing is kvm is not
>> > available (and indeed, they likely expect it to match upstream). So
>> > the patch should only change behaviour when -enable-kvm is specified.
>>
>> Like this?
>>
>> upstream qemu | default |-enable-kvm
>> ----------------+-----------+-----------
>> KVM available | disabled | enabled
>> KVM unavailable | disabled | fail
>>
>> qemu-kvm | default |-enable-kvm| -no-kvm
>> ----------------+-----------+-----------+-----------
>> KVM available | enabled* | enabled | disabled
>> KVM unavailable | disabled | fail | disabled
>>
>> * differs from upstream
>
> Yes.
Thanks. I'll cook up a new patch.
- [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Anthony Liguori, 2010/12/15
- [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Markus Armbruster, 2010/12/15
- [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Avi Kivity, 2010/12/21
- [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Richard W.M. Jones, 2010/12/21
- [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Richard W.M. Jones, 2010/12/21
- [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Markus Armbruster, 2010/12/21
- Re: [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Anthony Liguori, 2010/12/21
- Re: [Qemu-devel] Re: [PATCH] qemu-kvm: Switch to upstream -enable-kvm semantics, Alexander Graf, 2010/12/21