RFR: 8038934: Remove prefix allocated_ from methods and variables in Metaspace

Jon Masamitsu jon.masamitsu at oracle.com
Tue Apr 1 21:59:09 UTC 2014


Looks good.  Thanks for the fix.

Reviewed.

Jon

On 4/1/14 8:02 AM, Erik Helin wrote:
> Hi all,
>
> this patch removes the prefix "allocated_" from some members in the 
> class MetaspaceAux.
>
> From the beginning, there were only functions named used_in_bytes() 
> and capacity_in_bytes(). These functions iterated over the 
> ClassLoaderDataGraph to reach all metaspaces and then iterated over 
> the chunk list, which made them quite slow.
>
> Later, the functions allocated_used_bytes() and 
> allocated_capacity_bytes() were added that used accounting to be able 
> to return a value faster and without using a lock. When these 
> functions were added, the functions used_bytes() and capacity_bytes() 
> were renamed to used_bytes_slow() and capacity_bytes_slow() to clarify 
> the difference (the slow functions were later made private).
>
> Since the old, slow functions end in "_slow", we can drop the prefix 
> "allocated_" from the accounting functions.
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8038934/webrev.00/
>
> Thanks,
> Erik



More information about the hotspot-dev mailing list