[jdk11u-dev] RFR: 8214972: Uses of klass_holder() except GC need to apply GC barriers

[Francesco Andreuzzi]

On Wed, 9 Jul 2025 18:09:29 GMT, Paul Hohensee <phh at openjdk.org> wrote:

>> I've been working on a backport to fix [JDK-8339725](https://bugs.openjdk.org/browse/JDK-8339725). The fix attached to the ticket is not enough to fix the problem on jdk11. 
>> 
>> The missing backport is JDK-8214972, which I apply in this PR. JDK-8210321 is also needed to provide `ClassLoaderData::holder_no_keepalive`.
>> 
>> Tier1 tests completed successfully.
>
> This looks ok, but why is it needed as a prereq for [JDK-8339725](https://bugs.openjdk.org/browse/JDK-8339725)?
> 
> Given that 11u is old, why do you want to backport it? Do you have production use issues? I recommend running both tier1 and tier2, with both fastdebug and release JDKs.

@phohensee I ran the following test suites:
- release
   + tier1: all test pass
   + tier2: all test pass except for:
      - `ConnectionReuseTest::testConnReuse`
      - `DirectIOTest`
- fastdebug
   + tier1: all test pass
   + tier2: all test pass except for:
      - `ConnectionReuseTest::testConnReuse`
      - `DirectIOTest`

The failures don't seem to be related to my changes, possibly flaky tests? I found tickets related to them (e.g. [JDK-8305900](https://bugs.openjdk.org/browse/JDK-8305900)) which haven't been backported to jdk11.

-------------

PR Comment: https://git.openjdk.org/jdk11u-dev/pull/3059#issuecomment-3054335533