RFR(XXS): 8044531: Event based tracing locks to rank as leafs where possible

[Markus Grönlund]

Thanks Dan!

 

I will remove the +1

 

Cheers

Markus

 

 

From: Daniel D. Daugherty 
Sent: den 2 juni 2014 16:48
To: Markus Grönlund
Cc: serviceability-dev at openjdk.java.net serviceability-dev at openjdk.java.net; hotspot-runtime-dev
Subject: Re: RFR(XXS): 8044531: Event based tracing locks to rank as leafs where possible

 

On 6/2/14 8:41 AM, Markus Grönlund wrote:



Greetings,

 

Kindly asking for review for the following small change:

 

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Emgronlun/8044531/webrev01/"http://cr.openjdk.java.net/~mgronlun/8044531/webrev01/

 
src/share/vm/runtime/mutexLocker.cpp
    line 285   def(JfrStream_lock               , Mutex,   nonleaf+1,   true);
        Now that the other locks in the logical group that were
        nonleaf are now leaf, I think you can make this one 'nonleaf'
        without the '+1'.

Thumbs up.
 
Dan
 



Bug: https://bugs.openjdk.java.net/browse/JDK-8044531 

 

Thanks

Markus

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20140602/ab3d45a6/attachment-0001.html>