G1CollectedHeap::par_iterate_regions_array_part_from is moved from G1CollectionSet::iterate_part_from in JDK-8254167.
The parameters and logic of this method could be refined a bit to make it more straight.
"This method essentially apply a closure to all elements in [offset, length) of regions. I wonder if we can pass ®ions[offset] and length-offset to this method. Then this method can work on the whole array, and figuring out the starting index for each worker becomes trivial, worker_id * array_len / #workers. It's not obvious to me why the calculation in this patch guarantees balanced start index."
The parameters and logic of this method could be refined a bit to make it more straight.
"This method essentially apply a closure to all elements in [offset, length) of regions. I wonder if we can pass ®ions[offset] and length-offset to this method. Then this method can work on the whole array, and figuring out the starting index for each worker becomes trivial, worker_id * array_len / #workers. It's not obvious to me why the calculation in this patch guarantees balanced start index."
- relates to
-
JDK-8254167 G1: Record regions where evacuation failed to provide targeted iteration
-
- Resolved
-
