[Mono-dev] [PATCH] appdomain-unload crashes

[Rodrigo Kumpera]

Guys,

Attached is the final version of the patch.

It can timeout properly - thou the appdomain unload code doesn't bother
about it.


Cheers,
Rodrigo


On Mon, Nov 10, 2008 at 11:24 AM, Rodrigo Kumpera <kumpera at gmail.com> wrote:

> Yes, you're right, there is a race condition between pushing the the domain
> ref, decrementing the counter
> and checking if the domain needs to be unloaded.
>
>
>
> On Sun, Nov 9, 2008 at 6:19 PM, Zoltan Varga <vargaz at gmail.com> wrote:
>
>> Hi,
>>
>>  Looks good. Isn't there a race between the check:
>> +                       if (domain->state == MONO_APPDOMAIN_UNLOADED ||
>> domain->state ==
>> MONO_APPDOMAIN_UNLOADING) {
>> and the
>> +                               mono_thread_push_appdomain_ref (domain);
>> call ?
>>
>>               Zoltan
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ximian.com/pipermail/mono-devel-list/attachments/20081110/2e292888/attachment.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: take3.diff
Type: text/x-patch
Size: 11827 bytes
Desc: not available
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20081110/2e292888/attachment.bin