Open smlambert opened 1 year ago
So hb_subset_context_t is a Harfbuzz symbol: https://github.com/harfbuzz/harfbuzz/blob/60db142784bb145426e4a02635bbad4f2f260d16/src/hb-subset.hh#L40 and I am suspecting the change in Harfbuzz version 7.0.1 might be the culprit: https://github.com/openjdk/jdk17u-dev/commit/f969f811a71603f39e4ab42f1362271b7a4d303d i'm trying to figure why we don't have the right Harfbuzz dynamic library?
As a wild guess, i'm going to try re-building, without this optimization: https://github.com/adoptium/jdk17u/blob/3197a9b02992c9f95154b4b5e2b420e82f420cf2/make/modules/java.desktop/lib/Awt2dLibraries.gmk#L494 As the failing symbol is in that source.
See harfbuzz issue: https://github.com/harfbuzz/harfbuzz/issues/4138
@ShelleyLambert @tellison The harfbuzz issue https://github.com/harfbuzz/harfbuzz/issues/4138 is still open, with no resolution yet. Openj9 have reverted Harfbuzz 7.0.1 to 4.4.1, https://github.com/eclipse-openj9/openj9/issues/17759#issuecomment-1644562760 We need to make the decision on whether to do the same?
Note, the revert is several hundred files...
I'm going to have a go at reverting this change: https://github.com/adoptium/jdk17u/blob/3197a9b02992c9f95154b4b5e2b420e82f420cf2/src/java.desktop/share/native/libharfbuzz/hb-subset.cc#L47 which seems a bit too much, and will see if the original problem can be work'd around some other way
The root compilation failure in https://github.com/adoptium/jdk17u/blob/3197a9b02992c9f95154b4b5e2b420e82f420cf2/src/java.desktop/share/native/libharfbuzz/hb-ot-post-table-v2subset.hh#L81 is described in this comment: https://github.com/harfbuzz/harfbuzz/issues/4138#issuecomment-1472854889 Which fails specifically with Xlc 16 on AIX. This needs focus from Harfbuzz upstream, who require access to an AIX box to debug..
Raised OpenJDK bug for this issue: https://bugs.openjdk.org/browse/JDK-8313240
Build with new Harfbuzz patch completes successfully: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-aix-ppc64-temurin/276/
Running jdk_beans_1 group to confirm: https://ci.adoptium.net/job/Grinder/7411/
jdk_beans_1 now passes: https://ci.adoptium.net/job/Grinder/7411/ So fix looks good.
So the patch applied to harfbuzz 7.0.1 does not compile with jdk-17.0.9+9 based on Harfbuzz 7.2.0.
I am trying a Harfbuzz 8.2.2 base on jdk-17.0.9+9, to verify a complete upgrade at least builds: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-aix-ppc64-temurin/321/
FYI I believe these are the changes OpenJ9 used to fix Harfbuzz 7.20 for AIX.
https://github.com/ibmruntimes/openj9-openjdk-jdk17/commit/55e75c010 https://github.com/ibmruntimes/openj9-openjdk-jdk17/commit/66679e2dc
FYI I believe these are the changes OpenJ9 used to fix Harfbuzz 7.20 for AIX.
ibmruntimes/openj9-openjdk-jdk17@55e75c010 ibmruntimes/openj9-openjdk-jdk17@66679e2dc
Thank you @pshipton that 2nd commit is probably the one I am missing, i'll give that a try, thanks
test build with additional commit patch: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-aix-ppc64-temurin/323/
So the patch applied to harfbuzz 7.0.1 does not compile with jdk-17.0.9+9 based on Harfbuzz 7.2.0.
I am trying a Harfbuzz 8.2.2 base on jdk-17.0.9+9, to verify a complete upgrade at least builds: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-aix-ppc64-temurin/321/
Upgrading harfbuzz is not straightfoward, as it's not a straight copy....
test build with additional commit patch: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-aix-ppc64-temurin/323/
Successful build, so patch looks good from compilation perspective.
Plan to exclude testcase until the upstream fix lands for April CPU, check via https://ci.adoptium.net/job/Grinder/9026 what failures relate to this issue.
Appears the testcase is already excluded as Grinder/9026 is green. So will return to this ahead of April, to see what can be re-enabled, moving to a later iteration.
jdk_beans testcases fail on JDK11 AIX with java.lang.UnsatisfiedLinkError rtld: 0712-001 Symbol _ZNK2OT10postV2Tail6subsetEP19hb_subset_context_t was referenced from module /home/jenkins/workspace/Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_1/openjdkbinary/j2sdk-image/lib/libfontmanager.so(), but a runtime definition of the symbol wa
Test Info Test Name: jdk_beans_1 Test Duration: 11 min 45 sec Machine: test-osuosl-aix72-ppc64-6 TRSS link for the test output: https://trss.adoptium.net/output/test?id=64b8535d17052c6715871023
Build Info Build Name: Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_1 Jenkins Build start time: Jul 19 2023, 07:15 am Jenkins Build URL: https://ci.adoptium.net/job/Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_1/78/ TRSS link for the build: https://trss.adoptium.net/allTestsInfo?buildId=64b8527817052c6715870b1d
Java Version openjdk version "11.0.20" 2023-07-18 OpenJDK Runtime Environment Temurin-11.0.20+8 (build 11.0.20+8) OpenJDK 64-Bit Server VM Temurin-11.0.20+8 (build 11.0.20+8, mixed mode)
This test has been failed 3 times since Jul 01 2023, 02:20 pm Java Version when the issue first seen openjdk version "11.0.20-beta" 2023-07-18 OpenJDK Runtime Environment Temurin-11.0.20+7-202307011712 (build 11.0.20-beta+7-202307011712) OpenJDK 64-Bit Server VM Temurin-11.0.20+7-202307011712 (build 11.0.20-beta+7-202307011712, mixed mode) Jenkins Build URL: https://ci.adoptium.net/job/Test_openjdk11_hs_extended.openjdk_ppc64_aix_testList_1/76/
The test failed on machine test-osuosl-aix72-ppc64-6 2 times The test failed on machine test-osuosl-aix72-ppc64-5 1 times
Rerun in Grinder