review request (S): 6966222: G1: simplify TaskQueue overflow handling
John Coomes
John.Coomes at oracle.com
Wed Aug 4 00:11:37 UTC 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
review :-).
Many thanks.
-John
> 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
mailing list