RFR (XS): 8201365: Remove G1Policy::should_process_references()
shade at redhat.com
Tue Apr 10 14:18:01 UTC 2018
On 04/10/2018 04:06 PM, Thomas Schatzl wrote:
> On Tue, 2018-04-10 at 15:31 +0200, Aleksey Shipilev wrote:
>> On 04/10/2018 02:26 PM, Thomas Schatzl wrote:
>> Patch looks good.
>> But, don't you actually want to control refprocessing, especially
>> when pause times are concerned?
> the removed code does not control reference processing at all, it
> simply always enables it; this is some renmant of some removed closed
> code. The change is simple enough to be added again if somebody wants
> to experiment with that.
> Regarding pause times, in the future all but the parts that actually
> copy data (finalizers) for global reference processing could be done
> concurrently too.
> Maybe at some point in the future finalizers will be removed (they are
> deprecated already), so that issue may go away.
> Only clearing out referent fields may be predictable enough for the
> reference processing after young gcs (or moving it to some concurrent
> phase too).
OK, I see. I was asking that because we have Shenandoah users who control reference processing
frequency, and thus balance pause times when refprocessing is not yet concurrent. I would understand
if that is ruled not to be the priority in current G1.
More information about the hotspot-gc-dev