Suggestion needed to port the fix to JDK17 and JDK11S

[Shruthi .]

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.Shruthi1 at ibm.com>
Sent: Friday, May 23, 2025 6:27 PM
To: Alan Bateman <alan.bateman at oracle.com>; net-dev at openjdk.org <net-dev at openjdk.org>
Cc: Syed Moinudeen <smoinud1 at in.ibm.com>; SREENIVAS MAKINEEDI <sreeni at 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.Shruthi1 at ibm.com>
Sent: Tuesday, April 8, 2025 6:03 PM
To: Alan Bateman <alan.bateman at oracle.com>; net-dev at openjdk.org <net-dev at openjdk.org>
Cc: Syed Moinudeen <smoinud1 at in.ibm.com>; SREENIVAS MAKINEEDI <sreeni at 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.bateman at oracle.com>
Sent: Friday, March 14, 2025 3:00 PM
To: Shruthi . <Shruthi.Shruthi1 at ibm.com>; net-dev at openjdk.org <net-dev at openjdk.org>
Cc: Syed Moinudeen <smoinud1 at in.ibm.com>; SREENIVAS MAKINEEDI <sreeni at 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/net-dev/attachments/20250531/0cd8d4ef/attachment-0001.htm>