Hi Alan, :-) Sure, will do.
-Yuri -----Original Message----- From: Alan Bateman [mailto:[email protected]] Sent: Friday, October 11, 2013 12:17 AM To: Yuri Gaevsky Cc: [email protected] 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
