RFR [8055338]: (process) Add instrumentation to help diagnose JDK-6573254

[Daniel D. Daugherty]

On 8/19/14 6:57 AM, Ivan Gerasimov wrote:
> Hello again!
>
> I updated the patch to cover situations when the exiting thread isn't 
> daemon.
> I also added load_acquire/store_release for the sake of accuracy, even 
> though on Windows they seem to add nothing to the volatile access.
>
> http://cr.openjdk.java.net/~igerasim/8055338/1/webrev/

src/os/windows/vm/os_windows.cpp
     No comments.

src/share/vm/runtime/java.cpp
     No comments.

Thumbs up. I like this version even more.

Dan


>
> If the updated patch looks Okay, I'll need a sponsor to push it.
>
> Sincerely yours,
> Ivan
>
> On 19.08.2014 6:42, David Holmes wrote:
>> On 19/08/2014 10:12 AM, Ioi Lam wrote:
>>> With the Windows/x86/x64 memory model, is the write to
>>> vm_getting_terminated guaranteed to be observable by java_start()?
>>
>> In the general case, not immediately. For the threads actually of 
>> interest the logic that tells the threads to terminate happens after 
>> the write to the flag, and that logic contains sufficient 
>> "synchronization" that if the termination logic is correct then the 
>> flag must also be visible.
>>
>> David
>>
>>> - Ioi
>>>
>>> On 8/18/14, 2:19 PM, Ivan Gerasimov wrote:
>>>> Hello!
>>>>
>>>> This is a request to temporarily add some instrumentation code to
>>>> hotspot to help diagnose the intermittent failure on Windows, which
>>>> results in a wrong exit code of (sub-)process.
>>>>
>>>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8055338
>>>> WEBREV: http://cr.openjdk.java.net/~igerasim/8055338/0/webrev/
>>>>
>>>> Sincerely yours,
>>>> Ivan
>>>>
>>>
>>
>>
>