[runtime]Add the marshal lock to the lock tracer.

data/lock-decoder/LockTracerDecoder.cs

@@ -122,6 +122,8 @@ public enum Lock {
 	DomainAssembliesLock,
 	DomainJitCodeHashLock,
 	IcallLock,
+	AssemblyBindingLock,
+	MarshalLock,
 }
 
 public class SimLock

mono/metadata/lock-tracer.h

@@ -17,6 +17,7 @@ typedef enum {
 	DomainJitCodeHashLock,
 	IcallLock,
 	AssemblyBindingLock,
+	MarshalLock
 } RuntimeLocks;
 
 #ifdef LOCK_TRACER

mono/metadata/marshal.c

@@ -72,11 +72,12 @@ typedef struct _MonoRemotingMethods MonoRemotingMethods;
 /* 
  * This mutex protects the various marshalling related caches in MonoImage
  * and a few other data structures static to this file.
- * Note that when this lock is held it is not possible to take other runtime
- * locks like the loader lock.
+ *
+ * The marshal lock is a non-recursive complex lock that sits below the domain lock in the
+ * runtime locking latice. Which means it can take simple locks suck as the image lock.
  */
-#define mono_marshal_lock() EnterCriticalSection (&marshal_mutex)
-#define mono_marshal_unlock() LeaveCriticalSection (&marshal_mutex)
+#define mono_marshal_lock() mono_locks_acquire (&marshal_mutex, MarshalLock)
+#define mono_marshal_unlock() mono_locks_release (&marshal_mutex, MarshalLock)
 static CRITICAL_SECTION marshal_mutex;
 static gboolean marshal_mutex_initialized;