On Sun, 11 Oct 2026 19:17:45 GMT, Vladimir Petko <[email protected]> wrote:
>> Implement suggestions in https://bugs.openjdk.org/browse/JDK-8387577. >> This PR uses "approximate" pc that matches the registration offset of oops >> map. >> >> >> Testing - tier1 and tier2 test run >> >> make test TEST=":tier1 :tier2" CONF=linux-s390x-server-fastdebug >> ============================== >> Test summary >> ============================== >> TEST TOTAL PASS FAIL ERROR >> SKIP >>>> jtreg:test/hotspot/jtreg:tier1 3887 3411 8 0 >>>> 468 << >> jtreg:test/jdk:tier1 2656 2605 0 0 >> 51 >> jtreg:test/langtools:tier1 4775 4763 0 0 >> 12 >> jtreg:test/jaxp:tier1 0 0 0 0 >> 0 >> jtreg:test/lib-test:tier1 41 41 0 0 >> 0 >>>> jtreg:test/hotspot/jtreg:tier2 1059 846 3 0 >>>> 210 << >>>> jtreg:test/jdk:tier2 4609 4351 2 0 >>>> 256 << >> jtreg:test/langtools:tier2 14 12 0 0 >> 2 >> jtreg:test/jaxp:tier2 524 523 0 0 >> 1 >> jtreg:test/docs:tier2 4 0 0 0 >> 4 >> ============================== >> TEST FAILURE >> >> Failed tests also fail on master branch for the same reason, >> test/hotspot/jtreg/runtime/ErrorHandling/CreateCoredumpOnCrash.java` passes >> after enabling `ulimit -c unlimited`. >> >> >> ============================== >> Test summary >> ============================== >> TEST TOTAL PASS FAIL ERROR >> SKIP >> >> jtreg:test/hotspot/jtreg/compiler/c2/igvn/TestOptimizeTrichotomy.java#Xcomp >>>> 1 0 1 0 >>>> 0 << >> >> jtreg:test/hotspot/jtreg/compiler/c2/igvn/TestOptimizeTrichotomy.java#vanilla >>>> 1 0 1 0 >>>> 0 << >> >> jtreg:test/hotspot/jtreg/compiler/escapeAnalysis/TestBCEscapeAnalyzerOverflow.java >>>> 1 0 1 0 >>>> 0 << >> jtreg:test/hotspot/jtreg/compiler/gcbarriers/TestZGCBarrierElision.java#Z >>>> 1 0 1 0 >>>> 0 << >> >> jtreg:test/hotspot/jtreg/compiler/loopopts/superword/TestDoNotFilterNaNSummands.java >>>> ... > > Vladimir Petko has updated the pull request incrementally with two additional > commits since the last revision: > > - pd_last_frame(): add comment on null pc > - Revert "pd_last_frame(): assign z_common_abi->return_pc for runtime stub" > > This reverts commit c1c83d267e1bd475c7ef7032284c3e3a9545bf99. src/hotspot/os_cpu/linux_s390/javaThread_linux_s390.cpp line 41: > 39: // They stored it in frame::z_common_abi return_pc. > 40: // frame reads it in frame::setup called from the frame > 41: // constructor. Suggestion: // pc == nullptr is a valid case for C2 runtime stubs // as they do not set last_Java_pc. // C2 stubs set their caller's SP as last_Java_sp and store their return pc // in the caller's frame::z_common_abi::return_pc. // where frame::setup() finds and reads it. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/33262#discussion_r4245211754
