JDK-8219650 : [TESTBUG] Fix crashes in new test hotspot gtest "test_print_hex_dump"
  • Type: Bug
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: 11,12,13
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2019-02-25
  • Updated: 2020-01-13
  • Resolved: 2019-03-06
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 11 JDK 12 JDK 13
11.0.3Fixed 12.0.2Fixed 13 b11Fixed
Related Reports
Relates :  
JDK-8217994 - os::print_hex_dump should be more resilient against unreadable memory
Description
The new test "test_print_hex_dump", introduced with JDK-8217994 has the issue, that it could cause access to an unaligned address. We've observed crashes on Solaris. Furthermore, the test should run as TEST_VM, because it uses VM features such as ResourceMark. This won't be a problem if the complete test suite is run but for standalone usage, it'll cause problems.

---

After discussing this, we thing that 
- test should use TEST_VM
- the alignment problems are real issues and should be fixed in os::print_hex_dump();

---

Update: for the record, we had the following issues:

- Test did not use TEST_VM so signal handlers could be missing depending on test order
- os::print_hex_dump() had alignment problems.
- On Windows, the gtest will not work since SEH frame is missing. This is a general problem for gtests and I opened a separate issue for this (JDK-8220220)
- On AIX, we can read from zero page (address 0) and therefore the test must use a different unreadable address
Comments
Thank you Christoph :)
08-03-2019
Addition to the Fix Request for jdk11: I would like to request a push into the jdk11u repository (11.0.3) since it fixes problems that we see in rampdown testing.
08-03-2019
Fix Request JDK-8217994 causes gtests to fail on various platforms for various reasons (Solaris, AIX, Windows32). Fix applies cleanly to both 11u and 12u, passes tier1 testing, and new gtest on all platforms.
06-03-2019