https://bugzilla.novell.com/show_bug.cgi?id=432388
User [EMAIL PROTECTED] added comment https://bugzilla.novell.com/show_bug.cgi?id=432388#c4 --- Comment #4 from Massimiliano Mantione <[EMAIL PROTECTED]> 2008-10-08 03:30:09 MDT --- I should check the ChangeLog, bit IIRC the "thread attach" was needed because it could happen that the shutdown happened inside a thread that was not attached. The function that printd the statistical profiler data needs to be called in a thread that has a "small ID", otherwise the profiler cannot get the JIT info for the IP saved in the buffer. The new profiler has proper code to deal with this (which has been nontrivial to set up), so my preferred "fix" would be to switch to the new profiler. Unfortunately I still did not build it on Windows, so this "fix" is not applicable :-( I would vote against this patch because it would fix this case and break another one. BTW, I checked the ChangeLog, and it turns out that the patch that introduced the call to "mono_thread_attach" did not come from me, so my comment above could be wrong. Given a test case or the previous problem I could try to adapt the new profiler's logic to the old one, or anyway to solve the problem properly. But simply applying this patch would likely cause a regression. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. _______________________________________________ mono-bugs maillist - mono-bugs@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-bugs