RFR: 8041626: [Event Request] Shutdown reason

David Holmes david.holmes at oracle.com
Fri Feb 16 02:44:13 UTC 2018


Looks fine to me.

Thanks,
David
-----

On 15/02/2018 11:35 PM, Robin Westberg wrote:
> Hi again,
> 
> Here’s the (hopefully) final version:
> 
> Full: http://cr.openjdk.java.net/~rwestberg/8041626/webrev.02/ <http://cr.openjdk.java.net/~rwestberg/8041626/webrev.02/>
> Incremental: http://cr.openjdk.java.net/~rwestberg/8041626/webrev.01-02/ <http://cr.openjdk.java.net/~rwestberg/8041626/webrev.01-02/>
> 
> Best regards,
> Robin
> 
>> On 14 Feb 2018, at 16:15, Robin Westberg <robin.westberg at oracle.com> wrote:
>>
>> Hi Roger,
>>
>>> On 13 Feb 2018, at 16:17, Roger Riggs <Roger.Riggs at Oracle.com> wrote:
>>>
>>> Hi Robin,
>>>
>>> It looks like the status argument to BeforeHalt is discarded in JVM_BeforeHalt
>>> and is not inserted into the event.
>>> That suggests it should be removed all the way back to Shutdown.beforeHalt.
>>
>> You are right, my thinking was that the interface wouldn’t need to be changed if we decided to revisit the event in the future and add the status code. But that is perhaps unlikely, so can certainly remove the argument for now.
>>
>> Best regards,
>> Robin
>>
>>>
>>> Roger
>>>
>>>
>>>
>>> On 2/13/2018 9:59 AM, Robin Westberg wrote:
>>>> Hi Alan,
>>>>
>>>>> On 12 Feb 2018, at 09:02, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>>>>>
>>>>>
>>>>>
>>>>> On 12/02/2018 07:07, David Holmes wrote:
>>>>>>> Okay, I’ve removed the code related to the status field, certainly makes the change a bit less intrusive.
>>>>>>>
>>>>>>> Updated webrev: http://cr.openjdk.java.net/~rwestberg/8041626/webrev.01/
>>>>>>> Incremental: http://cr.openjdk.java.net/~rwestberg/8041626/webrev.00-01/
>>>>>> This looks much cleaner/neater to me - thanks.
>>>>>>
>>>>> The updates to Runtime/Shutdown seems okay.
>>>> Thanks for reviewing!
>>>>
>>>> Best regards,
>>>> Robin
>>>>
>>>>> -Alan
>>>
>>
> 


More information about the hotspot-runtime-dev mailing list