RFR (M): 8146987: Improve Parallel GC Full GC by caching results of live_words_in_range() [Was: Re: [PATCH] enhancement to ParallelScavenge Full GC]
Thomas Schatzl
thomas.schatzl at oracle.com
Fri Jan 15 11:43:18 UTC 2016
Hi all,
On Thu, 2016-01-14 at 13:04 -0800, Jon Masamitsu wrote:
> http://cr.openjdk.java.net/~tschatzl/8146987/webrev.1/src/share/vm/gc
> /parallel/psCompactionManager.hpp.frames.html
>
> Please initialize these in the constructor
>
> 112 HeapWord* _last_query_beg;
> 113 oop _last_query_obj;
> 114 size_t _last_query_ret;
>
> http://cr.openjdk.java.net/~tschatzl/8146987/webrev.1/src/share/vm/oo
> ps/oop.hpp.udiff.html
>
> Any reason to use "pc" as the parameter name here and
> not "cm" as is used elsewhere?
>
> - void pc_update_contents();
> + void pc_update_contents(ParCompactionManager* pc);
>
new webrevs (also including Mikael's comments) at
http://cr.openjdk.java.net/~tschatzl/8146987/webrev.2/ (full)
http://cr.openjdk.java.net/~tschatzl/8146987/webrev.1_to_2/ (diff)
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list