[9] RFR 8146653: Debug version missing in hs_err files and on internal version after Verona
Daniel D. Daugherty
daniel.daugherty at oracle.com
Sat Jan 16 00:07:38 UTC 2016
On 1/15/16 4:31 PM, Alejandro E Murillo wrote:
> Hi Dan, thanks a lot for the prompt review
I figured you wanted this done quickly since it is causing
nightly analysis confusion... :-)
> I applied all the changes, revised webrev (with proper links) is here:
>
> http://cr.openjdk.java.net/~amurillo/9/8146653/index.html
Forgot to remind about updating copyright years in the first round...
src/share/vm/runtime/vm_version.hpp
No comments.
src/share/vm/runtime/vm_version.cpp
No comments.
src/share/vm/utilities/vmError.cpp
No comments.
Thumbs up!
Dan
>
> Thanks
> Alejandro
>
> On 1/15/2016 3:09 PM, Daniel D. Daugherty wrote:
>> On 1/15/16 2:51 PM, Alejandro E Murillo wrote:
>>>
>>> Please review the following fix for JDK-8146653
>>> <https://bugs.openjdk.java.net/browse/JDK-8146653>:
>>> Debug version missing in hs_err files and on internal version after
>>> Verona
>>>
>>> Webrev: http://cr.openjdk.java.net/~amurillo/9/8146653/
>>
>> Your version of webrev generated broken nav links for
>> the "frames" version. Shows 404...
>>
>>
>> src/share/vm/runtime/vm_version.cpp
>> L242: return strcmp(DEBUG_LEVEL, "release") ?
>> L243: VMNAME " (" DEBUG_LEVEL " " INTERNAL_VERSION_SUFFIX
>> L244: :
>> L245: VMNAME " (" INTERNAL_VERSION_SUFFIX;
>>
>> No implied booleans on this part: strcmp(DEBUG_LEVEL, "release")
>>
>> Also I think this would be more HotSpot style:
>>
>> return strcmp(DEBUG_LEVEL, "release") == 0
>> ? VMNAME " (" INTERNAL_VERSION_SUFFIX
>> : VMNAME " (" DEBUG_LEVEL " " INTERNAL_VERSION_SUFFIX;
>>
>> where the "usual case" is first and the indenting is more
>> HotSpot like...
>>
>> L258: return strcmp(DEBUG_LEVEL, "release") ? DEBUG_LEVEL " " :
>> "";
>>
>> Similarly here:
>>
>> return strcmp(DEBUG_LEVEL, "release") == 0 ? "" : DEBUG_LEVEL;
>>
>> src/share/vm/runtime/vm_version.hpp
>> No comments.
>>
>> src/share/vm/utilities/vmError.cpp
>> L234: const char* runtime_version =
>> JDK_Version::runtime_version() != NULL ?
>> L235: JDK_Version::runtime_version() : "";
>> L236: const char* jdk_debug_level =
>> Abstract_VM_Version::alternate_jdk_debug_level() != NULL ?
>> L237: Abstract_VM_Version::alternate_jdk_debug_level() : "";
>>
>> L237 indent needs three more spaces. So does L235, but that's
>> not your issue.
>>
>> L239: st->print_cr("# JRE version: %s (%s) (%sbuild %s)",
>> runtime_name, buf,
>> L240: jdk_debug_level, runtime_version);
>>
>> L240 indent has one too many spaces.
>>
>> Dan
>>
>>>
>>> Background:
>>> Verona removed the debug info from version related system properties
>>> and that's now specified on a new system property called jdk.debug.
>>> The version header spitted in hs_err files was updated to include
>>> debug info based on that change.
>>> That info was also missing on the output for -Xinternalversion
>>>
>>> Thanks
>>>
>>
>
More information about the hotspot-dev
mailing list