commit ee8b7021da823e11e3c6ea9063ebb2ab50406a3e
author bellard <bellard@c046a42c-6fe2-441c-8c8c-71466251a162> Tue Feb 03 23:35:10 2004 +0000
committer bellard <bellard@c046a42c-6fe2-441c-8c8c-71466251a162> Tue Feb 03 23:35:10 2004 +0000
tree 91b71a0497d72e147a40d26dbed74c9b7d2347ad
parent 625976dac8a2233233c7a5943d2b970ac29ef8ad

temporary interrupt locking fix (need rework)


git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@598 c046a42c-6fe2-441c-8c8c-71466251a162

cpu-exec.c

diff --git a/cpu-exec.c b/cpu-exec.c
index bd0cbb3..4e2d77f 100644
--- a/cpu-exec.c
+++ b/cpu-exec.c
@@ -142,6 +142,7 @@
     /* prepare setjmp context for exception handling */
     for(;;) {
         if (setjmp(env->jmp_env) == 0) {
+            env->current_tb = NULL;
             /* if an exception is pending, we execute it here */
             if (env->exception_index >= 0) {
                 if (env->exception_index >= EXCP_INTERRUPT) {

exec-all.h

diff --git a/exec-all.h b/exec-all.h
index 9c7e5ed..407e963 100644
--- a/exec-all.h
+++ b/exec-all.h
@@ -85,7 +85,7 @@
 void tb_invalidate_page_range(target_ulong start, target_ulong end);
 void tlb_flush_page(CPUState *env, uint32_t addr);
 void tlb_flush_page_write(CPUState *env, uint32_t addr);
-void tlb_flush(CPUState *env);
+void tlb_flush(CPUState *env, int flush_global);
 int tlb_set_page(CPUState *env, uint32_t vaddr, uint32_t paddr, int prot, 
                  int is_user, int is_softmmu);
 

exec.c

diff --git a/exec.c b/exec.c
index 9104efb..4bf87e3 100644
--- a/exec.c
+++ b/exec.c
@@ -260,10 +260,6 @@
            nb_tbs, 
            nb_tbs > 0 ? (code_gen_ptr - code_gen_buffer) / nb_tbs : 0);
 #endif
-    /* must reset current TB so that interrupts cannot modify the
-       links while we are modifying them */
-    env->current_tb = NULL;
-
     nb_tbs = 0;
     for(i = 0;i < CODE_GEN_HASH_SIZE; i++)
         tb_hash[i] = NULL;
@@ -970,13 +966,16 @@
 void cpu_interrupt(CPUState *env, int mask)
 {
     TranslationBlock *tb;
+    static int interrupt_lock;
     
     env->interrupt_request |= mask;
     /* if the cpu is currently executing code, we must unlink it and
        all the potentially executing TB */
     tb = env->current_tb;
-    if (tb) {
+    if (tb && !testandset(&interrupt_lock)) {
+        env->current_tb = NULL;
         tb_reset_jump_recursive(tb);
+        interrupt_lock = 0;
     }
 }
 
@@ -998,7 +997,9 @@
 
 #if !defined(CONFIG_USER_ONLY)
 
-void tlb_flush(CPUState *env)
+/* NOTE: if flush_global is true, also flush global entries (not
+   implemented yet) */
+void tlb_flush(CPUState *env, int flush_global)
 {
     int i;
 
@@ -1293,7 +1294,7 @@
 
 #else
 
-void tlb_flush(CPUState *env)
+void tlb_flush(CPUState *env, int flush_global)
 {
 }