2006-04-16 Zoltan Varga <vargaz@gmail.com>
[mono.git] / mono / mini / tramp-x86.c
index 83bad9893f295264661c381bbba9f6bfa2360c59..f2904793225970a2b2209930a8541689f27c317b 100644 (file)
 #include <glib.h>
 
 #include <mono/metadata/appdomain.h>
+#include <mono/metadata/metadata-internals.h>
 #include <mono/metadata/marshal.h>
 #include <mono/metadata/tabledefs.h>
-#include <mono/arch/x86/x86-codegen.h>
+#include <mono/metadata/mono-debug.h>
 #include <mono/metadata/mono-debug-debugger.h>
+#include <mono/arch/x86/x86-codegen.h>
 
 #ifdef HAVE_VALGRIND_MEMCHECK_H
 #include <valgrind/memcheck.h>
 #include "mini.h"
 #include "mini-x86.h"
 
-typedef enum {
-       MONO_TRAMPOLINE_GENERIC,
-       MONO_TRAMPOLINE_JUMP,
-       MONO_TRAMPOLINE_CLASS_INIT
-} MonoTrampolineType;
-
-/* adapt to mini later... */
-#define mono_jit_share_code (1)
+static guint8* nullified_class_init_trampoline;
 
 /*
- * Address of the x86 trampoline code.  This is used by the debugger to check
- * whether a method is a trampoline.
- */
-guint8 *mono_generic_trampoline_code = NULL;
-
-/*
- * get_unbox_trampoline:
+ * mono_arch_get_unbox_trampoline:
  * @m: method pointer
  * @addr: pointer to native code for @m
  *
@@ -47,16 +36,19 @@ guint8 *mono_generic_trampoline_code = NULL;
  * this argument. This method returns a pointer to a trampoline which does
  * unboxing before calling the method
  */
-static gpointer
-get_unbox_trampoline (MonoMethod *m, gpointer addr)
+gpointer
+mono_arch_get_unbox_trampoline (MonoMethod *m, gpointer addr)
 {
        guint8 *code, *start;
        int this_pos = 4;
+       MonoDomain *domain = mono_domain_get ();
 
-       if (!m->signature->ret->byref && MONO_TYPE_ISSTRUCT (m->signature->ret))
+       if (!mono_method_signature (m)->ret->byref && MONO_TYPE_ISSTRUCT (mono_method_signature (m)->ret))
                this_pos = 8;
            
-       start = code = g_malloc (16);
+       mono_domain_lock (domain);
+       start = code = mono_code_manager_reserve (domain->code_mp, 16);
+       mono_domain_unlock (domain);
 
        x86_alu_membase_imm (code, X86_ADD, X86_ESP, this_pos, sizeof (MonoObject));
        x86_jump_code (code, addr);
@@ -65,42 +57,9 @@ get_unbox_trampoline (MonoMethod *m, gpointer addr)
        return start;
 }
 
-/**
- * x86_magic_trampoline:
- * @eax: saved x86 register 
- * @ecx: saved x86 register 
- * @edx: saved x86 register 
- * @esi: saved x86 register 
- * @edi: saved x86 register 
- * @ebx: saved x86 register
- * @code: pointer into caller code
- * @method: the method to translate
- *
- * This method is called by the trampoline functions for virtual
- * methods. It inspects the caller code to find the address of the
- * vtable slot, then calls the JIT compiler and writes the address
- * of the compiled method back to the vtable. All virtual methods 
- * are called with: x86_call_membase (inst, basereg, disp). We always
- * use 32 bit displacement to ensure that the length of the call 
- * instruction is 6 bytes. We need to get the value of the basereg 
- * and the constant displacement.
- */
-static gpointer
-x86_magic_trampoline (int eax, int ecx, int edx, int esi, int edi, 
-                     int ebx, guint8 *code, MonoMethod *m)
+void
+mono_arch_patch_callsite (guint8 *code, guint8 *addr)
 {
-       guint8 reg;
-       gint32 disp;
-       char *o;
-       gpointer addr;
-
-       addr = mono_compile_method (m);
-       g_assert (addr);
-
-       /* the method was jumped to */
-       if (!code)
-               return addr;
-
        /* go to the start of the call instruction
         *
         * address_byte = (m << 6) | (o << 3) | reg
@@ -109,116 +68,28 @@ x86_magic_trampoline (int eax, int ecx, int edx, int esi, int edi,
         * 0xff m=2,o=2 imm32
         */
        code -= 6;
-       if ((code [1] != 0xe8) && (code [3] == 0xff) && ((code [4] & 0x18) == 0x10) && ((code [4] >> 6) == 1)) {
-               reg = code [4] & 0x07;
-               disp = (signed char)code [5];
-       } else {
-               if ((code [0] == 0xff) && ((code [1] & 0x18) == 0x10) && ((code [1] >> 6) == 2)) {
-                       reg = code [1] & 0x07;
-                       disp = *((gint32*)(code + 2));
-               } else if ((code [1] == 0xe8)) {
-                       MonoJitInfo *ji = 
-                               mono_jit_info_table_find (mono_domain_get (), code);
-                       MonoJitInfo *target_ji = 
-                               mono_jit_info_table_find (mono_domain_get (), addr);
-
-                       /* The first part of the condition means an icall without a wrapper */
-                       if ((!target_ji && m->addr) || mono_method_same_domain (ji, target_ji)) {
-                               gboolean do_patch = TRUE;
+       if ((code [1] == 0xe8)) {
+               if (!mono_running_on_valgrind ()) {
+                       InterlockedExchange ((gint32*)(code + 2), (guint)addr - ((guint)code + 1) - 5);
 
 #ifdef HAVE_VALGRIND_MEMCHECK_H
-                               if (RUNNING_ON_VALGRIND)
-                                       do_patch = FALSE;
+                               /* Tell valgrind to recompile the patched code */
+                               //VALGRIND_DISCARD_TRANSLATIONS (code + 2, code + 6);
 #endif
-
-                               if (do_patch) {
-                                       InterlockedExchange ((gint32*)(code + 2), (guint)addr - ((guint)code + 1) - 5);
-
-#ifdef HAVE_VALGRIND_MEMCHECK_H
-                                       /* Tell valgrind to recompile the patched code */
-                                       VALGRIND_DISCARD_TRANSLATIONS (code + 2, code + 6);
-#endif
-                               }
-                       }
-                       return addr;
-               } else if ((code [4] == 0xff) && (((code [5] >> 6) & 0x3) == 0) && (((code [5] >> 3) & 0x7) == 2)) {
-                       /*
-                        * This is a interface call: should check the above code can't catch it earlier 
-                        * 8b 40 30   mov    0x30(%eax),%eax
-                        * ff 10      call   *(%eax)
-                        */
-                       disp = 0;
-                       reg = code [5] & 0x07;
-               } else {
-                       printf ("Invalid trampoline sequence: %x %x %x %x %x %x %x\n", code [0], code [1], code [2], code [3],
-                               code [4], code [5], code [6]);
-                       g_assert_not_reached ();
                }
-       }
-
-       switch (reg) {
-       case X86_EAX:
-               o = (gpointer)eax;
-               break;
-       case X86_EDX:
-               o = (gpointer)edx;
-               break;
-       case X86_ECX:
-               o = (gpointer)ecx;
-               break;
-       case X86_ESI:
-               o = (gpointer)esi;
-               break;
-       case X86_EDI:
-               o = (gpointer)edi;
-               break;
-       case X86_EBX:
-               o = (gpointer)ebx;
-               break;
-       default:
+       } else {
+               printf ("Invalid trampoline sequence: %x %x %x %x %x %x %x\n", code [0], code [1], code [2], code [3],
+                               code [4], code [5], code [6]);
                g_assert_not_reached ();
        }
-
-       o += disp;
-
-       if (m->klass->valuetype)
-               addr = get_unbox_trampoline (m, addr);
-
-       *((gpointer *)o) = addr;
-
-       return addr;
 }
 
-/**
- * x86_class_init_trampoline:
- * @eax: saved x86 register 
- * @ecx: saved x86 register 
- * @edx: saved x86 register 
- * @esi: saved x86 register 
- * @edi: saved x86 register 
- * @ebx: saved x86 register
- * @code: pointer into caller code
- * @vtable: the type to initialize
- *
- * This method calls mono_runtime_class_init () to run the static constructor
- * for the type, then patches the caller code so it is not called again.
- */
-static void
-x86_class_init_trampoline (int eax, int ecx, int edx, int esi, int edi, 
-                                                  int ebx, guint8 *code, MonoVTable *vtable)
+void
+mono_arch_nullify_class_init_trampoline (guint8 *code, gssize *regs)
 {
-       mono_runtime_class_init (vtable);
-
        code -= 5;
        if (code [0] == 0xe8) {
-               gboolean do_patch = TRUE;
-
-#ifdef HAVE_VALGRIND_MEMCHECK_H
-               if (RUNNING_ON_VALGRIND)
-                       do_patch = FALSE;
-#endif
-
-               if (do_patch) {
+               if (!mono_running_on_valgrind ()) {
                        guint32 ops;
                        /*
                         * Thread safe code patching using the algorithm from the paper
@@ -238,8 +109,7 @@ x86_class_init_trampoline (int eax, int ecx, int edx, int esi, int edi,
 
                        /* Then atomically change the first 4 bytes to a nop as well */
                        ops = 0x90909090;
-                       InterlockedExchange ((guint32*)code, ops);
-
+                       InterlockedExchange ((gint32*)code, ops);
 #ifdef HAVE_VALGRIND_MEMCHECK_H
                        /* FIXME: the calltree skin trips on the self modifying code above */
 
@@ -247,45 +117,71 @@ x86_class_init_trampoline (int eax, int ecx, int edx, int esi, int edi,
                        //VALGRIND_DISCARD_TRANSLATIONS (code, code + 8);
 #endif
                }
-       }
-       else
-               if (code [0] == 0x90 || code [0] == 0xeb)
-                       /* Already changed by another thread */
-                       ;
-               else {
+       } else if (code [0] == 0x90 || code [0] == 0xeb) {
+               /* Already changed by another thread */
+               ;
+       } else if ((code [-1] == 0xff) && (x86_modrm_reg (code [0]) == 0x2)) {
+               /* call *<OFFSET>(<REG>) -> Call made from AOT code */
+               gpointer *vtable_slot;
+
+               vtable_slot = mono_arch_get_vcall_slot_addr (code + 5, (gpointer*)regs);
+               g_assert (vtable_slot);
+
+               *vtable_slot = nullified_class_init_trampoline;
+       } else {
                        printf ("Invalid trampoline sequence: %x %x %x %x %x %x %x\n", code [0], code [1], code [2], code [3],
                                code [4], code [5], code [6]);
                        g_assert_not_reached ();
                }
 }
 
-static guchar*
-create_trampoline_code (MonoTrampolineType tramp_type)
+void
+mono_arch_patch_delegate_trampoline (guint8 *code, guint8 *tramp, gssize *regs, guint8 *addr)
 {
-       guint8 *buf, *code;
-       static guint8* generic_jump_trampoline = NULL;
-       static guint8 *generic_class_init_trampoline = NULL;
-
-       switch (tramp_type) {
-       case MONO_TRAMPOLINE_GENERIC:
-               if (mono_generic_trampoline_code)
-                       return mono_generic_trampoline_code;
-               break;
-       case MONO_TRAMPOLINE_JUMP:
-               if (generic_jump_trampoline)
-                       return generic_jump_trampoline;
-               break;
-       case MONO_TRAMPOLINE_CLASS_INIT:
-               if (generic_class_init_trampoline)
-                       return generic_class_init_trampoline;
-               break;
+       guint32 reg;
+       guint32 disp;
+
+       if ((code [-3] == 0xff) && (x86_modrm_reg (code [-2]) == 0x2) && (x86_modrm_mod (code [-2]) == 0x1)) {
+               /* call *[reg+disp8] */
+               reg = x86_modrm_rm (code [-2]);
+               disp = *(guint8*)(code - 1);
+               //printf ("B: [%%r%d+0x%x]\n", reg, disp);
+       }
+       else {
+               int i;
+
+               for (i = -16; i < 0; ++i)
+                       printf ("%d ", code [i]);
+               printf ("\n");
+               g_assert_not_reached ();
        }
 
-       code = buf = g_malloc (256);
-       /* save caller save regs because we need to do a call */ 
+       *(gpointer*)(((guint32)(regs [reg])) + disp) = addr;
+}
+
+guchar*
+mono_arch_create_trampoline_code (MonoTrampolineType tramp_type)
+{
+       guint8 *buf, *code;
+       int pushed_args;
+
+       code = buf = mono_global_codeman_reserve (256);
+
+       /* Note that there is a single argument to the trampoline
+        * and it is stored at: esp + pushed_args * sizeof (gpointer)
+        * the ret address is at: esp + (pushed_args + 1) * sizeof (gpointer)
+        */
+       /* Put all registers into an array on the stack */
+       x86_push_reg (buf, X86_EDI);
+       x86_push_reg (buf, X86_ESI);
+       x86_push_reg (buf, X86_EBP);
+       x86_push_reg (buf, X86_ESP);
+       x86_push_reg (buf, X86_EBX);
        x86_push_reg (buf, X86_EDX);
-       x86_push_reg (buf, X86_EAX);
        x86_push_reg (buf, X86_ECX);
+       x86_push_reg (buf, X86_EAX);
+
+       pushed_args = 8;
 
        /* save LMF begin */
 
@@ -293,15 +189,30 @@ create_trampoline_code (MonoTrampolineType tramp_type)
        if (tramp_type == MONO_TRAMPOLINE_JUMP)
                x86_push_imm (buf, 0);
        else
-               x86_push_membase (buf, X86_ESP, 16);
+               x86_push_membase (buf, X86_ESP, (pushed_args + 1) * sizeof (gpointer));
+
+       pushed_args++;
 
-       x86_push_reg (buf, X86_EBX);
-       x86_push_reg (buf, X86_EDI);
-       x86_push_reg (buf, X86_ESI);
        x86_push_reg (buf, X86_EBP);
+       x86_push_reg (buf, X86_ESI);
+       x86_push_reg (buf, X86_EDI);
+       x86_push_reg (buf, X86_EBX);
+
+       pushed_args += 4;
 
        /* save method info */
-       x86_push_membase (buf, X86_ESP, 32);
+       x86_push_membase (buf, X86_ESP, pushed_args * sizeof (gpointer));
+
+       pushed_args++;
+
+       /* the stack is correctly aligned to 16 bytes because pushed_args is 14
+        * and there is the extra trampoline arg + the return ip pushed by call
+        * FIXME: Note that if an exception happens while some args are pushed
+        * on the stack, the stack will be misaligned.
+        */
+#ifdef __APPLE__
+       g_assert (pushed_args == 14);
+#endif
        /* get the address of lmf for the current thread */
        x86_call_code (buf, mono_get_lmf_addr);
        /* push lmf */
@@ -312,27 +223,61 @@ create_trampoline_code (MonoTrampolineType tramp_type)
        x86_mov_membase_reg (buf, X86_EAX, 0, X86_ESP, 4);
        /* save LFM end */
 
+       pushed_args += 2;
+
+       /* starting the call sequence */
+#ifdef __APPLE__
+       /* changing esp to keep the stack aligned */
+       x86_alu_reg_imm (buf, X86_SUB, X86_ESP, 8);
+       pushed_args += 2;
+#endif
+
+       /* FIXME: Push the trampoline address */
+       x86_push_imm (buf, 0);
+
+       pushed_args++;
+
        /* push the method info */
-       x86_push_membase (buf, X86_ESP, 44);
+       x86_push_membase (buf, X86_ESP, pushed_args * sizeof (gpointer));
+
+       pushed_args++;
+
        /* push the return address onto the stack */
        if (tramp_type == MONO_TRAMPOLINE_JUMP)
                x86_push_imm (buf, 0);
        else
-               x86_push_membase (buf, X86_ESP, 52);
+               x86_push_membase (buf, X86_ESP, (pushed_args + 1) * sizeof (gpointer));
+       pushed_args++;
+       /* push the address of the register array */
+       x86_lea_membase (buf, X86_EAX, X86_ESP, (pushed_args - 8) * sizeof (gpointer));
+       x86_push_reg (buf, X86_EAX);
 
-       /* save all register values */
-       x86_push_reg (buf, X86_EBX);
-       x86_push_reg (buf, X86_EDI);
-       x86_push_reg (buf, X86_ESI);
-       x86_push_membase (buf, X86_ESP, 64); /* EDX */
-       x86_push_membase (buf, X86_ESP, 64); /* ECX */
-       x86_push_membase (buf, X86_ESP, 64); /* EAX */
+       pushed_args++;
+
+#ifdef __APPLE__
+       /* check the stack is aligned after the ret ip is pushed */
+       /*x86_mov_reg_reg (buf, X86_EDX, X86_ESP, 4);
+       x86_alu_reg_imm (buf, X86_AND, X86_EDX, 15);
+       x86_alu_reg_imm (buf, X86_CMP, X86_EDX, 0);
+       x86_branch_disp (buf, X86_CC_Z, 3, FALSE);
+       x86_breakpoint (buf);*/
+#endif
 
        if (tramp_type == MONO_TRAMPOLINE_CLASS_INIT)
-               x86_call_code (buf, x86_class_init_trampoline);
+               x86_call_code (buf, mono_class_init_trampoline);
+       else if (tramp_type == MONO_TRAMPOLINE_AOT)
+               x86_call_code (buf, mono_aot_trampoline);
+       else if (tramp_type == MONO_TRAMPOLINE_DELEGATE)
+               x86_call_code (buf, mono_delegate_trampoline);
        else
-               x86_call_code (buf, x86_magic_trampoline);
-       x86_alu_reg_imm (buf, X86_ADD, X86_ESP, 8*4);
+               x86_call_code (buf, mono_magic_trampoline);
+
+#ifdef __APPLE__
+       /* account for the alignment above */
+       x86_alu_reg_imm (buf, X86_ADD, X86_ESP, 6*4);
+#else
+       x86_alu_reg_imm (buf, X86_ADD, X86_ESP, 4*4);
+#endif
 
        /* restore LMF start */
        /* ebx = previous_lmf */
@@ -344,15 +289,21 @@ create_trampoline_code (MonoTrampolineType tramp_type)
        /* discard method info */
        x86_pop_reg (buf, X86_ESI);
        /* restore caller saved regs */
-       x86_pop_reg (buf, X86_EBP);
-       x86_pop_reg (buf, X86_ESI);
-       x86_pop_reg (buf, X86_EDI);
        x86_pop_reg (buf, X86_EBX);
+       x86_pop_reg (buf, X86_EDI);
+       x86_pop_reg (buf, X86_ESI);
+       x86_pop_reg (buf, X86_EBP);
+
        /* discard save IP */
        x86_alu_reg_imm (buf, X86_ADD, X86_ESP, 4);             
        /* restore LMF end */
 
-       x86_alu_reg_imm (buf, X86_ADD, X86_ESP, 16);
+       /* Restore caller saved registers */
+       x86_mov_reg_membase (buf, X86_ECX, X86_ESP, 1 * 4, 4);
+       x86_mov_reg_membase (buf, X86_EDX, X86_ESP, 2 * 4, 4);
+
+       /* Pop saved reg array + method ptr */
+       x86_alu_reg_imm (buf, X86_ADD, X86_ESP, 9 * 4);
 
        if (tramp_type == MONO_TRAMPOLINE_CLASS_INIT)
                x86_ret (buf);
@@ -362,16 +313,10 @@ create_trampoline_code (MonoTrampolineType tramp_type)
 
        g_assert ((buf - code) <= 256);
 
-       switch (tramp_type) {
-       case MONO_TRAMPOLINE_GENERIC:
-               mono_generic_trampoline_code = code;
-               break;
-       case MONO_TRAMPOLINE_JUMP:
-               generic_jump_trampoline = code;
-               break;
-       case MONO_TRAMPOLINE_CLASS_INIT:
-               generic_class_init_trampoline = code;
-               break;
+       if (tramp_type == MONO_TRAMPOLINE_CLASS_INIT) {
+               /* Initialize the nullified class init trampoline used in the AOT case */
+               nullified_class_init_trampoline = buf = mono_global_codeman_reserve (16);
+               x86_ret (buf);
        }
 
        return code;
@@ -380,129 +325,50 @@ create_trampoline_code (MonoTrampolineType tramp_type)
 #define TRAMPOLINE_SIZE 10
 
 gpointer
-mono_arch_create_jump_trampoline (MonoMethod *method)
+mono_arch_create_specific_trampoline (gpointer arg1, MonoTrampolineType tramp_type, MonoDomain *domain, guint32 *code_len)
 {
        guint8 *code, *buf, *tramp;
-
-       if (method->iflags & METHOD_IMPL_ATTRIBUTE_SYNCHRONIZED)
-               return mono_arch_create_jump_trampoline (mono_marshal_get_synchronized_wrapper (method));
-
-       /* icalls use method->addr */
-       if ((method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) ||
-           (method->flags & METHOD_ATTRIBUTE_PINVOKE_IMPL)) {
-               MonoMethod *nm;
-               
-               if (!method->addr) {
-                       if (method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL)
-                               method->addr = mono_lookup_internal_call (method);
-                       if (method->flags & METHOD_ATTRIBUTE_PINVOKE_IMPL)
-                               mono_lookup_pinvoke_call (method, NULL, NULL);
-               }
-#ifdef MONO_USE_EXC_TABLES
-               if (mono_method_blittable (method)) {
-                       return method->addr;
-               } else {
-#endif
-                       nm = mono_marshal_get_native_wrapper (method);
-                       return mono_compile_method (nm);
-#ifdef MONO_USE_EXC_TABLES
-               }
-#endif
-       }
        
-       tramp = create_trampoline_code (MONO_TRAMPOLINE_JUMP);
+       tramp = mono_get_trampoline_code (tramp_type);
 
-       code = buf = g_malloc (TRAMPOLINE_SIZE);
-       x86_push_imm (buf, method);
-       x86_jump_code (buf, tramp);
-       g_assert ((buf - code) <= TRAMPOLINE_SIZE);
-
-       mono_jit_stats.method_trampolines++;
-
-       return code;
-
-}
-
-/**
- * mono_arch_create_jit_trampoline:
- * @method: pointer to the method info
- *
- * Creates a trampoline function for virtual methods. If the created
- * code is called it first starts JIT compilation of method,
- * and then calls the newly created method. I also replaces the
- * corresponding vtable entry (see x86_magic_trampoline).
- * 
- * Returns: a pointer to the newly created code 
- */
-gpointer
-mono_arch_create_jit_trampoline (MonoMethod *method)
-{
-       guint8 *code, *buf, *tramp;
-
-       /* previously created trampoline code */
-       if (method->info)
-               return method->info;
-
-       if (method->iflags & METHOD_IMPL_ATTRIBUTE_SYNCHRONIZED)
-               return mono_arch_create_jit_trampoline (mono_marshal_get_synchronized_wrapper (method));
-
-       tramp = create_trampoline_code (MONO_TRAMPOLINE_GENERIC);
+       mono_domain_lock (domain);
+       code = buf = mono_code_manager_reserve (domain->code_mp, TRAMPOLINE_SIZE);
+       mono_domain_unlock (domain);
 
-       code = buf = g_malloc (TRAMPOLINE_SIZE);
-       x86_push_imm (buf, method);
+       x86_push_imm (buf, arg1);
        x86_jump_code (buf, tramp);
        g_assert ((buf - code) <= TRAMPOLINE_SIZE);
 
-       /* store trampoline address */
-       method->info = code;
+       mono_arch_flush_icache (code, buf - code);
 
        mono_jit_stats.method_trampolines++;
 
+       if (code_len)
+               *code_len = buf - code;
+
        return code;
 }
 
-/**
- * mono_arch_create_class_init_trampoline:
- *  @vtable: the type to initialize
- *
- * Creates a trampoline function to run a type initializer. 
- * If the trampoline is called, it calls mono_runtime_class_init with the
- * given vtable, then patches the caller code so it does not get called any
- * more.
- * 
- * Returns: a pointer to the newly created code 
- */
-gpointer
-mono_arch_create_class_init_trampoline (MonoVTable *vtable)
+void
+mono_arch_invalidate_method (MonoJitInfo *ji, void *func, gpointer func_arg)
 {
-       guint8 *code, *buf, *tramp;
-
-       tramp = create_trampoline_code (MONO_TRAMPOLINE_CLASS_INIT);
-
-       code = buf = g_malloc (TRAMPOLINE_SIZE);
-       x86_push_imm (buf, vtable);
-       x86_jump_code (buf, tramp);
-       g_assert ((buf - code) <= TRAMPOLINE_SIZE);
+       /* FIXME: This is not thread safe */
+       guint8 *code = ji->code_start;
 
-       mono_jit_stats.method_trampolines++;
-
-       return code;
+       x86_push_imm (code, func_arg);
+       x86_call_code (code, (guint8*)func);
 }
 
 /*
  * This method is only called when running in the Mono Debugger.
  */
-gpointer
-mono_debugger_create_notification_function (gpointer *notification_address)
+guint8 *
+mono_debugger_create_notification_function ()
 {
-       guint8 *ptr, *buf;
+       guint8 *buf, *code;
 
-       ptr = buf = g_malloc0 (16);
+       code = buf = mono_global_codeman_reserve (2);
        x86_breakpoint (buf);
-       if (notification_address)
-               *notification_address = buf;
        x86_ret (buf);
-
-       return ptr;
+       return code;
 }
-