RFR: 8194681: G1 uses young free cset time when reporting non-young free cset times

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR: 8194681: G1 uses young free cset time when reporting non-young free cset times

Leo Korinth
Hi,

G1 uses young free cset time when reporting non-young free cset times.
This patch is fixing the typo.

Bug:
https://bugs.openjdk.java.net/browse/JDK-8194681

Webrev:
http://cr.openjdk.java.net/~lkorinth/8194681/00/

Testing:
- hs-tier1, hs-tier2

Thanks,
Leo
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8194681: G1 uses young free cset time when reporting non-young free cset times

Thomas Schatzl
Hi,

On Tue, 2018-01-09 at 15:41 +0100, Leo Korinth wrote:

> Hi,
>
> G1 uses young free cset time when reporting non-young free cset
> times.
> This patch is fixing the typo.
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8194681
>
> Webrev:
> http://cr.openjdk.java.net/~lkorinth/8194681/00/
>
> Testing:
> - hs-tier1, hs-tier2

  looks good.

Thomas

Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8194681: G1 uses young free cset time when reporting non-young free cset times

Kim Barrett
In reply to this post by Leo Korinth
> On Jan 9, 2018, at 9:41 AM, Leo Korinth <[hidden email]> wrote:
>
> Hi,
>
> G1 uses young free cset time when reporting non-young free cset times. This patch is fixing the typo.
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8194681
>
> Webrev:
> http://cr.openjdk.java.net/~lkorinth/8194681/00/
>
> Testing:
> - hs-tier1, hs-tier2
>
> Thanks,
> Leo

Looks good.

Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8194681: G1 uses young free cset time when reporting non-young free cset times

Leo Korinth
Thanks for the reviews Kim and Thomas!

/Leo

On 09/01/18 18:15, Kim Barrett wrote:

>> On Jan 9, 2018, at 9:41 AM, Leo Korinth <[hidden email]> wrote:
>>
>> Hi,
>>
>> G1 uses young free cset time when reporting non-young free cset times. This patch is fixing the typo.
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8194681
>>
>> Webrev:
>> http://cr.openjdk.java.net/~lkorinth/8194681/00/
>>
>> Testing:
>> - hs-tier1, hs-tier2
>>
>> Thanks,
>> Leo
>
> Looks good.
>