RFR(xs): 8221517: G1: Reserved page size for heap can be wrong
Thomas Schatzl
thomas.schatzl at oracle.com
Mon Apr 1 10:37:51 UTC 2019
Hi Sangheon,
On Fri, 2019-03-29 at 09:54 -0700, sangheon.kim at oracle.com wrote:
> Hi Thomas,
>
> Thanks for the review.
>
> On 3/29/19 3:46 AM, Thomas Schatzl wrote:
> > Hi Sangheon,
> >
> > On Thu, 2019-03-28 at 20:47 -0700, sangheon.kim at oracle.com wrote:
> > > Hi all,
> > > I added a new jtreg test to check this problem. It relies on the
> > > error message when printed if reserving memory with large page
> > > fails.
> > > Unfortunately, Solaris has a bug that the error message is
> > > printed even in success. To address this problem I filed 'JDK-
> > > 8221526: Solaris: large page reservation failure message is
> > > printed even in success.' and the test is excluded on Solaris.
> >
> > - g1CollectedHeap.cpp:1578: s/alginment/alignment ,
> > s/ResesrvedSpace/ReservedSpace
> >
> > - Could you rewrite line 1581 with:
> >
> > page_size = MIN2(rs.alignment(), os::large_page_size());
> >
> > That would be a lot shorter and as understandable than the three
> > extra lines imho.
> >
> > - TestLargePageUseForHeap.java:28: maybe add "... is allocated
> > using large pages of the appropriate size if available" to the
> > @summary tag.
>
> All fixed, thanks!
>
> http://cr.openjdk.java.net/~sangheki/8221517/webrev.1
> http://cr.openjdk.java.net/~sangheki/8221517/webrev.1_inc_0
> Testing: local build
still looks good.
Thomas
More information about the hotspot-gc-dev
mailing list