RFR(XS) 8224247: AllocateOldGenAt fires assertion failure

[Kharbas, Kishor]

Thank you!
-Kishor

> -----Original Message-----
> From: sangheon.kim at oracle.com [mailto:sangheon.kim at oracle.com]
> Sent: Monday, June 3, 2019 2:40 PM
> To: Kharbas, Kishor <kishor.kharbas at intel.com>
> Cc: Thomas Schatzl <thomas.schatzl at oracle.com>; hotspot-gc-
> dev at openjdk.java.net
> Subject: Re: RFR(XS) 8224247: AllocateOldGenAt fires assertion failure
> 
> Hi Kishor,
> 
> On 6/3/19 11:27 AM, Kharbas, Kishor wrote:
> > Thanks Thomas and Sangheon for the review. Tier1 tests also pass at my
> end.
> > @Sangheon, can you sponsor the patch?
> > I have the changeset at
> http://cr.openjdk.java.net/~kkharbas/8224247/8224247.patch.
> Done.
> 
> Thanks,
> Sangheon
> 
> 
> >
> > Thanks,
> > Kishor
> >
> >> -----Original Message-----
> >> From: Thomas Schatzl [mailto:thomas.schatzl at oracle.com]
> >> Sent: Wednesday, May 29, 2019 12:11 PM
> >> To: Kharbas, Kishor <kishor.kharbas at intel.com>; hotspot-gc-
> >> dev at openjdk.java.net
> >> Subject: Re: RFR(XS) 8224247: AllocateOldGenAt fires assertion
> >> failure
> >>
> >> Hi,
> >>
> >> On Tue, 2019-05-28 at 15:40 +0000, Kharbas, Kishor wrote:
> >>> Hi,
> >>>
> >>> Requesting review for,
> >>> CR : https://bugs.openjdk.java.net/browse/JDK-8224247
> >>> Webrev : http://cr.openjdk.java.net/~kkharbas/8224247/webrev.00
> >>>
> >>> [...]
> >>> The fix is just adding a new method and using it during
> >>> initialization of G1 and PS heap, but the existing method's name is
> >>> more appropriate for the new method. So I renamed existing method.
> >>> This would also avoid confusion about these methods down the line.
> >>>
> >>> Please suggest alternated approach if you see one.
> >>>
> >>    looks good.
> >>
> >> Thomas