RFR: 8273967: gtest os.dll_address_to_function_and_library_name_vm fails on macOS12

Daniel D.Daugherty dcubed at openjdk.java.net
Wed Nov 3 19:30:12 UTC 2021


On Wed, 3 Nov 2021 19:08:38 GMT, Daniel D. Daugherty <dcubed at openjdk.org> 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


More information about the serviceability-dev mailing list