2004-05-25 Atsushi Enomoto <atsushi@ximian.com>
[mono.git] / mono / metadata / marshal.c
index 1eabd529d4d8e667dc7a9fcd2f5800777400a395..7f04c457615669b678256a4d84482d6b68d008fc 100644 (file)
 #include "metadata/appdomain.h"
 #include "mono/metadata/debug-helpers.h"
 #include "mono/metadata/threadpool.h"
+#include "mono/metadata/threads.h"
 #include "mono/metadata/monitor.h"
 #include <string.h>
+#include <errno.h>
 
 //#define DEBUG_RUNTIME_CODE
 
@@ -76,6 +78,27 @@ static MonoDisHelper marshal_dh = {
 };
 #endif 
 
+/* This mutex protects the various marshalling related caches in MonoImage */
+static CRITICAL_SECTION marshal_mutex;
+
+/* Maps wrapper methods to the methods they wrap */
+static MonoGHashTable *wrapper_hash;
+
+static guint32 last_error_tls_id;
+
+void
+mono_marshal_init (void)
+{
+       static gboolean module_initialized = FALSE;
+
+       if (!module_initialized) {
+               module_initialized = TRUE;
+               InitializeCriticalSection (&marshal_mutex);
+               wrapper_hash = mono_g_hash_table_new (NULL, NULL);
+               last_error_tls_id = TlsAlloc ();
+       }
+}
+
 gpointer
 mono_delegate_to_ftnptr (MonoDelegate *delegate)
 {
@@ -111,6 +134,21 @@ mono_delegate_to_ftnptr (MonoDelegate *delegate)
        return delegate->delegate_trampoline;
 }
 
+MonoDelegate*
+mono_ftnptr_to_delegate (MonoClass *klass, gpointer ftn)
+{
+       MonoDelegate *d;
+
+       d = (MonoDelegate*)mono_object_new (mono_domain_get (), klass);
+
+       /* FIXME: Add a managed->native wrapper */
+       g_assert_not_reached ();
+
+       mono_delegate_ctor ((MonoObject*)d, NULL, ftn);
+
+       return d;
+}
+
 gpointer
 mono_array_to_savearray (MonoArray *array)
 {
@@ -145,12 +183,12 @@ mono_string_utf8_to_builder (MonoStringBuilder *sb, char *text)
        l = strlen (text);
 
        ut = g_utf8_to_utf16 (text, l, NULL, &items_written, &error);
-
-       if (items_written > sb->capacity)
-               items_written = sb->capacity;
+       
+       if (items_written > mono_stringbuilder_capacity (sb))
+               items_written = mono_stringbuilder_capacity (sb);
        
        if (!error) {
-               memcpy (sb->chars->vector, ut, items_written * 2);
+               memcpy (mono_string_chars (sb->str), ut, items_written * 2);
                sb->length = items_written;
        } else 
                g_error_free (error);
@@ -158,22 +196,51 @@ mono_string_utf8_to_builder (MonoStringBuilder *sb, char *text)
        g_free (ut);
 }
 
+void
+mono_string_utf16_to_builder (MonoStringBuilder *sb, gunichar2 *text)
+{
+       guint32 len;
+
+       if (!sb || !text)
+               return;
+
+       g_assert (mono_string_chars (sb->str) == text);
+
+       for (len = 0; text [len] != 0; ++len)
+               ;
+
+       sb->length = len;
+}
+
 gpointer
 mono_string_builder_to_utf8 (MonoStringBuilder *sb)
 {
-       char *res;
+       GError *error = NULL;
+       glong *res;
 
        if (!sb)
                return NULL;
 
-       res = g_malloc (sb->capacity + 1);
+       res = g_malloc0 (mono_stringbuilder_capacity (sb) + 1);
 
-       /* fixme: copy the content of the string builder? */
-       res [0] = 0;
+       g_utf16_to_utf8 (mono_string_chars (sb->str), sb->length, NULL, res, &error);
+       if (error) {
+               g_error_free (error);
+               mono_raise_exception (mono_get_exception_execution_engine ("Failed to convert StringBuilder from utf16 to utf8"));
+       }
 
        return res;
 }
 
+gpointer
+mono_string_builder_to_utf16 (MonoStringBuilder *sb)
+{
+       if (!sb)
+               return NULL;
+
+       return mono_string_chars (sb->str);
+}
+
 gpointer
 mono_string_to_ansibstr (MonoString *string_obj)
 {
@@ -408,7 +475,7 @@ mono_mb_emit_i2 (MonoMethodBuilder *mb, gint16 data)
 void
 mono_mb_emit_ldstr (MonoMethodBuilder *mb, char *str)
 {
-        mono_mb_emit_byte (mb, CEE_LDSTR);
+       mono_mb_emit_byte (mb, CEE_LDSTR);
        mono_mb_emit_i4 (mb, mono_mb_add_data (mb, str));
 }
 
@@ -498,50 +565,71 @@ mono_mb_emit_icon (MonoMethodBuilder *mb, gint32 value)
        }
 }
 
+guint32
+mono_mb_emit_branch (MonoMethodBuilder *mb, guint8 op)
+{
+       guint32 res;
+       mono_mb_emit_byte (mb, op);
+       res = mb->pos;
+       mono_mb_emit_i4 (mb, 0);
+       return res;
+}
+
+static void
+mono_mb_emit_calli (MonoMethodBuilder *mb, MonoMethodSignature *sig)
+{
+       mono_mb_emit_byte (mb, CEE_CALLI);
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, sig));
+}
+
 void
 mono_mb_emit_managed_call (MonoMethodBuilder *mb, MonoMethod *method, MonoMethodSignature *opt_sig)
 {
-       if (!opt_sig)
-               opt_sig = method->signature;
        mono_mb_emit_byte (mb, CEE_PREFIX1);
        mono_mb_emit_byte (mb, CEE_LDFTN);
        mono_mb_emit_i4 (mb, mono_mb_add_data (mb, method));
-       mono_mb_emit_byte (mb, CEE_CALLI);
-       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, opt_sig));
+       mono_mb_emit_calli (mb, opt_sig ? opt_sig : method->signature);
 }
 
 void
 mono_mb_emit_native_call (MonoMethodBuilder *mb, MonoMethodSignature *sig, gpointer func)
 {
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_SAVE_LMF);
        mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
        mono_mb_emit_byte (mb, CEE_MONO_LDPTR);
        mono_mb_emit_i4 (mb, mono_mb_add_data (mb, func));
-       mono_mb_emit_byte (mb, CEE_CALLI);
-       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, sig));
+       mono_mb_emit_calli (mb, sig);
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_RESTORE_LMF);
 }
 
 void
-mono_mb_emit_exception (MonoMethodBuilder *mb)
+mono_mb_emit_exception (MonoMethodBuilder *mb, const char *exc_name, const char *msg)
 {
        /* fixme: we need a better way to throw exception,
         * supporting several exception types and messages */
-       static MonoMethod *missing_method_ctor = NULL;
+       MonoMethod *ctor = NULL;
 
-       if (!missing_method_ctor) {
-               MonoClass *mme = mono_class_from_name (mono_defaults.corlib, "System", "MissingMethodException");
-               int i;
-               mono_class_init (mme);
-               for (i = 0; i < mme->method.count; ++i) {
-                       if (strcmp (mme->methods [i]->name, ".ctor") == 0 && mme->methods [i]->signature->param_count == 0) {
-                               missing_method_ctor = mme->methods [i];
-                               break;
-                       }
+       MonoClass *mme = mono_class_from_name (mono_defaults.corlib, "System", exc_name);
+       int i;
+       mono_class_init (mme);
+       for (i = 0; i < mme->method.count; ++i) {
+               if (strcmp (mme->methods [i]->name, ".ctor") == 0 && mme->methods [i]->signature->param_count == 0) {
+                       ctor = mme->methods [i];
+                       break;
                }
        }
+       g_assert (ctor);
        mono_mb_emit_byte (mb, CEE_NEWOBJ);
-       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, missing_method_ctor));
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, ctor));
+       if (msg != NULL) {
+               mono_mb_emit_byte (mb, CEE_DUP);
+               mono_mb_emit_ldflda (mb, G_STRUCT_OFFSET (MonoException, message));
+               mono_mb_emit_ldstr (mb, (char*)msg);
+               mono_mb_emit_byte (mb, CEE_STIND_I);
+       }
        mono_mb_emit_byte (mb, CEE_THROW);
-       
 }
 
 void
@@ -559,25 +647,35 @@ emit_ptr_to_str_conv (MonoMethodBuilder *mb, MonoType *type, MonoMarshalConv con
 {
        switch (conv) {
        case MONO_MARSHAL_CONV_BOOL_I4:
+               mono_mb_emit_byte (mb, CEE_LDLOC_1);
                mono_mb_emit_byte (mb, CEE_LDLOC_0);
-               mono_mb_emit_byte (mb, CEE_LDIND_I);
+               mono_mb_emit_byte (mb, CEE_LDIND_I4);
                mono_mb_emit_byte (mb, CEE_BRFALSE_S);
-               mono_mb_emit_byte (mb, 5);
-               mono_mb_emit_byte (mb, CEE_LDLOC_1);
+               mono_mb_emit_byte (mb, 3);
                mono_mb_emit_byte (mb, CEE_LDC_I4_1);
-               mono_mb_emit_byte (mb, CEE_STIND_I1);
                mono_mb_emit_byte (mb, CEE_BR_S);
-               mono_mb_emit_byte (mb, 3);
+               mono_mb_emit_byte (mb, 1);
+               mono_mb_emit_byte (mb, CEE_LDC_I4_0);
+               mono_mb_emit_byte (mb, CEE_STIND_I1);
+               break;
+       case MONO_MARSHAL_CONV_BOOL_VARIANTBOOL:
                mono_mb_emit_byte (mb, CEE_LDLOC_1);
+               mono_mb_emit_byte (mb, CEE_LDLOC_0);
+               mono_mb_emit_byte (mb, CEE_LDIND_I2);
+               mono_mb_emit_byte (mb, CEE_BRFALSE_S);
+               mono_mb_emit_byte (mb, 3);
+               mono_mb_emit_byte (mb, CEE_LDC_I4_1);
+               mono_mb_emit_byte (mb, CEE_BR_S);
+               mono_mb_emit_byte (mb, 1);
                mono_mb_emit_byte (mb, CEE_LDC_I4_0);
                mono_mb_emit_byte (mb, CEE_STIND_I1);
                break;
        case MONO_MARSHAL_CONV_ARRAY_BYVALARRAY: {
-               MonoClass *eclass;
+               MonoClass *eclass = NULL;
                int esize;
 
                if (type->type == MONO_TYPE_SZARRAY) {
-                       eclass = mono_class_from_mono_type (type->data.type);
+                       eclass = type->data.klass;
                } else {
                        g_assert_not_reached ();
                }
@@ -667,14 +765,15 @@ emit_ptr_to_str_conv (MonoMethodBuilder *mb, MonoType *type, MonoMarshalConv con
                // know if thats the correct behaviour
                break;
        }
+       case MONO_MARSHAL_CONV_ARRAY_LPARRAY:
+               g_error ("Structure field of type %s can't be marshalled as LPArray", mono_class_from_mono_type (type)->name);
+               break;
        case MONO_MARSHAL_CONV_STR_LPWSTR:
        case MONO_MARSHAL_CONV_STR_BSTR:
        case MONO_MARSHAL_CONV_STR_ANSIBSTR:
        case MONO_MARSHAL_CONV_STR_TBSTR:
        case MONO_MARSHAL_CONV_ARRAY_SAVEARRAY:
-       case MONO_MARSHAL_CONV_ARRAY_LPARRAY:
        case MONO_MARSHAL_CONV_STR_BYVALWSTR: 
-       case MONO_MARSHAL_CONV_BOOL_VARIANTBOOL:
        default:
                g_warning ("marshaling conversion %d not implemented", conv);
                g_assert_not_reached ();
@@ -694,6 +793,13 @@ emit_str_to_ptr_conv (MonoMethodBuilder *mb, MonoType *type, MonoMarshalConv con
                mono_mb_emit_byte (mb, CEE_LDIND_U1);
                mono_mb_emit_byte (mb, CEE_STIND_I4);
                break;
+       case MONO_MARSHAL_CONV_BOOL_VARIANTBOOL:
+               mono_mb_emit_byte (mb, CEE_LDLOC_1);
+               mono_mb_emit_byte (mb, CEE_LDLOC_0);
+               mono_mb_emit_byte (mb, CEE_LDIND_U1);
+               mono_mb_emit_byte (mb, CEE_NEG);
+               mono_mb_emit_byte (mb, CEE_STIND_I2);
+               break;
        case MONO_MARSHAL_CONV_STR_LPWSTR:
        case MONO_MARSHAL_CONV_STR_LPSTR:
        case MONO_MARSHAL_CONV_STR_LPTSTR:
@@ -743,19 +849,19 @@ emit_str_to_ptr_conv (MonoMethodBuilder *mb, MonoType *type, MonoMarshalConv con
                break;
        }
        case MONO_MARSHAL_CONV_ARRAY_BYVALARRAY: {
-               MonoClass *eclass;
+               MonoClass *eclass = NULL;
                int esize;
 
                if (type->type == MONO_TYPE_SZARRAY) {
-                       eclass = mono_class_from_mono_type (type->data.type);
+                       eclass = type->data.klass;
                } else {
                        g_assert_not_reached ();
                }
 
-               if (eclass->valuetype)
-                       esize = mono_class_native_size (eclass, NULL);
-               else
-                       esize = sizeof (gpointer);
+               if (eclass->valuetype)
+                       esize = mono_class_native_size (eclass, NULL);
+               else
+                       esize = sizeof (gpointer);
 
                if (!usize) 
                        break;
@@ -817,10 +923,13 @@ emit_str_to_ptr_conv (MonoMethodBuilder *mb, MonoType *type, MonoMarshalConv con
                mono_mb_patch_addr_s (mb, pos, mb->pos - pos - 1);
                break;
        }
-       case MONO_MARSHAL_CONV_BOOL_VARIANTBOOL:
-       default:
-               g_warning ("marshalling conversion %d not implemented", conv);
-               g_assert_not_reached ();
+       default: {
+               char *msg = g_strdup_printf ("marshalling conversion %d not implemented", conv);
+               MonoException *exc = mono_get_exception_not_implemented (msg);
+               g_warning (msg);
+               g_free (msg);
+               mono_raise_exception (exc);
+       }
        }
 }
 
@@ -835,6 +944,9 @@ emit_struct_conv (MonoMethodBuilder *mb, MonoClass *klass, gboolean to_object)
 
        info = mono_marshal_load_type_info (klass);
 
+       if (info->native_size == 0)
+               return;
+
        if (klass->blittable) {
                mono_mb_emit_byte (mb, CEE_LDLOC_1);
                mono_mb_emit_byte (mb, CEE_LDLOC_0);
@@ -861,9 +973,13 @@ emit_struct_conv (MonoMethodBuilder *mb, MonoClass *klass, gboolean to_object)
                        msize = klass->instance_size - info->fields [i].field->offset;
                        usize = info->native_size - info->fields [i].offset;
                } else {
-                       msize = klass->fields [i + 1].offset - info->fields [i].field->offset;
+                       msize = info->fields [i + 1].field->offset - info->fields [i].field->offset;
                        usize = info->fields [i + 1].offset - info->fields [i].offset;
                }
+               if ((msize < 0) || (usize < 0))
+                       /* This happens with GC aware auto layout */
+                       g_error ("Type %s which is passed to unmanaged code must have a StructLayout attribute", mono_type_full_name (&klass->byval_arg));
+
                g_assert ((msize >= 0) && (usize >= 0));
 
                switch (conv) {
@@ -902,6 +1018,7 @@ emit_struct_conv (MonoMethodBuilder *mb, MonoClass *klass, gboolean to_object)
                                break;
                        case MONO_TYPE_I2:
                        case MONO_TYPE_U2:
+                       case MONO_TYPE_CHAR:
                                mono_mb_emit_byte (mb, CEE_LDLOC_1);
                                mono_mb_emit_byte (mb, CEE_LDLOC_0);
                                mono_mb_emit_byte (mb, CEE_LDIND_I2);
@@ -959,6 +1076,29 @@ emit_struct_conv (MonoMethodBuilder *mb, MonoClass *klass, gboolean to_object)
        }
 }
 
+static void
+emit_thread_interrupt_checkpoint (MonoMethodBuilder *mb)
+{
+       static MonoMethodSignature *state_check_sig = NULL;
+       int pos_noabort;
+       
+       if (!state_check_sig) {
+               state_check_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 0);
+               state_check_sig->ret = &mono_defaults.void_class->byval_arg;
+               state_check_sig->pinvoke = 0;
+       }
+       
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_LDPTR);
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, (gpointer) mono_thread_interruption_request_flag ()));
+       mono_mb_emit_byte (mb, CEE_LDIND_I4);
+       pos_noabort = mono_mb_emit_branch (mb, CEE_BRFALSE);
+
+       mono_mb_emit_native_call (mb, state_check_sig, mono_thread_interruption_checkpoint);
+       
+       mono_mb_patch_addr (mb, pos_noabort, mb->pos - (pos_noabort + 4));
+}
+
 static MonoAsyncResult *
 mono_delegate_begin_invoke (MonoDelegate *delegate, gpointer *params)
 {
@@ -972,6 +1112,33 @@ mono_delegate_begin_invoke (MonoDelegate *delegate, gpointer *params)
 
        g_assert (delegate);
 
+       if (delegate->target && mono_object_class (delegate->target) == mono_defaults.transparent_proxy_class) {
+
+               MonoTransparentProxy* tp = (MonoTransparentProxy *)delegate->target;
+               if (!tp->remote_class->proxy_class->contextbound || tp->rp->context != (MonoObject *) mono_context_get ()) {
+
+                       // If the target is a proxy, make a direct call. Is proxy's work
+                       // to make the call asynchronous.
+
+                       MonoAsyncResult *ares;
+                       MonoObject *exc;
+                       MonoArray *out_args;
+                       HANDLE handle;
+                       method = delegate->method_info->method;
+
+                       msg = mono_method_call_message_new (mono_marshal_method_from_wrapper (method), params, NULL, &async_callback, &state);
+                       handle = CreateEvent (NULL, TRUE, FALSE, NULL);
+                       ares = mono_async_result_new (mono_domain_get (), handle, state, handle);
+                       ares->async_delegate = (MonoObject *)delegate;
+                       ares->async_callback = (MonoObject *)async_callback;
+                       msg->async_result = ares;
+                       msg->call_type = CallType_BeginInvoke;
+
+                       mono_remoting_invoke ((MonoObject *)tp->rp, msg, &exc, &out_args);
+                       return ares;
+               }
+       }
+
        klass = delegate->object.vtable->klass;
 
        method = mono_get_delegate_invoke (klass);
@@ -986,7 +1153,6 @@ mono_delegate_begin_invoke (MonoDelegate *delegate, gpointer *params)
        g_assert (method != NULL);
 
        im = mono_get_delegate_invoke (method->klass);
-       
        msg = mono_method_call_message_new (method, params, im, &async_callback, &state);
 
        return mono_thread_pool_add ((MonoObject *)delegate, msg, async_callback, state);
@@ -1057,6 +1223,94 @@ mono_signature_to_name (MonoMethodSignature *sig, const char *prefix)
        return result;
 }
 
+/**
+ * mono_marshal_get_string_encoding:
+ *
+ *  Return the string encoding which should be used for a given parameter.
+ */
+static MonoMarshalNative
+mono_marshal_get_string_encoding (MonoMethodPInvoke *piinfo, MonoMarshalSpec *spec)
+{
+       /* First try the parameter marshal info */
+       if (spec) {
+               if (spec->native == MONO_NATIVE_LPARRAY) {
+                       if (spec->data.array_data.elem_type != 0)
+                               return spec->data.array_data.elem_type;
+               }
+               else
+                       return spec->native;
+       }
+
+       if (!piinfo)
+               return MONO_NATIVE_LPSTR;
+
+       /* Then try the method level marshal info */
+       switch (piinfo->piflags & PINVOKE_ATTRIBUTE_CHAR_SET_MASK) {
+       case PINVOKE_ATTRIBUTE_CHAR_SET_ANSI:
+               return MONO_NATIVE_LPSTR;
+       case PINVOKE_ATTRIBUTE_CHAR_SET_UNICODE:
+               return MONO_NATIVE_LPWSTR;
+       case PINVOKE_ATTRIBUTE_CHAR_SET_AUTO:
+               return MONO_NATIVE_LPTSTR;
+       default:
+               return MONO_NATIVE_LPSTR;
+       }
+}
+
+static inline MonoMethod*
+mono_marshal_find_in_cache (GHashTable *cache, gpointer key)
+{
+       MonoMethod *res;
+
+       EnterCriticalSection (&marshal_mutex);
+       res = g_hash_table_lookup (cache, key);
+       LeaveCriticalSection (&marshal_mutex);
+       return res;
+}
+
+/* Create the method from the builder and place it in the cache */
+static inline MonoMethod*
+mono_mb_create_and_cache (GHashTable *cache, gpointer key,
+                                                          MonoMethodBuilder *mb, MonoMethodSignature *sig,
+                                                          int max_stack)
+{
+       MonoMethod *res;
+
+       EnterCriticalSection (&marshal_mutex);
+       res = g_hash_table_lookup (cache, key);
+       if (!res) {
+               /* This does not acquire any locks */
+               res = mono_mb_create_method (mb, sig, max_stack);
+               g_hash_table_insert (cache, key, res);
+               mono_g_hash_table_insert (wrapper_hash, res, key);
+       }
+       else
+               /* Somebody created it before us */
+               ;
+       LeaveCriticalSection (&marshal_mutex);
+
+       return res;
+}              
+
+MonoMethod *
+mono_marshal_method_from_wrapper (MonoMethod *wrapper)
+{
+       MonoMethod *res;
+
+       if (wrapper->wrapper_type == MONO_WRAPPER_NONE)
+               return wrapper;
+
+       EnterCriticalSection (&marshal_mutex);
+       res = mono_g_hash_table_lookup (wrapper_hash, wrapper);
+       LeaveCriticalSection (&marshal_mutex);
+
+       if (res && wrapper->wrapper_type == MONO_WRAPPER_REMOTING_INVOKE_WITH_CHECK)
+               /* See mono_marshal_get_remoting_invoke_with_check */
+               return (MonoMethod*)((char*)res - 1);
+       else
+               return res;
+}
+
 MonoMethod *
 mono_marshal_get_delegate_begin_invoke (MonoMethod *method)
 {
@@ -1074,17 +1328,15 @@ mono_marshal_get_delegate_begin_invoke (MonoMethod *method)
        sig = method->signature;
 
        cache = method->klass->image->delegate_begin_invoke_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, sig)))
+       if ((res = mono_marshal_find_in_cache (cache, sig)))
                return res;
 
        g_assert (sig->hasthis);
 
        if (!csig) {
-               int sigsize = sizeof (MonoMethodSignature) + 2 * sizeof (MonoType *);
-               csig = g_malloc0 (sigsize);
+               csig = mono_metadata_signature_alloc (method->klass->image, 2);
 
                /* MonoAsyncResult * begin_invoke (MonoDelegate *delegate, gpointer params[]) */
-               csig->param_count = 2;
                csig->ret = &mono_defaults.object_class->byval_arg;
                csig->params [0] = &mono_defaults.object_class->byval_arg;
                csig->params [1] = &mono_defaults.int_class->byval_arg;
@@ -1101,11 +1353,11 @@ mono_marshal_get_delegate_begin_invoke (MonoMethod *method)
        mono_mb_emit_ldarg (mb, 0);
        mono_mb_emit_ldloc (mb, params_var);
        mono_mb_emit_native_call (mb, csig, mono_delegate_begin_invoke);
+       emit_thread_interrupt_checkpoint (mb);
        mono_mb_emit_byte (mb, CEE_RET);
 
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (cache, sig, mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
-       g_hash_table_insert (cache, sig, res);
        return res;
 }
 
@@ -1146,7 +1398,16 @@ mono_delegate_end_invoke (MonoDelegate *delegate, gpointer *params)
        ares = mono_array_get (msg->args, gpointer, sig->param_count - 1);
        g_assert (ares);
 
-       res = mono_thread_pool_finish (ares, &out_args, &exc);
+       if (delegate->target && mono_object_class (delegate->target) == mono_defaults.transparent_proxy_class) {
+               MonoTransparentProxy* tp = (MonoTransparentProxy *)delegate->target;
+               msg = (MonoMethodMessage *)mono_object_new (domain, mono_defaults.mono_method_message_class);
+               mono_message_init (domain, msg, delegate->method_info, NULL);
+               msg->call_type = CallType_EndInvoke;
+               msg->async_result = ares;
+               res = mono_remoting_invoke ((MonoObject *)tp->rp, msg, &exc, &out_args);
+       }
+       else
+               res = mono_thread_pool_finish (ares, &out_args, &exc);
 
        if (exc) {
                char *strace = mono_string_to_utf8 (((MonoException*)exc)->stack_trace);
@@ -1270,17 +1531,15 @@ mono_marshal_get_delegate_end_invoke (MonoMethod *method)
        sig = method->signature;
 
        cache = method->klass->image->delegate_end_invoke_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, sig)))
+       if ((res = mono_marshal_find_in_cache (cache, sig)))
                return res;
 
        g_assert (sig->hasthis);
 
        if (!csig) {
-               int sigsize = sizeof (MonoMethodSignature) + 2 * sizeof (MonoType *);
-               csig = g_malloc0 (sigsize);
+               csig = mono_metadata_signature_alloc (method->klass->image, 2);
 
                /* MonoObject *end_invoke (MonoDelegate *delegate, gpointer params[]) */
-               csig->param_count = 2;
                csig->ret = &mono_defaults.object_class->byval_arg;
                csig->params [0] = &mono_defaults.object_class->byval_arg;
                csig->params [1] = &mono_defaults.int_class->byval_arg;
@@ -1297,6 +1556,7 @@ mono_marshal_get_delegate_end_invoke (MonoMethod *method)
        mono_mb_emit_ldarg (mb, 0);
        mono_mb_emit_ldloc (mb, params_var);
        mono_mb_emit_native_call (mb, csig, mono_delegate_end_invoke);
+       emit_thread_interrupt_checkpoint (mb);
 
        if (sig->ret->type == MONO_TYPE_VOID) {
                mono_mb_emit_byte (mb, CEE_POP);
@@ -1304,9 +1564,9 @@ mono_marshal_get_delegate_end_invoke (MonoMethod *method)
        } else
                mono_mb_emit_restore_result (mb, sig->ret);
 
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (cache, sig,
+                                                                                mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
-       g_hash_table_insert (cache, sig, res);
 
        return res;
 }
@@ -1327,7 +1587,7 @@ mono_remoting_wrapper (MonoMethod *method, gpointer *params)
        /* skip the this pointer */
        params++;
 
-       if (this->klass->contextbound && this->rp->context == (MonoObject *) mono_context_get ())
+       if (this->remote_class->proxy_class->contextbound && this->rp->context == (MonoObject *) mono_context_get ())
        {
                int i;
                MonoMethodSignature *sig = method->signature;
@@ -1346,7 +1606,7 @@ mono_remoting_wrapper (MonoMethod *method, gpointer *params)
                        }
                }
 
-               return mono_runtime_invoke (method, this->rp->unwrapped_server, mparams, NULL);
+               return mono_runtime_invoke (method, this, mparams, NULL);
        }
 
        msg = mono_method_call_message_new (method, params, NULL, NULL, NULL);
@@ -1383,7 +1643,7 @@ mono_marshal_get_remoting_invoke (MonoMethod *method)
                return method;
 
        cache = method->klass->image->remoting_invoke_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, method)))
+       if ((res = mono_marshal_find_in_cache (cache, method)))
                return res;
 
        if (!csig) {
@@ -1404,6 +1664,7 @@ mono_marshal_get_remoting_invoke (MonoMethod *method)
        mono_mb_emit_i4 (mb, mono_mb_add_data (mb, method));
        mono_mb_emit_ldloc (mb, params_var);
        mono_mb_emit_native_call (mb, csig, mono_remoting_wrapper);
+       emit_thread_interrupt_checkpoint (mb);
 
        if (sig->ret->type == MONO_TYPE_VOID) {
                mono_mb_emit_byte (mb, CEE_POP);
@@ -1412,9 +1673,9 @@ mono_marshal_get_remoting_invoke (MonoMethod *method)
                 mono_mb_emit_restore_result (mb, sig->ret);
        }
 
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (cache, method, mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
-       g_hash_table_insert (cache, method, res);
+
        return res;
 }
 
@@ -1438,7 +1699,7 @@ mono_marshal_get_remoting_invoke_with_check (MonoMethod *method)
        g_assert (sig->hasthis);
 
        cache = method->klass->image->remoting_invoke_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, (char *)method + 1)))
+       if ((res = mono_marshal_find_in_cache (cache, (char *)method + 1)))
                return res;
 
        mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_REMOTING_INVOKE_WITH_CHECK);
@@ -1462,9 +1723,10 @@ mono_marshal_get_remoting_invoke_with_check (MonoMethod *method)
        mono_mb_emit_managed_call (mb, method, method->signature);
        mono_mb_emit_byte (mb, CEE_RET);
 
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (cache, (char*)method + 1,
+                                                                                mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
-       g_hash_table_insert (cache, (char *)method + 1, res);
+
        return res;
 }
 
@@ -1475,7 +1737,7 @@ MonoMethod *
 mono_marshal_get_delegate_invoke (MonoMethod *method)
 {
        MonoMethodSignature *sig, *static_sig;
-       int i, sigsize;
+       int i;
        MonoMethodBuilder *mb;
        MonoMethod *res;
        GHashTable *cache;
@@ -1488,11 +1750,10 @@ mono_marshal_get_delegate_invoke (MonoMethod *method)
        sig = method->signature;
 
        cache = method->klass->image->delegate_invoke_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, sig)))
+       if ((res = mono_marshal_find_in_cache (cache, sig)))
                return res;
 
-       sigsize = sizeof (MonoMethodSignature) + sig->param_count * sizeof (MonoType *);
-       static_sig = g_memdup (sig, sigsize);
+       static_sig = mono_metadata_signature_dup (sig);
        static_sig->hasthis = 0;
 
        name = mono_signature_to_name (sig, "invoke");
@@ -1509,7 +1770,10 @@ mono_marshal_get_delegate_invoke (MonoMethod *method)
          *     prev.Invoke( args .. );
         * return this.<target>( args .. );
          */
-
+       
+       /* this wrapper can be used in unmanaged-managed transitions */
+       emit_thread_interrupt_checkpoint (mb);
+       
        /* get this->prev */
        mono_mb_emit_ldarg (mb, 0);
        mono_mb_emit_ldflda (mb, G_STRUCT_OFFSET (MonoMulticastDelegate, prev));
@@ -1575,54 +1839,66 @@ mono_marshal_get_delegate_invoke (MonoMethod *method)
        mono_mb_patch_addr (mb, pos1, mb->pos - (pos1 + 4));
        mono_mb_emit_byte (mb, CEE_RET);
 
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (cache, sig,
+                                                                                mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
 
-       g_hash_table_insert (cache, sig, res);
-
        return res;     
 }
 
 /*
  * generates IL code for the runtime invoke function 
- * MonoObject *runtime_invoke (MonoObject *this, void **params, MonoObject **exc)
+ * MonoObject *runtime_invoke (MonoObject *this, void **params, MonoObject **exc, void* method)
  *
  * we also catch exceptions if exc != null
  */
 MonoMethod *
 mono_marshal_get_runtime_invoke (MonoMethod *method)
 {
-       MonoMethodSignature *sig, *csig;
+       MonoMethodSignature *sig, *csig, *callsig;
        MonoExceptionClause *clause;
        MonoMethodHeader *header;
        MonoMethodBuilder *mb;
        MonoMethod *res;
        GHashTable *cache;
        static MonoString *string_dummy = NULL;
-       int i, pos, sigsize;
+       int i, pos;
+       char *name;
 
        g_assert (method);
+       
+       if (method->string_ctor) {
+               static MonoMethodSignature *strsig = NULL;
+               if (!strsig) {
+                       strsig = mono_metadata_signature_dup (method->signature);
+                       strsig->ret = &mono_defaults.string_class->byval_arg;
+               }
+               
+               callsig = strsig;
+       } else
+               callsig = method->signature;
 
        cache = method->klass->image->runtime_invoke_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, method)))
+       if ((res = mono_marshal_find_in_cache (cache, callsig)))
                return res;
-       
+
        /* to make it work with our special string constructors */
        if (!string_dummy)
                string_dummy = mono_string_new_wrapper ("dummy");
 
        sig = method->signature;
 
-       sigsize = sizeof (MonoMethodSignature) + 3 * sizeof (MonoType *);
-       csig = g_malloc0 (sigsize);
+       csig = mono_metadata_signature_alloc (method->klass->image, 4);
 
-       csig->param_count = 3;
        csig->ret = &mono_defaults.object_class->byval_arg;
        csig->params [0] = &mono_defaults.object_class->byval_arg;
        csig->params [1] = &mono_defaults.int_class->byval_arg;
        csig->params [2] = &mono_defaults.int_class->byval_arg;
+       csig->params [3] = &mono_defaults.int_class->byval_arg;
 
-       mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_RUNTIME_INVOKE);
+       name = mono_signature_to_name (callsig, "runtime_invoke");
+       mb = mono_mb_new (method->klass, name,  MONO_WRAPPER_RUNTIME_INVOKE);
+       g_free (name);
 
        /* allocate local 0 (object) tmp */
        mono_mb_add_local (mb, &mono_defaults.object_class->byval_arg);
@@ -1722,17 +1998,9 @@ handle_enum:
                        g_assert_not_reached ();
                }               
        }
-
-       if (method->string_ctor) {
-               MonoMethodSignature *strsig;
-
-               sigsize = sizeof (MonoMethodSignature) + sig->param_count * sizeof (MonoType *);
-               strsig = g_memdup (sig, sigsize);
-               strsig->ret = &mono_defaults.string_class->byval_arg;
-
-               mono_mb_emit_managed_call (mb, method, strsig);         
-       } else 
-               mono_mb_emit_managed_call (mb, method, NULL);
+       
+       mono_mb_emit_ldarg (mb, 3);
+       mono_mb_emit_calli (mb, callsig);
 
        if (sig->ret->byref) {
                /* fixme: */
@@ -1807,6 +2075,9 @@ handle_enum:
        mono_mb_emit_ldloc (mb, 1);
        mono_mb_emit_byte (mb, CEE_STIND_I);
 
+       mono_mb_emit_byte (mb, CEE_LDC_I4_0);
+       mono_mb_emit_stloc (mb, 0);
+
        mono_mb_emit_byte (mb, CEE_LEAVE);
        mono_mb_emit_i4 (mb, 0);
 
@@ -1817,15 +2088,13 @@ handle_enum:
        mono_mb_emit_ldloc (mb, 0);
        mono_mb_emit_byte (mb, CEE_RET);
        
-       res = mono_mb_create_method (mb, csig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (cache, callsig, mb, csig, sig->param_count + 16);
        mono_mb_free (mb);
 
        header = ((MonoMethodNormal *)res)->header;
        header->num_clauses = 1;
        header->clauses = clause;
 
-       g_hash_table_insert (cache, method, res);
-
        return res;     
 }
 
@@ -1837,22 +2106,32 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
 {
        MonoMethodSignature *sig, *csig;
        MonoMethodBuilder *mb;
-       MonoClass *klass;
+       MonoClass *klass = NULL;
        MonoMethod *res;
        GHashTable *cache;
-       int i, pos, sigsize, *tmp_locals;
+       int i, pos = 0, *tmp_locals;
+       static MonoMethodSignature *alloc_sig = NULL;
+       int retobj_var = 0;
 
        g_assert (method != NULL);
        g_assert (!method->signature->pinvoke);
 
        cache = method->klass->image->managed_wrapper_cache;
-       if (!this && (res = (MonoMethod *)g_hash_table_lookup (cache, method)))
+       if (!this && (res = mono_marshal_find_in_cache (cache, method)))
                return res;
 
+       /* Under MS, the allocation should be done using CoTaskMemAlloc */
+       if (!alloc_sig) {
+               alloc_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 1);
+               alloc_sig->params [0] = &mono_defaults.int_class->byval_arg;
+               alloc_sig->ret = &mono_defaults.int_class->byval_arg;
+               alloc_sig->pinvoke = 1;
+       }
+
        if (this) {
                /* fime: howto free that memory ? */
        }
-       
+
        sig = method->signature;
 
        mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_NATIVE_TO_MANAGED);
@@ -1864,15 +2143,27 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
        /* allocate local 2 (boolean) delete_old */
        mono_mb_add_local (mb, &mono_defaults.boolean_class->byval_arg);
 
-       mono_mb_emit_byte (mb, CEE_LDNULL);
+       if (!MONO_TYPE_IS_VOID(sig->ret)) {
+               /* allocate local 3 to store the return value */
+               mono_mb_add_local (mb, sig->ret);
+       }
+
+       mono_mb_emit_icon (mb, 0);
        mono_mb_emit_byte (mb, CEE_STLOC_2);
 
        /* we copy the signature, so that we can modify it */
-       sigsize = sizeof (MonoMethodSignature) + sig->param_count * sizeof (MonoType *);
-       csig = g_memdup (sig, sigsize);
+       csig = mono_metadata_signature_dup (sig);
        csig->hasthis = 0;
        csig->pinvoke = 1;
 
+#ifdef PLATFORM_WIN32
+       /* 
+        * Under windows, delegates passed to native code must use the STDCALL
+        * calling convention.
+        */
+       csig->call_convention = MONO_CALL_STDCALL;
+#endif
+
        /* fixme: howto handle this ? */
        if (sig->hasthis) {
 
@@ -1903,6 +2194,8 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                        MonoMethod *marshal_native_to_managed;
                        MonoMethod *get_instance;
 
+                       /* FIXME: Call CleanUpNativeData after the call */
+
                        mtype = mono_reflection_type_from_name (spec->data.custom_data.custom_name, method->klass->image);
                        g_assert (mtype != NULL);
                        mklass = mono_class_from_mono_type (mtype);
@@ -1945,6 +2238,76 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                }
 
                switch (t->type) {
+               case MONO_TYPE_CLASS: {
+                       klass = t->data.klass;
+
+                       tmp_locals [i] = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
+
+                       if (klass->delegate) {
+                               g_assert (!t->byref);
+                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                               mono_mb_emit_byte (mb, CEE_MONO_LDPTR);
+                               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+                               mono_mb_emit_ldarg (mb, i);
+                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                               mono_mb_emit_byte (mb, CEE_MONO_FUNC2);
+                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_FTN_DEL);
+                               mono_mb_emit_stloc (mb, tmp_locals [i]);
+                               break;
+                       }
+
+                       /* FIXME: Raise a MarshalDirectiveException here */
+                       g_assert ((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) != TYPE_ATTRIBUTE_AUTO_LAYOUT);
+
+                       if (t->attrs & PARAM_ATTRIBUTE_OUT) {
+                               mono_mb_emit_byte (mb, CEE_LDNULL);
+                               mono_mb_emit_stloc (mb, tmp_locals [i]);
+                               break;
+                       }
+
+                       /* Set src */
+                       mono_mb_emit_ldarg (mb, i);
+                       if (t->byref) {
+                               int pos2;
+
+                               /* Check for NULL and raise an exception */
+                               mono_mb_emit_byte (mb, CEE_BRTRUE);
+                               pos2 = mb->pos;
+                               mono_mb_emit_i4 (mb, 0);
+
+                               mono_mb_emit_exception (mb, "ArgumentNullException", NULL);
+
+                               mono_mb_patch_addr (mb, pos2, mb->pos - (pos2 + 4));
+                               mono_mb_emit_ldarg (mb, i);
+                               mono_mb_emit_byte (mb, CEE_LDIND_I);
+                       }                               
+
+                       mono_mb_emit_byte (mb, CEE_STLOC_0);
+
+                       mono_mb_emit_byte (mb, CEE_LDC_I4_0);
+                       mono_mb_emit_stloc (mb, tmp_locals [i]);
+
+                       mono_mb_emit_byte (mb, CEE_LDLOC_0);
+                       mono_mb_emit_byte (mb, CEE_BRFALSE);
+                       pos = mb->pos;
+                       mono_mb_emit_i4 (mb, 0);
+
+                       /* Create and set dst */
+                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                       mono_mb_emit_byte (mb, CEE_MONO_NEWOBJ);        
+                       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+                       mono_mb_emit_stloc (mb, tmp_locals [i]);
+                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                       mono_mb_emit_icon (mb, sizeof (MonoObject));
+                       mono_mb_emit_byte (mb, CEE_ADD);
+                       mono_mb_emit_byte (mb, CEE_STLOC_1); 
+
+                       /* emit valuetype conversion code */
+                       emit_struct_conv (mb, klass, TRUE);
+
+                       mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
+                       break;
+               }
                case MONO_TYPE_VALUETYPE:
                        
                        klass = sig->params [i]->data.klass;
@@ -1976,7 +2339,9 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                        if (t->byref)
                                mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
                        break;
-               case MONO_TYPE_STRING:
+               case MONO_TYPE_STRING: {
+                       MonoMarshalNative encoding = mono_marshal_get_string_encoding (NULL, spec);
+
                        if (t->byref)
                                continue;
 
@@ -1986,9 +2351,26 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                        mono_mb_emit_ldarg (mb, i);
                        mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
                        mono_mb_emit_byte (mb, CEE_MONO_FUNC1);
-                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_LPSTR_STR);
+
+                       switch (encoding) {
+                       case MONO_NATIVE_LPWSTR:
+                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_LPWSTR_STR);
+                               break;
+                       case MONO_NATIVE_LPSTR:
+                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_LPSTR_STR);
+                               break;
+                       default: {
+                                       char *msg = g_strdup_printf ("string marshalling conversion %d not implemented", encoding);
+                                       MonoException *exc = mono_get_exception_not_implemented (msg);
+                                       g_warning (msg);
+                                       g_free (msg);
+                                       mono_raise_exception (exc);
+                               }
+                       }
+
                        mono_mb_emit_stloc (mb, tmp_locals [i]);
                        break;  
+               }
                case MONO_TYPE_ARRAY:
                case MONO_TYPE_SZARRAY:
                        if (t->byref)
@@ -2034,6 +2416,11 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                        }
                        break;  
                case MONO_TYPE_CLASS:  
+                       if (t->byref)
+                               mono_mb_emit_ldloc_addr (mb, tmp_locals [i]);
+                       else
+                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                       break;
                case MONO_TYPE_ARRAY:
                case MONO_TYPE_SZARRAY:
                case MONO_TYPE_OBJECT:
@@ -2062,11 +2449,13 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                }
        }
 
+       emit_thread_interrupt_checkpoint (mb);
        mono_mb_emit_managed_call (mb, method, NULL);
 
        if (!sig->ret->byref) { 
                switch (sig->ret->type) {
                case MONO_TYPE_VOID:
+                       break;
                case MONO_TYPE_BOOLEAN:
                case MONO_TYPE_I1:
                case MONO_TYPE_U1:
@@ -2082,7 +2471,7 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                case MONO_TYPE_I8:
                case MONO_TYPE_U8:
                case MONO_TYPE_OBJECT:
-                       mono_mb_emit_byte (mb, CEE_RET);
+                       mono_mb_emit_byte (mb, CEE_STLOC_3);
                        break;
                case MONO_TYPE_STRING:          
                        csig->ret = &mono_defaults.int_class->byval_arg;
@@ -2090,10 +2479,9 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                        mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
                        mono_mb_emit_byte (mb, CEE_MONO_FUNC1);
                        mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPSTR);
-                       mono_mb_emit_byte (mb, CEE_RET);
+                       mono_mb_emit_byte (mb, CEE_STLOC_3);
                        break;
-               case MONO_TYPE_VALUETYPE: {
-                       int tmp;
+               case MONO_TYPE_VALUETYPE:
                        klass = sig->ret->data.klass;
                        if (((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) == TYPE_ATTRIBUTE_EXPLICIT_LAYOUT) ||
                            klass->blittable || klass->enumtype)
@@ -2107,21 +2495,55 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                        mono_mb_emit_byte (mb, CEE_STLOC_0);
                        /* allocate space for the native struct and
                         * store the address into dst_ptr */
-                       tmp = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
-                       g_assert (tmp);
+                       retobj_var = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
+                       g_assert (retobj_var);
                        mono_mb_emit_icon (mb, mono_class_native_size (klass, NULL));
-                       mono_mb_emit_byte (mb, CEE_PREFIX1);
-                       mono_mb_emit_byte (mb, CEE_LOCALLOC);
+                       mono_mb_emit_byte (mb, CEE_CONV_I);
+                       mono_mb_emit_native_call (mb, alloc_sig, mono_marshal_alloc);
+                       emit_thread_interrupt_checkpoint (mb);
                        mono_mb_emit_byte (mb, CEE_STLOC_1);
                        mono_mb_emit_byte (mb, CEE_LDLOC_1);
-                       mono_mb_emit_stloc (mb, tmp);
+                       mono_mb_emit_stloc (mb, retobj_var);
 
                        /* emit valuetype conversion code */
                        emit_struct_conv (mb, klass, FALSE);
-                       mono_mb_emit_ldloc (mb, tmp);
-                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-                       mono_mb_emit_byte (mb, CEE_MONO_RETOBJ);
-                       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+                       break;
+               case MONO_TYPE_CLASS: {
+                       int pos2;
+
+                       klass = sig->ret->data.klass;
+
+                       /* FIXME: Raise a MarshalDirectiveException here */
+                       g_assert ((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) != TYPE_ATTRIBUTE_AUTO_LAYOUT);
+
+                       mono_mb_emit_byte (mb, CEE_STLOC_0);
+                       /* Check for null */
+                       mono_mb_emit_byte (mb, CEE_LDLOC_0);
+                       pos = mono_mb_emit_branch (mb, CEE_BRTRUE);
+                       mono_mb_emit_byte (mb, CEE_LDNULL);
+                       mono_mb_emit_byte (mb, CEE_STLOC_3);
+                       pos2 = mono_mb_emit_branch (mb, CEE_BR);
+
+                       mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
+
+                       /* Set src */
+                       mono_mb_emit_byte (mb, CEE_LDLOC_0);
+                       mono_mb_emit_icon (mb, sizeof (MonoObject));
+                       mono_mb_emit_byte (mb, CEE_ADD);
+                       mono_mb_emit_byte (mb, CEE_STLOC_0);
+
+                       /* Allocate and set dest */
+                       mono_mb_emit_icon (mb, mono_class_native_size (klass, NULL));
+                       mono_mb_emit_byte (mb, CEE_CONV_I);
+                       mono_mb_emit_native_call (mb, alloc_sig, mono_marshal_alloc);
+                       emit_thread_interrupt_checkpoint (mb);
+                       mono_mb_emit_byte (mb, CEE_DUP);
+                       mono_mb_emit_byte (mb, CEE_STLOC_1);
+                       mono_mb_emit_byte (mb, CEE_STLOC_3);
+
+                       emit_struct_conv (mb, klass, FALSE);
+
+                       mono_mb_patch_addr (mb, pos2, mb->pos - (pos2 + 4));
                        break;
                }
                default:
@@ -2129,14 +2551,79 @@ mono_marshal_get_managed_wrapper (MonoMethod *method, MonoObject *this, MonoMars
                        g_assert_not_reached ();
                }
        } else {
+               mono_mb_emit_byte (mb, CEE_STLOC_3);
+       }
+
+       /* Convert byref arguments back */
+       for (i = 0; i < sig->param_count; i ++) {
+               MonoType *t = sig->params [i];
+
+               if (!t->byref)
+                       break;
+
+               switch (t->type) {
+               case MONO_TYPE_CLASS: {
+                       int pos2;
+
+                       klass = t->data.klass;
+
+                       /* Check for null */
+                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                       pos = mono_mb_emit_branch (mb, CEE_BRTRUE);
+                       mono_mb_emit_ldarg (mb, i);
+                       mono_mb_emit_byte (mb, CEE_LDC_I4_0);
+                       mono_mb_emit_byte (mb, CEE_STIND_I);
+                       pos2 = mono_mb_emit_branch (mb, CEE_BR);
+                       
+                       mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));                      
+                       
+                       /* Set src */
+                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                       mono_mb_emit_icon (mb, sizeof (MonoObject));
+                       mono_mb_emit_byte (mb, CEE_ADD);
+                       mono_mb_emit_byte (mb, CEE_STLOC_0);
+
+                       /* Allocate and set dest */
+                       mono_mb_emit_icon (mb, mono_class_native_size (klass, NULL));
+                       mono_mb_emit_byte (mb, CEE_CONV_I);
+                       mono_mb_emit_native_call (mb, alloc_sig, mono_marshal_alloc);
+                       emit_thread_interrupt_checkpoint (mb);
+                       mono_mb_emit_byte (mb, CEE_STLOC_1);
+
+                       /* Update argument pointer */
+                       mono_mb_emit_ldarg (mb, i);
+                       mono_mb_emit_byte (mb, CEE_LDLOC_1);
+                       mono_mb_emit_byte (mb, CEE_STIND_I);
+
+                       /* emit valuetype conversion code */
+                       emit_struct_conv (mb, klass, FALSE);
+
+                       mono_mb_patch_addr (mb, pos2, mb->pos - (pos2 + 4));
+                       break;
+               }
+               }
+       }
+
+       if (retobj_var) {
+               mono_mb_emit_ldloc (mb, retobj_var);
+               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+               mono_mb_emit_byte (mb, CEE_MONO_RETOBJ);
+               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+       }
+       else {
+               if (!MONO_TYPE_IS_VOID(sig->ret))
+                       mono_mb_emit_byte (mb, CEE_LDLOC_3);
                mono_mb_emit_byte (mb, CEE_RET);
        }
 
-       res = mono_mb_create_method (mb, csig, sig->param_count + 16);
+       if (!this)
+               res = mono_mb_create_and_cache (cache, method,
+                                                                                        mb, csig, sig->param_count + 16);
+       else
+               res = mono_mb_create_method (mb, csig, sig->param_count + 16);
        mono_mb_free (mb);
 
-       if (!this)
-               g_hash_table_insert (cache, method, res);
+       //printf ("CODE FOR %s: \n%s.\n", mono_method_full_name (res, TRUE), mono_disasm_code (0, res, ((MonoMethodNormal*)res)->header->code, ((MonoMethodNormal*)res)->header->code + ((MonoMethodNormal*)res)->header->code_size));
 
        return res;
 }
@@ -2158,11 +2645,7 @@ mono_marshal_get_ldfld_wrapper (MonoType *type)
        MonoClass *klass;
        static GHashTable *ldfld_hash = NULL; 
        char *name;
-       int t, pos0, pos1;
-
-       if (!ldfld_hash) 
-               ldfld_hash = g_hash_table_new (NULL, NULL);
-
+       int t, pos0, pos1 = 0;
 
        t = type->type;
 
@@ -2187,7 +2670,12 @@ mono_marshal_get_ldfld_wrapper (MonoType *type)
                klass = mono_defaults.int_class;
        }
 
-       if ((res = g_hash_table_lookup (ldfld_hash, klass)))
+       EnterCriticalSection (&marshal_mutex);
+       if (!ldfld_hash) 
+               ldfld_hash = g_hash_table_new (NULL, NULL);
+       res = g_hash_table_lookup (ldfld_hash, klass);
+       LeaveCriticalSection (&marshal_mutex);
+       if (res)
                return res;
 
        name = g_strdup_printf ("__ldfld_wrapper_%s.%s", klass->name_space, klass->name); 
@@ -2218,6 +2706,7 @@ mono_marshal_get_ldfld_wrapper (MonoType *type)
        csig->pinvoke = 1;
 
        mono_mb_emit_native_call (mb, csig, mono_load_remote_field_new);
+       emit_thread_interrupt_checkpoint (mb);
 
        if (klass->valuetype) {
                mono_mb_emit_byte (mb, CEE_UNBOX);
@@ -2289,11 +2778,10 @@ mono_marshal_get_ldfld_wrapper (MonoType *type)
 
        mono_mb_emit_byte (mb, CEE_RET);
        
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (ldfld_hash, klass,
+                                                                                mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
        
-       g_hash_table_insert (ldfld_hash, klass, res);
-
        return res;
 }
 
@@ -2316,9 +2804,6 @@ mono_marshal_get_stfld_wrapper (MonoType *type)
        char *name;
        int t, pos;
 
-       if (!stfld_hash) 
-               stfld_hash = g_hash_table_new (NULL, NULL);
-
        t = type->type;
 
        if (!type->byref) {
@@ -2342,7 +2827,12 @@ mono_marshal_get_stfld_wrapper (MonoType *type)
                klass = mono_defaults.int_class;
        }
 
-       if ((res = g_hash_table_lookup (stfld_hash, klass)))
+       EnterCriticalSection (&marshal_mutex);
+       if (!stfld_hash) 
+               stfld_hash = g_hash_table_new (NULL, NULL);
+       res = g_hash_table_lookup (stfld_hash, klass);
+       LeaveCriticalSection (&marshal_mutex);
+       if (res)
                return res;
 
        name = g_strdup_printf ("__stfld_wrapper_%s.%s", klass->name_space, klass->name); 
@@ -2381,6 +2871,7 @@ mono_marshal_get_stfld_wrapper (MonoType *type)
        csig->pinvoke = 1;
 
        mono_mb_emit_native_call (mb, csig, mono_store_remote_field_new);
+       emit_thread_interrupt_checkpoint (mb);
 
        mono_mb_emit_byte (mb, CEE_RET);
 
@@ -2441,17 +2932,16 @@ mono_marshal_get_stfld_wrapper (MonoType *type)
 
        mono_mb_emit_byte (mb, CEE_RET);
        
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (stfld_hash, klass,
+                                                                       mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
        
-       g_hash_table_insert (stfld_hash, klass, res);
-
        return res;
 }
 
 /*
  * generates IL code for the icall wrapper (the generated method
- * calls the unamnaged code in func)
+ * calls the unmanaged code in func)
  */
 MonoMethod *
 mono_marshal_get_icall_wrapper (MonoMethodSignature *sig, const char *name, gconstpointer func)
@@ -2459,8 +2949,8 @@ mono_marshal_get_icall_wrapper (MonoMethodSignature *sig, const char *name, gcon
        MonoMethodSignature *csig;
        MonoMethodBuilder *mb;
        MonoMethod *res;
-       int i, sigsize;
-
+       int i;
+       
        g_assert (sig->pinvoke);
 
        mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_MANAGED_TO_NATIVE);
@@ -2468,7 +2958,6 @@ mono_marshal_get_icall_wrapper (MonoMethodSignature *sig, const char *name, gcon
        mb->method->save_lmf = 1;
 
        /* we copy the signature, so that we can modify it */
-       sigsize = sizeof (MonoMethodSignature) + sig->param_count * sizeof (MonoType *);
 
        if (sig->hasthis)
                mono_mb_emit_byte (mb, CEE_LDARG_0);
@@ -2477,10 +2966,10 @@ mono_marshal_get_icall_wrapper (MonoMethodSignature *sig, const char *name, gcon
                mono_mb_emit_ldarg (mb, i + sig->hasthis);
 
        mono_mb_emit_native_call (mb, sig, (gpointer) func);
-
+       emit_thread_interrupt_checkpoint (mb);
        mono_mb_emit_byte (mb, CEE_RET);
 
-       csig = g_memdup (sig, sigsize);
+       csig = mono_metadata_signature_dup (sig);
        csig->pinvoke = 0;
 
        res = mono_mb_create_method (mb, csig, csig->param_count + 16);
@@ -2489,9 +2978,12 @@ mono_marshal_get_icall_wrapper (MonoMethodSignature *sig, const char *name, gcon
        return res;
 }
 
-/*
+/**
+ * mono_marshal_get_native_wrapper:
+ * @method: The MonoMethod to wrap.
+ *
  * generates IL code for the pinvoke wrapper (the generated method
- * calls the unamnage code in method->addr)
+ * calls the unmanaged code in method->addr)
  */
 MonoMethod *
 mono_marshal_get_native_wrapper (MonoMethod *method)
@@ -2505,38 +2997,43 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
        MonoClass *klass;
        gboolean pinvoke = FALSE;
        int i, pos, argnum, *tmp_locals;
-       int type, sigsize;
+       int type;
+       const char *exc_class = "MissingMethodException";
+       const char *exc_arg = NULL;
 
        g_assert (method != NULL);
        g_assert (method->signature->pinvoke);
 
        cache = method->klass->image->native_wrapper_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, method)))
+       if ((res = mono_marshal_find_in_cache (cache, method)))
                return res;
 
        sig = method->signature;
-       sigsize = sizeof (MonoMethodSignature) + sig->param_count * sizeof (MonoType *);
 
        if (!(method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) &&
            (method->flags & METHOD_ATTRIBUTE_PINVOKE_IMPL))
                pinvoke = TRUE;
 
+       if (!method->addr) {
+               if (pinvoke)
+                       mono_lookup_pinvoke_call (method, &exc_class, &exc_arg);
+               else
+                       method->addr = mono_lookup_internal_call (method);
+       }
+
        mb = mono_mb_new (method->klass, method->name, MONO_WRAPPER_MANAGED_TO_NATIVE);
 
        mb->method->save_lmf = 1;
 
-       if (pinvoke && !method->addr)
-               mono_lookup_pinvoke_call (method);
-
        piinfo = (MonoMethodPInvoke *)method;
 
        if (!method->addr) {
-               mono_mb_emit_exception (mb);
-               csig = g_memdup (sig, sigsize);
+               mono_mb_emit_exception (mb, exc_class, exc_arg);
+               csig = mono_metadata_signature_dup (sig);
                csig->pinvoke = 0;
-               res = mono_mb_create_method (mb, csig, csig->param_count + 16);
+               res = mono_mb_create_and_cache (cache, method,
+                                                                               mb, csig, csig->param_count + 16);
                mono_mb_free (mb);
-               g_hash_table_insert (cache, method, res);
                return res;
        }
 
@@ -2545,7 +3042,7 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
 
                /* hack - string constructors returns a value */
                if (method->string_ctor) {
-                       csig = g_memdup (sig, sigsize);
+                       csig = mono_metadata_signature_dup (sig);
                        csig->ret = &mono_defaults.string_class->byval_arg;
                } else
                        csig = sig;
@@ -2558,14 +3055,14 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
 
                g_assert (method->addr);
                mono_mb_emit_native_call (mb, csig, method->addr);
-
+               emit_thread_interrupt_checkpoint (mb);
                mono_mb_emit_byte (mb, CEE_RET);
 
-               csig = g_memdup (csig, sigsize);
+               csig = mono_metadata_signature_dup (csig);
                csig->pinvoke = 0;
-               res = mono_mb_create_method (mb, csig, csig->param_count + 16);
+               res = mono_mb_create_and_cache (cache, method,
+                                                                               mb, csig, csig->param_count + 16);
                mono_mb_free (mb);
-               g_hash_table_insert (cache, method, res);
                return res;
        }
 
@@ -2577,7 +3074,7 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
        /* pinvoke: we need to convert the arguments if necessary */
 
        /* we copy the signature, so that we can set pinvoke to 0 */
-       csig = g_memdup (sig, sigsize);
+       csig = mono_metadata_signature_dup (sig);
        csig->pinvoke = 1;
 
        /* we allocate local for use with emit_struct_conv() */
@@ -2597,8 +3094,19 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                mono_mb_add_local (mb, sig->ret);
        }
 
+       if (mspecs [0] && mspecs [0]->native == MONO_NATIVE_CUSTOM) {
+               /* Return type custom marshaling */
+               /*
+                * Since we can't determine the return type of the unmanaged function,
+                * we assume it returns a pointer, and pass that pointer to
+                * MarshalNativeToManaged.
+                */
+               csig->ret = &mono_defaults.int_class->byval_arg;
+       }
+
        /* we first do all conversions */
        tmp_locals = alloca (sizeof (int) * sig->param_count);
+
        for (i = 0; i < sig->param_count; i ++) {
                MonoType *t = sig->params [i];
                MonoMarshalSpec *spec = mspecs [i + 1];
@@ -2612,6 +3120,8 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                        MonoMethod *marshal_managed_to_native;
                        MonoMethod *get_instance;
 
+                       /* FIXME: Call CleanUpNativeData after the call */
+
                        mtype = mono_reflection_type_from_name (spec->data.custom_data.custom_name, method->klass->image);
                        g_assert (mtype != NULL);
                        mklass = mono_class_from_mono_type (mtype);
@@ -2628,6 +3138,7 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                        case MONO_TYPE_STRING:
                        case MONO_TYPE_ARRAY:
                        case MONO_TYPE_SZARRAY:
+                       case MONO_TYPE_VALUETYPE:
                                if (t->byref)
                                        break;
 
@@ -2639,12 +3150,24 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                                mono_mb_emit_i4 (mb, mono_mb_add_data (mb, get_instance));
                                
                                mono_mb_emit_ldarg (mb, argnum);
-                               
+
+                               if (t->type == MONO_TYPE_VALUETYPE) {
+                                       /*
+                                        * Since we can't determine the type of the argument, we
+                                        * will assume the unmanaged function takes a pointer.
+                                        */
+                                       csig->params [i] = &mono_defaults.int_class->byval_arg;
+
+                                       mono_mb_emit_byte (mb, CEE_BOX);
+                                       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, mono_class_from_mono_type (t)));
+                               }
+
                                mono_mb_emit_byte (mb, CEE_CALLVIRT);
                                mono_mb_emit_i4 (mb, mono_mb_add_data (mb, marshal_managed_to_native));
-                               
+
                                mono_mb_emit_stloc (mb, tmp_locals [i]);
                                break;
+
                        default:
                                g_warning ("custom marshalling of type %x is currently not supported", t->type);
                                g_assert_not_reached ();
@@ -2652,8 +3175,23 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                        }
                        continue;
                }
-               
 
+               if (spec && spec->native == MONO_NATIVE_ASANY) {
+                       MonoMarshalNative encoding = mono_marshal_get_string_encoding (piinfo, NULL);
+
+                       g_assert (t->type == MONO_TYPE_OBJECT);
+                       g_assert (!t->byref);
+
+                       tmp_locals [i] = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
+                       mono_mb_emit_ldarg (mb, argnum);
+                       mono_mb_emit_icon (mb, encoding);
+                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                       mono_mb_emit_byte (mb, CEE_MONO_FUNC2);
+                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_ASANY);
+                       mono_mb_emit_stloc (mb, tmp_locals [i]);
+                       continue;
+               }
+                       
                switch (t->type) {
                case MONO_TYPE_VALUETYPE:                       
                        klass = t->data.klass;
@@ -2696,7 +3234,9 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                        if (t->byref)
                                mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
                        break;
-               case MONO_TYPE_STRING:
+               case MONO_TYPE_STRING: {
+                       MonoMarshalNative encoding = mono_marshal_get_string_encoding (piinfo, spec);
+
                        csig->params [argnum] = &mono_defaults.int_class->byval_arg;
                        tmp_locals [i] = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
 
@@ -2712,38 +3252,29 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
 
                        mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
                        mono_mb_emit_byte (mb, CEE_MONO_FUNC1);
-
-                       if (spec) {
-                               switch (spec->native) {
-                               case MONO_NATIVE_LPWSTR:
-                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPWSTR);
-                                       break;
-                               case MONO_NATIVE_LPSTR:
-                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPSTR);
-                                       break;
-                               default:
-                                       g_warning ("marshalling conversion %d not implemented", spec->native);
-                                       g_assert_not_reached ();
-                               }
-                       } else {
-                               switch (piinfo->piflags & PINVOKE_ATTRIBUTE_CHAR_SET_MASK) {
-                               case PINVOKE_ATTRIBUTE_CHAR_SET_ANSI:
-                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPSTR);
-                                       break;
-                               case PINVOKE_ATTRIBUTE_CHAR_SET_UNICODE:
-                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPWSTR);
-                                       break;
-                               case PINVOKE_ATTRIBUTE_CHAR_SET_AUTO:
-                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPTSTR);
-                                       break;
-                               default:
-                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPSTR);
-                                       break;                                  
+                       
+                       switch (encoding) {
+                       case MONO_NATIVE_LPWSTR:
+                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPWSTR);
+                               break;
+                       case MONO_NATIVE_LPSTR:
+                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPSTR);
+                               break;
+                       case MONO_NATIVE_LPTSTR:
+                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STR_LPTSTR);
+                               break;
+                       default: {
+                                       char *msg = g_strdup_printf ("string marshalling conversion %d not implemented", encoding);
+                                       MonoException *exc = mono_get_exception_not_implemented (msg);
+                                       g_warning (msg);
+                                       g_free (msg);
+                                       mono_raise_exception (exc);
                                }
                        }
 
                        mono_mb_emit_stloc (mb, tmp_locals [i]);
                        break;
+               }
                case MONO_TYPE_CLASS:
                case MONO_TYPE_OBJECT:
                        klass = t->data.klass;
@@ -2759,11 +3290,32 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                                mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_DEL_FTN);
                                mono_mb_emit_stloc (mb, tmp_locals [i]);
                        } else if (klass == mono_defaults.stringbuilder_class) {
+                               MonoMarshalNative encoding = mono_marshal_get_string_encoding (piinfo, spec);
+
                                g_assert (!t->byref);
                                mono_mb_emit_ldarg (mb, argnum);
                                mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
                                mono_mb_emit_byte (mb, CEE_MONO_FUNC1);
-                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_SB_LPSTR);
+
+                               switch (encoding) {
+                               case MONO_NATIVE_LPWSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_SB_LPWSTR);
+                                       break;
+                               case MONO_NATIVE_LPSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_SB_LPSTR);
+                                       break;
+                               case MONO_NATIVE_LPTSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_SB_LPTSTR);
+                                       break;
+                               default: {
+                                       char *msg = g_strdup_printf ("stringbuilder marshalling conversion %d not implemented", encoding);
+                                       MonoException *exc = mono_get_exception_not_implemented (msg);
+                                       g_warning (msg);
+                                       g_free (msg);
+                                       mono_raise_exception (exc);
+                               }
+                               }
+
                                mono_mb_emit_stloc (mb, tmp_locals [i]);
                        } else {
                                mono_mb_emit_byte (mb, CEE_LDNULL);
@@ -2823,16 +3375,136 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                        csig->params [argnum] = &mono_defaults.int_class->byval_arg;
                        tmp_locals [i] = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
 
-                       mono_mb_emit_ldarg (mb, argnum);
-                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-                       mono_mb_emit_byte (mb, CEE_MONO_FUNC1);
-                       if (klass->element_class == mono_defaults.string_class) 
-                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STRARRAY_STRLPARRAY);
-                       else
+                       if (klass->element_class == mono_defaults.string_class) {
+                               MonoMarshalNative encoding = mono_marshal_get_string_encoding (piinfo, spec);
+
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                               mono_mb_emit_byte (mb, CEE_MONO_FUNC1);
+
+                               switch (encoding) {
+                               case MONO_NATIVE_LPWSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STRARRAY_STRWLPARRAY);
+                                       break;
+                               case MONO_NATIVE_LPSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_STRARRAY_STRLPARRAY);
+                                       break;
+                               default: {
+                                       char *msg = g_strdup_printf ("string array marshalling conversion %d not implemented", encoding);
+                                       MonoException *exc = mono_get_exception_not_implemented (msg);
+                                       g_warning (msg);
+                                       g_free (msg);
+                                       mono_raise_exception (exc);
+                               }
+                               }
+                               mono_mb_emit_stloc (mb, tmp_locals [i]);
+                       }
+                       else if (klass->element_class->blittable) {
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                               mono_mb_emit_byte (mb, CEE_MONO_FUNC1);
+
                                mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_ARRAY_LPARRAY);
+                               mono_mb_emit_stloc (mb, tmp_locals [i]);
+                       }
+                       else {
+                               MonoClass *eklass;
+                               guint32 label1, label2, label3;
+                               int index_var, dest_ptr;
+
+                               dest_ptr = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
+
+                               /* Check null */
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_stloc (mb, tmp_locals [i]);
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, CEE_BRFALSE);
+                               label1 = mb->pos;
+                               mono_mb_emit_i4 (mb, 0);
+
+                               /* allocate space for the native struct and store the address */
+                               eklass = klass->element_class;
+                               mono_mb_emit_icon (mb, mono_class_native_size (eklass, NULL));
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, CEE_LDLEN);
+                               mono_mb_emit_byte (mb, CEE_MUL);
+                               mono_mb_emit_byte (mb, CEE_PREFIX1);
+                               mono_mb_emit_byte (mb, CEE_LOCALLOC);
+                               mono_mb_emit_stloc (mb, tmp_locals [i]);
+
+                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                               mono_mb_emit_stloc (mb, dest_ptr);
+
+                               /* Emit marshalling loop */
+                               index_var = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);                                
+                               mono_mb_emit_byte (mb, CEE_LDC_I4_0);
+                               mono_mb_emit_stloc (mb, index_var);
+                               label2 = mb->pos;
+                               mono_mb_emit_ldloc (mb, index_var);
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, CEE_LDLEN);
+                               mono_mb_emit_byte (mb, CEE_BGE);
+                               label3 = mb->pos;
+                               mono_mb_emit_i4 (mb, 0);
+
+                               /* Emit marshalling code */
+
+                               /* set the src_ptr */
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_ldloc (mb, index_var);
+                               mono_mb_emit_byte (mb, CEE_LDELEMA);
+                               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, eklass));
+                               mono_mb_emit_byte (mb, CEE_STLOC_0);
+
+                               /* set dst_ptr */
+                               mono_mb_emit_ldloc (mb, dest_ptr);
+                               mono_mb_emit_byte (mb, CEE_STLOC_1);
+
+                               /* emit valuetype conversion code */
+                               emit_struct_conv (mb, eklass, FALSE);
+
+                               mono_mb_emit_add_to_local (mb, index_var, 1);
+                               mono_mb_emit_add_to_local (mb, dest_ptr, mono_class_native_size (eklass, NULL));
+
+                               mono_mb_emit_byte (mb, CEE_BR);
+                               mono_mb_emit_i4 (mb, label2 - (mb->pos + 4));
+
+                               mono_mb_patch_addr (mb, label1, mb->pos - (label1 + 4));
+                               mono_mb_patch_addr (mb, label3, mb->pos - (label3 + 4));
+                       }
+
+                       break;
+               case MONO_TYPE_BOOLEAN: {
+                       MonoType *local_type;
+                       int variant_bool = 0;
+                       if (!t->byref)
+                               continue;
+                       if (spec == NULL) {
+                               local_type = &mono_defaults.int32_class->byval_arg;
+                       } else {
+                               switch (spec->native) {
+                               case MONO_NATIVE_I1:
+                                       local_type = &mono_defaults.byte_class->byval_arg;
+                                       break;
+                               case MONO_NATIVE_VARIANTBOOL:
+                                       local_type = &mono_defaults.int16_class->byval_arg;
+                                       variant_bool = 1;
+                                       break;
+                               default:
+                                       g_warning ("marshalling bool as native type %x is currently not supported", spec->native);
+                               break;
+                               }
+                       }
+                       csig->params [argnum] = &mono_defaults.int_class->byval_arg;
+                       tmp_locals [i] = mono_mb_add_local (mb, local_type);
+                       mono_mb_emit_ldarg (mb, argnum);
+                       mono_mb_emit_byte (mb, CEE_LDIND_I1);
+                       if (variant_bool)
+                               mono_mb_emit_byte (mb, CEE_NEG);
                        mono_mb_emit_stloc (mb, tmp_locals [i]);
                        break;
                }
+               }
        }
 
        /* push all arguments */
@@ -2842,80 +3514,103 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
 
        for (i = 0; i < sig->param_count; i++) {
                MonoType *t = sig->params [i];
-               
-               argnum = i + sig->hasthis;
+               MonoMarshalSpec *spec = mspecs [i + 1];
 
-               switch (t->type) {
-               case MONO_TYPE_BOOLEAN:
-                       if (t->byref)
-                               g_warning ("byref boolean marshalling not inplemented");
-                       mono_mb_emit_ldarg (mb, argnum);
-                       break;
-               case MONO_TYPE_I1:
-               case MONO_TYPE_U1:
-               case MONO_TYPE_I2:
-               case MONO_TYPE_U2:
-               case MONO_TYPE_I4:
-               case MONO_TYPE_U4:
-               case MONO_TYPE_I:
-               case MONO_TYPE_U:
-               case MONO_TYPE_PTR:
-               case MONO_TYPE_R4:
-               case MONO_TYPE_R8:
-               case MONO_TYPE_I8:
-               case MONO_TYPE_U8:
-                       mono_mb_emit_ldarg (mb, argnum);
-                       break;
-               case MONO_TYPE_VALUETYPE:
-                       klass = sig->params [i]->data.klass;
-                       if (((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) == TYPE_ATTRIBUTE_EXPLICIT_LAYOUT) ||
-                           klass->blittable || klass->enumtype) {
-                               mono_mb_emit_ldarg (mb, argnum);
-                               break;
-                       }                       
-                       g_assert (tmp_locals [i]);
+               if (spec && spec->native == MONO_NATIVE_CUSTOM) {
                        mono_mb_emit_ldloc (mb, tmp_locals [i]);
-                       if (!t->byref) {
-                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-                               mono_mb_emit_byte (mb, CEE_MONO_LDNATIVEOBJ);
-                               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
-                       }
-                       break;
-               case MONO_TYPE_STRING:
-               case MONO_TYPE_CLASS:
-               case MONO_TYPE_OBJECT:
-                       g_assert (tmp_locals [i]);
-                       if (t->byref) 
-                               mono_mb_emit_ldloc_addr (mb, tmp_locals [i]);
-                       else
-                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
-                       break;
-               case MONO_TYPE_CHAR:
-                       /* fixme: dont know how to marshal that. We cant simply
-                        * convert it to a one byte UTF8 character, because an
-                        * unicode character may need more that one byte in UTF8 */
-                       mono_mb_emit_ldarg (mb, argnum);
-                       break;
-               case MONO_TYPE_ARRAY:
-               case MONO_TYPE_SZARRAY:
-                       if (t->byref) {
+               }
+               else
+               if (spec && spec->native == MONO_NATIVE_ASANY) {
+                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+               }
+               else {
+                       argnum = i + sig->hasthis;
+
+                       switch (t->type) {
+                       case MONO_TYPE_BOOLEAN:
+                               if (t->byref) {
+                                       g_assert (tmp_locals [i]);
+                                       mono_mb_emit_ldloc_addr (mb, tmp_locals [i]);
+                               } else
+                                       mono_mb_emit_ldarg (mb, argnum);
+                               break;
+                       case MONO_TYPE_I1:
+                       case MONO_TYPE_U1:
+                       case MONO_TYPE_I2:
+                       case MONO_TYPE_U2:
+                       case MONO_TYPE_I4:
+                       case MONO_TYPE_U4:
+                       case MONO_TYPE_I:
+                       case MONO_TYPE_U:
+                       case MONO_TYPE_PTR:
+                       case MONO_TYPE_R4:
+                       case MONO_TYPE_R8:
+                       case MONO_TYPE_I8:
+                       case MONO_TYPE_U8:
                                mono_mb_emit_ldarg (mb, argnum);
-                       } else {
+                               break;
+                       case MONO_TYPE_VALUETYPE:
+                               klass = sig->params [i]->data.klass;
+                               if (((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) == TYPE_ATTRIBUTE_EXPLICIT_LAYOUT) ||
+                                       klass->blittable || klass->enumtype) {
+                                       mono_mb_emit_ldarg (mb, argnum);
+                                       break;
+                               }                       
                                g_assert (tmp_locals [i]);
                                mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                               if (!t->byref) {
+                                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                                       mono_mb_emit_byte (mb, CEE_MONO_LDNATIVEOBJ);
+                                       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+                               }
+                               break;
+                       case MONO_TYPE_STRING:
+                       case MONO_TYPE_CLASS:
+                       case MONO_TYPE_OBJECT:
+                               g_assert (tmp_locals [i]);
+                               if (t->byref) 
+                                       mono_mb_emit_ldloc_addr (mb, tmp_locals [i]);
+                               else
+                                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                               break;
+                       case MONO_TYPE_CHAR:
+                               /* fixme: dont know how to marshal that. We cant simply
+                                * convert it to a one byte UTF8 character, because an
+                                * unicode character may need more that one byte in UTF8 */
+                               mono_mb_emit_ldarg (mb, argnum);
+                               break;
+                       case MONO_TYPE_ARRAY:
+                       case MONO_TYPE_SZARRAY:
+                               if (t->byref) {
+                                       mono_mb_emit_ldarg (mb, argnum);
+                               } else {
+                                       g_assert (tmp_locals [i]);
+                                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                               }
+                               break;
+                       case MONO_TYPE_TYPEDBYREF:
+                       case MONO_TYPE_FNPTR:
+                       default:
+                               g_warning ("type 0x%02x unknown", t->type);     
+                               g_assert_not_reached ();
                        }
-                       break;
-               case MONO_TYPE_TYPEDBYREF:
-               case MONO_TYPE_FNPTR:
-               default:
-                       g_warning ("type 0x%02x unknown", t->type);     
-                       g_assert_not_reached ();
                }
        }                       
 
        /* call the native method */
        mono_mb_emit_native_call (mb, csig, method->addr);
 
+       /* Set LastError if needed */
+       if (piinfo->piflags & PINVOKE_ATTRIBUTE_SUPPORTS_LAST_ERROR) {
+               MonoMethodSignature *lasterr_sig;
+
+               lasterr_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 0);
+               lasterr_sig->ret = &mono_defaults.void_class->byval_arg;
+               lasterr_sig->pinvoke = 1;
+
+               mono_mb_emit_native_call (mb, lasterr_sig, mono_marshal_set_last_error);
+       }               
+
        /* convert the result */
        if (!sig->ret->byref) {
                MonoMarshalSpec *spec = mspecs [0];
@@ -2943,18 +3638,31 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                        case MONO_TYPE_STRING:
                        case MONO_TYPE_ARRAY:
                        case MONO_TYPE_SZARRAY:
-                               mono_mb_emit_byte (mb, CEE_STLOC_3);
+                       case MONO_TYPE_VALUETYPE:
+                               if (type == MONO_TYPE_VALUETYPE) {
+                                       /* local 3 can't hold a pointer */
+                                       mono_mb_emit_byte (mb, CEE_STLOC_0);
+                               }
+                               else
+                                       mono_mb_emit_byte (mb, CEE_STLOC_3);
 
                                mono_mb_emit_ldstr (mb, spec->data.custom_data.cookie);
 
                                mono_mb_emit_byte (mb, CEE_CALL);
                                mono_mb_emit_i4 (mb, mono_mb_add_data (mb, get_instance));
-                               
-                               mono_mb_emit_byte (mb, CEE_LDLOC_3);
+
+                               if (type == MONO_TYPE_VALUETYPE)
+                                       mono_mb_emit_byte (mb, CEE_LDLOC_0);
+                               else
+                                       mono_mb_emit_byte (mb, CEE_LDLOC_3);
                                
                                mono_mb_emit_byte (mb, CEE_CALLVIRT);
                                mono_mb_emit_i4 (mb, mono_mb_add_data (mb, marshal_native_to_managed));
                                
+                               if (type == MONO_TYPE_VALUETYPE) {
+                                       mono_mb_emit_byte (mb, CEE_UNBOX);
+                                       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, mono_class_from_mono_type (sig->ret)));
+                               }
                                mono_mb_emit_byte (mb, CEE_STLOC_3);
                                break;
                        default:
@@ -2994,7 +3702,7 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                                        type = sig->ret->data.klass->enum_basetype->type;
                                        goto handle_enum;
                                }
-                               
+
                                if (((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) == TYPE_ATTRIBUTE_EXPLICIT_LAYOUT) ||
                                    klass->blittable) {
                                        mono_mb_emit_byte (mb, CEE_STLOC_3);
@@ -3041,10 +3749,45 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                                mono_mb_emit_byte (mb, CEE_MONO_FREE);
 #endif
                                break;
-                       case MONO_TYPE_ARRAY:
-                       case MONO_TYPE_SZARRAY:
                        case MONO_TYPE_CLASS:
                        case MONO_TYPE_OBJECT:
+                               klass = sig->ret->data.klass;
+
+                               /* set src */
+                               mono_mb_emit_byte (mb, CEE_STLOC_0);
+
+                               mono_mb_emit_byte (mb, CEE_LDNULL);
+                               mono_mb_emit_byte (mb, CEE_STLOC_3);
+
+
+                               mono_mb_emit_byte (mb, CEE_LDLOC_0);
+                               mono_mb_emit_byte (mb, CEE_BRFALSE);
+                               pos = mb->pos;
+                               mono_mb_emit_i4 (mb, 0);
+
+                               /* allocate result object */
+
+                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                               mono_mb_emit_byte (mb, CEE_MONO_NEWOBJ);        
+                               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+                               mono_mb_emit_byte (mb, CEE_STLOC_3);
+                               
+                               /* set dst  */
+
+                               mono_mb_emit_byte (mb, CEE_LDLOC_3);
+                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                               mono_mb_emit_byte (mb, CEE_MONO_OBJADDR);
+                               mono_mb_emit_icon (mb, sizeof (MonoObject));
+                               mono_mb_emit_byte (mb, CEE_ADD);
+                               mono_mb_emit_byte (mb, CEE_STLOC_1);
+                                                       
+                               /* emit conversion code */
+                               emit_struct_conv (mb, klass, TRUE);
+
+                               mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
+                               break;
+                       case MONO_TYPE_ARRAY:
+                       case MONO_TYPE_SZARRAY:
                                /* fixme: we need conversions here */
                                mono_mb_emit_byte (mb, CEE_STLOC_3);
                                break;
@@ -3063,12 +3806,74 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                mono_mb_emit_byte (mb, CEE_STLOC_3);
        }
 
+       /* 
+        * Need to call this after converting the result since MONO_VTADDR needs 
+        * to be adjacent to the call instruction.
+        */
+       emit_thread_interrupt_checkpoint (mb);
+
        /* we need to convert byref arguments back and free string arrays */
        for (i = 0; i < sig->param_count; i++) {
                MonoType *t = sig->params [i];
-               
+               MonoMarshalSpec *spec = mspecs [i + 1];
+
                argnum = i + sig->hasthis;
 
+               if (spec && spec->native == MONO_NATIVE_CUSTOM) {
+                       MonoType *mtype;
+                       MonoClass *mklass;
+                       MonoMethod *get_instance;
+                       MonoMethod *cleanup_native;
+
+                       mtype = mono_reflection_type_from_name (spec->data.custom_data.custom_name, method->klass->image);
+                       g_assert (mtype != NULL);
+                       mklass = mono_class_from_mono_type (mtype);
+                       g_assert (mklass != NULL);
+
+                       get_instance = mono_find_method_by_name (mklass, "GetInstance", 1);
+                       g_assert (get_instance);
+                       cleanup_native = mono_find_method_by_name (mklass, "CleanUpNativeData", 1);
+                       g_assert (get_instance);
+                       
+                       switch (t->type) {
+                       case MONO_TYPE_CLASS:
+                       case MONO_TYPE_OBJECT:
+                       case MONO_TYPE_STRING:
+                       case MONO_TYPE_ARRAY:
+                       case MONO_TYPE_SZARRAY:
+                       case MONO_TYPE_VALUETYPE:
+                               mono_mb_emit_ldstr (mb, spec->data.custom_data.cookie);
+
+                               mono_mb_emit_byte (mb, CEE_CALL);
+                               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, get_instance));
+                               
+                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
+
+                               mono_mb_emit_byte (mb, CEE_CALLVIRT);
+                               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, cleanup_native));
+
+                               break;
+
+                       default:
+                               g_warning ("custom marshalling of type %x is currently not supported", t->type);
+                               g_assert_not_reached ();
+                               break;
+                       }
+                       continue;
+               }
+
+               if (spec && spec->native == MONO_NATIVE_ASANY) {
+                       MonoMarshalNative encoding = mono_marshal_get_string_encoding (piinfo, NULL);
+
+                       mono_mb_emit_ldarg (mb, argnum);
+                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                       mono_mb_emit_icon (mb, encoding);
+                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                       mono_mb_emit_byte (mb, CEE_MONO_PROC3);
+                       mono_mb_emit_byte (mb, MONO_MARSHAL_FREE_ASANY);
+                       continue;
+               }
+               
                switch (t->type) {
                case MONO_TYPE_STRING:
                        if (t->byref && (t->attrs & PARAM_ATTRIBUTE_OUT)) {
@@ -3087,22 +3892,46 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                case MONO_TYPE_CLASS:
                case MONO_TYPE_OBJECT:                  
                        if (t->data.klass == mono_defaults.stringbuilder_class) {
+                               MonoMarshalNative encoding = mono_marshal_get_string_encoding (piinfo, spec);
+                               gboolean need_free = TRUE;
+
                                g_assert (!t->byref);
                                mono_mb_emit_ldarg (mb, argnum);
                                mono_mb_emit_ldloc (mb, tmp_locals [i]);
                                mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
                                mono_mb_emit_byte (mb, CEE_MONO_PROC2);
-                               mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_LPSTR_SB);
-                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
-                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-                               mono_mb_emit_byte (mb, CEE_MONO_FREE);
+
+                               switch (encoding) {
+                               case MONO_NATIVE_LPWSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_LPWSTR_SB);
+                                       /* 
+                                        * mono_string_builder_to_utf16 does not allocate a 
+                                        * new buffer, so no need to free it.
+                                        */
+                                       need_free = FALSE;
+                                       break;
+                               case MONO_NATIVE_LPSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_LPSTR_SB);
+                                       break;
+                               case MONO_NATIVE_LPTSTR:
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_CONV_LPTSTR_SB);
+                                       break;
+                               default:
+                                       g_assert_not_reached ();
+                               }
+
+                               if (need_free) {
+                                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                                       mono_mb_emit_byte (mb, CEE_MONO_FREE);
+                               }
                                break;
                        }
                        
-                       if (!t->byref)
+                       if (!(t->byref || (t->attrs & PARAM_ATTRIBUTE_OUT)))
                                continue;
 
-                       if (t->attrs & PARAM_ATTRIBUTE_OUT) {
+                       if (t->byref && (t->attrs & PARAM_ATTRIBUTE_OUT)) {
                                /* allocate a new object new object */
                                mono_mb_emit_ldarg (mb, argnum);
                                mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
@@ -3113,7 +3942,10 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
 
                        /* dst = *argument */
                        mono_mb_emit_ldarg (mb, argnum);
-                       mono_mb_emit_byte (mb, CEE_LDIND_I);
+
+                       if (t->byref)
+                               mono_mb_emit_byte (mb, CEE_LDIND_I);
+
                        mono_mb_emit_byte (mb, CEE_STLOC_1);
 
                        mono_mb_emit_byte (mb, CEE_LDLOC_1);
@@ -3144,73 +3976,443 @@ mono_marshal_get_native_wrapper (MonoMethod *method)
                            klass->blittable || klass->enumtype)
                                break;
 
-                       /* dst = argument */
-                       mono_mb_emit_ldarg (mb, argnum);
-                       mono_mb_emit_byte (mb, CEE_STLOC_1);
+                       /* dst = argument */
+                       mono_mb_emit_ldarg (mb, argnum);
+                       mono_mb_emit_byte (mb, CEE_STLOC_1);
+
+                       mono_mb_emit_byte (mb, CEE_LDLOC_1);
+                       mono_mb_emit_byte (mb, CEE_BRFALSE);
+                       pos = mb->pos;
+                       mono_mb_emit_i4 (mb, 0);
+
+                       /* src = tmp_locals [i] */
+                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                       mono_mb_emit_byte (mb, CEE_STLOC_0);
+
+                       /* emit valuetype conversion code */
+                       emit_struct_conv (mb, klass, TRUE);
+                       
+                       mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
+                       break;
+               case MONO_TYPE_SZARRAY:
+                       if (t->byref)
+                               continue;
+                       klass = mono_class_from_mono_type (t);
+
+                       if (klass->element_class == mono_defaults.string_class) {
+                               MonoMarshalNative encoding = mono_marshal_get_string_encoding (piinfo, spec);
+                               g_assert (tmp_locals [i]);
+
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, CEE_BRFALSE);
+                               pos = mb->pos;
+                               mono_mb_emit_i4 (mb, 0);
+
+                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
+
+                               switch (encoding) {
+                               case MONO_NATIVE_LPWSTR:
+                                       /* 
+                                        * The array elements point to the managed string data so 
+                                        * they don't need to be freed.
+                                        */
+                                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                                       mono_mb_emit_byte (mb, CEE_MONO_FREE);
+                                       break;
+                               default:
+                                       mono_mb_emit_ldarg (mb, argnum);
+                                       mono_mb_emit_byte (mb, CEE_LDLEN);                              
+                                       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+                                       mono_mb_emit_byte (mb, CEE_MONO_PROC2);
+                                       mono_mb_emit_byte (mb, MONO_MARSHAL_FREE_ARRAY);
+                                       break;                                  
+                               }
+
+                               mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
+                       }
+
+                       /* Character arrays are implicitly marshalled as [Out] */
+                       if ((klass->element_class == mono_defaults.char_class) || (t->attrs & PARAM_ATTRIBUTE_OUT)) {
+                               /* FIXME: Optimize blittable case */
+                               MonoClass *eklass;
+                               guint32 label1, label2, label3;
+                               int index_var, src_ptr;
+
+                               eklass = klass->element_class;
+                               src_ptr = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);
+
+                               /* Check null */
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, CEE_BRFALSE);
+                               label1 = mb->pos;
+                               mono_mb_emit_i4 (mb, 0);
+
+                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                               mono_mb_emit_stloc (mb, src_ptr);
+
+                               /* Emit marshalling loop */
+                               index_var = mono_mb_add_local (mb, &mono_defaults.int_class->byval_arg);                                
+                               mono_mb_emit_byte (mb, CEE_LDC_I4_0);
+                               mono_mb_emit_stloc (mb, index_var);
+                               label2 = mb->pos;
+                               mono_mb_emit_ldloc (mb, index_var);
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_byte (mb, CEE_LDLEN);
+                               mono_mb_emit_byte (mb, CEE_BGE);
+                               label3 = mb->pos;
+                               mono_mb_emit_i4 (mb, 0);
+
+                               /* Emit marshalling code */
+
+                               /* set the src_ptr */
+                               mono_mb_emit_ldloc (mb, src_ptr);
+                               mono_mb_emit_byte (mb, CEE_STLOC_0);
+
+                               /* set dst_ptr */
+                               mono_mb_emit_ldarg (mb, argnum);
+                               mono_mb_emit_ldloc (mb, index_var);
+                               mono_mb_emit_byte (mb, CEE_LDELEMA);
+                               mono_mb_emit_i4 (mb, mono_mb_add_data (mb, eklass));
+                               mono_mb_emit_byte (mb, CEE_STLOC_1);
+
+                               /* emit valuetype conversion code */
+                               emit_struct_conv (mb, eklass, TRUE);
+
+                               mono_mb_emit_add_to_local (mb, index_var, 1);
+                               mono_mb_emit_add_to_local (mb, src_ptr, mono_class_native_size (eklass, NULL));
+
+                               mono_mb_emit_byte (mb, CEE_BR);
+                               mono_mb_emit_i4 (mb, label2 - (mb->pos + 4));
+
+                               mono_mb_patch_addr (mb, label1, mb->pos - (label1 + 4));
+                               mono_mb_patch_addr (mb, label3, mb->pos - (label3 + 4));
+                       }
+                       break;
+               case MONO_TYPE_BOOLEAN:
+                       if (!t->byref)
+                               continue;
+                       mono_mb_emit_ldarg (mb, argnum);
+                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
+                       if (mspecs [i + 1] != NULL && mspecs [i + 1]->native == MONO_NATIVE_VARIANTBOOL)
+                               mono_mb_emit_byte (mb, CEE_NEG);
+                       mono_mb_emit_byte (mb, CEE_STIND_I1);
+               }
+       }
+
+       if (!MONO_TYPE_IS_VOID(sig->ret))
+               mono_mb_emit_byte (mb, CEE_LDLOC_3);
+
+       mono_mb_emit_byte (mb, CEE_RET);
+
+       csig = mono_metadata_signature_dup (sig);
+       csig->pinvoke = 0;
+       res = mono_mb_create_and_cache (cache, method,
+                                                                       mb, csig, csig->param_count + 16);
+       mono_mb_free (mb);
+
+       for (i = sig->param_count; i >= 0; i--)
+               g_free (mspecs [i]);
+       g_free (mspecs);
+
+       //printf ("CODE FOR %s: \n%s.\n", mono_method_full_name (res, TRUE), mono_disasm_code (0, res, ((MonoMethodNormal*)res)->header->code, ((MonoMethodNormal*)res)->header->code + ((MonoMethodNormal*)res)->header->code_size));
+
+       return res;
+}
+
+void
+mono_upgrade_remote_class_wrapper (MonoReflectionType *rtype, MonoTransparentProxy *tproxy);
+
+static MonoReflectionType *
+type_from_handle (MonoType *handle)
+{
+       MonoDomain *domain = mono_domain_get (); 
+       MonoClass *klass = mono_class_from_mono_type (handle);
+
+       MONO_ARCH_SAVE_REGS;
+
+       mono_class_init (klass);
+       return mono_type_get_object (domain, handle);
+}
+
+/*
+ * mono_marshal_get_isinst:
+ * @klass: the type of the field
+ *
+ * This method generates a function which can be used to check if an object is
+ * an instance of the given type, icluding the case where the object is a proxy.
+ * The generated function has the following signature:
+ * MonoObject* __isinst_wrapper_ (MonoObject *obj)
+ */
+MonoMethod *
+mono_marshal_get_isinst (MonoClass *klass)
+{
+       static MonoMethodSignature *isint_sig = NULL;
+       static GHashTable *isinst_hash = NULL; 
+       MonoMethod *res;
+       int pos_was_ok, pos_failed, pos_end, pos_end2;
+       char *name;
+       MonoMethodBuilder *mb;
+
+       EnterCriticalSection (&marshal_mutex);
+       if (!isinst_hash) 
+               isinst_hash = g_hash_table_new (NULL, NULL);
+       
+       res = g_hash_table_lookup (isinst_hash, klass);
+       LeaveCriticalSection (&marshal_mutex);
+       if (res)
+               return res;
+
+       if (!isint_sig) {
+               isint_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 1);
+               isint_sig->params [0] = &mono_defaults.object_class->byval_arg;
+               isint_sig->ret = &mono_defaults.object_class->byval_arg;
+               isint_sig->pinvoke = 0;
+       }
+       
+       name = g_strdup_printf ("__isinst_wrapper_%s", klass->name); 
+       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_UNKNOWN);
+       g_free (name);
+       
+       mb->method->save_lmf = 1;
+
+       /* check if the object is a proxy that needs special cast */
+       mono_mb_emit_ldarg (mb, 0);
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_CISINST);
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+
+       /* The result of MONO_ISINST can be:
+               0) the type check succeeded
+               1) the type check did not succeed
+               2) a CanCastTo call is needed */
+       
+       mono_mb_emit_byte (mb, CEE_DUP);
+       pos_was_ok = mono_mb_emit_branch (mb, CEE_BRFALSE);
+
+       mono_mb_emit_byte (mb, CEE_LDC_I4_2);
+       pos_failed = mono_mb_emit_branch (mb, CEE_BNE_UN);
+       
+       /* get the real proxy from the transparent proxy*/
+
+       mono_mb_emit_ldarg (mb, 0);
+       mono_mb_emit_managed_call (mb, mono_marshal_get_proxy_cancast (klass), NULL);
+       pos_end = mono_mb_emit_branch (mb, CEE_BR);
+       
+       /* fail */
+       
+       mono_mb_patch_addr (mb, pos_failed, mb->pos - (pos_failed + 4));
+       mono_mb_emit_byte (mb, CEE_LDNULL);
+       pos_end2 = mono_mb_emit_branch (mb, CEE_BR);
+       
+       /* success */
+       
+       mono_mb_patch_addr (mb, pos_was_ok, mb->pos - (pos_was_ok + 4));
+       mono_mb_emit_byte (mb, CEE_POP);
+       mono_mb_emit_ldarg (mb, 0);
+       
+       /* the end */
+       
+       mono_mb_patch_addr (mb, pos_end, mb->pos - (pos_end + 4));
+       mono_mb_patch_addr (mb, pos_end2, mb->pos - (pos_end2 + 4));
+       mono_mb_emit_byte (mb, CEE_RET);
+
+       res = mono_mb_create_and_cache (isinst_hash, klass, mb, isint_sig, isint_sig->param_count + 16);
+       mono_mb_free (mb);
+
+       return res;
+}
+
+/*
+ * mono_marshal_get_castclass:
+ * @klass: the type of the field
+ *
+ * This method generates a function which can be used to cast an object to
+ * an instance of the given type, icluding the case where the object is a proxy.
+ * The generated function has the following signature:
+ * MonoObject* __castclass_wrapper_ (MonoObject *obj)
+ */
+MonoMethod *
+mono_marshal_get_castclass (MonoClass *klass)
+{
+       static MonoMethodSignature *castclass_sig = NULL;
+       static GHashTable *castclass_hash = NULL; 
+       MonoMethod *res;
+       int pos_was_ok, pos_was_ok2;
+       char *name;
+       MonoMethodBuilder *mb;
+
+       EnterCriticalSection (&marshal_mutex);
+       if (!castclass_hash) 
+               castclass_hash = g_hash_table_new (NULL, NULL);
+       
+       res = g_hash_table_lookup (castclass_hash, klass);
+       LeaveCriticalSection (&marshal_mutex);
+       if (res)
+               return res;
+
+       if (!castclass_sig) {
+               castclass_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 1);
+               castclass_sig->params [0] = &mono_defaults.object_class->byval_arg;
+               castclass_sig->ret = &mono_defaults.object_class->byval_arg;
+               castclass_sig->pinvoke = 0;
+       }
+       
+       name = g_strdup_printf ("__castclass_wrapper_%s", klass->name); 
+       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_UNKNOWN);
+       g_free (name);
+       
+       mb->method->save_lmf = 1;
+
+       /* check if the object is a proxy that needs special cast */
+       mono_mb_emit_ldarg (mb, 0);
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_CCASTCLASS);
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, klass));
+
+       /* The result of MONO_ISINST can be:
+               0) the cast is valid
+               1) cast of unknown proxy type
+               or an exception if the cast is is invalid
+       */
+       
+       pos_was_ok = mono_mb_emit_branch (mb, CEE_BRFALSE);
+
+       /* get the real proxy from the transparent proxy*/
 
-                       mono_mb_emit_byte (mb, CEE_LDLOC_1);
-                       mono_mb_emit_byte (mb, CEE_BRFALSE);
-                       pos = mb->pos;
-                       mono_mb_emit_i4 (mb, 0);
+       mono_mb_emit_ldarg (mb, 0);
+       mono_mb_emit_managed_call (mb, mono_marshal_get_proxy_cancast (klass), NULL);
+       pos_was_ok2 = mono_mb_emit_branch (mb, CEE_BRTRUE);
+       
+       /* fail */
+       mono_mb_emit_exception (mb, "InvalidCastException", NULL);
+       
+       /* success */
+       mono_mb_patch_addr (mb, pos_was_ok, mb->pos - (pos_was_ok + 4));
+       mono_mb_patch_addr (mb, pos_was_ok2, mb->pos - (pos_was_ok2 + 4));
+       mono_mb_emit_ldarg (mb, 0);
+       
+       /* the end */
+       mono_mb_emit_byte (mb, CEE_RET);
 
-                       /* src = tmp_locals [i] */
-                       mono_mb_emit_ldloc (mb, tmp_locals [i]);
-                       mono_mb_emit_byte (mb, CEE_STLOC_0);
+       res = mono_mb_create_and_cache (castclass_hash, klass, mb, castclass_sig, castclass_sig->param_count + 16);
+       mono_mb_free (mb);
 
-                       /* emit valuetype conversion code */
-                       emit_struct_conv (mb, klass, TRUE);
-                       
-                       mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
-                       break;
-               case MONO_TYPE_SZARRAY:
-                       if (t->byref)
-                               continue;
-                       klass = mono_class_from_mono_type (t);
-                       
-                       if (klass->element_class == mono_defaults.string_class) {
-                               g_assert (tmp_locals [i]);
+       return res;
+}
 
-                               mono_mb_emit_ldarg (mb, argnum);
-                               mono_mb_emit_byte (mb, CEE_BRFALSE);
-                               pos = mb->pos;
-                               mono_mb_emit_i4 (mb, 0);
+MonoMethod *
+mono_marshal_get_proxy_cancast (MonoClass *klass)
+{
+       static MonoMethodSignature *from_handle_sig = NULL;
+       static MonoMethodSignature *upgrade_proxy_sig = NULL;
+       static MonoMethodSignature *isint_sig = NULL;
+       static GHashTable *proxy_isinst_hash = NULL; 
+       MonoMethod *res;
+       int pos_failed, pos_end;
+       char *name;
+       MonoMethod *can_cast_to;
+       MonoMethodDesc *desc;
+       MonoMethodBuilder *mb;
 
-                               mono_mb_emit_ldloc (mb, tmp_locals [i]);
-                               mono_mb_emit_ldarg (mb, argnum);
-                               mono_mb_emit_byte (mb, CEE_LDLEN);                              
-                               mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
-                               mono_mb_emit_byte (mb, CEE_MONO_PROC2);
-                               mono_mb_emit_byte (mb, MONO_MARSHAL_FREE_ARRAY);
+       EnterCriticalSection (&marshal_mutex);
+       if (!proxy_isinst_hash) 
+               proxy_isinst_hash = g_hash_table_new (NULL, NULL);
+       
+       res = g_hash_table_lookup (proxy_isinst_hash, klass);
+       LeaveCriticalSection (&marshal_mutex);
+       if (res)
+               return res;
 
-                               mono_mb_patch_addr (mb, pos, mb->pos - (pos + 4));
-                       }
+       if (!isint_sig) {
+               upgrade_proxy_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 2);
+               upgrade_proxy_sig->params [0] = &mono_defaults.object_class->byval_arg;
+               upgrade_proxy_sig->params [1] = &mono_defaults.object_class->byval_arg;
+               upgrade_proxy_sig->ret = &mono_defaults.void_class->byval_arg;
+               upgrade_proxy_sig->pinvoke = 1;
 
-                       break;
-               }
+               from_handle_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 1);
+               from_handle_sig->params [0] = &mono_defaults.object_class->byval_arg;
+               from_handle_sig->ret = &mono_defaults.object_class->byval_arg;
+       
+               isint_sig = mono_metadata_signature_alloc (mono_defaults.corlib, 1);
+               isint_sig->params [0] = &mono_defaults.object_class->byval_arg;
+               isint_sig->ret = &mono_defaults.object_class->byval_arg;
+               isint_sig->pinvoke = 0;
        }
+       
+       name = g_strdup_printf ("__proxy_isinst_wrapper_%s", klass->name); 
+       mb = mono_mb_new (mono_defaults.object_class, name, MONO_WRAPPER_UNKNOWN);
+       g_free (name);
+       
+       mb->method->save_lmf = 1;
 
+       /* get the real proxy from the transparent proxy*/
+       mono_mb_emit_ldarg (mb, 0);
+       mono_mb_emit_ldflda (mb, G_STRUCT_OFFSET (MonoTransparentProxy, rp));
+       mono_mb_emit_byte (mb, CEE_LDIND_I);
+       
+       /* get the refletion type from the type handle */
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_LDPTR);
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, &klass->byval_arg));
+       mono_mb_emit_native_call (mb, from_handle_sig, type_from_handle);
+       
+       mono_mb_emit_ldarg (mb, 0);
+       
+       /* make the call to CanCastTo (type, ob) */
+       desc = mono_method_desc_new ("IRemotingTypeInfo:CanCastTo", FALSE);
+       can_cast_to = mono_method_desc_search_in_class (desc, mono_defaults.iremotingtypeinfo_class);
+       g_assert (can_cast_to);
+       mono_method_desc_free (desc);
+       mono_mb_emit_byte (mb, CEE_CALLVIRT);
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, can_cast_to));
 
-       if (!MONO_TYPE_IS_VOID(sig->ret))
-               mono_mb_emit_byte (mb, CEE_LDLOC_3);
+       
+       pos_failed = mono_mb_emit_branch (mb, CEE_BRFALSE);
 
+       /* Upgrade the proxy vtable by calling: mono_upgrade_remote_class_wrapper (type, ob)*/
+       mono_mb_emit_byte (mb, MONO_CUSTOM_PREFIX);
+       mono_mb_emit_byte (mb, CEE_MONO_LDPTR);
+       mono_mb_emit_i4 (mb, mono_mb_add_data (mb, &klass->byval_arg));
+       mono_mb_emit_native_call (mb, from_handle_sig, type_from_handle);
+       mono_mb_emit_ldarg (mb, 0);
+       
+       mono_mb_emit_native_call (mb, upgrade_proxy_sig, mono_upgrade_remote_class_wrapper);
+       emit_thread_interrupt_checkpoint (mb);
+       
+       mono_mb_emit_ldarg (mb, 0);
+       pos_end = mono_mb_emit_branch (mb, CEE_BR);
+       
+       /* fail */
+       
+       mono_mb_patch_addr (mb, pos_failed, mb->pos - (pos_failed + 4));
+       mono_mb_emit_byte (mb, CEE_LDNULL);
+       
+       /* the end */
+       
+       mono_mb_patch_addr (mb, pos_end, mb->pos - (pos_end + 4));
        mono_mb_emit_byte (mb, CEE_RET);
 
-       csig = g_memdup (sig, sigsize);
-       csig->pinvoke = 0;
-       res = mono_mb_create_method (mb, csig, csig->param_count + 16);
+       res = mono_mb_create_and_cache (proxy_isinst_hash, klass, mb, isint_sig, isint_sig->param_count + 16);
        mono_mb_free (mb);
 
-       g_hash_table_insert (cache, method, res);
-
-       for (i = sig->param_count; i >= 0; i--)
-               g_free (mspecs [i]);
-       g_free (mspecs);
-
        return res;
 }
 
-/*
+void
+mono_upgrade_remote_class_wrapper (MonoReflectionType *rtype, MonoTransparentProxy *tproxy)
+{
+       MonoClass *klass;
+       klass = mono_class_from_mono_type (rtype->type);
+       mono_upgrade_remote_class (((MonoObject*)tproxy)->vtable->domain, tproxy->remote_class, klass);
+       ((MonoObject*)tproxy)->vtable = tproxy->remote_class->vtable;
+}
+
+/**
+ * mono_marshal_get_struct_to_ptr:
+ * @klass:
+ *
  * generates IL code for StructureToPtr (object structure, IntPtr ptr, bool fDeleteOld)
  */
 MonoMethod *
@@ -3270,7 +4472,10 @@ mono_marshal_get_struct_to_ptr (MonoClass *klass)
        return res;
 }
 
-/*
+/**
+ * mono_marshal_get_ptr_to_struct:
+ * @klass:
+ *
  * generates IL code for PtrToStructure (IntPtr src, object structure)
  */
 MonoMethod *
@@ -3326,18 +4531,6 @@ mono_marshal_get_ptr_to_struct (MonoClass *klass)
        return res;
 }
 
-static MonoReflectionType *
-type_from_handle (MonoType *handle)
-{
-       MonoDomain *domain = mono_domain_get (); 
-       MonoClass *klass = mono_class_from_mono_type (handle);
-
-       MONO_ARCH_SAVE_REGS;
-
-       mono_class_init (klass);
-       return mono_type_get_object (domain, handle);
-}
-
 /*
  * generates IL code for the synchronized wrapper: the generated method
  * calls METHOD while locking 'this' or the parent type.
@@ -3361,7 +4554,7 @@ mono_marshal_get_synchronized_wrapper (MonoMethod *method)
                return method;
 
        cache = method->klass->image->synchronized_cache;
-       if ((res = (MonoMethod *)g_hash_table_lookup (cache, method)))
+       if ((res = mono_marshal_find_in_cache (cache, method)))
                return res;
 
        sig = method->signature;
@@ -3446,15 +4639,14 @@ mono_marshal_get_synchronized_wrapper (MonoMethod *method)
                mono_mb_emit_ldloc (mb, ret_local);
        mono_mb_emit_byte (mb, CEE_RET);
 
-       res = mono_mb_create_method (mb, sig, sig->param_count + 16);
+       res = mono_mb_create_and_cache (cache, method,
+                                                                       mb, sig, sig->param_count + 16);
        mono_mb_free (mb);
 
        header = ((MonoMethodNormal *)res)->header;
        header->num_clauses = 1;
        header->clauses = clause;
 
-       g_hash_table_insert (cache, method, res);
-
        return res;     
 }
 
@@ -3518,6 +4710,44 @@ mono_marshal_string_array (MonoArray *array)
        return result;
 }
 
+void *
+mono_marshal_string_array_to_unicode (MonoArray *array)
+{
+       gunichar2 **result;
+       int i, len;
+
+       if (!array)
+               return NULL;
+
+       len = mono_array_length (array);
+
+       result = g_malloc (sizeof (gunichar2 *) * (len + 1));
+       for (i = 0; i < len; ++i) {
+               MonoString *s = (MonoString *)mono_array_get (array, gpointer, i);
+               result [i] = s ? mono_string_chars (s) : NULL;
+       }
+       /* null terminate the array */
+       result [i] = NULL;
+
+       return result;
+}
+
+/**
+ * mono_marshal_set_last_error:
+ *
+ * This function is invoked to set the last error value from a P/Invoke call
+ * which has SetLastError set.
+ */
+void
+mono_marshal_set_last_error (void)
+{
+#ifdef WIN32
+       TlsSetValue (last_error_tls_id, (gpointer)GetLastError ());
+#else
+       TlsSetValue (last_error_tls_id, (gpointer)errno);
+#endif
+}
+
 void
 ves_icall_System_Runtime_InteropServices_Marshal_copy_to_unmanaged (MonoArray *src, gint32 start_index,
                                                                    gpointer dest, gint32 length)
@@ -3566,6 +4796,22 @@ ves_icall_System_Runtime_InteropServices_Marshal_copy_from_unmanaged (gpointer s
        memcpy (dest_addr, src, length * element_size);
 }
 
+#if NO_UNALIGNED_ACCESS
+#define RETURN_UNALIGNED(type, addr) \
+       { \
+               type val; \
+               memcpy(&val, p + offset, sizeof(val)); \
+               return val; \
+       }
+#define WRITE_UNALIGNED(type, addr, val) \
+       memcpy(addr, &val, sizeof(type))
+#else
+#define RETURN_UNALIGNED(type, addr) \
+       return *(type*)(p + offset);
+#define WRITE_UNALIGNED(type, addr, val) \
+       (*(type *)(addr) = (val))
+#endif
+
 gpointer
 ves_icall_System_Runtime_InteropServices_Marshal_ReadIntPtr (gpointer ptr, gint32 offset)
 {
@@ -3573,7 +4819,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_ReadIntPtr (gpointer ptr, gint3
 
        MONO_ARCH_SAVE_REGS;
 
-       return *(gpointer*)(p + offset);
+       RETURN_UNALIGNED(gpointer, p + offset);
 }
 
 unsigned char
@@ -3593,7 +4839,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_ReadInt16 (gpointer ptr, gint32
 
        MONO_ARCH_SAVE_REGS;
 
-       return *(gint16*)(p + offset);
+       RETURN_UNALIGNED(gint16, p + offset);
 }
 
 gint32
@@ -3603,7 +4849,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_ReadInt32 (gpointer ptr, gint32
 
        MONO_ARCH_SAVE_REGS;
 
-       return *(gint32*)(p + offset);
+       RETURN_UNALIGNED(gint32, p + offset);
 }
 
 gint64
@@ -3613,7 +4859,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_ReadInt64 (gpointer ptr, gint32
 
        MONO_ARCH_SAVE_REGS;
 
-       return *(gint64*)(p + offset);
+       RETURN_UNALIGNED(gint64, p + offset);
 }
 
 void
@@ -3633,7 +4879,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_WriteIntPtr (gpointer ptr, gint
 
        MONO_ARCH_SAVE_REGS;
 
-       *(gpointer*)(p + offset) = val;
+       WRITE_UNALIGNED(gpointer, p + offset, val);
 }
 
 void
@@ -3643,7 +4889,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_WriteInt16 (gpointer ptr, gint3
 
        MONO_ARCH_SAVE_REGS;
 
-       *(gint16*)(p + offset) = val;
+       WRITE_UNALIGNED(gint16, p + offset, val);
 }
 
 void
@@ -3653,7 +4899,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_WriteInt32 (gpointer ptr, gint3
 
        MONO_ARCH_SAVE_REGS;
 
-       *(gint32*)(p + offset) = val;
+       WRITE_UNALIGNED(gint32, p + offset, val);
 }
 
 void
@@ -3663,7 +4909,7 @@ ves_icall_System_Runtime_InteropServices_Marshal_WriteInt64 (gpointer ptr, gint3
 
        MONO_ARCH_SAVE_REGS;
 
-       *(gint64*)(p + offset) = val;
+       WRITE_UNALIGNED(gint64, p + offset, val);
 }
 
 MonoString *
@@ -3671,7 +4917,10 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringAnsi (char *ptr)
 {
        MONO_ARCH_SAVE_REGS;
 
-       return mono_string_new (mono_domain_get (), ptr);
+       if (ptr == NULL)
+               return mono_string_new (mono_domain_get (), "");
+       else
+               return mono_string_new (mono_domain_get (), ptr);
 }
 
 MonoString *
@@ -3679,7 +4928,10 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringAnsi_len (char *ptr,
 {
        MONO_ARCH_SAVE_REGS;
 
-       return mono_string_new_len (mono_domain_get (), ptr, len);
+       if (ptr == NULL)
+               return mono_string_new (mono_domain_get (), "");
+       else
+               return mono_string_new_len (mono_domain_get (), ptr, len);
 }
 
 MonoString *
@@ -3691,6 +4943,9 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringUni (guint16 *ptr)
 
        MONO_ARCH_SAVE_REGS;
 
+       if (ptr == NULL)
+               return mono_string_new (mono_domain_get (), "");
+
        while (*t++)
                len++;
 
@@ -3704,7 +4959,10 @@ ves_icall_System_Runtime_InteropServices_Marshal_PtrToStringUni_len (guint16 *pt
 
        MONO_ARCH_SAVE_REGS;
 
-       return mono_string_new_utf16 (domain, ptr, len);
+       if (ptr == NULL)
+               return mono_string_new (mono_domain_get (), "");
+       else
+               return mono_string_new_utf16 (domain, ptr, len);
 }
 
 MonoString *
@@ -3723,19 +4981,34 @@ ves_icall_System_Runtime_InteropServices_Marshal_GetLastWin32Error (void)
 {
        MONO_ARCH_SAVE_REGS;
 
-       return (GetLastError ());
+       return ((guint32)TlsGetValue (last_error_tls_id));
 }
 
 guint32 
 ves_icall_System_Runtime_InteropServices_Marshal_SizeOf (MonoReflectionType *rtype)
 {
        MonoClass *klass;
+       MonoType *type;
+       guint32 layout;
 
        MONO_ARCH_SAVE_REGS;
 
        MONO_CHECK_ARG_NULL (rtype);
 
-       klass = mono_class_from_mono_type (rtype->type);
+       type = rtype->type;
+       klass = mono_class_from_mono_type (type);
+       layout = (klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK);
+
+       if (layout == TYPE_ATTRIBUTE_AUTO_LAYOUT) {
+               gchar *msg;
+               MonoException *exc;
+
+               msg = g_strdup_printf ("Type %s cannot be marshaled as an unmanaged structure.", klass->name);
+               exc = mono_get_exception_argument ("t", msg);
+               g_free (msg);
+               mono_raise_exception (exc);
+       }
+
 
        return mono_class_native_size (klass, NULL);
 }
@@ -3858,7 +5131,10 @@ ves_icall_System_Runtime_InteropServices_Marshal_StringToHGlobalUni (MonoString
 {
        MONO_ARCH_SAVE_REGS;
 
-       return g_memdup (mono_string_chars (string), mono_string_length (string)*2);
+       if (string == NULL)
+               return NULL;
+       else
+               return g_memdup (mono_string_chars (string), mono_string_length (string)*2);
 }
 
 static void
@@ -3919,10 +5195,34 @@ ves_icall_System_Runtime_InteropServices_Marshal_DestroyStructure (gpointer src,
        mono_struct_delete_old (klass, (char *)src);
 }
 
+void*
+ves_icall_System_Runtime_InteropServices_Marshal_AllocCoTaskMem (int size)
+{
+       /* FIXME: Call AllocCoTaskMem under windows */
+       MONO_ARCH_SAVE_REGS;
+
+       return g_try_malloc ((gulong)size);
+}
+
+void
+ves_icall_System_Runtime_InteropServices_Marshal_FreeCoTaskMem (void *ptr)
+{
+       /* FIXME: Call FreeCoTaskMem under windows */
+       MONO_ARCH_SAVE_REGS;
+
+       g_free (ptr);
+}
+
+void*
+ves_icall_System_Runtime_InteropServices_Marshal_UnsafeAddrOfPinnedArrayElement (MonoArray *arrayobj, int index)
+{
+       return mono_array_addr_with_size (arrayobj, mono_array_element_size (arrayobj->obj.vtable->klass), index);
+}
+
 MonoMarshalType *
 mono_marshal_load_type_info (MonoClass* klass)
 {
-       int i, j, count = 0, native_size = 0;
+       int i, j, count = 0, native_size = 0, min_align = 1;
        MonoMarshalType *info;
        guint32 layout;
 
@@ -3937,6 +5237,8 @@ mono_marshal_load_type_info (MonoClass* klass)
        for (i = 0; i < klass->field.count; ++i) {
                if (klass->fields [i].type->attrs & FIELD_ATTRIBUTE_STATIC)
                        continue;
+               if (mono_field_is_deleted (&klass->fields [i]))
+                       continue;
                count++;
        }
 
@@ -3962,18 +5264,27 @@ mono_marshal_load_type_info (MonoClass* klass)
                if (klass->fields [i].type->attrs & FIELD_ATTRIBUTE_STATIC)
                        continue;
 
+               if (mono_field_is_deleted (&klass->fields [i]))
+                       continue;
                if (klass->fields [i].type->attrs & FIELD_ATTRIBUTE_HAS_FIELD_MARSHAL)
                        mono_metadata_field_info (klass->image, klass->field.first + i, 
                                                  NULL, NULL, &info->fields [j].mspec);
 
                info->fields [j].field = &klass->fields [i];
 
+               if ((klass->field.count == 1) && (klass->instance_size == sizeof (MonoObject)) &&
+                       (strcmp (klass->fields [i].name, "$PRIVATE$") == 0)) {
+                       /* This field is a hack inserted by MCS to empty structures */
+                       continue;
+               }
+
                switch (layout) {
                case TYPE_ATTRIBUTE_AUTO_LAYOUT:
                case TYPE_ATTRIBUTE_SEQUENTIAL_LAYOUT:
                        size = mono_marshal_type_size (klass->fields [i].type, info->fields [j].mspec, 
                                                       &align, TRUE, klass->unicode);
-                       align = klass->packing_size ? MIN (klass->packing_size, align): align;  
+                       align = klass->packing_size ? MIN (klass->packing_size, align): align;
+                       min_align = MAX (align, min_align);
                        info->fields [j].offset = info->native_size;
                        info->fields [j].offset += align - 1;
                        info->fields [j].offset &= ~(align - 1);
@@ -3992,9 +5303,9 @@ mono_marshal_load_type_info (MonoClass* klass)
                info->native_size = MAX (native_size, info->native_size);
        }
 
-       if (info->native_size & (klass->min_align - 1)) {
-               info->native_size += klass->min_align - 1;
-               info->native_size &= ~(klass->min_align - 1);
+       if (info->native_size & (min_align - 1)) {
+               info->native_size += min_align - 1;
+               info->native_size &= ~(min_align - 1);
        }
 
        return klass->marshal_info;
@@ -4093,6 +5404,11 @@ mono_type_native_stack_size (MonoType *t, gint *align)
        return 0;
 }
 
+/* __alignof__ returns the preferred alignment of values not the actual alignment used by
+   the compiler so is wrong e.g. for Linux where doubles are aligned on a 4 byte boundary
+   but __alignof__ returns 8 - using G_STRUCT_OFFSET works better */
+#define ALIGNMENT(type) G_STRUCT_OFFSET(struct { char c; type x; }, x)
+
 gint32
 mono_marshal_type_size (MonoType *type, MonoMarshalSpec *mspec, gint32 *align, 
                        gboolean as_field, gboolean unicode)
@@ -4110,6 +5426,7 @@ mono_marshal_type_size (MonoType *type, MonoMarshalSpec *mspec, gint32 *align,
                return 1;
        case MONO_NATIVE_I2:
        case MONO_NATIVE_U2:
+       case MONO_NATIVE_VARIANTBOOL:
                *align = 2;
                return 2;
        case MONO_NATIVE_I4:
@@ -4119,13 +5436,13 @@ mono_marshal_type_size (MonoType *type, MonoMarshalSpec *mspec, gint32 *align,
                return 4;
        case MONO_NATIVE_I8:
        case MONO_NATIVE_U8:
-               *align = 4;
+               *align = ALIGNMENT(guint64);
                return 8;
        case MONO_NATIVE_R4:
                *align = 4;
                return 4;
        case MONO_NATIVE_R8:
-               *align = 4;
+               *align = ALIGNMENT(double);
                return 8;
        case MONO_NATIVE_INT:
        case MONO_NATIVE_UINT:
@@ -4141,10 +5458,9 @@ mono_marshal_type_size (MonoType *type, MonoMarshalSpec *mspec, gint32 *align,
        case MONO_NATIVE_IDISPATCH:
        case MONO_NATIVE_INTERFACE:
        case MONO_NATIVE_ASANY:
-       case MONO_NATIVE_VARIANTBOOL:
        case MONO_NATIVE_FUNC:
        case MONO_NATIVE_LPSTRUCT:
-               *align =  4;
+               *align = ALIGNMENT(gpointer);
                return sizeof (gpointer);
        case MONO_NATIVE_STRUCT: 
                klass = mono_class_from_mono_type (type);
@@ -4175,3 +5491,119 @@ mono_marshal_type_size (MonoType *type, MonoMarshalSpec *mspec, gint32 *align,
        return 0;
 }
 
+gpointer
+mono_marshal_asany (MonoObject *o, MonoMarshalNative string_encoding)
+{
+       MonoType *t;
+       MonoClass *klass;
+
+       if (o == NULL)
+               return NULL;
+
+       t = &o->vtable->klass->byval_arg;
+       switch (t->type) {
+       case MONO_TYPE_I4:
+       case MONO_TYPE_U4:
+       case MONO_TYPE_PTR:
+       case MONO_TYPE_I1:
+       case MONO_TYPE_U1:
+       case MONO_TYPE_BOOLEAN:
+       case MONO_TYPE_I2:
+       case MONO_TYPE_U2:
+       case MONO_TYPE_CHAR:
+       case MONO_TYPE_I8:
+       case MONO_TYPE_U8:
+       case MONO_TYPE_R4:
+       case MONO_TYPE_R8:
+               return mono_object_unbox (o);
+               break;
+       case MONO_TYPE_STRING:
+               switch (string_encoding) {
+               case MONO_NATIVE_LPWSTR:
+                       return mono_string_to_utf16 ((MonoString*)o);
+                       break;
+               case MONO_NATIVE_LPSTR:
+                       return mono_string_to_utf8 ((MonoString*)o);
+                       break;
+               default:
+                       g_warning ("marshaling conversion %d not implemented", string_encoding);
+                       g_assert_not_reached ();
+               }
+               break;
+       case MONO_TYPE_CLASS:
+       case MONO_TYPE_VALUETYPE: {
+               MonoMethod *method;
+               gpointer pa [3];
+               gpointer res;
+               MonoBoolean delete_old = FALSE;
+
+               klass = t->data.klass;
+
+               if ((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) == TYPE_ATTRIBUTE_AUTO_LAYOUT)
+                       break;
+
+               if (((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) == TYPE_ATTRIBUTE_EXPLICIT_LAYOUT) ||
+                       klass->blittable || klass->enumtype)
+                       return mono_object_unbox (o);
+
+               res = g_malloc0 (mono_class_native_size (klass, NULL));
+
+               method = mono_marshal_get_struct_to_ptr (o->vtable->klass);
+
+               pa [0] = o;
+               pa [1] = &res;
+               pa [2] = &delete_old;
+
+               mono_runtime_invoke (method, NULL, pa, NULL);
+
+               return res;
+       }
+       }
+
+       mono_raise_exception (mono_get_exception_argument ("", "No PInvoke conversion exists for value passed to Object-typed parameter."));
+
+       return NULL;
+}
+
+void
+mono_marshal_free_asany (MonoObject *o, gpointer ptr, MonoMarshalNative string_encoding)
+{
+       MonoType *t;
+       MonoClass *klass;
+
+       /* FIXME: Free embedded data as well */
+
+       if (o == NULL)
+               return;
+
+       t = &o->vtable->klass->byval_arg;
+       switch (t->type) {
+       case MONO_TYPE_STRING:
+               switch (string_encoding) {
+               case MONO_NATIVE_LPWSTR:
+                       g_free (ptr);
+                       break;
+               case MONO_NATIVE_LPSTR:
+                       g_free (ptr);
+                       break;
+               default:
+                       g_warning ("marshaling conversion %d not implemented", string_encoding);
+                       g_assert_not_reached ();
+               }
+               break;
+       case MONO_TYPE_CLASS:
+       case MONO_TYPE_VALUETYPE: {
+               klass = t->data.klass;
+
+               if (((klass->flags & TYPE_ATTRIBUTE_LAYOUT_MASK) == TYPE_ATTRIBUTE_EXPLICIT_LAYOUT) ||
+                       klass->blittable || klass->enumtype)
+                       break;
+
+               g_free (ptr);
+               break;
+       }
+       default:
+               break;
+       }
+}
+