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