[Qemu-devel] [PATCH] x86: Reboot CPU on triple fault - Version 4

[Kevin Wolf <kwolf@suse.de>]

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

And another version which implements the -d cpu_reset suggested by
Jamie. Extending -d is probably better than introducing a completely new
option.

Signed-off-by: Kevin Wolf <kwolf@suse.de>

[-- Attachment #2: qemu-triple-fault-v4.patch --]
[-- Type: text/x-patch, Size: 2918 bytes --]

Index: dyngen-exec.h
===================================================================
--- dyngen-exec.h	(Revision 4215)
+++ dyngen-exec.h	(Arbeitskopie)
@@ -86,6 +86,7 @@
 extern int fprintf(FILE *, const char *, ...);
 extern int fputs(const char *, FILE *);
 extern int printf(const char *, ...);
+extern FILE *stderr;
 #undef NULL
 #define NULL 0
 
Index: exec.c
===================================================================
--- exec.c	(Revision 4215)
+++ exec.c	(Arbeitskopie)
@@ -1259,6 +1259,8 @@
 #ifdef TARGET_I386
     { CPU_LOG_PCALL, "pcall",
       "show protected mode far calls/returns/exceptions" },
+    { CPU_LOG_RESET, "cpu_reset",
+      "show CPU state before CPU resets" },
 #endif
 #ifdef DEBUG_IOPORT
     { CPU_LOG_IOPORT, "ioport",
Index: target-i386/helper.c
===================================================================
--- target-i386/helper.c	(Revision 4215)
+++ target-i386/helper.c	(Arbeitskopie)
@@ -1231,6 +1231,10 @@
     }
 }
 
+/* This should come from sysemu.h - if we could include it here... */
+void qemu_system_reset_request(void);
+extern int warn_on_triple_fault;
+
 /*
  * Check nested exceptions and change to double or triple fault if
  * needed. It should only be called, if this is not an interrupt.
@@ -1248,8 +1252,19 @@
         fprintf(logfile, "check_exception old: %x new %x\n",
                 env->old_exception, intno);
 
-    if (env->old_exception == EXCP08_DBLE)
-        cpu_abort(env, "triple fault");
+    if (env->old_exception == EXCP08_DBLE) {
+        if(env->intercept & INTERCEPT_SVM_MASK) {
+            /* most probably the virtual machine should not
+               be shut down but rather caught by the VMM */
+            vmexit(SVM_EXIT_SHUTDOWN, 0);
+        }
+        if (loglevel & CPU_LOG_RESET) {
+            fprintf(stderr, "qemu: warning: triple fault\n");
+            fprintf(logfile, "qemu: warning: triple fault\n");
+        }
+        qemu_system_reset_request();
+        return EXCP_HLT;
+    }
 
     if ((first_contributory && second_contributory)
         || (env->old_exception == EXCP0E_PAGE &&
Index: target-i386/helper2.c
===================================================================
--- target-i386/helper2.c	(Revision 4215)
+++ target-i386/helper2.c	(Arbeitskopie)
@@ -362,6 +362,10 @@
 void cpu_reset(CPUX86State *env)
 {
     int i;
+        
+    if (loglevel & CPU_LOG_RESET) {
+        cpu_dump_state(env, logfile, fprintf, X86_DUMP_FPU | X86_DUMP_CCOP);
+    }
 
     memset(env, 0, offsetof(CPUX86State, breakpoints));
 
Index: cpu-all.h
===================================================================
--- cpu-all.h	(Revision 4215)
+++ cpu-all.h	(Arbeitskopie)
@@ -779,6 +779,7 @@
 #define CPU_LOG_PCALL      (1 << 6)
 #define CPU_LOG_IOPORT     (1 << 7)
 #define CPU_LOG_TB_CPU     (1 << 8)
+#define CPU_LOG_RESET      (1 << 9)
 
 /* define log items */
 typedef struct CPULogItem {