RFR: 8234741: enhance os::get_core_path on macOS

David Holmes david.holmes at oracle.com
Tue Nov 26 09:32:51 UTC 2019


On 26/11/2019 6:36 pm, Baesken, Matthias wrote:
> OK I'll do a separate change .
> 
> Are you fine with the rest of the patch ?

Sorry not an area I'm familiar with. Hoping one of our mac fans can 
chime in on this one. I agree in principle but don't know the details.

Probably going to be a slow week for reviews due to US Thanksgiving 
holidays.

Cheers,
David

> Best regards, Matthias
> 
> 
> 
>>
>> On 25/11/2019 11:44 pm, Baesken, Matthias wrote:
>>> Hello,
>>>
>>> Currently the macOS implementation of os::get_core_path just displays
>> the default core file location.
>>> However it does not handle/show other locations set by the sysctl
>> parameter "kern.corefile" . This is enhanced by this change .
>>> I also take care of  handling %P  which is used a lot for  the  pid-placeholder
>> on macOS in the  "kern.corefile"  parameter .
>>>
>>>
>>> ( additionally  the change  contains a one-liner adjustment in
>> src/java.desktop/macosx/native/libawt_lwawt/awt/AWTView.h  to be able
>> to compile  again  on older macOs versions)
>>
>> That issue is being discussed elsewhere by the client folk and should
>> not be part of this change.
>>
>> David
>>
>>>
>>> Bug / webrev :
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8234741
>>>
>>> http://cr.openjdk.java.net/~mbaesken/webrevs/8234741.0/
>>>
>>> Thanks, Matthias
>>>


More information about the hotspot-dev mailing list