[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the ho
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it |
Date: |
Wed, 03 Oct 2012 11:02:05 +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 2012-10-01 18:20, Anthony Liguori wrote:
> Jan Kiszka <address@hidden> writes:
>
>> If we built a target for a host that supports KVM in principle, set the
>> default accelerator to KVM as well. This also means the start of QEMU
>> will fail to start if KVM support turns out to be unavailable at
>> runtime.
>>
>> Signed-off-by: Jan Kiszka <address@hidden>
>> ---
>> kvm-all.c | 1 +
>> kvm-stub.c | 1 +
>> kvm.h | 1 +
>> vl.c | 4 ++--
>> 4 files changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/kvm-all.c b/kvm-all.c
>> index 92a7137..4d5f86c 100644
>> --- a/kvm-all.c
>> +++ b/kvm-all.c
>> @@ -103,6 +103,7 @@ struct KVMState
>> #endif
>> };
>>
>> +bool kvm_configured = true;
>> KVMState *kvm_state;
>> bool kvm_kernel_irqchip;
>> bool kvm_async_interrupts_allowed;
>> diff --git a/kvm-stub.c b/kvm-stub.c
>> index 3c52eb5..86a6451 100644
>> --- a/kvm-stub.c
>> +++ b/kvm-stub.c
>> @@ -17,6 +17,7 @@
>> #include "gdbstub.h"
>> #include "kvm.h"
>>
>> +bool kvm_configured;
>> KVMState *kvm_state;
>> bool kvm_kernel_irqchip;
>> bool kvm_async_interrupts_allowed;
>> diff --git a/kvm.h b/kvm.h
>> index dea2998..9936e5f 100644
>> --- a/kvm.h
>> +++ b/kvm.h
>> @@ -22,6 +22,7 @@
>> #include <linux/kvm.h>
>> #endif
>>
>> +extern bool kvm_configured;
>> extern int kvm_allowed;
>> extern bool kvm_kernel_irqchip;
>> extern bool kvm_async_interrupts_allowed;
>> diff --git a/vl.c b/vl.c
>> index 8d305ca..f557bd1 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -2215,8 +2215,8 @@ static int configure_accelerator(void)
>> }
>>
>> if (p == NULL) {
>> - /* Use the default "accelerator", tcg */
>> - p = "tcg";
>> + /* The default accelerator depends on the availability of KVM. */
>> + p = kvm_configured ? "kvm" : "tcg";
>> }
>
> How about making this an arch_init() function call and then using a #if
> defined(KVM_CONFIG) in arch_init.c?
>
> I hate to introduce another global variable if we can avoid it...
Hacked too quickly. In fact, kvm_configured is simply kvm_available().
However, resistance appear to be too high here.
Jan
>
> Otherwise:
>
> Acked-by: Anthony Liguori <address@hidden>
>
> Blue/Aurelien, any objections?
>
> Regards,
>
> Anthony Liguori
>
>>
>> while (!accel_initialised && *p != '\0') {
>> --
>> 1.7.3.4
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to address@hidden
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility, (continued)
- Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility, Paolo Bonzini, 2012/10/01
- Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility, Jan Kiszka, 2012/10/01
- Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility, Anthony Liguori, 2012/10/01
- [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Jan Kiszka, 2012/10/01
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Anthony Liguori, 2012/10/01
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Aurelien Jarno, 2012/10/01
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it,
Jan Kiszka <=
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Blue Swirl, 2012/10/03
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Peter Maydell, 2012/10/03
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Alexander Graf, 2012/10/04
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Anthony Liguori, 2012/10/04
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Alexander Graf, 2012/10/04
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Peter Maydell, 2012/10/05
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Andreas Färber, 2012/10/08
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Alexander Graf, 2012/10/08
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Andreas Färber, 2012/10/01
- Re: [Qemu-devel] [PATCH] kvm: Set default accelerator to "kvm" if the host supports it, Daniel P. Berrange, 2012/10/01