public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Simon Kagstrom <simon.kagstrom-6Uc+yI3S9P8@public.gmane.org>
To: KVM <kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
Cc: Baruch Even <baruch-6P1Dz+XQpLLYtjvyW6yDsg@public.gmane.org>
Subject: [PATCH] 2/2: Fall back to plain Qemu if KVM is not present, report errors from libkvm
Date: Tue, 14 Nov 2006 11:36:32 +0100	[thread overview]
Message-ID: <871wo69uxb.wl%simon.kagstrom@bth.se> (raw)
In-Reply-To: <45598F42.5090400-atKUWr5tajBWk0Htik3J/w@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 193 bytes --]

This patch implements fall-back to plain Qemu if KVM is not
present. Some logic has been moved into kvm_qemu_init.

Signed-off-by: Simon Kagstrom <simon.kagstrom-6Uc+yI3S9P8@public.gmane.org>


[-- Attachment #2: qemu-kvm-start-transparent.patch --]
[-- Type: application/octet-stream, Size: 2651 bytes --]

diff -u ../orig/kvm-2/qemu/qemu-kvm.c qemu/qemu-kvm.c
--- ../orig/kvm-2/qemu/qemu-kvm.c	2006-11-05 10:18:52.000000000 +0100
+++ qemu/qemu-kvm.c	2006-11-14 11:04:19.000000000 +0100
@@ -450,10 +450,20 @@
     .io_window = kvm_io_window,
 };
 
-void kvm_qemu_init()
+int kvm_qemu_init()
 {
+    /* Try to initialize kvm */
     kvm_context = kvm_init(&qemu_kvm_ops, saved_env);
-    kvm_create(kvm_context, phys_ram_size, (void**)&phys_ram_base);
+    if (!kvm_context) {
+      	return -1;
+    }
+    phys_ram_size += KVM_EXTRA_PAGES * 4096;
+    if (kvm_create(kvm_context, phys_ram_size, (void**)&phys_ram_base) < 0) {
+	kvm_finalize(kvm_context);
+	return -1;
+    }
+
+    return 0;
 }
 
 int kvm_update_debugger(CPUState *env)
diff -u ../orig/kvm-2/qemu/qemu-kvm.h qemu/qemu-kvm.h
--- ../orig/kvm-2/qemu/qemu-kvm.h	2006-10-16 11:01:02.000000000 +0200
+++ qemu/qemu-kvm.h	2006-11-14 10:00:48.000000000 +0100
@@ -3,7 +3,9 @@
 
 #include "kvmctl.h"
 
-void kvm_qemu_init(void);
+#define KVM_EXTRA_PAGES 3
+
+int kvm_qemu_init(void);
 void kvm_load_registers(CPUState *env);
 int kvm_cpu_exec(CPUState *env);
 int kvm_update_debugger(CPUState *env);
diff -u ../orig/kvm-2/qemu/vl.c qemu/vl.c
--- ../orig/kvm-2/qemu/vl.c	2006-11-05 10:18:52.000000000 +0100
+++ qemu/vl.c	2006-11-14 11:07:17.000000000 +0100
@@ -6099,20 +6099,24 @@
     }
 
     /* init the memory */
-#if USE_KVM
     phys_ram_size = ram_size + vga_ram_size + bios_size;
+#if USE_KVM
+    /* Initialize kvm */
     if (kvm_allowed) {
-	    phys_ram_size += KVM_EXTRA_PAGES * 4096;
-	    kvm_qemu_init();
-    } else {
-	    phys_ram_base = qemu_vmalloc(phys_ram_size);
-	    if (!phys_ram_base) {
-		    fprintf(stderr, "Could not allocate physical memory\n");
-		    exit(1);
-	    }
+	if (kvm_qemu_init() < 0) {
+	    fprintf(stderr, "Could not initialize KVM\n");
+	    kvm_allowed = 0;
+	}
+    }
+    /* Fall back to normal qemu if KVM fails */
+    if (!kvm_allowed) {
+	phys_ram_base = qemu_vmalloc(phys_ram_size);
+	if (!phys_ram_base) {
+	    fprintf(stderr, "Could not allocate physical memory\n");
+	    exit(1);
+	}
     }
 #else
-    phys_ram_size = ram_size + vga_ram_size + bios_size;
     phys_ram_base = qemu_vmalloc(phys_ram_size);
     if (!phys_ram_base) {
         fprintf(stderr, "Could not allocate physical memory\n");
diff -u ../orig/kvm-2/qemu/vl.h qemu/vl.h
--- ../orig/kvm-2/qemu/vl.h	2006-11-05 10:18:52.000000000 +0100
+++ qemu/vl.h	2006-11-14 09:48:31.000000000 +0100
@@ -162,10 +162,6 @@
 #define BIOS_SIZE ((256 + 64) * 1024)
 #endif
 
-#if USE_KVM
-#define KVM_EXTRA_PAGES 3
-#endif
-
 /* keyboard/mouse support */
 
 #define MOUSE_EVENT_LBUTTON 0x01

[-- Attachment #3: Type: text/plain, Size: 373 bytes --]

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642

[-- Attachment #4: Type: text/plain, Size: 186 bytes --]

_______________________________________________
kvm-devel mailing list
kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/kvm-devel

  parent reply	other threads:[~2006-11-14 10:36 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-14  7:26 qemu-kvm Baruch Even
     [not found] ` <20061114072601.GD28027-xGn4Jn0woyz+OtfAA3OxFg@public.gmane.org>
2006-11-14  8:28   ` qemu-kvm Simon Kagstrom
2006-11-14  8:50   ` qemu-kvm Avi Kivity
     [not found]     ` <4559834F.80905-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2006-11-14  9:28       ` qemu-kvm Simon Kagstrom
     [not found]         ` <8764di9y30.wl%simon.kagstrom-6Uc+yI3S9P8@public.gmane.org>
2006-11-14  9:35           ` qemu-kvm Baruch Even
     [not found]             ` <20061114093537.GH28027-xGn4Jn0woyz+OtfAA3OxFg@public.gmane.org>
2006-11-14 10:30               ` [PATCH] 1/2: Fall back to plain Qemu if KVM is not present, report errors from libkvm Simon Kagstrom
2006-11-14  9:41           ` qemu-kvm Avi Kivity
     [not found]             ` <45598F42.5090400-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2006-11-14 10:36               ` Simon Kagstrom [this message]
     [not found]                 ` <871wo69uxb.wl%simon.kagstrom-6Uc+yI3S9P8@public.gmane.org>
2006-11-14 11:44                   ` [PATCH] 2/2: Fall back to plain Qemu if KVM is not present, report errors from libkvm Avi Kivity
     [not found]                     ` <4559AC1F.9080304-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2006-11-14 12:08                       ` Simon Kagstrom
     [not found]                         ` <87y7qe8c43.wl%simon.kagstrom-6Uc+yI3S9P8@public.gmane.org>
2006-11-14 12:24                           ` Avi Kivity
     [not found]                             ` <4559B574.5000000-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2006-11-14 13:04                               ` Simon Kagstrom
     [not found]                                 ` <87wt5y89ik.wl%simon.kagstrom-6Uc+yI3S9P8@public.gmane.org>
2006-11-14 13:24                                   ` Avi Kivity
     [not found]                                     ` <4559C393.70001-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2006-11-14 13:39                                       ` Simon Kagstrom
     [not found]                                         ` <87veli87vf.wl%simon.kagstrom-6Uc+yI3S9P8@public.gmane.org>
2006-11-14 13:55                                           ` Avi Kivity

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=871wo69uxb.wl%simon.kagstrom@bth.se \
    --to=simon.kagstrom-6uc+yi3s9p8@public.gmane.org \
    --cc=baruch-6P1Dz+XQpLLYtjvyW6yDsg@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox