RFR (S): 8066566: Refactor ParNewGeneration to contain ParNewTracer
Marcus Larsson
marcus.larsson at oracle.com
Wed Dec 10 14:51:36 UTC 2014
Hi again,
Updated the patch with some cleanups to use 'const ParNewTracer*' rather
than 'ParNewTracer&' where appropriate.
Webrev:
http://cr.openjdk.java.net/~mlarsson/8066566/webrev.02/
Incremental:
http://cr.openjdk.java.net/~mlarsson/8066566/webrev.01-02/
Thanks,
Marcus
On 08/12/14 12:29, Marcus Larsson wrote:
> Hi Erik,
>
> On 06/12/14 10:57, Erik Helin wrote:
>> Hi Marcus,
>>
>> On 2014-12-04 13:05, Marcus Larsson wrote:
>>> Hi,
>>>
>>> I would like reviews for the following small patch.
>>>
>>> Like the bug description says, the ParNewTracer is stack-allocated
>>> during ParNewGeneration::collect(), forcing us to send the tracer as an
>>> argument to functions that might need it. This patch moves the tracer,
>>> and makes it a field in ParNewGeneration instead.
>>>
>>> The change includes adding a getter for this field, even though it is
>>> not used, however, future changes adding PLAB tracing [1] will require
>>> this.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~mlarsson/8066566/webrev.00/
>>
>> parNewGeneration.hpp
>> + ParNewTracer& gc_tracer() {
>> + return _gc_tracer;
>> + }
>> +
>>
>> Can this getter return a const ParNewTracer* instead? I believe most
>> of the methods on ParNewTracer should be const.
>
> Done.
>
> Incremental:
> http://cr.openjdk.java.net/~mlarsson/8066566/webrev.00-01/
>
> New webrev:
> http://cr.openjdk.java.net/~mlarsson/8066566/webrev.01/
>
> Thanks for looking at this!
> Marcus
>
>>
>> Thanks,
>> Erik
>>
>>> Bug:
>>> https://bugs.openjdk.java.net/browse/JDK-8066566
>>>
>>> Testing:
>>> jprt
>>>
>>> Thanks,
>>> Marcus
>>>
>>> [1]: https://bugs.openjdk.java.net/browse/JDK-8055845
>
More information about the hotspot-gc-dev
mailing list