[jdk17u-dev] Integrated: 8309613: [Windows] hs_err files sometimes miss information about the code containing the error

[Martin Doerr]

On Wed, 21 Jun 2023 14:04:47 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> Backport of [JDK-8309613](https://bugs.openjdk.org/browse/JDK-8309613). Requires manual integration, but it's pretty straightforward.
> 
> Manually tested on Windows like in the upstream PR (without hsdis provided):
> 
> ---------------  T H R E A D  ---------------
> 
> Current thread (0x000000c5550feee0):  JavaThread "main" [_thread_in_Java, id=628356, stack(0x000000c5547a0000,0x000000c5548a000
> 0)]
> 
> Stack: [0x000000c5547a0000,0x000000c5548a0000]
> Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, C=native code)
> V  [jvm.dll+0x64c869]  os::platform_print_native_stack+0xd9  (os_windows_x86.cpp:235)
> V  [jvm.dll+0x7f6467]  VMError::report+0xc67  (vmerror.cpp:852)
> V  [jvm.dll+0x7f7e1e]  VMError::report_and_die+0x79e  (vmerror.cpp:1710)
> V  [jvm.dll+0x236ce7]  report_fatal+0x97  (debug.cpp:302)
> V  [jvm.dll+0x586086]  MacroAssembler::debug64+0x96  (macroassembler_x86.cpp:865)
> C  0x000000c564791247
> 
> The last pc belongs to nmethod (printed below).
> 
> Compiled method (c2)     498   32       4       java.lang.Object::<init> (1 bytes)
>  total in heap  [0x000000c564791090,0x000000c5647912b0] = 544
>  relocation     [0x000000c5647911e8,0x000000c564791200] = 24
>  main code      [0x000000c564791200,0x000000c564791260] = 96
>  stub code      [0x000000c564791260,0x000000c564791278] = 24
>  metadata       [0x000000c564791278,0x000000c564791280] = 8
>  scopes data    [0x000000c564791280,0x000000c564791288] = 8
>  scopes pcs     [0x000000c564791288,0x000000c5647912a8] = 32
>  dependencies   [0x000000c5647912a8,0x000000c5647912b0] = 8
> 
> [Constant Pool (empty)]
> 
> [MachCode]
> [Entry Point]
>   # {method} {0x0000000800443ff8} '<init>' '()V' in 'java/lang/Object'
>   #           [sp+0x20]  (sp of caller)
>   0x000000c564791200: 448b 5208 | 49bb 0000 | 0000 0800 | 0000 4d03 | d349 3bc2 
> 
>   0x000000c564791214: ;   {runtime_call ic_miss_stub}
>   0x000000c564791214: 0f85 66fe | 8fff 6690 | 0f1f 4000 
> [Verified Entry Point]
>   0x000000c564791220: 4881 ec18 | 0000 0048 | 896c 2410 
> 
>   0x000000c56479122c: ;   {external_word}
>   0x000000c56479122c: 48b9 1806 | b598 fa7f | 0000 4883 
> 
>   0x000000c564791238: ;   {runtime_call MacroAssembler::debug64}
>   0x000000c564791238: e4f0 49ba | f05f 7598 | fa7f 0000 | 41ff d2f4 | f4f4 f4f4 | f4f4 f4f4 | f4f4 f4f4 | f4f4 f4f4 
>   0x000000c564791258: f4f4 f4f4 | f4f4 f4f4 
> [Exception Handler]
>   0x000000c564791260: ;   {no_reloc}
>   0x000000c564791260: e99b 5d91 | ffe8 0000 | 0000 4883 
> 
>   0x000000c56479126c: ;   {runtime_call DeoptimizationBlob}
>   0x000000c56479126c: 2c24 05e9 | 2c5a...

This pull request has now been integrated.

Changeset: 8dae82c1
Author:    Martin Doerr <mdoerr at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/8dae82c19648077b120772cde69c27ed6a067b2a
Stats:     51 lines in 7 files changed: 41 ins; 0 del; 10 mod

8309613: [Windows] hs_err files sometimes miss information about the code containing the error

Reviewed-by: phh
Backport-of: bd79db3930f192f6742e29a63a6d1c3bc3dd3385

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

PR: https://git.openjdk.org/jdk17u-dev/pull/1482