Hi Kornel,<br><br>Your patch have a few issues. First, it does a lot of different stuff in the same patch, second no Changelog entries are provided.<br><br>My suggestion is that yo split it in the following pieces and add Changelog entries so they can properly reviewed:<br>
<br>-The change to mono_gc_init (). This affects all platforms and require more reviewer effort that the rest.<br><br>-SEH for reading mapped images. Can this change break cygwin/mingw builds?<br><br>-coree/vtable fixups stuff. This one requires less reviewing as your&#39;re the author of it and it doesn&#39;t affect other targets.<br>
<br>Thanks,<br>Rodrigo<br><br><br><div class="gmail_quote">2009/5/5 Kornél Pál <span dir="ltr">&lt;<a href="mailto:kornelpal@gmail.com">kornelpal@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi,<br>
<br>
The attached patch implements using trampolines for vtable fixups that delay assembly loading. I believe that this is the correct solution.<br>
<br>
This patch also modifies:<br>
<br>
1) EXE image is only fixed up when using driver.c; embedded mono.dll will not improperly tamper the image used for version initialization.<br>
<br>
2) Use SEH with MS VC++ for reading mapped image. (GCC has no support for that:( )<br>
<br>
3) Disallow unloading mono.dll after mscoree.dll was fixed up to prevent calling unmapped functions.<br>
<br>
4) Remove WaitForSingleObjectEx in mono_gc_init () by modifying mono_thread_create_internal to return the thread object.<br>
<br>
This latter also affects other platform. I would like to ask you to test it. I wasn&#39;t able to reproduce any deadlock related to this. (I only found a deadlock with socket accept on Windows that was discussed earlier on the list.) If you can reproduce a deadlock related to finalizer thread I am willing to help solving that but I need the exact locaions of deadlocked thread stack traces.<br>
<font color="#888888">
<br>
Kornél<br>
</font><br>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div><br>