RFR(S): 8186437: Lock held when compiler thread creation is aborted.

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Tue Aug 22 14:38:24 UTC 2017


Hi Thomas, 

thanks for the review!

The control flow is just as in JVM_StartThread, there even an 
extra variable is maintained for the error case that is used only once.

Best regards,
  Goetz. 


> -----Original Message-----
> From: Thomas Stüfe [mailto:thomas.stuefe at gmail.com]
> Sent: Dienstag, 22. August 2017 15:37
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Cc: hotspot-compiler-dev at openjdk.java.net
> Subject: Re: RFR(S): 8186437: Lock held when compiler thread creation is
> aborted.
> 
> Hi Goetz,
> 
> 
> 
> On Tue, Aug 22, 2017 at 3:19 PM, Lindenmaier, Goetz
> <goetz.lindenmaier at sap.com <mailto:goetz.lindenmaier at sap.com> > wrote:
> 
> 
> 	Hi,
> 
> 	Could I please get reviews for this small change? I also please need a
> sponsor.
> 	http://cr.openjdk.java.net/~goetz/wr17/8186437-compThrLock/
> <http://cr.openjdk.java.net/~goetz/wr17/8186437-compThrLock/>
> 
> 	When the VM is aborted because compiler thread creation fails (seen
> in
> 	TestOprionsWithRanges with huge stack size) the Thread_lock was not
> 	released.
> 
> 
> 
> 
> Seems fine, if a bit difficult to read. Alternatively, one could have unlocked
> manually - after all, we are never leaving this function, so the ~MutexLocker.
> That would have preserved the control.
> 
> Cheers, Thomas
> 
> 
> 
> 
> 
> 	Best regards,
> 	  Goetz.
> 
> 



More information about the hotspot-compiler-dev mailing list