Hi Alan, I wanted to follow up on the update I shared earlier I’ve successfully backported the preClose re-order changes to JDK 17 and ran the Race.java test case 500 times. It passed consistently without any failures. For validation, I also ran the test without the patch, and it failed on the first iteration. I ran the tests under java/nio to see if there is any regression and all the testcases are passing. Currently, we are running the tests under java/net.
* Do you recommend running any additional test groups? * Also, can we proceed with porting the preClose re-order fix to headstream? Please let me know your thoughts. Thanks Shruthi ________________________________ From: Shruthi . <shruthi.shrut...@ibm.com> Sent: Friday, May 23, 2025 6:27 PM To: Alan Bateman <alan.bate...@oracle.com>; net-dev@openjdk.org <net-dev@openjdk.org> Cc: Syed Moinudeen <smoin...@in.ibm.com>; SREENIVAS MAKINEEDI <sre...@us.ibm.com> Subject: Re: [EXTERNAL] Re: Suggestion needed to port the fix to JDK17 and JDK11S Hi Alan, I was able to backport the preClose re-order changes to JDK17 and I executed the Race.java testcase 500 times, and it passed consistently without any failures. I also tested Race.java without the patch and it fails in the first iteration it self. We are running the tests under java/net and java/nio to check if there is any regression. Do you think I should run any other tests? Also, can we proceed with porting preClose re-order fix to headstream? Thanks Shruthi ________________________________ From: Shruthi . <shruthi.shrut...@ibm.com> Sent: Tuesday, April 8, 2025 6:03 PM To: Alan Bateman <alan.bate...@oracle.com>; net-dev@openjdk.org <net-dev@openjdk.org> Cc: Syed Moinudeen <smoin...@in.ibm.com>; SREENIVAS MAKINEEDI <sre...@us.ibm.com> Subject: Re: [EXTERNAL] Re: Suggestion needed to port the fix to JDK17 and JDK11S Hi Alan, I was able to successfully build the OpenJDK headstream with the preClose reordering patch, which you have shared: pending_signals.patch<https://bugs.openjdk.org/secure/attachment/113740/pending_signals.patch>. To validate the build, I executed the Race.java testcase 500 times, and it passed consistently without any failures. For comparison, I ran the same testcase without the patch, and it failed on the 86th iteration. Thanks Shruthi ________________________________ From: Alan Bateman <alan.bate...@oracle.com> Sent: Friday, March 14, 2025 3:00 PM To: Shruthi . <shruthi.shrut...@ibm.com>; net-dev@openjdk.org <net-dev@openjdk.org> Cc: Syed Moinudeen <smoin...@in.ibm.com>; SREENIVAS MAKINEEDI <sre...@us.ibm.com> Subject: [EXTERNAL] Re: Suggestion needed to port the fix to JDK17 and JDK11S On 14/03/2025 09: 22, Shruthi . wrote: The issue which we are seeing is not reproducible in JDK21 or higher version Okay, so you might be chasing a different issue? For the JDK then maybe we need more stress tests for async close, maybe that On 14/03/2025 09:22, Shruthi . wrote: The issue which we are seeing is not reproducible in JDK21 or higher version Okay, so you might be chasing a different issue? For the JDK then maybe we need more stress tests for async close, maybe that would help you reproduce it with main line. -Alan
