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






Reply via email to