Request for review (S): 7022037: Pause when exiting if a debugger is attached on windows

[Staffan Larsen]

[resend]

 

Right - I see the problem. We've used the same code in JRockit forever, but of course we had the luxury of introducing the flag early and all of our tests were written to be aware of it...

 

How about, as others suggested, we enable it only when the hotspot.exe launcher is used? This is the use-case I am interested in, and I don't think any tests would be using it.

 

Thanks,

/Staffan

 

 

From: Daniel D. Daugherty 
Sent: den 24 februari 2011 4:11
To: hotspot-runtime-dev at openjdk.java.net
Subject: Re: Request for review (S): 7022037: Pause when exiting if a debugger is attached on windows

 

Trying again since I got a bounce from the Oracle mailer...

On 2/24/2011 7:49 AM, Staffan Larsen wrote: 

Does the method IsDebuggerPresent() distinguish between Visual
Studio,
windbg, cdbg, adplus and so one?
    

 
No, I think it works for all of them. See: http://msdn.microsoft.com/en-us/library/ms680345(VS.85).aspx
 
  

Can we automatically turn this future off when debugging remote JVM
or windows service?
    

 
I'm not sure how to do that, but I added an option to turn it off manually: -XX:-PauseAtExit
  


So for tests where we are intentionally causing a crash in order
to attach to the crash dump, we will need to add this new option.
I'm thinking about SA tests here.

More likely we will need:

    -XX:+IgnoreUnrecognizedVMOptions -XX:-PauseAtExit

in order for the tests to work with VMs that don't have the option.

Dan





 
  

Can we detect JVM launched by custom launcher (not a java.exe) and
turn this future off in this case (custom launcher could have no console)?
    

 
Don't know how to do that automatically. I'm actually mostly interested in having it turned on for the hotspot.exe launcher.
 
/Staffan
 
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20110224/d253ea84/attachment-0001.html