[PATCH] 7052549 "(aio) AssertionError in sun.nio.ch.PendingIoCache.clearPendingIoMap (win)"

[Yuri Gaevsky]

Hi Alan, :-)

Sure, will do.

-Yuri

-----Original Message-----
From: Alan Bateman [mailto:Alan.Bateman at oracle.com] 
Sent: Friday, October 11, 2013 12:17 AM
To: Yuri Gaevsky
Cc: jdk7u-dev at openjdk.java.net
Subject: Re: [PATCH] 7052549 "(aio) AssertionError in sun.nio.ch.PendingIoCache.clearPendingIoMap (win)"

On 10/10/2013 20:56, Yuri Gaevsky wrote:
> Hello,
>
> I'm trying to follow [1] in order to propose a patch for JDK-7052549 
> [2] into the JDK 7 Updates and hopefully this is the correct mailing list for it.
>
> The code in clearPendingIoMap()  doesn't use 'synchronized' access to 
> internal data as all other methods do and the 'assert 
> Thread.holdsLock(this)' statement in the method can fail when "-esa" 
> command-line option enabled. The proposed patch moves all accesses to 
> 'pendingIoMap' and 'closePending' under synchronized block, and removes the 'assert !result.isDone()' statement.
>
> The patch has been verified by JCK testrun with '-esa' option.
>
Hi Yuri - looks like an issue is following you :-)

Can you send it to nio-dev to review? Changes need to be in the main line (currently jdk8) before being back ported.

-Alan