RFR 4947890 : Minimize JNI upcalls in system property initialization

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Mon Nov 19 12:29:29 UTC 2018


On 2018-11-16 21:36, Erik Joelsson wrote:
> Thanks, looks good to me now.
And to me.

Looks like a nice cleanup in general!

/Magnus

>
> /Erik
>
> On 2018-11-16 12:02, Roger Riggs wrote:
>> Hi Erik,
>>
>> Yes, that is removed.
>> Webrev updated in place.
>>
>> Thanks, Roger
>>
>> http://cr.openjdk.java.net/~rriggs/webrev-props-only-raw/
>>
>> On 11/16/2018 02:37 PM, Erik Joelsson wrote:
>>> Does this mean we can remove $(VERSION_CFLAGS) from System.c in 
>>> make/lib/CoreLibraries.gmk?
>>>
>>> Otherwise this looks good from a build point of view.
>>>
>>> /Erik
>>>
>>> On 2018-11-16 10:49, Roger Riggs wrote:
>>>> Updates to include the suggestions made by Mandy and Brent:
>>>>
>>>>  - Move the build-time properties from native code to the 
>>>> VersionProps.java.template
>>>>    including java.vendor.* and java.specification.* properties, 
>>>> plus the java.class.version (major.minor)
>>>>    This included two changes to the build that generates source of 
>>>> VersionProps.java.
>>>>
>>>>  - Updated the StaticProperty initialization to be explicitly 
>>>> invoked by initProperties.
>>>>
>>>>  - Makes separate calls to native to retrieve the platform 
>>>> properties and VM/command line properties
>>>>
>>>>  - (The hotspot function for JVM_GetProperties are unchanged)
>>>>
>>>> Webrev:
>>>> http://cr.openjdk.java.net/~rriggs/webrev-props-only-raw/
>>>>
>>>> Issue:
>>>> https://bugs.openjdk.java.net/browse/JDK-4947890
>>>>
>>>> Thanks, Roger
>>>>
>>>>
>>



More information about the core-libs-dev mailing list