Open pshipton opened 4 years ago
@hangshao0 I think the test needs to be updated for Java 11 to handle the location difference between regular and groupAccess.
I have an impression @LongyuZhang has fixed something similar (cache location issue) on z/OS recently. (I guess it was on Java 8 ? I cannot recall the details).
Discussed with @hangshao0 about this failure. Previous fix passed OpenJ9 11, for this issue about IBM 11, will add extra condition for the cache location. Related to Runtimes/backlog/issue/306
The build been tested in job/Test_openjdk11_ibm_extended.functional_s390x_zos_Personal_testList_0/
are built from
job/Build_JDK11_s390x_zos_Nightly
Looking at the latest build: job/Build_JDK11_s390x_zos_Nightly/488/consoleFull
, I see -DOPENJ9_BUILD
is incorrectly set:
04:47:43 cxx -DOPENJ9_BUILD -Wc,debug -Wc,"langlvl(extended)" -Wc,lp64 -+ -DIPv6_FUNCTION_SUPPORT -DJ9ZOS390 -DLONGLONG -DJ9VM_TIERED_CODE_CACHE -D_ALL_SOURCE -D_XOPEN_SOURCE_EXTENDED -DIBM_ATOE -D_POSIX_SOURCE -D_ISOC99_SOURCE -D__STDC_LIMIT_MACROS -I../omr/util/a2e/headers -O3 -Wc,"ARCH(10)" -Wc,"TUNE(10)" -Wl,compat=ZOSV2R3 -Wc,"convlit(ISO8859-1),xplink,rostring,FLOAT(IEEE,FOLD,AFP),enum(4)" -Wa,goff -Wc,NOANSIALIAS -Wc,"inline(auto,noreport,600,5000)" -Wc,"SERVICE(j20200810)" -Wc,"TARGET(zOSV2R3)" -Wc,list,offset -DJ9ZOS39064 -Wc,lp64 -Wa,"SYSPARM(BIT64)" -I. -I../include -I../oti -I../shared_util -I../verbose -I../shared_common/include -I../shared -I../nls -I../omr/include_core -DTR_HOST_S390 -c shrinit.cpp > shrinit.asmlist
The-DOPENJ9_BUILD
should not be set for an IBM build. The default cache location is affected by -DOPENJ9_BUILD
.
The build been tested in
job/Test_openjdk11_ibm_extended.functional_s390x_zos_Personal_testList_0/
are built fromjob/Build_JDK11_s390x_zos_Nightly
Looking at the latest build:
job/Build_JDK11_s390x_zos_Nightly/488/consoleFull
, I see-DOPENJ9_BUILD
is incorrectly set:
04:47:43 cxx -DOPENJ9_BUILD -Wc,debug -Wc,"langlvl(extended)" -Wc,lp64 -+ -DIPv6_FUNCTION_SUPPORT -DJ9ZOS390 -DLONGLONG -DJ9VM_TIERED_CODE_CACHE -D_ALL_SOURCE -D_XOPEN_SOURCE_EXTENDED -DIBM_ATOE -D_POSIX_SOURCE -D_ISOC99_SOURCE -D__STDC_LIMIT_MACROS -I../omr/util/a2e/headers -O3 -Wc,"ARCH(10)" -Wc,"TUNE(10)" -Wl,compat=ZOSV2R3 -Wc,"convlit(ISO8859-1),xplink,rostring,FLOAT(IEEE,FOLD,AFP),enum(4)" -Wa,goff -Wc,NOANSIALIAS -Wc,"inline(auto,noreport,600,5000)" -Wc,"SERVICE(j20200810)" -Wc,"TARGET(zOSV2R3)" -Wc,list,offset -DJ9ZOS39064 -Wc,lp64 -Wa,"SYSPARM(BIT64)" -I. -I../include -I../oti -I../shared_util -I../verbose -I../shared_common/include -I../shared -I../nls -I../omr/include_core -DTR_HOST_S390 -c shrinit.cpp > shrinit.asmlist
The
-DOPENJ9_BUILD
should not be set for an IBM build. The default cache location is affected by-DOPENJ9_BUILD
.
Opened Issue of above mentioned -DOPENJ9_BUILD
setting on Internal GitHub runtimes/openj9-openjdk-jdk11-zos/issues/360. Thanks.
@hangshao0 @LongyuZhang -DOPENJ9_BUILD
needs to be set for all Java 11+ builds, even IBM variants. We need to find another way to fix the test. Something like isOpen() && isZos()
should work.
@LongyuZhang, you may want to modify your change https://github.com/eclipse/openj9/pull/10368 according to the comment above.
@hangshao0 @LongyuZhang
-DOPENJ9_BUILD
needs to be set for all Java 11+ builds, even IBM variants. We need to find another way to fix the test. Something likeisOpen() && isZos()
should work.
Hi @pshipton, if all Java 11+ all use -DOPENJ9_BUILD
, does it mean ibm 11+ use this setting on all platforms, not only on zos? So far Hang and I tried to change is openj9
check to is not ibm 8
check, and it passed ibm zos 11. Thanks.
if all Java 11+ all use -DOPENJ9_BUILD, does it mean ibm 11+ use this setting on all platforms
I think the answer is yes.
Correct, -DOPENJ9_BUILD should be set for all builds which aren't IBM Java 8. It's enabled via the extensions repos, and only IBM Java 8 doesn't use the OpenJDK extensions.
I don't think this is intermittent, but the assert at OSCachesysv.cpp:1213 doesn't always occur. I think the assert is a separate issue.
11.0.8 stream job/Test_openjdk11_ibm_extended.functional_s390x_zos_Personal_testList_0/17 testSCCacheManagement_0 fyrec520
head stream: job/Test_openjdk11_ibm_extended.functional_s390x_zos_Personal_testList_0/18/ fyrec520