On Wed, 3 Nov 2021 19:08:38 GMT, Daniel D. Daugherty <[email protected]> wrote:
>> src/hotspot/os/bsd/os_bsd.cpp line 902:
>>
>>> 900: return false;
>>> 901: }
>>> 902: #endif
>>
>> We use dladdr() in several places in this code. I wonder whether it would
>> make sense to fix all of those with a wrapper instead:
>>
>> static int my_dladdr(const void* addr, Dl_info* info) {
>> if (addr != (void*)-1) {
>> return dladdr(addr, info);
>> } else {
>> // add comment here
>> return 0;
>> }
>> }
>> #define dladdr my_dladdr
>
> I'll take a look at the other calls to dladdr(). I'm trying to limit what I
> change
> here to things that actually failed in our test on macOS12 on X64 and aarch64.
I took a quick look at the other calls to `dladdr()` in
src/hotspot/os/bsd/os_bsd.cpp
and I'm not comfortable with changing those uses without having a specific test
case that I can use to investigate those code paths.
We are fairly early in our testing on macOS12 so I may run into a reason to
revisit
this choice down the road.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6193
