Implement passing of this as the first argument for ia64 correctly.
[mono.git] / mono / mini / debugger-agent.c
index dc2c96486c65727a04a06b614e18a4ff6730fec5..86315e332230fc0320f999ba32301853d5016f39 100644 (file)
@@ -1,10 +1,10 @@
 /*
- * debugger-agent.c: Debugger back-end module
+ * debugger-agent.c: Soft Debugger back-end module
  *
  * Author:
  *   Zoltan Varga (vargaz@gmail.com)
  *
- * (C) 2009 Novell, Inc.
+ * Copyright 2009-2010 Novell, Inc.
  */
 
 #include <config.h>
@@ -14,6 +14,9 @@
 #ifdef HAVE_SYS_TYPES_H
 #include <sys/types.h>
 #endif
+#ifdef HAVE_SYS_SELECT_H
+#include <sys/select.h>
+#endif
 #ifdef HAVE_SYS_SOCKET_H
 #include <sys/socket.h>
 #endif
@@ -67,6 +70,7 @@ int WSAAPI getnameinfo(const struct sockaddr*,socklen_t,char*,DWORD,
 #include <mono/metadata/gc-internal.h>
 #include <mono/metadata/threads-types.h>
 #include <mono/metadata/socket-io.h>
+#include <mono/metadata/assembly.h>
 #include <mono/utils/mono-semaphore.h>
 #include "debugger-agent.h"
 #include "mini.h"
@@ -99,6 +103,7 @@ typedef struct {
        GSList *onthrow;
        int timeout;
        char *launch;
+       gboolean embedding;
 } AgentConfig;
 
 typedef struct
@@ -117,7 +122,9 @@ typedef struct
        gboolean has_ctx;
 } StackFrame;
 
-typedef struct
+typedef struct _InvokeData InvokeData;
+
+struct _InvokeData
 {
        int id;
        int flags;
@@ -132,7 +139,9 @@ typedef struct
        MonoMethod *method;
        gpointer *args;
        guint32 suspend_count;
-} InvokeData;
+
+       InvokeData *last_invoke;
+};
 
 typedef struct {
        MonoContext ctx;
@@ -153,7 +162,7 @@ typedef struct {
         * Points to data about a pending invoke which needs to be executed after the thread
         * resumes.
         */
-       InvokeData *invoke;
+       InvokeData *pending_invoke;
        /*
         * Set to TRUE if this thread is suspended in suspend_current () or it is executing
         * native code.
@@ -208,6 +217,13 @@ typedef struct {
         * The callee address of the last mono_runtime_invoke call
         */
        gpointer invoke_addr;
+
+       gboolean abort_requested;
+
+       /*
+        * The current mono_runtime_invoke invocation.
+        */
+       InvokeData *invoke;
 } DebuggerTlsData;
 
 /* 
@@ -217,7 +233,7 @@ typedef struct {
 #define HEADER_LENGTH 11
 
 #define MAJOR_VERSION 2
-#define MINOR_VERSION 1
+#define MINOR_VERSION 2
 
 typedef enum {
        CMD_SET_VM = 1,
@@ -266,7 +282,9 @@ typedef enum {
        ERR_NOT_IMPLEMENTED = 100,
        ERR_NOT_SUSPENDED = 101,
        ERR_INVALID_ARGUMENT = 102,
-       ERR_UNLOADED = 103
+       ERR_UNLOADED = 103,
+       ERR_NO_INVOCATION = 104,
+       ERR_ABSENT_INFORMATION = 105
 } ErrorCode;
 
 typedef enum {
@@ -318,14 +336,17 @@ typedef enum {
        CMD_VM_RESUME = 4,
        CMD_VM_EXIT = 5,
        CMD_VM_DISPOSE = 6,
-       CMD_VM_INVOKE_METHOD = 7
+       CMD_VM_INVOKE_METHOD = 7,
+       CMD_VM_SET_PROTOCOL_VERSION = 8,
+       CMD_VM_ABORT_INVOKE = 9
 } CmdVM;
 
 typedef enum {
        CMD_THREAD_GET_FRAME_INFO = 1,
        CMD_THREAD_GET_NAME = 2,
        CMD_THREAD_GET_STATE = 3,
-       CMD_THREAD_GET_INFO = 4
+       CMD_THREAD_GET_INFO = 4,
+       CMD_THREAD_GET_ID = 5
 } CmdThread;
 
 typedef enum {
@@ -344,7 +365,8 @@ typedef enum {
        CMD_APPDOMAIN_GET_ASSEMBLIES = 3,
        CMD_APPDOMAIN_GET_ENTRY_ASSEMBLY = 4,
        CMD_APPDOMAIN_CREATE_STRING = 5,
-       CMD_APPDOMAIN_GET_CORLIB = 6
+       CMD_APPDOMAIN_GET_CORLIB = 6,
+       CMD_APPDOMAIN_CREATE_BOXED_VALUE = 7,
 } CmdAppDomain;
 
 typedef enum {
@@ -383,7 +405,8 @@ typedef enum {
        CMD_TYPE_GET_PROPERTIES = 9,
        CMD_TYPE_GET_CATTRS = 10,
        CMD_TYPE_GET_FIELD_CATTRS = 11,
-       CMD_TYPE_GET_PROPERTY_CATTRS = 12
+       CMD_TYPE_GET_PROPERTY_CATTRS = 12,
+       CMD_TYPE_GET_SOURCE_FILES_2 = 13,
 } CmdType;
 
 typedef enum {
@@ -512,6 +535,8 @@ static HANDLE debugger_thread_handle;
 
 static int log_level;
 
+static gboolean embedding;
+
 static FILE *log_file;
 
 /* Classes whose class load event has been sent */
@@ -520,6 +545,9 @@ static GHashTable *loaded_classes;
 /* Assemblies whose assembly load event has no been sent yet */
 static GPtrArray *pending_assembly_loads;
 
+/* Types whose type load event has no been sent yet */
+static GPtrArray *pending_type_loads;
+
 /* Whenever the debugger thread has exited */
 static gboolean debugger_thread_exited;
 
@@ -540,6 +568,15 @@ static gpointer ss_invoke_addr = NULL;
 static int ss_count;
 #endif
 
+/* The protocol version of the client */
+static int major_version, minor_version;
+
+/* Whenever the variables above are set by the client */
+static gboolean protocol_version_set;
+
+/* A hash table containing all active domains */
+static GHashTable *domains;
+
 static void transport_connect (const char *host, int port);
 
 static guint32 WINAPI debugger_thread (void *arg);
@@ -548,14 +585,16 @@ static void runtime_initialized (MonoProfiler *prof);
 
 static void runtime_shutdown (MonoProfiler *prof);
 
-static void thread_startup (MonoProfiler *prof, intptr_t tid);
+static void thread_startup (MonoProfiler *prof, uintptr_t tid);
 
-static void thread_end (MonoProfiler *prof, intptr_t tid);
+static void thread_end (MonoProfiler *prof, uintptr_t tid);
 
 static void appdomain_load (MonoProfiler *prof, MonoDomain *domain, int result);
 
 static void appdomain_unload (MonoProfiler *prof, MonoDomain *domain);
 
+static void invalidate_each_thread (gpointer key, gpointer value, gpointer user_data);
+
 static void assembly_load (MonoProfiler *prof, MonoAssembly *assembly, int result);
 
 static void assembly_unload (MonoProfiler *prof, MonoAssembly *assembly);
@@ -574,6 +613,12 @@ static void stop_single_stepping (void);
 
 static void suspend_current (void);
 
+static void clear_event_requests_for_assembly (MonoAssembly *assembly);
+
+static void clear_breakpoints_for_domain (MonoDomain *domain);
+
+static void clear_types_for_assembly (MonoAssembly *assembly);
+
 /* Submodule init/cleanup */
 static void breakpoints_init (void);
 static void breakpoints_cleanup (void);
@@ -685,6 +730,8 @@ mono_debugger_agent_parse_options (char *options)
                        agent_config.timeout = atoi (arg + 8);
                } else if (strncmp (arg, "launch=", 7) == 0) {
                        agent_config.launch = g_strdup (arg + 7);
+               } else if (strncmp (arg, "embedding=", 10) == 0) {
+                       agent_config.embedding = atoi (arg + 10) == 1;
                } else {
                        print_usage ();
                        exit (1);
@@ -736,20 +783,24 @@ mono_debugger_agent_init (void)
 
        debugger_tls_id = TlsAlloc ();
 
-       thread_to_tls = mono_g_hash_table_new (NULL, NULL);
-       MONO_GC_REGISTER_ROOT (thread_to_tls);
+       thread_to_tls = mono_g_hash_table_new_type (NULL, NULL, MONO_HASH_KEY_GC);
+       MONO_GC_REGISTER_ROOT_FIXED (thread_to_tls);
 
        tid_to_thread = mono_g_hash_table_new_type (NULL, NULL, MONO_HASH_VALUE_GC);
-       MONO_GC_REGISTER_ROOT (tid_to_thread);
+       MONO_GC_REGISTER_ROOT_FIXED (tid_to_thread);
 
        tid_to_thread_obj = mono_g_hash_table_new_type (NULL, NULL, MONO_HASH_VALUE_GC);
-       MONO_GC_REGISTER_ROOT (tid_to_thread_obj);
+       MONO_GC_REGISTER_ROOT_FIXED (tid_to_thread_obj);
 
        loaded_classes = g_hash_table_new (mono_aligned_addr_hash, NULL);
        pending_assembly_loads = g_ptr_array_new ();
+       pending_type_loads = g_ptr_array_new ();
+       domains = g_hash_table_new (mono_aligned_addr_hash, NULL);
 
        log_level = agent_config.log_level;
 
+       embedding = agent_config.embedding;
+
        if (agent_config.log_file) {
                log_file = fopen (agent_config.log_file, "w+");
                if (!log_file) {
@@ -855,7 +906,7 @@ mono_debugger_agent_cleanup (void)
        //WaitForSingleObject (debugger_thread_handle, INFINITE);
        if (GetCurrentThreadId () != debugger_thread_id) {
                mono_mutex_lock (&debugger_thread_exited_mutex);
-               if (!debugger_thread_exited) {
+               while (!debugger_thread_exited) {
 #ifdef HOST_WIN32
                        if (WAIT_TIMEOUT == WaitForSingleObject(debugger_thread_exited_cond, 0)) {
                                mono_mutex_unlock (&debugger_thread_exited_mutex);
@@ -1061,6 +1112,14 @@ transport_connect (const char *host, int port)
                exit (1);
        }
 
+       /*
+        * To support older clients, the client sends its protocol version after connecting
+        * using a command. Until that is received, default to our protocol version.
+        */
+       major_version = MAJOR_VERSION;
+       minor_version = MINOR_VERSION;
+       protocol_version_set = FALSE;
+
        /* 
         * Set TCP_NODELAY on the socket so the client receives events/command
         * results immediately.
@@ -1343,26 +1402,42 @@ static ObjRef*
 get_objref (MonoObject *obj)
 {
        ObjRef *ref;
+       GSList *reflist = NULL, *l;
+       int hash = 0;
 
        if (obj == NULL)
                return 0;
 
-#ifdef HAVE_SGEN_GC
-       NOT_IMPLEMENTED;
-#endif
-
-       /* Use a hash table with masked pointers to internalize object references */
-       /* FIXME: This can grow indefinitely */
        mono_loader_lock ();
 
        if (!obj_to_objref)
                obj_to_objref = g_hash_table_new (NULL, NULL);
+       
+       /* FIXME: The tables can grow indefinitely */
 
-       ref = g_hash_table_lookup (obj_to_objref, GINT_TO_POINTER (~((gsize)obj)));
-       /* ref might refer to a different object with the same addr which was GCd */
-       if (ref && mono_gchandle_get_target (ref->handle) == obj) {
-               mono_loader_unlock ();
-               return ref;
+       if (mono_gc_is_moving ()) {
+               /*
+                * Objects can move, so use a hash table mapping hash codes to lists of
+                * ObjRef structures.
+                */
+               hash = mono_object_hash (obj);
+
+               reflist = g_hash_table_lookup (obj_to_objref, GINT_TO_POINTER (hash));
+               for (l = reflist; l; l = l->next) {
+                       ref = l->data;
+                       if (ref && mono_gchandle_get_target (ref->handle) == obj) {
+                               mono_loader_unlock ();
+                               return ref;
+                       }
+               }
+       } else {
+               /* Use a hash table with masked pointers to internalize object references */
+               ref = g_hash_table_lookup (obj_to_objref, GINT_TO_POINTER (~((gsize)obj)));
+               /* ref might refer to a different object with the same addr which was GCd */
+               if (ref && mono_gchandle_get_target (ref->handle) == obj) {
+                       mono_loader_unlock ();
+                       return ref;
+               }
        }
 
        ref = g_new0 (ObjRef, 1);
@@ -1370,7 +1445,13 @@ get_objref (MonoObject *obj)
        ref->handle = mono_gchandle_new_weakref (obj, FALSE);
 
        g_hash_table_insert (objrefs, GINT_TO_POINTER (ref->id), ref);
-       g_hash_table_insert (obj_to_objref, GINT_TO_POINTER (~((gsize)obj)), ref);
+
+       if (mono_gc_is_moving ()) {
+               reflist = g_slist_append (reflist, ref);
+               g_hash_table_insert (obj_to_objref, GINT_TO_POINTER (hash), reflist);
+       } else {
+               g_hash_table_insert (obj_to_objref, GINT_TO_POINTER (~((gsize)obj)), ref);
+       }
 
        mono_loader_unlock ();
 
@@ -1533,6 +1614,10 @@ mono_debugger_agent_free_domain_info (MonoDomain *domain)
                        }
                }
        }
+
+       mono_loader_lock ();
+       g_hash_table_remove (domains, domain);
+       mono_loader_unlock ();
 }
 
 static int
@@ -1852,7 +1937,8 @@ mono_debugger_agent_thread_interrupt (void *sigctx, MonoJitInfo *ji)
 
                        // FIXME: printf is not signal safe, but this is only used during
                        // debugger debugging
-                       DEBUG (1, printf ("[%p] Received interrupt while at %p, treating as suspended.\n", (gpointer)GetCurrentThreadId (), mono_arch_ip_from_context (sigctx)));
+                       if (sigctx)
+                               DEBUG (1, printf ("[%p] Received interrupt while at %p, treating as suspended.\n", (gpointer)GetCurrentThreadId (), mono_arch_ip_from_context (sigctx)));
                        //save_thread_context (&ctx);
 
                        if (!tls->thread)
@@ -1955,7 +2041,7 @@ notify_thread (gpointer key, gpointer value, gpointer user_data)
 #ifdef HOST_WIN32
        QueueUserAPC (notify_thread_apc, thread->handle, NULL);
 #else
-       pthread_kill ((pthread_t) tid, mono_thread_get_abort_signal ());
+       mono_thread_kill (thread, mono_thread_get_abort_signal ());
 #endif
 }
 
@@ -2182,10 +2268,10 @@ suspend_current (void)
 
        DEBUG(1, fprintf (log_file, "[%p] Resumed.\n", (gpointer)GetCurrentThreadId ()));
 
-       if (tls->invoke) {
+       if (tls->pending_invoke) {
                /* Save the original context */
-               tls->invoke->has_ctx = TRUE;
-               memcpy (&tls->invoke->ctx, &tls->ctx, sizeof (MonoContext));
+               tls->pending_invoke->has_ctx = TRUE;
+               memcpy (&tls->pending_invoke->ctx, &tls->ctx, sizeof (MonoContext));
 
                invoke_method ();
        }
@@ -2364,7 +2450,7 @@ process_frame (StackFrameInfo *info, MonoContext *ctx, gpointer user_data)
                if (info->type == FRAME_TYPE_DEBUGGER_INVOKE) {
                        /* Mark the last frame as an invoke frame */
                        if (ud->frames)
-                               ((StackFrame*)ud->frames->data)->flags |= FRAME_FLAG_DEBUGGER_INVOKE;
+                               ((StackFrame*)g_slist_last (ud->frames)->data)->flags |= FRAME_FLAG_DEBUGGER_INVOKE;
                }
                return FALSE;
        }
@@ -2382,7 +2468,7 @@ process_frame (StackFrameInfo *info, MonoContext *ctx, gpointer user_data)
                info->il_offset = mono_debug_il_offset_from_address (method, info->domain, info->native_offset);
        }
 
-       DEBUG (1, fprintf (stderr, "\tFrame: %s %d %d %d\n", mono_method_full_name (method, TRUE), info->native_offset, info->il_offset, info->managed));
+       DEBUG (1, fprintf (log_file, "\tFrame: %s %d %d %d\n", mono_method_full_name (method, TRUE), info->native_offset, info->il_offset, info->managed));
 
        if (!info->managed && method->wrapper_type != MONO_WRAPPER_DYNAMIC_METHOD) {
                /*
@@ -2754,7 +2840,7 @@ runtime_shutdown (MonoProfiler *prof)
 }
 
 static void
-thread_startup (MonoProfiler *prof, intptr_t tid)
+thread_startup (MonoProfiler *prof, uintptr_t tid)
 {
        MonoInternalThread *thread = mono_thread_internal_current ();
        MonoInternalThread *old_thread;
@@ -2795,7 +2881,7 @@ thread_startup (MonoProfiler *prof, intptr_t tid)
        // FIXME: Free this somewhere
        tls = g_new0 (DebuggerTlsData, 1);
        tls->resume_event = CreateEvent (NULL, FALSE, FALSE, NULL);
-       MONO_GC_REGISTER_ROOT (tls->thread);
+       MONO_GC_REGISTER_ROOT_SINGLE (tls->thread);
        tls->thread = thread;
        TlsSetValue (debugger_tls_id, tls);
 
@@ -2816,7 +2902,7 @@ thread_startup (MonoProfiler *prof, intptr_t tid)
 }
 
 static void
-thread_end (MonoProfiler *prof, intptr_t tid)
+thread_end (MonoProfiler *prof, uintptr_t tid)
 {
        MonoInternalThread *thread;
        DebuggerTlsData *tls = NULL;
@@ -2844,15 +2930,34 @@ thread_end (MonoProfiler *prof, intptr_t tid)
 static void
 appdomain_load (MonoProfiler *prof, MonoDomain *domain, int result)
 {
+       mono_loader_lock ();
+       g_hash_table_insert (domains, domain, domain);
+       mono_loader_unlock ();
+
        process_profiler_event (EVENT_KIND_APPDOMAIN_CREATE, domain);
 }
 
 static void
 appdomain_unload (MonoProfiler *prof, MonoDomain *domain)
 {
+       /* Invalidate each thread's frame stack */
+       mono_g_hash_table_foreach (thread_to_tls, invalidate_each_thread, NULL);
+       clear_breakpoints_for_domain (domain);
        process_profiler_event (EVENT_KIND_APPDOMAIN_UNLOAD, domain);
 }
 
+/*
+ * invalidate_each_thread:
+ *
+ *   A GHFunc to invalidate frames.
+ *   value must be a DebuggerTlsData*
+ */
+static void
+invalidate_each_thread (gpointer key, gpointer value, gpointer user_data)
+{
+       invalidate_frames (value);
+}
+
 static void
 assembly_load (MonoProfiler *prof, MonoAssembly *assembly, int result)
 {
@@ -2866,6 +2971,9 @@ static void
 assembly_unload (MonoProfiler *prof, MonoAssembly *assembly)
 {
        process_profiler_event (EVENT_KIND_ASSEMBLY_UNLOAD, assembly);
+
+       clear_event_requests_for_assembly (assembly);
+       clear_types_for_assembly (assembly);
 }
 
 static void
@@ -2899,7 +3007,7 @@ end_runtime_invoke (MonoProfiler *prof, MonoMethod *method)
        gpointer stackptr = __builtin_frame_address (1);
 #endif
 
-       if (ss_req == NULL || stackptr != ss_invoke_addr || ss_req->thread != mono_thread_internal_current ())
+       if (!embedding || ss_req == NULL || stackptr != ss_invoke_addr || ss_req->thread != mono_thread_internal_current ())
                return;
 
        /*
@@ -2922,6 +3030,21 @@ end_runtime_invoke (MonoProfiler *prof, MonoMethod *method)
        mono_loader_unlock ();
 }
 
+static void
+send_type_load (MonoClass *klass)
+{
+       gboolean type_load = FALSE;
+
+       mono_loader_lock ();
+       if (!g_hash_table_lookup (loaded_classes, klass)) {
+               type_load = TRUE;
+               g_hash_table_insert (loaded_classes, klass, klass);
+       }
+       mono_loader_unlock ();
+       if (type_load)
+               process_profiler_event (EVENT_KIND_TYPE_LOAD, klass);
+}
+
 static void
 jit_end (MonoProfiler *prof, MonoMethod *method, MonoJitInfo *jinfo, int result)
 {
@@ -2931,8 +3054,6 @@ jit_end (MonoProfiler *prof, MonoMethod *method, MonoJitInfo *jinfo, int result)
         * loader lock held. They could also occur in the debugger thread.
         * Same for assembly load events.
         */
-       gboolean type_load = FALSE;
-
        while (TRUE) {
                MonoAssembly *assembly = NULL;
 
@@ -2950,14 +3071,30 @@ jit_end (MonoProfiler *prof, MonoMethod *method, MonoJitInfo *jinfo, int result)
                        break;
        }
 
-       mono_loader_lock ();
-       if (!g_hash_table_lookup (loaded_classes, method->klass)) {
-               type_load = TRUE;
-               g_hash_table_insert (loaded_classes, method->klass, method->klass);
+       if (!vm_start_event_sent) {
+               /* Save these so they can be sent after the vm start event */
+               mono_loader_lock ();
+               g_ptr_array_add (pending_type_loads, method->klass);
+               mono_loader_unlock ();
+       } else {
+               /* Send all pending type load events */
+               MonoClass *klass;
+               while (TRUE) {
+                       klass = NULL;
+                       mono_loader_lock ();
+                       if (pending_type_loads->len > 0) {
+                               klass = g_ptr_array_index (pending_type_loads, 0);
+                               g_ptr_array_remove_index (pending_type_loads, 0);
+                       }
+                       mono_loader_unlock ();
+                       if (klass)
+                               send_type_load (klass);
+                       else
+                               break;
+               }
+
+               send_type_load (method->klass);
        }
-       mono_loader_unlock ();
-       if (type_load)
-               process_profiler_event (EVENT_KIND_TYPE_LOAD, method->klass);
 
        if (!result)
                add_pending_breakpoints (method, jinfo);
@@ -2973,8 +3110,8 @@ jit_end (MonoProfiler *prof, MonoMethod *method, MonoJitInfo *jinfo, int result)
 typedef struct {
        long il_offset, native_offset;
        guint8 *ip;
-       gboolean pending, entry;
        MonoJitInfo *ji;
+       MonoDomain *domain;
 } BreakpointInstance;
 
 /*
@@ -2989,7 +3126,6 @@ typedef struct {
         */
        MonoMethod *method;
        long il_offset;
-       gboolean pending, entry;
        EventRequest *req;
        /* 
         * A list of BreakpointInstance structures describing where the breakpoint
@@ -3011,22 +3147,6 @@ breakpoints_init (void)
        bp_locs = g_hash_table_new (NULL, NULL);
 }      
 
-static void
-breakpoints_cleanup (void)
-{
-       int i;
-
-       mono_loader_lock ();
-
-       for (i = 0; i < breakpoints->len; ++i)
-               g_free (g_ptr_array_index (breakpoints, i));
-
-       g_ptr_array_free (breakpoints, TRUE);
-       g_hash_table_destroy (bp_locs);
-
-       mono_loader_unlock ();
-}
-
 /*
  * insert_breakpoint:
  *
@@ -3034,10 +3154,10 @@ breakpoints_cleanup (void)
  * JI.
  */
 static void
-insert_breakpoint (MonoSeqPointInfo *seq_points, MonoJitInfo *ji, MonoBreakpoint *bp)
+insert_breakpoint (MonoSeqPointInfo *seq_points, MonoDomain *domain, MonoJitInfo *ji, MonoBreakpoint *bp)
 {
        int i, count;
-       gint32 il_offset, native_offset;
+       gint32 il_offset = -1, native_offset;
        BreakpointInstance *inst;
 
        native_offset = 0;
@@ -3049,14 +3169,16 @@ insert_breakpoint (MonoSeqPointInfo *seq_points, MonoJitInfo *ji, MonoBreakpoint
                        break;
        }
 
-       if (i == seq_points->len)
+       if (i == seq_points->len) {
                /* Have to handle this somehow */
-               NOT_IMPLEMENTED;
+               g_error ("Unable to insert breakpoint at %s:%d, seq_points=%d\n", mono_method_full_name (ji->method, TRUE), bp->il_offset, seq_points->len);
+       }
 
        inst = g_new0 (BreakpointInstance, 1);
        inst->native_offset = native_offset;
        inst->ip = (guint8*)ji->code_start + native_offset;
        inst->ji = ji;
+       inst->domain = domain;
 
        mono_loader_lock ();
 
@@ -3073,6 +3195,8 @@ insert_breakpoint (MonoSeqPointInfo *seq_points, MonoJitInfo *ji, MonoBreakpoint
                NOT_IMPLEMENTED;
 #endif
        }
+
+       DEBUG(1, fprintf (log_file, "[dbg] Inserted breakpoint at %s:0x%x.\n", mono_method_full_name (ji->method, TRUE), (int)il_offset));      
 }
 
 static void
@@ -3098,6 +3222,12 @@ remove_breakpoint (BreakpointInstance *inst)
 #endif
 }      
 
+static inline gboolean
+bp_matches_method (MonoBreakpoint *bp, MonoMethod *method)
+{
+       return (!bp->method || method == bp->method || (method->is_inflated && ((MonoMethodInflated*)method)->declaring == bp->method));
+}
+
 /*
  * add_pending_breakpoints:
  *
@@ -3106,7 +3236,7 @@ remove_breakpoint (BreakpointInstance *inst)
 static void
 add_pending_breakpoints (MonoMethod *method, MonoJitInfo *ji)
 {
-       int i;
+       int i, j;
        MonoSeqPointInfo *seq_points;
        MonoDomain *domain;
 
@@ -3119,8 +3249,19 @@ add_pending_breakpoints (MonoMethod *method, MonoJitInfo *ji)
 
        for (i = 0; i < breakpoints->len; ++i) {
                MonoBreakpoint *bp = g_ptr_array_index (breakpoints, i);
+               gboolean found = FALSE;
+
+               if (!bp_matches_method (bp, method))
+                       continue;
 
-               if (bp->pending && (bp->method == method || !bp->method || (method->is_inflated && ((MonoMethodInflated*)method)->declaring == bp->method))) {
+               for (j = 0; j < bp->children->len; ++j) {
+                       BreakpointInstance *inst = g_ptr_array_index (bp->children, j);
+
+                       if (inst->ji == ji)
+                               found = TRUE;
+               }
+
+               if (!found) {
                        mono_domain_lock (domain);
                        seq_points = g_hash_table_lookup (domain_jit_info (domain)->seq_points, ji->method);
                        mono_domain_unlock (domain);
@@ -3129,7 +3270,7 @@ add_pending_breakpoints (MonoMethod *method, MonoJitInfo *ji)
                                continue;
                        g_assert (seq_points);
 
-                       insert_breakpoint (seq_points, ji, bp);
+                       insert_breakpoint (seq_points, domain, ji, bp);
                }
        }
 
@@ -3152,26 +3293,41 @@ set_bp_in_method (MonoDomain *domain, MonoMethod *method, MonoSeqPointInfo *seq_
        }
        g_assert (code);
 
-       insert_breakpoint (seq_points, ji, bp);
+       insert_breakpoint (seq_points, domain, ji, bp);
 }
 
+typedef struct
+{
+       MonoBreakpoint *bp;
+       MonoDomain *domain;
+} SetBpUserData;
+
 static void
 set_bp_in_method_cb (gpointer key, gpointer value, gpointer user_data)
 {
        MonoMethod *method = key;
        MonoSeqPointInfo *seq_points = value;
-       MonoBreakpoint *bp = user_data;
-       MonoDomain *domain = mono_domain_get ();
+       SetBpUserData *ud = user_data;
+       MonoBreakpoint *bp = ud->bp;
+       MonoDomain *domain = ud->domain;
 
-       if (bp->method) {
-               if (method->is_inflated && ((MonoMethodInflated*)method)->declaring == bp->method) {
-                       /* Generic instance */
-                       set_bp_in_method (domain, method, seq_points, bp);
-               }
-       } else {
-               /* Method entry/exit */
+       if (bp_matches_method (bp, method))
                set_bp_in_method (domain, method, seq_points, bp);
-       }
+}
+
+static void
+set_bp_in_domain (gpointer key, gpointer value, gpointer user_data)
+{
+       MonoDomain *domain = key;
+       MonoBreakpoint *bp = user_data;
+       SetBpUserData ud;
+
+       ud.bp = bp;
+       ud.domain = domain;
+
+       mono_domain_lock (domain);
+       g_hash_table_foreach (domain_jit_info (domain)->seq_points, set_bp_in_method_cb, &ud);
+       mono_domain_unlock (domain);
 }
 
 /*
@@ -3185,13 +3341,10 @@ set_bp_in_method_cb (gpointer key, gpointer value, gpointer user_data)
 static MonoBreakpoint*
 set_breakpoint (MonoMethod *method, long il_offset, EventRequest *req)
 {
-       MonoSeqPointInfo *seq_points;
-       MonoDomain *domain;
        MonoBreakpoint *bp;
 
-       // FIXME: 
+       // FIXME:
        // - suspend/resume the vm to prevent code patching problems
-       // - appdomains
        // - multiple breakpoints on the same location
        // - dynamic methods
        // - races
@@ -3204,25 +3357,11 @@ set_breakpoint (MonoMethod *method, long il_offset, EventRequest *req)
 
        DEBUG(1, fprintf (log_file, "[dbg] Setting %sbreakpoint at %s:0x%x.\n", (req->event_kind == EVENT_KIND_STEP) ? "single step " : "", method ? mono_method_full_name (method, TRUE) : "<all>", (int)il_offset));
 
-       domain = mono_domain_get ();
-       mono_domain_lock (domain);
-       if (method) {
-               seq_points = g_hash_table_lookup (domain_jit_info (domain)->seq_points, method);
-               if (seq_points) {
-                       set_bp_in_method (domain, method, seq_points, bp);
-               } else {
-                       if (method->is_generic)
-                               /* There might be already JITted instances */
-                               g_hash_table_foreach (domain_jit_info (domain)->seq_points, set_bp_in_method_cb, bp);
+       mono_loader_lock ();
 
-                       /* Not yet JITted */
-                       bp->pending = TRUE;
-               }
-       } else {
-               g_hash_table_foreach (domain_jit_info (domain)->seq_points, set_bp_in_method_cb, bp);
-               bp->pending = TRUE;
-       }
-       mono_domain_unlock (domain);
+       g_hash_table_foreach (domains, set_bp_in_domain, bp);
+
+       mono_loader_unlock ();
 
        mono_loader_lock ();
        g_ptr_array_add (breakpoints, bp);
@@ -3253,6 +3392,79 @@ clear_breakpoint (MonoBreakpoint *bp)
        g_free (bp);
 }
 
+static void
+breakpoints_cleanup (void)
+{
+       int i;
+
+       mono_loader_lock ();
+       i = 0;
+       while (i < event_requests->len) {
+               EventRequest *req = g_ptr_array_index (event_requests, i);
+
+               if (req->event_kind == EVENT_KIND_BREAKPOINT) {
+                       clear_breakpoint (req->info);
+                       g_ptr_array_remove_index_fast (event_requests, i);
+                       g_free (req);
+               } else {
+                       i ++;
+               }
+       }
+
+       for (i = 0; i < breakpoints->len; ++i)
+               g_free (g_ptr_array_index (breakpoints, i));
+
+       g_ptr_array_free (breakpoints, TRUE);
+       g_hash_table_destroy (bp_locs);
+
+       breakpoints = NULL;
+       bp_locs = NULL;
+
+       mono_loader_unlock ();
+}
+
+/*
+ * clear_breakpoints_for_domain:
+ *
+ *   Clear breakpoint instances which reference DOMAIN.
+ */
+static void
+clear_breakpoints_for_domain (MonoDomain *domain)
+{
+       int i, j;
+
+       /* This could be called after shutdown */
+       if (!breakpoints)
+               return;
+
+       mono_loader_lock ();
+       for (i = 0; i < breakpoints->len; ++i) {
+               MonoBreakpoint *bp = g_ptr_array_index (breakpoints, i);
+
+               j = 0;
+               while (j < bp->children->len) {
+                       BreakpointInstance *inst = g_ptr_array_index (bp->children, j);
+
+                       if (inst->domain == domain) {
+                               remove_breakpoint (inst);
+
+                               g_free (inst);
+
+                               g_ptr_array_remove_index_fast (bp->children, j);
+                       } else {
+                               j ++;
+                       }
+               }
+       }
+       mono_loader_unlock ();
+}
+
+static gboolean
+breakpoint_matches_assembly (MonoBreakpoint *bp, MonoAssembly *assembly)
+{
+       return bp->method && bp->method->klass->image->assembly == assembly;
+}
+
 static void
 process_breakpoint_inner (DebuggerTlsData *tls, MonoContext *ctx)
 {
@@ -3889,13 +4101,25 @@ ss_destroy (SingleStepReq *req)
 
 void
 mono_debugger_agent_handle_exception (MonoException *exc, MonoContext *throw_ctx, 
-                                                                         MonoContext *catch_ctx)
+                                     MonoContext *catch_ctx)
 {
        int suspend_policy;
        GSList *events;
        MonoJitInfo *ji;
        EventInfo ei;
 
+       if (thread_to_tls != NULL) {
+               MonoInternalThread *thread = mono_thread_internal_current ();
+               DebuggerTlsData *tls;
+
+               mono_loader_lock ();
+               tls = mono_g_hash_table_lookup (thread_to_tls, thread);
+               mono_loader_unlock ();
+
+               if (tls && tls->abort_requested)
+                       return;
+       }
+
        memset (&ei, 0, sizeof (EventInfo));
 
        /* Just-In-Time debugging */
@@ -4120,7 +4344,9 @@ decode_value (MonoType *t, MonoDomain *domain, guint8 *addr, guint8 *buf, guint8
                !(t->type == MONO_TYPE_U && type == MONO_TYPE_VALUETYPE) &&
                !(t->type == MONO_TYPE_PTR && type == MONO_TYPE_I8) &&
                !(t->type == MONO_TYPE_GENERICINST && type == MONO_TYPE_VALUETYPE)) {
-               DEBUG(1, fprintf (log_file, "[%p] Expected value of type 0x%0x, got 0x%0x.\n", (gpointer)GetCurrentThreadId (), t->type, type));
+               char *name = mono_type_full_name (t);
+               DEBUG(1, fprintf (log_file, "[%p] Expected value of type %s, got 0x%0x.\n", (gpointer)GetCurrentThreadId (), name, type));
+               g_free (name);
                return ERR_INVALID_ARGUMENT;
        }
 
@@ -4342,6 +4568,82 @@ clear_event_request (int req_id, int etype)
        mono_loader_unlock ();
 }
 
+static gboolean
+event_req_matches_assembly (EventRequest *req, MonoAssembly *assembly)
+{
+       if (req->event_kind == EVENT_KIND_BREAKPOINT)
+               return breakpoint_matches_assembly (req->info, assembly);
+       else {
+               int i, j;
+
+               for (i = 0; i < req->nmodifiers; ++i) {
+                       Modifier *m = &req->modifiers [i];
+
+                       if (m->kind == MOD_KIND_EXCEPTION_ONLY && m->data.exc_class && m->data.exc_class->image->assembly == assembly)
+                               return TRUE;
+                       if (m->kind == MOD_KIND_ASSEMBLY_ONLY && m->data.assemblies) {
+                               for (j = 0; m->data.assemblies [j]; ++j)
+                                       if (m->data.assemblies [j] == assembly)
+                                               return TRUE;
+                       }
+               }
+       }
+
+       return FALSE;
+}
+
+/*
+ * clear_event_requests_for_assembly:
+ *
+ *   Clear all events requests which reference ASSEMBLY.
+ */
+static void
+clear_event_requests_for_assembly (MonoAssembly *assembly)
+{
+       int i;
+       gboolean found;
+
+       mono_loader_lock ();
+       found = TRUE;
+       while (found) {
+               found = FALSE;
+               for (i = 0; i < event_requests->len; ++i) {
+                       EventRequest *req = g_ptr_array_index (event_requests, i);
+
+                       if (event_req_matches_assembly (req, assembly)) {
+                               clear_event_request (req->id, req->event_kind);
+                               found = TRUE;
+                               break;
+                       }
+               }
+       }
+       mono_loader_unlock ();
+}
+
+/*
+ * type_comes_from_assembly:
+ *
+ *   GHRFunc that returns TRUE if klass comes from assembly
+ */
+static gboolean
+type_comes_from_assembly (gpointer klass, gpointer also_klass, gpointer assembly)
+{
+       return (mono_class_get_image ((MonoClass*)klass) == mono_assembly_get_image ((MonoAssembly*)assembly));
+}
+
+/*
+ * clear_types_for_assembly:
+ *
+ *   Clears types from loaded_classes for a given assembly
+ */
+static void
+clear_types_for_assembly (MonoAssembly *assembly)
+{
+       mono_loader_lock ();
+       g_hash_table_foreach_remove (loaded_classes, type_comes_from_assembly, assembly);
+       mono_loader_unlock ();
+}
+
 static void
 add_thread (gpointer key, gpointer value, gpointer user_data)
 {
@@ -4364,7 +4666,7 @@ do_invoke_method (DebuggerTlsData *tls, Buffer *buf, InvokeData *invoke)
        MonoObject *this, *res, *exc;
        MonoDomain *domain;
        guint8 *this_buf;
-#ifdef MONO_ARCH_HAVE_FIND_JIT_INFO_EXT
+#ifdef MONO_ARCH_SOFT_DEBUG_SUPPORTED
        MonoLMFExt ext;
 #endif
 
@@ -4387,9 +4689,17 @@ do_invoke_method (DebuggerTlsData *tls, Buffer *buf, InvokeData *invoke)
                this_buf = g_alloca (mono_class_instance_size (m->klass));
        else
                this_buf = g_alloca (sizeof (MonoObject*));
-       err = decode_value (&m->klass->byval_arg, domain, this_buf, p, &p, end);
-       if (err)
-               return err;
+       if (m->klass->valuetype && (m->flags & METHOD_ATTRIBUTE_STATIC)) {
+               /* Should be null */
+               int type = decode_byte (p, &p, end);
+               if (type != VALUE_TYPE_ID_NULL)
+                       return ERR_INVALID_ARGUMENT;
+               memset (this_buf, 0, mono_class_instance_size (m->klass));
+       } else {
+               err = decode_value (&m->klass->byval_arg, domain, this_buf, p, &p, end);
+               if (err)
+                       return err;
+       }
 
        if (!m->klass->valuetype)
                this = *(MonoObject**)this_buf;
@@ -4451,7 +4761,7 @@ do_invoke_method (DebuggerTlsData *tls, Buffer *buf, InvokeData *invoke)
         * Add an LMF frame to link the stack frames on the invoke method with our caller.
         */
        /* FIXME: Move this to arch specific code */
-#ifdef MONO_ARCH_HAVE_FIND_JIT_INFO_EXT
+#ifdef MONO_ARCH_SOFT_DEBUG_SUPPORTED
        if (invoke->has_ctx) {
                MonoLMF **lmf_addr;
 
@@ -4517,7 +4827,7 @@ do_invoke_method (DebuggerTlsData *tls, Buffer *buf, InvokeData *invoke)
 
        tls->disable_breakpoints = FALSE;
 
-#ifdef MONO_ARCH_HAVE_FIND_JIT_INFO_EXT
+#ifdef MONO_ARCH_SOFT_DEBUG_SUPPORTED
        if (invoke->has_ctx)
                mono_set_lmf ((gpointer)(((gssize)ext.lmf.previous_lmf) & ~3));
 #endif
@@ -4530,7 +4840,7 @@ do_invoke_method (DebuggerTlsData *tls, Buffer *buf, InvokeData *invoke)
 /*
  * invoke_method:
  *
- *   Invoke the method given by tls->invoke in the current thread.
+ *   Invoke the method given by tls->pending_invoke in the current thread.
  */
 static void
 invoke_method (void)
@@ -4549,9 +4859,21 @@ invoke_method (void)
        tls = TlsGetValue (debugger_tls_id);
        g_assert (tls);
 
-       invoke = tls->invoke;
+       /*
+        * Store the `InvokeData *' in `tls->invoke' until we're done with
+        * the invocation, so CMD_VM_ABORT_INVOKE can check it.
+        */
+
+       mono_loader_lock ();
+
+       invoke = tls->pending_invoke;
        g_assert (invoke);
-       tls->invoke = NULL;
+       tls->pending_invoke = NULL;
+
+       invoke->last_invoke = tls->invoke;
+       tls->invoke = invoke;
+
+       mono_loader_unlock ();
 
        tls->frames_up_to_date = FALSE;
 
@@ -4581,6 +4903,24 @@ invoke_method (void)
 
        DEBUG (1, printf ("[%p] Invoke finished, resume_count = %d.\n", (gpointer)GetCurrentThreadId (), tls->resume_count));
 
+       /*
+        * Take the loader lock to avoid race conditions with CMD_VM_ABORT_INVOKE:
+        *
+        * It is possible that ves_icall_System_Threading_Thread_Abort () was called
+        * after the mono_runtime_invoke() already returned, but it doesn't matter
+        * because we reset the abort here.
+        */
+
+       mono_loader_lock ();
+
+       if (tls->abort_requested)
+               mono_thread_internal_reset_abort (tls->thread);
+
+       tls->invoke = tls->invoke->last_invoke;
+       tls->abort_requested = FALSE;
+
+       mono_loader_unlock ();
+
        g_free (invoke->p);
        g_free (invoke);
 
@@ -4620,6 +4960,13 @@ vm_commands (int command, int id, guint8 *p, guint8 *end, Buffer *buf)
                g_free (version);
                break;
        }
+       case CMD_VM_SET_PROTOCOL_VERSION: {
+               major_version = decode_int (p, &p, end);
+               minor_version = decode_int (p, &p, end);
+               protocol_version_set = TRUE;
+               DEBUG(1, fprintf (log_file, "[dbg] Protocol version %d.%d, client protocol version %d.%d.\n", MAJOR_VERSION, MINOR_VERSION, major_version, minor_version));
+               break;
+       }
        case CMD_VM_ALL_THREADS: {
                // FIXME: Domains
                mono_loader_lock ();
@@ -4703,9 +5050,9 @@ vm_commands (int command, int id, guint8 *p, guint8 *end, Buffer *buf)
                        args [0] = g_malloc (sizeof (int));
                        *(int*)(args [0]) = exit_code;
 
-                       tls->invoke = g_new0 (InvokeData, 1);
-                       tls->invoke->method = exit_method;
-                       tls->invoke->args = args;
+                       tls->pending_invoke = g_new0 (InvokeData, 1);
+                       tls->pending_invoke->method = exit_method;
+                       tls->pending_invoke->args = args;
 
                        while (suspend_count > 0)
                                resume_vm ();
@@ -4768,15 +5115,15 @@ vm_commands (int command, int id, guint8 *p, guint8 *end, Buffer *buf)
                 * Store the invoke data into tls, the thread will execute it after it is
                 * resumed.
                 */
-               if (tls->invoke)
+               if (tls->pending_invoke)
                        NOT_IMPLEMENTED;
-               tls->invoke = g_new0 (InvokeData, 1);
-               tls->invoke->id = id;
-               tls->invoke->flags = flags;
-               tls->invoke->p = g_malloc (end - p);
-               memcpy (tls->invoke->p, p, end - p);
-               tls->invoke->endp = tls->invoke->p + (end - p);
-               tls->invoke->suspend_count = suspend_count;
+               tls->pending_invoke = g_new0 (InvokeData, 1);
+               tls->pending_invoke->id = id;
+               tls->pending_invoke->flags = flags;
+               tls->pending_invoke->p = g_malloc (end - p);
+               memcpy (tls->pending_invoke->p, p, end - p);
+               tls->pending_invoke->endp = tls->pending_invoke->p + (end - p);
+               tls->pending_invoke->suspend_count = suspend_count;
 
                if (flags & INVOKE_FLAG_SINGLE_THREADED)
                        resume_thread (THREAD_TO_INTERNAL (thread));
@@ -4784,6 +5131,49 @@ vm_commands (int command, int id, guint8 *p, guint8 *end, Buffer *buf)
                        resume_vm ();
                break;
        }
+       case CMD_VM_ABORT_INVOKE: {
+               int objid = decode_objid (p, &p, end);
+               MonoThread *thread;
+               DebuggerTlsData *tls;
+               int invoke_id, err;
+
+               err = get_object (objid, (MonoObject**)&thread);
+               if (err)
+                       return err;
+
+               invoke_id = decode_int (p, &p, end);
+
+               mono_loader_lock ();
+               tls = mono_g_hash_table_lookup (thread_to_tls, THREAD_TO_INTERNAL (thread));
+               g_assert (tls);
+
+               if (tls->abort_requested) {
+                       mono_loader_unlock ();
+                       break;
+               }
+
+               /*
+                * Check whether we're still inside the mono_runtime_invoke() and that it's
+                * actually the correct invocation.
+                *
+                * Careful, we do not stop the thread that's doing the invocation, so we can't
+                * inspect its stack.  However, invoke_method() also acquires the loader lock
+                * when it's done, so we're safe here.
+                *
+                */
+
+               if (!tls->invoke || (tls->invoke->id != invoke_id)) {
+                       mono_loader_unlock ();
+                       return ERR_NO_INVOCATION;
+               }
+
+               tls->abort_requested = TRUE;
+
+               ves_icall_System_Threading_Thread_Abort (THREAD_TO_INTERNAL (thread), NULL);
+               mono_loader_unlock ();
+               break;
+       }
+
        default:
                return ERR_NOT_IMPLEMENTED;
        }
@@ -4847,6 +5237,7 @@ event_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                                        return err;
                                req->modifiers [i].caught = decode_byte (p, &p, end);
                                req->modifiers [i].uncaught = decode_byte (p, &p, end);
+                               DEBUG(1, fprintf (log_file, "[dbg] \tEXCEPTION_ONLY filter (%s%s%s).\n", exc_class ? exc_class->name : "all", req->modifiers [i].caught ? ", caught" : "", req->modifiers [i].uncaught ? ", uncaught" : ""));
                                if (exc_class) {
                                        req->modifiers [i].data.exc_class = exc_class;
 
@@ -5015,6 +5406,30 @@ domain_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                buffer_add_objid (buf, (MonoObject*)o);
                break;
        }
+       case CMD_APPDOMAIN_CREATE_BOXED_VALUE: {
+               MonoClass *klass;
+               MonoDomain *domain2;
+               MonoObject *o;
+
+               domain = decode_domainid (p, &p, end, NULL, &err);
+               if (err)
+                       return err;
+               klass = decode_typeid (p, &p, end, &domain2, &err);
+               if (err)
+                       return err;
+
+               // FIXME:
+               g_assert (domain == domain2);
+
+               o = mono_object_new (domain, klass);
+
+               err = decode_value (&klass->byval_arg, domain, mono_object_unbox (o), p, &p, end);
+               if (err)
+                       return err;
+
+               buffer_add_objid (buf, o);
+               break;
+       }
        default:
                return ERR_NOT_IMPLEMENTED;
        }
@@ -5216,10 +5631,8 @@ buffer_add_cattrs (Buffer *buf, MonoDomain *domain, MonoImage *image, MonoClass
 }
 
 static ErrorCode
-type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
+type_commands_internal (int command, MonoClass *klass, MonoDomain *domain, guint8 *p, guint8 *end, Buffer *buf)
 {
-       MonoClass *klass;
-       MonoDomain *domain;
        MonoClass *nested;
        MonoType *type;
        gpointer iter;
@@ -5227,10 +5640,6 @@ type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
        int err, nnested;
        char *name;
 
-       klass = decode_typeid (p, &p, end, &domain, &err);
-       if (err)
-               return err;
-
        switch (command) {
        case CMD_TYPE_GET_INFO: {
                buffer_add_string (buf, klass->name_space);
@@ -5279,6 +5688,8 @@ type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                gpointer iter = NULL;
                MonoMethod *m;
 
+               mono_class_setup_methods (klass);
+
                nmethods = mono_class_num_methods (klass);
 
                buffer_add_int (buf, nmethods);
@@ -5451,7 +5862,10 @@ type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                                g_free (val);
                                return err;
                        }
-                       mono_field_static_set_value (vtable, f, val);
+                       if (MONO_TYPE_IS_REFERENCE (f->type))
+                               mono_field_static_set_value (vtable, f, *(gpointer*)val);
+                       else
+                               mono_field_static_set_value (vtable, f, val);
                        g_free (val);
                }
                break;
@@ -5461,7 +5875,8 @@ type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                buffer_add_objid (buf, o);
                break;
        }
-       case CMD_TYPE_GET_SOURCE_FILES: {
+       case CMD_TYPE_GET_SOURCE_FILES:
+       case CMD_TYPE_GET_SOURCE_FILES_2: {
                gpointer iter = NULL;
                MonoMethod *method;
                char *source_file, *base;
@@ -5475,6 +5890,8 @@ type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
 
                        if (minfo) {
                                mono_debug_symfile_get_line_numbers (minfo, &source_file, NULL, NULL, NULL);
+                               if (!source_file)
+                                       continue;
 
                                for (i = 0; i < files->len; ++i)
                                        if (!strcmp (g_ptr_array_index (files, i), source_file))
@@ -5488,9 +5905,13 @@ type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                buffer_add_int (buf, files->len);
                for (i = 0; i < files->len; ++i) {
                        source_file = g_ptr_array_index (files, i);
-                       base = g_path_get_basename (source_file);
-                       buffer_add_string (buf, base);
-                       g_free (base);
+                       if (command == CMD_TYPE_GET_SOURCE_FILES_2) {
+                               buffer_add_string (buf, source_file);
+                       } else {
+                               base = g_path_get_basename (source_file);
+                               buffer_add_string (buf, base);
+                               g_free (base);
+                       }
                        g_free (source_file);
                }
                g_ptr_array_free (files, TRUE);
@@ -5515,17 +5936,33 @@ type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
 }
 
 static ErrorCode
-method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
+type_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
 {
-       int err;
+       MonoClass *klass;
+       MonoDomain *old_domain;
        MonoDomain *domain;
-       MonoMethod *method;
-       MonoMethodHeader *header;
+       int err;
 
-       method = decode_methodid (p, &p, end, &domain, &err);
+       klass = decode_typeid (p, &p, end, &domain, &err);
        if (err)
                return err;
 
+       old_domain = mono_domain_get ();
+
+       mono_domain_set (domain, TRUE);
+
+       err = type_commands_internal (command, klass, domain, p, end, buf);
+
+       mono_domain_set (old_domain, TRUE);
+
+       return err;
+}
+
+static ErrorCode
+method_commands_internal (int command, MonoMethod *method, MonoDomain *domain, guint8 *p, guint8 *end, Buffer *buf)
+{
+       MonoMethodHeader *header;
+
        switch (command) {
        case CMD_METHOD_GET_NAME: {
                buffer_add_string (buf, method->name);
@@ -5555,6 +5992,7 @@ method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                        buffer_add_int (buf, header->code_size);
                        buffer_add_string (buf, "");
                        buffer_add_int (buf, 0);
+                       mono_metadata_free_mh (header);
                        break;
                }
 
@@ -5571,6 +6009,7 @@ method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                g_free (source_file);
                g_free (il_offsets);
                g_free (line_numbers);
+               mono_metadata_free_mh (header);
                break;
        }
        case CMD_METHOD_GET_PARAM_INFO: {
@@ -5601,8 +6040,7 @@ method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
        }
        case CMD_METHOD_GET_LOCALS_INFO: {
                int i, j, num_locals;
-               char **local_names;
-               int *local_indexes;
+               MonoDebugLocalsInfo *locals;
 
                header = mono_method_get_header (method);
                g_assert (header);
@@ -5614,25 +6052,38 @@ method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                        buffer_add_typeid (buf, domain, mono_class_from_mono_type (header->locals [i]));
 
                /* Names */
-               num_locals = mono_debug_lookup_locals (method, &local_names, &local_indexes);
+               locals = mono_debug_lookup_locals (method);
+               if (locals)
+                       num_locals = locals->num_locals;
+               else
+                       num_locals = 0;
                for (i = 0; i < header->num_locals; ++i) {
                        for (j = 0; j < num_locals; ++j)
-                               if (local_indexes [j] == i)
+                               if (locals->locals [j].index == i)
                                        break;
                        if (j < num_locals)
-                               buffer_add_string (buf, local_names [j]);
+                               buffer_add_string (buf, locals->locals [j].name);
                        else
                                buffer_add_string (buf, "");
                }
-               g_free (local_names);
-               g_free (local_indexes);
 
-               /* Live ranges */
-               /* FIXME: This works because we set debug_options.mdb_optimizations */
+               /* Scopes */
                for (i = 0; i < header->num_locals; ++i) {
-                       buffer_add_int (buf, 0);
-                       buffer_add_int (buf, header->code_size);
+                       for (j = 0; j < num_locals; ++j)
+                               if (locals->locals [j].index == i)
+                                       break;
+                       if (j < num_locals && locals->locals [j].block) {
+                               buffer_add_int (buf, locals->locals [j].block->start_offset);
+                               buffer_add_int (buf, locals->locals [j].block->end_offset);
+                       } else {
+                               buffer_add_int (buf, 0);
+                               buffer_add_int (buf, header->code_size);
+                       }
                }
+               mono_metadata_free_mh (header);
+
+               if (locals)
+                       mono_debug_symfile_free_locals (locals);
 
                break;
        }
@@ -5652,6 +6103,7 @@ method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                        for (i = 0; i < header->code_size; ++i)
                                buffer_add_byte (buf, header->code [i]);
                }
+               mono_metadata_free_mh (header);
                break;
        }
        case CMD_METHOD_RESOLVE_TOKEN: {
@@ -5722,6 +6174,29 @@ method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
        return ERR_NONE;
 }
 
+static ErrorCode
+method_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
+{
+       int err;
+       MonoDomain *old_domain;
+       MonoDomain *domain;
+       MonoMethod *method;
+
+       method = decode_methodid (p, &p, end, &domain, &err);
+       if (err)
+               return err;
+
+       old_domain = mono_domain_get ();
+
+       mono_domain_set (domain, TRUE);
+
+       err = method_commands_internal (command, method, domain, p, end, buf);
+
+       mono_domain_set (old_domain, TRUE);
+
+       return err;
+}
+
 static ErrorCode
 thread_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
 {
@@ -5800,6 +6275,9 @@ thread_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
        case CMD_THREAD_GET_INFO:
                buffer_add_byte (buf, thread->threadpool_thread);
                break;
+       case CMD_THREAD_GET_ID:
+               buffer_add_long (buf, (guint64)(gsize)thread);
+               break;
        default:
                return ERR_NOT_IMPLEMENTED;
        }
@@ -5852,7 +6330,9 @@ frame_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
 
        if (!frame->jit) {
                frame->jit = mono_debug_find_method (frame->method, frame->domain);
-               g_assert (frame->jit);
+               if (!frame->jit)
+                       /* This could happen for aot images with no jit debug info */
+                       return ERR_ABSENT_INFORMATION;
        }
        jit = frame->jit;
 
@@ -5882,6 +6362,7 @@ frame_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
                                add_var (buf, header->locals [pos], &jit->locals [pos], &frame->ctx, frame->domain, FALSE);
                        }
                }
+               mono_metadata_free_mh (header);
                break;
        }
        case CMD_STACK_FRAME_GET_THIS: {
@@ -5937,6 +6418,7 @@ frame_commands (int command, guint8 *p, guint8 *end, Buffer *buf)
 
                        set_var (t, var, &frame->ctx, frame->domain, val_buf);
                }
+               mono_metadata_free_mh (header);
                break;
        }
        default: