[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-gc-dev
Subject:    Re: RFR: 8202364: Add GCConfig::hs_err_name() to avoid GC-specific code in error reporting
From:       Per Liden <per.liden () oracle ! com>
Date:       2018-04-27 10:05:04
Message-ID: 0abff4f8-2293-5055-d6d7-29343edb2bc7 () oracle ! com
[Download RAW message or body]

On 04/27/2018 11:59 AM, Aleksey Shipilev wrote:
> On 04/27/2018 11:58 AM, Per Liden wrote:
>> On 04/27/2018 11:50 AM, Aleksey Shipilev wrote:
>>> On 04/27/2018 11:48 AM, Per Liden wrote:
>>>> On 04/27/2018 11:40 AM, Aleksey Shipilev wrote:
>>>>> On 04/27/2018 11:39 AM, Aleksey Shipilev wrote:
>>>>>> On 04/27/2018 11:33 AM, Per Liden wrote:
>>>>>>> The gc_mode() function in vmErrror.cpp currently tests which Use*GC flag is set and translates
>>>>>>> that
>>>>>>> into a name. We should instead call into GCConfig to get the name. This patch adds a new function
>>>>>>> (GCConfig::hs_err_name) for this purpose. This is not placed on CollectedHeap, since we might
>>>>>>> need
>>>>>>> this name before a CollectedHeap instance has been created, e.g. if we crash during VM startup.
>>>>>>>
>>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8202364
>>>>>>> Webrev: http://cr.openjdk.java.net/~pliden/8202364/webrev.0
>>>>>>
>>>>>> Looks good.
>>>>>
>>>>> Wait, except you want to keep the human-readable?
>>>>
>>>> I though it looked ugly with the space so I removed it. I don't think it makes it less readable.
>>>
>>> The line where GC name is injected already have spaces:
>>>
>>> Java VM: OpenJDK Server VM (slowdebug 10-internal+0-adhoc.user.shenandoah-jdk10, mixed mode, tiered,
>>> g1 gc, linux-x86)
>>>
>>> I prefer not to change formatting unless there is very compelling reason to do so.
>>
>> Sure, keeping the original names.
>>
>> http://cr.openjdk.java.net/~pliden/8202364/webrev.1
> 
> Excellent, go!
> 

Thanks for reviewing!

/Per
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic