RFR 8059533: (process) Make exiting process wait for exiting threads [win]

[Daniel D. Daugherty]

On 10/25/14 12:23 PM, Ivan Gerasimov wrote:
>
> On 25.10.2014 3:06, Daniel D. Daugherty wrote:
>> On 10/1/14 3:07 AM, Ivan Gerasimov wrote:
>>> Hello!
>>>
>>> The tests that continue to fail with wrong exit codes suggest that 
>>> the fix for JDK-8057744 wasn't sufficient.
>>> Here's another proposal, which expands the synchronized portion of 
>>> the code.
>>> It is proposed to make the exiting process wait for the threads that 
>>> have already started exiting.
>>> This should help to make sure that no thread is executing any 
>>> potentially racy code concurrently with the exiting process.
>>>
>>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8059533
>>> WEBREV: http://cr.openjdk.java.net/~igerasim/8059533/0/webrev/
>>
>> Finally got a chance to look at the official version of fix.
>>
>> Thumbs up!
>>
>> src/os/windows/vm/os_windows.cpp
>>     No comments.
>>
> Thank you Daniel!
>
> I assume the change needs the second hotspot reviewer?

Yes, HotSpot changes always need two reviewers. David Holmes
chimed in on this thread. You should ask him if he can be
counted as a reviewer.


> What would be the best time for pushing this fix?

Let's go for Wednesday again so we have a full week of testing
to evaluate this latest tweak.

Dan


>
> Sincerely yours,
> Ivan
>
>> Dan
>>
>> P.S.
>> We had another sighting of an exit_code == 60115 test failure
>> this past week so while your previous fix greatly reduced the
>> odds of this race, I'm looking forward to seeing this new
>> version in action...
>>
>>
>>
>>>
>>> Comments, suggestion are welcome!
>>>
>>> Sincerely yours,
>>> Ivan
>>
>>
>>
>