[jdk11u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent

[Andrew John Hughes]

On Wed, 3 Sep 2025 09:43:45 GMT, John Jiang <jjiang at openjdk.org> wrote:

> Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`.
> This backport is not clean, because:
> - Some files have different copyright notes between the repos
>   - test/hotspot/jtreg/compiler/debug/TraceIterativeGVN.java
>   - test/jdk/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java
>   - test/jdk/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java
> - Some files should not be modified due to there is not Tencent copyright notes
>   - src/hotspot/share/gc/shenandoah/c2/shenandoahSupport.cpp
>   - src/hotspot/share/gc/shenandoah/shenandoahControlThread.cpp
>   - test/hotspot/jtreg/compiler/c2/cr6865031/Test.java
>   - test/hotspot/jtreg/compiler/debug/TraceIterativeGVN.java
> - Some files doesn't exist in jdk11u
>   - src/hotspot/cpu/x86/stubGenerator_x86_64_exp.cpp
>   - src/hotspot/cpu/x86/stubGenerator_x86_64_log.cpp
>   - src/hotspot/cpu/x86/stubGenerator_x86_64_pow.cpp
>   - src/hotspot/share/gc/shenandoah/shenandoahGenerationalControlThread.cpp
>   - test/hotspot/jtreg/compiler/c1/TestRangeCheckEliminated.java
>   - test/hotspot/jtreg/compiler/c2/TestDuplicateSimpleLoopBackedge.java
>   - test/hotspot/jtreg/compiler/c2/irTests/TestAutoVectorization2DArray.java
>   - test/hotspot/jtreg/compiler/compilercontrol/TestConflictInlineCommands.java
>   - test/hotspot/jtreg/compiler/intrinsics/math/TestPow0Dot5Opt.java
>   - test/hotspot/jtreg/compiler/intrinsics/math/TestPow2Opt.java
>   - test/hotspot/jtreg/compiler/intrinsics/sha/cli/TestUseSHA3IntrinsicsOptionOnSupportedCPU.java
>   - test/hotspot/jtreg/compiler/intrinsics/sha/cli/TestUseSHA3IntrinsicsOptionOnUnsupportedCPU.java
>   - test/hotspot/jtreg/compiler/jvmci/errors/TestInvalidTieredStopAtLevel.java
>   - test/hotspot/jtreg/compiler/loopopts/TestLoopEndNodeEliminate.java
>   - test/hotspot/jtreg/compiler/oracle/TestInvalidCompileCommand.java
>   - test/hotspot/jtreg/compiler/print/TestTraceOptoParse.java
>   - test/hotspot/jtreg/compiler/vectorapi/TestIntrinsicBailOut.java
>   - test/hotspot/jtreg/compiler/vectorapi/TestVectorErgonomics.java
>   - test/hotspot/jtreg/compiler/vectorapi/VectorReinterpretTest.java
>   - test/hotspot/jtreg/gc/arguments/TestG1CompressedOops.java
>   - test/hotspot/jtreg/runtime/cds/appcds/FillerObjectLoadTest.java
>   - test/jdk/java/lang/Thread/virtual/ParkWithFixedThreadPool.java
>   - test/jdk/javax/net/ssl/DTLS/DTLSNamedGroups.java
>   - test/jdk/javax/net/ssl/DTLS/DTLSSignatureSchemes.java
>   - test/jdk/javax/net/ssl/SSLException/CheckSSLHandshakeException.java
>   - test/jdk/javax/net/ssl/SSL...

Rather than treat this as a backport, it seems to make more sense to ensure that all occurrences of 'THL A29' are removed from the code base for each release.

As such, along with sanity checking the patch itself, I compared the list of files covered by the patch with those in the repository containing 'THL A29':
~~~
$ grep -r 'THL A29' * 2>&1 | tee /tmp/thl.11u
$ diffstat -p0 /tmp/8364597.11u > /tmp/diffstat.11u
$ for file in $(cut -d : -f 1 /tmp/thl.11u ); do if ! grep -q ${file} /tmp/diffstat.11u ; then echo "${file} missing";  fi; done
~~~
With this, I confirmed that this change covers all cases (a much smaller number than 17u).

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

Marked as reviewed by andrew (Reviewer).

PR Review: https://git.openjdk.org/jdk11u-dev/pull/3086#pullrequestreview-3182896711