RFR(S): 7016955: G1: remove the is_zeroed parameter from the HeapRegion constructor
Jon Masamitsu
jon.masamitsu at oracle.com
Thu Sep 13 18:39:31 UTC 2012
Basically, it's deleting the G1OffsetTableContigSpace
version of initialize(), using the ContiguousSpace initialize()
and doing the little bit of extra work the constructor.
And, of course, dumping the is_zeroed parameter.
Looks good.
On 09/13/12 10:08, John Cuthbertson wrote:
> Hi Everyone,
>
> Can I have a volunteer to review the changes, which were contributed
> by Brandon Mitchell at Twitter, for this CR? The webrev can be found
> at: http://cr.openjdk.java.net/~johnc/7016955/webrev.0/
>
> Summary:
> When Tony removed the zero filling thread, the is_zeroed parameter in
> the HeapRegion constructor became unused. These changes remove that
> unused parameter and clean up the HeapRegion initialization code.
>
> Testing:
> SPECjvm98 and SPECjbb2005 on Linux (Brandon); GC test suite on solaris
> (x86 and sparc) and jprt (JohnC)
>
> Thanks,
>
> JohnC
More information about the hotspot-gc-dev
mailing list