On Sat, 17 Oct 2026 03:32:24 GMT, Yasumasa Suenaga <[email protected]> wrote:

>> Alpine Linux would not provide `.eh_frame` enough to unwind in both regular 
>> binary and debuginfo. We need to refer `.debug_frame` in debuginfo for 
>> unwinding.
>> 
>> This ticket also re-enable mixed jstack jtreg tests on Alpine if musl-c 
>> debuginfo is available because `.debug_frame` would be provided by musl-dbg 
>> package.
>> This change passed all of serviceability/sa jtreg tests on following 
>> environment:
>> 
>> - Fedora 44 x86_64
>> - Alpine Linux 3.23 (with musl-dbg) x86_64
>> - Rocky Linux 9.5 aarch64
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Yasumasa Suenaga has updated the pull request incrementally with four 
> additional commits since the last revision:
> 
>  - Update test condition
>  - Close FD for debuginfo when read_frame() failed
>  - Update conditions
>  - Add warning message

src/jdk.hotspot.agent/linux/native/libsaproc/dwarf.cpp line 376:

> 374:     } else if (_buf + length > end) {
> 375:       // unexpected: DWARF is broken?
> 376:       print_warning("DWARF: hitted end of buffer even though DwarfParser 
> does not reach the pc (0x%lx).\n", pc);

Suggestion:

      print_warning("DWARF: reached end of buffer even though DwarfParser did 
not find the pc (0x%lx).\n", pc);

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33375#discussion_r4318022776

Reply via email to