[9] RFR 8146653: Debug version missing in hs_err files and on internal version after Verona

[Alejandro E Murillo]

On 1/15/2016 5:07 PM, Daniel D. Daugherty wrote:
> 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'll get this in asap.
>
>> 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...
right. Will fix those
>
> 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!
Thanks
Alejandro
>
> 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
>>>>
>>>
>>
>

-- 
Alejandro