RFR: 8375477: CoreUtils support for SA tests should attempt to locate and unzip core files when they have been zipped [v2]

Chris Plummer cjplummer at openjdk.org
Mon Jan 26 19:37:39 UTC 2026


> The support for locating zipped (and subsequently unzipped) core files broke when a message in the hs_err file changed, causing CoreUtils.java to not detect the alternate location of the core file. It became broken 2 months ago by the changes for [JDK-8368551](https://bugs.openjdk.org/browse/JDK-8368551). That changed the hotspot message related to the core file location that CoreUtils was searching for:
> 
> 
>      if (core_pattern[0] == '|') {
>        written = jio_snprintf(buffer, bufferSize,
> - ""%s" (or dumping to %s/core.%d)",
> + ""%s" (alternatively, falling back to %s/core.%d)",
>                               &core_pattern[1], p, current_process_id());
> 
> 
> CoreUtils was checking for "or ", but it needs to check for "alternatively...".
> 
> Testing: I checked our CI runs, which used to skip all the SA core files tests on linux-x64 and linux-aarch64, and they are now running the tests and passing properly.

Chris Plummer has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:

 - Merge branch 'master' into 8375477_coreutils
   Merge
 - fix whitespace
 - get rid of unnecessary throws clause
 - fix zipped core file support

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/29284/files
  - new: https://git.openjdk.org/jdk/pull/29284/files/a79ef897..2ca1a223

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=29284&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29284&range=00-01

  Stats: 24318 lines in 663 files changed: 11478 ins; 4071 del; 8769 mod
  Patch: https://git.openjdk.org/jdk/pull/29284.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/29284/head:pull/29284

PR: https://git.openjdk.org/jdk/pull/29284


More information about the serviceability-dev mailing list