RFR: 8242181: [Linux] Show source information when printing native stack traces in hs_err files [v4]

Thomas Stuefe stuefe at openjdk.java.net
Mon Mar 28 13:19:23 UTC 2022


On Thu, 24 Feb 2022 08:14:34 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Christian Hagedorn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Make dwarf tag NOT_PRODUCT
>
> src/hotspot/share/utilities/decoder_elf.cpp line 67:
> 
>> 65:   if (!os::dll_address_to_library_name(pc, filepath, sizeof(filepath), &offset_in_library) || offset_in_library < 0) {
>> 66:     // Method not found. offset_in_library should not overflow.
>> 67:     log_develop_info(dwarf)("Did not find library for address " INTPTR_FORMAT, p2i(pc));
> 
> I know this has been discussed and decided, but I feel uncomfortable about this logging here. Also because it sets a precedent for using UL inside signal handling.

Note, if you do log, it would be nice to be precise and distinguish between dll_address_to_library_name returning false and returning an offset outside the library bounds.

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

PR: https://git.openjdk.java.net/jdk/pull/7126



More information about the build-dev mailing list