Request for reviews (XS): 6925573: vm/classfmt/lmt/mthnum001/mthnum00101m1/mthnum00101m1.html hangs since JDK7-b72 (-d64, solaris)
coleen.phillimore at oracle.com
Mon May 3 10:29:21 PDT 2010
Thank you for fixing this! It was on my list but I hadn't gotten to it yet.
I thought that we'd want to have a special narrow_oop_compare_fn() so
that we don't have to test if (UseCompressedOops) for each comparison,
but your testing shows no penalty for the frequent global value test.
To be clear though, in your table, the -UseCompressedOops case was
before your fix? If so, your fix looks great, and I'd be happy to check
it in after another review.
On 05/03/10 12:57, Volker Simonis wrote:
> I fixed 6925573 which was caused by the Compressed Oops change which
> changed the method sorting algorithm for class methods during
> classloading from quicksort to bubblesort.
> Could somebody please review the change at
> and submit it if is deemed to be suitable.
More information about the hotspot-runtime-dev