review request (S): 6966222: G1: simplify TaskQueue overflow handling
John.Coomes at oracle.com
Tue Aug 3 17:11:37 PDT 2010
Y. Srinivas Ramakrishna (y.s.ramakrishna at oracle.com) wrote:
> Also looks good.
Wow, you're on a roll--maybe I should get a few more things out for
> John Coomes wrote:
> > I (John.Coomes at oracle.com) wrote:
> >> I'd appreciate reviews of the changes to simplify TaskQueue overflow
> >> handling in G1.
> >> http://cr.openjdk.java.net/~jcoomes/6966222-taskqueue-g1/
> > I updated the webrev; the main difference is to format the output as a
> > table. See the webrev comments for more details.
> > I'd particularly like people to verify that the change in the duration
> > that the stats_lock() is held is correct (g1CollectedHeap.cpp).
> >> The changes are limited to 2 files, and more lines were deleted than
> >> added :-).
> > Unfortunately, the above two statements are no longer accurate :-(.
> > -John
More information about the hotspot-gc-dev