RFR: 8287001: Add warning message when fail to load hsdis libraries

Cesar Soares duke at openjdk.java.net
Wed Jun 1 00:49:38 UTC 2022


On Thu, 19 May 2022 06:37:28 GMT, Yuta Sato <duke at openjdk.java.net> wrote:

> When failing to load hsdis(Hot Spot Disassembler) library (because there is no library or hsdis.so is old and so on),
> there is no warning message (only can see info level messages if put -Xlog:os=info).
> This should show a warning message to tell the user that you failed to load libraries for hsdis.
> So I put a warning message to notify this.
> 
> e.g.
> `

FWIW it seems to me that `load_library` is indeed the best place to print the message.

src/hotspot/share/compiler/disassembler.cpp line 841:

> 839:                                           os::dll_lookup(_library, decode_instructions_virtual_name));
> 840:   } else {
> 841:     log_warning(os)("Try to load hsdis library failed");

NIT: I suggest adding the name of the file that was searched for and rewording this message a bit: "Failed to load hsdis library: hsdis-x86.so"

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

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


More information about the hotspot-compiler-dev mailing list