[rfc][icedtea-web] RH976833 ClassLoader deadlock

[Andrew Azores]

On 10/10/2013 10:35 AM, Jiri Vanek wrote:
> On 10/09/2013 03:50 PM, Andrew Azores wrote:
>> On 10/08/2013 04:42 PM, Andrew Azores wrote:
>>> On 10/04/2013 05:16 PM, Andrew Azores wrote:
>>>> (snip)
>>>>
>>>> If none of us are able to identify any issues with doing this 
>>>> simple lock replacement trick then
>>>> I'll make another push-candidate patch later.
>>>>
>>>> Thanks,
>>>>
>>>
>>> Seems nobody else has identified anything and neither have I, so the 
>>> patch is attached.
>>>
>>> Thanks,
>>>
>>
>> Oops, I forgot to attach the reproducer patch.
>>
>> It's a custom reproducer because the deadlock only seems to arise 
>> when the applet is loaded from
>> plain .class files, not in a JAR. The actual content of the two 
>> reproducers isn't really important -
>> I just decided to make them look more or less the same as the ones in 
>> the original bug report.
>>
>> Thanks,
>>
>
> I think more source codes are missing (only testcase here)
>
> Anyway - I strongly discourage you from close on Both.
>
> J.

The previous message in the thread contained the fix itself. I've 
attached it again here along with the reproducers, now using 
close_on_correct_end rather than close_on_both, and printing the magic 
string :)

Thanks,

-- 
Andrew A

-------------- next part --------------
A non-text attachment was scrubbed...
Name: classloader-deadlock.patch
Type: text/x-patch
Size: 8012 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20131010/d34c7e37/classloader-deadlock-0001.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: reproducer.patch
Type: text/x-patch
Size: 9411 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20131010/d34c7e37/reproducer-0001.patch