RFR (XXS): 8048088: Conservative maximum heap alignment should take vm_allocation_granularity into account

Thomas Schatzl thomas.schatzl at oracle.com
Thu Jul 10 09:26:25 UTC 2014


Hi Bengt,

On Mon, 2014-07-07 at 14:55 +0200, Bengt Rutisson wrote:
> On 2014-07-07 14:47, Thomas Schatzl wrote:
> > Hi all,
> >
> >    can I have reviews for the following small change? It fixes a at this
> > time benign issue when calculating the conservative maximum heap
> > alignment.
> >
> > In particular, it does not take the os::vm_allocation_granularity() into
> > account.
> > [..]
> > As it is a bug, although it is hidden by other code, and caught already
> > later too, I thought it would still be worth fixing this.
> >
> > This code touches runtime code, so I cc'ed hotspot-runtime-dev.
> >
> > Webbug:
> > http://cr.openjdk.java.net/~tschatzl/8048088/webrev/
> >
> > CR:
> > https://bugs.openjdk.java.net/browse/JDK-8048088
> 
> Looks good to me.

Thanks for the review.

Thomas




More information about the hotspot-gc-dev mailing list