From patchwork Thu Apr 15 19:08:29 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Blue Swirl X-Patchwork-Id: 50282 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 70B77B7D11 for ; Fri, 16 Apr 2010 05:13:49 +1000 (EST) Received: from localhost ([127.0.0.1]:53649 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O2UUT-0004V9-OE for incoming@patchwork.ozlabs.org; Thu, 15 Apr 2010 15:12:41 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1O2URP-00040e-Jm for qemu-devel@nongnu.org; Thu, 15 Apr 2010 15:09:31 -0400 Received: from [140.186.70.92] (port=35848 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O2UR1-00020m-Rr for qemu-devel@nongnu.org; Thu, 15 Apr 2010 15:09:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1O2UQS-0005o4-W9 for qemu-devel@nongnu.org; Thu, 15 Apr 2010 15:09:05 -0400 Received: from mail-pv0-f173.google.com ([74.125.83.173]:37823) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1O2UQQ-0005gx-05 for qemu-devel@nongnu.org; Thu, 15 Apr 2010 15:08:32 -0400 Received: by mail-pv0-f173.google.com with SMTP id 12so1059096pvd.4 for ; Thu, 15 Apr 2010 12:08:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:date:received:message-id :subject:from:to:content-type; bh=q/B6tdua2TyGaMzZlCVKQXUM5i5Ace/mTD0MkeJKV10=; b=aGJwjPqe2eguc4+5xUrYP+JhnQzDrDkScZil0xSAdOb/XwUo6iKXwgVfIJtRImS0H7 7YftCvxjCIslC3F3zfOyiGhQ2ht80ebKdxfZPKRWmNCFvbQhHLknnm0ROofZ1zo0bqhG dDJ99M4wGLCLgv2vcARozNLFCBI811AyG65u0= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:date:message-id:subject:from:to:content-type; b=S3Z/m+8k+fKJtU4LBd+6H3dwSVG6sd4M9H0znbAAWXhegquf6MAntgpvdsUFTeO3cO O4WsrKPE26er0ltDp+uoZsHt7OGS0cBhtNR/3gGni+9J/JFWEzRbdWpUNCw2Y5qzNLmO Fl74A8jDtOsB1vbcZ6FHkTmXuR2Sirl/soY+Y= MIME-Version: 1.0 Received: by 10.141.32.16 with HTTP; Thu, 15 Apr 2010 12:08:29 -0700 (PDT) Date: Thu, 15 Apr 2010 22:08:29 +0300 Received: by 10.141.107.9 with SMTP id j9mr908150rvm.177.1271358509203; Thu, 15 Apr 2010 12:08:29 -0700 (PDT) Message-ID: From: Blue Swirl To: qemu-devel X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) Subject: [Qemu-devel] [PATCH 1/3] provide a stub version of kvm-all.c if !CONFIG_KVM X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org This allows limited use of kvm functions (which will return ENOSYS) even in once-compiled modules. The patch also improves a bit the error messages for KVM initialization. Signed-off-by: Paolo Bonzini Signed-off-by: Blue Swirl --- Makefile.target | 2 + kvm-all.c | 6 ++- kvm-stub.c | 135 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ kvm.h | 13 +---- vl.c | 16 +++---- 5 files changed, 151 insertions(+), 21 deletions(-) create mode 100644 kvm-stub.c case QEMU_OPTION_usb: @@ -3511,12 +3507,14 @@ int main(int argc, char **argv, char **envp) exit(1); } - if (kvm_enabled()) { - int ret; - - ret = kvm_init(smp_cpus); + if (kvm_allowed) { + int ret = kvm_init(smp_cpus); if (ret < 0) { - fprintf(stderr, "failed to initialize KVM\n"); + if (!kvm_available()) { + printf("KVM not supported for this target\n"); + } else { + fprintf(stderr, "failed to initialize KVM: %s\n", strerror(-ret)); + } exit(1); } } diff --git a/Makefile.target b/Makefile.target index 95c9d23..34ceed6 100644 --- a/Makefile.target +++ b/Makefile.target @@ -1,6 +1,7 @@ # -*- Mode: makefile -*- GENERATED_HEADERS = config-target.h +CONFIG_NO_KVM = $(if $(subst n,,$(CONFIG_KVM)),n,y) include ../config-host.mak include config-devices.mak @@ -171,6 +172,7 @@ obj-y += vhost_net.o obj-$(CONFIG_VHOST_NET) += vhost.o obj-y += rwhandler.o obj-$(CONFIG_KVM) += kvm.o kvm-all.o +obj-$(CONFIG_NO_KVM) += kvm-stub.o LIBS+=-lz QEMU_CFLAGS += $(VNC_TLS_CFLAGS) diff --git a/kvm-all.c b/kvm-all.c index 7aa5e57..373fd34 100644 --- a/kvm-all.c +++ b/kvm-all.c @@ -1157,9 +1157,9 @@ int kvm_set_signal_mask(CPUState *env, const sigset_t *sigset) return r; } -#ifdef KVM_IOEVENTFD int kvm_set_ioeventfd_pio_word(int fd, uint16_t addr, uint16_t val, bool assign) { +#ifdef KVM_IOEVENTFD struct kvm_ioeventfd kick = { .datamatch = val, .addr = addr, @@ -1176,5 +1176,7 @@ int kvm_set_ioeventfd_pio_word(int fd, uint16_t addr, uint16_t val, bool assign) if (r < 0) return r; return 0; -} +#else + return -ENOSYS; #endif +} diff --git a/kvm-stub.c b/kvm-stub.c new file mode 100644 index 0000000..80bb908 --- /dev/null +++ b/kvm-stub.c @@ -0,0 +1,135 @@ +/* + * QEMU KVM stub + * + * Copyright Red Hat, Inc. 2010 + * + * Author: Paolo Bonzini + * + * This work is licensed under the terms of the GNU GPL, version 2 or later. + * See the COPYING file in the top-level directory. + * + */ + +#include "qemu-common.h" +#include "sysemu.h" +#include "hw/hw.h" +#include "gdbstub.h" +#include "kvm.h" + +int kvm_irqchip_in_kernel(void) +{ + return 0; +} + +int kvm_pit_in_kernel(void) +{ + return 0; +} + + +int kvm_init_vcpu(CPUState *env) +{ + return -ENOSYS; +} + +int kvm_log_start(target_phys_addr_t phys_addr, ram_addr_t size) +{ + return -ENOSYS; +} + +int kvm_log_stop(target_phys_addr_t phys_addr, ram_addr_t size) +{ + return -ENOSYS; +} + +int kvm_coalesce_mmio_region(target_phys_addr_t start, ram_addr_t size) +{ + return -ENOSYS; +} + +int kvm_uncoalesce_mmio_region(target_phys_addr_t start, ram_addr_t size) +{ + return -ENOSYS; +} + +int kvm_check_extension(KVMState *s, unsigned int extension) +{ + return 0; +} + +int kvm_init(int smp_cpus) +{ + return -ENOSYS; +} + +void kvm_flush_coalesced_mmio_buffer(void) +{ +} + +void kvm_cpu_synchronize_state(CPUState *env) +{ +} + +void kvm_cpu_synchronize_post_reset(CPUState *env) +{ +} + +void kvm_cpu_synchronize_post_init(CPUState *env) +{ +} + +int kvm_cpu_exec(CPUState *env) +{ + abort (); +} + +int kvm_has_sync_mmu(void) +{ + return 0; +} + +int kvm_has_vcpu_events(void) +{ + return 0; +} + +int kvm_has_robust_singlestep(void) +{ + return 0; +} + +void kvm_setup_guest_memory(void *start, size_t size) +{ +} + +int kvm_update_guest_debug(CPUState *env, unsigned long reinject_trap) +{ + tb_flush(env); + return 0; +} + +int kvm_insert_breakpoint(CPUState *current_env, target_ulong addr, + target_ulong len, int type) +{ + return -EINVAL; +} + +int kvm_remove_breakpoint(CPUState *current_env, target_ulong addr, + target_ulong len, int type) +{ + return -EINVAL; +} + +void kvm_remove_all_breakpoints(CPUState *current_env) +{ +} + +int kvm_set_signal_mask(CPUState *env, const sigset_t *sigset) +{ + abort(); +} + +int kvm_set_ioeventfd_pio_word(int fd, uint16_t addr, uint16_t val, bool assign) +{ + return -ENOSYS; +} diff --git a/kvm.h b/kvm.h index b90c67a..ae87d85 100644 --- a/kvm.h +++ b/kvm.h @@ -23,8 +23,9 @@ #include #endif -#ifdef CONFIG_KVM extern int kvm_allowed; + +#if defined CONFIG_KVM || !defined NEED_CPU_H #define kvm_enabled() (kvm_allowed) #else #define kvm_enabled() (0) @@ -167,15 +168,7 @@ static inline void cpu_synchronize_post_init(CPUState *env) } } -#if defined(KVM_IOEVENTFD) && defined(CONFIG_KVM) -int kvm_set_ioeventfd_pio_word(int fd, uint16_t adr, uint16_t val, bool assign); -#else -static inline -int kvm_set_ioeventfd_pio_word(int fd, uint16_t adr, uint16_t val, bool assign) -{ - return -ENOSYS; -} #endif -#endif +int kvm_set_ioeventfd_pio_word(int fd, uint16_t adr, uint16_t val, bool assign); #endif diff --git a/vl.c b/vl.c index 4fb55b8..b02efaf 100644 --- a/vl.c +++ b/vl.c @@ -3161,10 +3161,6 @@ int main(int argc, char **argv, char **envp) do_smbios_option(optarg); break; case QEMU_OPTION_enable_kvm: - if (!(kvm_available())) { - printf("Option %s not supported for this target\n", popt->name); - exit(1); - } kvm_allowed = 1; break;