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

[Jan Kiszka <jan.kiszka@siemens.com>]

Jan Kiszka wrote:
> To pick this up again, as the problem is still unfixed:
> 
> Kevin Wolf wrote:
>> Anthony Liguori schrieb:
>>>>   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;
>>>>   
>>> This is unnecessary.
>> Right. I'll fix this with the next version.
> 
> To accelerate this "a bit" ;), I attached a refreshed version of your
> patch.

And here is a version that even compiles (sorry).

I assume that we do not want to let a triple fault in SVM context
escalate to the guest's host context, right? So I left the check
in (while it was removed meanwhile from cpu_abort).

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 cpu-all.h             |    1 +
 exec.c                |    2 ++
 target-i386/helper.c  |   15 +++++++++++++--
 target-i386/helper2.c |    5 +++++
 4 files changed, 21 insertions(+), 2 deletions(-)

Index: b/exec.c
===================================================================
--- a/exec.c
+++ b/exec.c
@@ -1302,6 +1302,8 @@ CPULogItem cpu_log_items[] = {
 #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: b/target-i386/helper.c
===================================================================
--- a/target-i386/helper.c
+++ b/target-i386/helper.c
@@ -1261,6 +1261,9 @@ void do_interrupt(int intno, int is_int,
     }
 }
 
+/* This should come from sysemu.h - if we could include it here... */
+void qemu_system_reset_request(void);
+
 /*
  * Check nested exceptions and change to double or triple fault if
  * needed. It should only be called, if this is not an interrupt.
@@ -1278,8 +1281,16 @@ static int check_exception(int intno, in
         fprintf(logfile, "check_exception old: 0x%x new 0x%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)
+            helper_vmexit(SVM_EXIT_SHUTDOWN, 0);
+
+        if (loglevel & CPU_LOG_RESET)
+            fprintf(logfile, "Triple fault\n");
+
+        qemu_system_reset_request();
+        return EXCP_HLT;
+    }
 
     if ((first_contributory && second_contributory)
         || (env->old_exception == EXCP0E_PAGE &&
Index: b/target-i386/helper2.c
===================================================================
--- a/target-i386/helper2.c
+++ b/target-i386/helper2.c
@@ -363,6 +363,11 @@ void cpu_reset(CPUX86State *env)
 {
     int i;
 
+    if (loglevel & CPU_LOG_RESET) {
+        fprintf(logfile, "CPU Reset (CPU %d)\n", env->cpu_index);
+        cpu_dump_state(env, logfile, fprintf, X86_DUMP_FPU | X86_DUMP_CCOP);
+    }
+
     memset(env, 0, offsetof(CPUX86State, breakpoints));
 
     tlb_flush(env, 1);
Index: b/cpu-all.h
===================================================================
--- a/cpu-all.h
+++ b/cpu-all.h
@@ -824,6 +824,7 @@ target_phys_addr_t cpu_get_phys_page_deb
 #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 {