2008-08-18 Zoltan Varga <vargaz@gmail.com>
[mono.git] / mono / mini / debug-mini.c
index 00babd5b2825a253e4225cf9d4bcea52ed50550f..cc06068cf6d69785ffbd48179d14fab79f80a4a4 100644 (file)
@@ -33,13 +33,6 @@ typedef struct {
        MonoMethodDesc *desc;
 } MiniDebugBreakpointInfo;
 
-typedef struct
-{
-       guint64 index;
-       MonoMethod *method;
-       MonoDebugMethodAddressList *address_list;
-} MiniDebugMethodBreakpointInfo;
-
 typedef struct
 {
        MonoDebugMethodJitInfo *jit;
@@ -75,9 +68,6 @@ struct _MonoDebuggerThreadInfo {
 
 MonoDebuggerThreadInfo *mono_debugger_thread_table = NULL;
 
-static void
-mono_debugger_check_breakpoints (MonoMethod *method, MonoDebugMethodAddress *debug_info);
-
 static inline void
 record_line_number (MiniDebugMethodInfo *info, guint32 address, guint32 offset)
 {
@@ -89,16 +79,6 @@ record_line_number (MiniDebugMethodInfo *info, guint32 address, guint32 offset)
        g_array_append_val (info->line_numbers, lne);
 }
 
-static void
-mono_debug_free_method_jit_info (MonoDebugMethodJitInfo *jit)
-{
-       g_free (jit->line_numbers);
-       g_free (jit->lexical_blocks);
-       g_free (jit->this_var);
-       g_free (jit->params);
-       g_free (jit->locals);
-       g_free (jit);
-}
 
 void
 mono_debug_init_method (MonoCompile *cfg, MonoBasicBlock *start_block, guint32 breakpoint_id)
@@ -143,7 +123,10 @@ write_variable (MonoInst *inst, MonoDebugVarInfo *var)
 
        if (inst->opcode == OP_REGVAR)
                var->index = inst->dreg | MONO_DEBUG_VAR_ADDRESS_MODE_REGISTER;
-       else {
+       else if (inst->flags & MONO_INST_IS_DEAD) {
+               // FIXME:
+               var->index = 0 | MONO_DEBUG_VAR_ADDRESS_MODE_REGISTER;
+       } else {
                /* the debug interface needs fixing to allow 0(%base) address */
                var->index = inst->inst_basereg | MONO_DEBUG_VAR_ADDRESS_MODE_REGOFFSET;
                var->offset = inst->inst_offset;
@@ -278,21 +261,22 @@ mono_debug_close_method (MonoCompile *cfg)
        jit->epilogue_begin = cfg->epilog_begin;
        jit->code_size = cfg->code_len;
 
-       record_line_number (info, jit->epilogue_begin, header->code_size);
+       if (jit->epilogue_begin)
+                  record_line_number (info, jit->epilogue_begin, header->code_size);
 
        jit->num_params = sig->param_count;
        jit->params = g_new0 (MonoDebugVarInfo, jit->num_params);
 
        for (i = 0; i < jit->num_locals; i++)
-               write_variable (cfg->varinfo [cfg->locals_start + i], &jit->locals [i]);
+               write_variable (cfg->locals [i], &jit->locals [i]);
 
        if (sig->hasthis) {
                jit->this_var = g_new0 (MonoDebugVarInfo, 1);
-               write_variable (cfg->varinfo [0], jit->this_var);
+               write_variable (cfg->args [0], jit->this_var);
        }
 
        for (i = 0; i < jit->num_params; i++)
-               write_variable (cfg->varinfo [i + sig->hasthis], &jit->params [i]);
+               write_variable (cfg->args [i + sig->hasthis], &jit->params [i]);
 
        jit->num_line_numbers = info->line_numbers->len;
        jit->line_numbers = g_new0 (MonoDebugLineNumberEntry, jit->num_line_numbers);
@@ -456,18 +440,17 @@ serialize_variable (MonoDebugVarInfo *var, guint8 *p, guint8 **endbuf)
 void
 mono_debug_serialize_debug_info (MonoCompile *cfg, guint8 **out_buf, guint32 *buf_len)
 {
-       MiniDebugMethodInfo *info;
        MonoDebugMethodJitInfo *jit;
        guint32 size, prev_offset, prev_native_offset;
        guint8 *buf, *p;
        int i;
 
-       info = (MiniDebugMethodInfo *) cfg->debug_info;
-       if (!info || !info->jit) {
+       /* Can't use cfg->debug_info as it is freed by close_method () */
+       jit = mono_debug_find_method (cfg->method, mono_domain_get ());
+       if (!jit) {
                *buf_len = 0;
                return;
        }
-       jit = info->jit;
 
        size = ((jit->num_params + jit->num_locals + 1) * 10) + (jit->num_line_numbers * 10) + 64;
        p = buf = g_malloc (size);
@@ -674,81 +657,7 @@ mono_debug_print_vars (gpointer ip, gboolean only_arguments)
                        print_var_info (&jit->locals [i], i, "", "Local");
                }
        }
-}
-
-
-/*
- * Debugger breakpoint interface.
- *
- * This interface is used to insert breakpoints on methods which are not yet JITed.
- * The debugging code keeps a list of all such breakpoints and automatically inserts the
- * breakpoint when the method is JITed.
- */
-
-static GPtrArray *method_breakpoints = NULL;
-
-MonoDebugMethodAddressList *
-mono_debugger_insert_method_breakpoint (MonoMethod *method, guint64 index)
-{
-       MiniDebugMethodBreakpointInfo *info;
-
-       info = g_new0 (MiniDebugMethodBreakpointInfo, 1);
-       info->method = method;
-       info->index = index;
-
-       info->address_list = mono_debug_lookup_method_addresses (method);
-
-       if (!method_breakpoints)
-               method_breakpoints = g_ptr_array_new ();
-
-       g_ptr_array_add (method_breakpoints, info);
-
-       return info->address_list;
-}
-
-int
-mono_debugger_remove_method_breakpoint (guint64 index)
-{
-       int i;
-
-       if (!method_breakpoints)
-               return 0;
-
-       for (i = 0; i < method_breakpoints->len; i++) {
-               MiniDebugMethodBreakpointInfo *info = g_ptr_array_index (method_breakpoints, i);
-
-               if (info->index != index)
-                       continue;
-
-               g_ptr_array_remove (method_breakpoints, info);
-               g_free (info->address_list);
-               g_free (info);
-               return 1;
-       }
-
-       return 0;
-}
-
-static void
-mono_debugger_check_breakpoints (MonoMethod *method, MonoDebugMethodAddress *debug_info)
-{
-       int i;
-
-       if (!method_breakpoints)
-               return;
-
-       if (method->is_inflated)
-               method = ((MonoMethodInflated *) method)->declaring;
-
-       for (i = 0; i < method_breakpoints->len; i++) {
-               MiniDebugMethodBreakpointInfo *info = g_ptr_array_index (method_breakpoints, i);
-
-               if (method != info->method)
-                       continue;
-
-               mono_debugger_event (MONO_DEBUGGER_EVENT_JIT_BREAKPOINT,
-                                    (guint64) (gsize) debug_info, info->index);
-       }
+       mono_debug_free_method_jit_info (jit);
 }
 
 /*
@@ -851,6 +760,8 @@ mono_debugger_thread_created (gsize tid, MonoThread *thread, MonoJitTlsData *jit
        if (mono_debug_format == MONO_DEBUG_FORMAT_NONE)
                return;
 
+       mono_debugger_lock ();
+
        mono_thread_get_stack_bounds (&staddr, &stsize);
 
        info = g_new0 (MonoDebuggerThreadInfo, 1);
@@ -869,6 +780,8 @@ mono_debugger_thread_created (gsize tid, MonoThread *thread, MonoJitTlsData *jit
 
        mono_debugger_event (MONO_DEBUGGER_EVENT_THREAD_CREATED,
                             tid, (guint64) (gsize) info);
+
+       mono_debugger_unlock ();
 #endif /* MONO_DEBUGGER_SUPPORTED */
 }
 
@@ -881,6 +794,8 @@ mono_debugger_thread_cleanup (MonoJitTlsData *jit_tls)
        if (mono_debug_format == MONO_DEBUG_FORMAT_NONE)
                return;
 
+       mono_debugger_lock ();
+
        for (ptr = &mono_debugger_thread_table; *ptr; ptr = &(*ptr)->next) {
                MonoDebuggerThreadInfo *info = *ptr;
 
@@ -894,6 +809,8 @@ mono_debugger_thread_cleanup (MonoJitTlsData *jit_tls)
                g_free (info);
                break;
        }
+
+       mono_debugger_unlock ();
 #endif
 }
 
@@ -907,6 +824,8 @@ mono_debugger_extended_notification (MonoDebuggerEvent event, guint64 data, guin
        if (!mono_debug_using_mono_debugger ())
                return;
 
+       mono_debugger_lock ();
+
        for (ptr = &mono_debugger_thread_table; *ptr; ptr = &(*ptr)->next) {
                MonoDebuggerThreadInfo *info = *ptr;
 
@@ -918,6 +837,8 @@ mono_debugger_extended_notification (MonoDebuggerEvent event, guint64 data, guin
 
                mono_debugger_event (event, data, arg);
        }
+
+       mono_debugger_unlock ();
 #endif
 }