RFR (S): 8129573: CollectedHeap::fill_with_objects() needs to use multiple arrays in 32 bit mode too
tom.benson at oracle.com
Wed Jun 24 13:08:27 UTC 2015
Looks good to me, though I would say "may be required" rather than "are
required" in the comment.
On 6/24/2015 5:40 AM, Thomas Schatzl wrote:
> Hi all,
> can I have reviews for this small bugfix that got exposed with latest
> changes in JDK-8042668?
> With that change, we cannot assume that only one object is required to
> fill a given memory range, as filler_array_max_size is actually 512k
> with G1 now on 32 bit systems (i.e. typically you get 1M regions).
> The change just removes the #ifdef __LP64 and fixes the comment.
> The problem itself actually only got exposed with the upcoming change
> JDK-8129558, where GCBasher acts as a good reproducable test case.
> JPRT (running gcbasher + containing JDK-8129558)
More information about the hotspot-gc-dev