On 11/20/13 12:42 PM, Tristan Yan wrote:
Thanks Chris and Daniel
I change 'clientCtx' to final and error with volatile.

Unless I'm mistaken this should not compile because of line 60:

  44     static final SSLContext clientCtx = null;
  ...
60 clientCtx = new SimpleSSLContext(System.getProperty("test.src")).get();

best regards,

-- daniel


http://cr.openjdk.java.net/~ewang/tristan/JDK-7036666/webrev.00/
I'm appreciated Chris be my sponsor for this.
Thank you.
Tristan

-----邮件原件-----
发件人: Chris Hegarty
发送时间: Wednesday, November 20, 2013 6:49 PM
收件人: Daniel Fuchs; Tristan Yan; net-dev@openjdk.java.net
主题: Re: RFR for JDK-703666 test/com/sun/net/httpserver/Test9a.java fails 
intermittently

On 20/11/13 10:19, Daniel Fuchs wrote:
Hi Tristan,

I believe you should also declare the 'error' flag as well as the
'clientCtx' variable volatile since they are set by a thread and read
by another.

Good catch Daniel. Tristan, can you update the webrev and create a changeset. I 
can then sponsor this change for you.

Thanks,
-Chris.




best regards,

-- daniel

On 11/20/13 5:59 AM, Tristan Yan wrote:
/Hi All/

/I am working on https://bugs.openjdk.java.net/browse/JDK-7036666,
this bug was opened 3 years ago and test was put into ProblemList. I
ran a
1000 times loop to try to reproduce the failure but I got no luck.
Also I use same jdk binaries run all regression tests a couple of
hundred time I don’t see any failure on this test either./

/I suggest we bring this test back from ProblemList and add a debug
info for only possible failure place./

//

/http://cr.openjdk.java.net/~ewang/tristan/JDK-7036666/webrev.00//

//

/Please let me know if you have any suggestion or comments./

//

/Tristan Yan(Haibo Yan)/

//



Reply via email to