RFR (S): 8133047: Rename G1ParScanThreadState::_queue_num to _worker_id

Thomas Schatzl thomas.schatzl at oracle.com
Wed Aug 5 14:35:06 UTC 2015


Hi,

On Wed, 2015-08-05 at 16:26 +0200, Jesper Wilhelmsson wrote:
> Looks good.
> 
> There is another use of queue_num in 
> G1CollectedHeap::preserve_mark_during_evac_failure(). Was that one intentionally 
> left?
> 

Overlooked, the code was not in G1ParScanThreadState so I did not look.
I do think that again worker id is expected here.

New webrevs:
http://cr.openjdk.java.net/~tschatzl/8133047/webrev.1 (full)
http://cr.openjdk.java.net/~tschatzl/8133047/webrev.0_to_1 (diff)

Thanks,
  Thomas






More information about the hotspot-gc-dev mailing list