RFR 8191987: TLAB Refills for PPC64

Doerr, Martin martin.doerr at sap.com
Thu Dec 21 15:06:38 UTC 2017


Hi Jc,

excellent. I didn't really like the usage of SkipIfEqualZero in the current code. I prefer your version.
I can push it after we got a 2nd review.

Thanks and best regards,
Martin


-----Original Message-----
From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf Of JC Beyler
Sent: Mittwoch, 20. Dezember 2017 22:04
To: hotspot-dev at openjdk.java.net
Subject: Re: RFR 8191987: TLAB Refills for PPC64

Please see the updated webrev :)


WebRev: http://cr.openjdk.java.net/~jcbeyler/8191987/webrev.03/

On Wed, Dec 20, 2017 at 12:30 PM, JC Beyler <jcbeyler at google.com> wrote:
> Hi all,
>
> The TLAB and the inline contiguous allocations handling are different
> for each architecture. On certain architectures, TLAB is never
> actually never refilled (ref:
> https://bugs.openjdk.java.net/browse/JDK-8190862).
>
> The PPC patch really only removes the code that augments the TLAB
> waste limit since that is already done in the common code slow path.
>
> Due to a double jump, there is an additional change to have code from
> SkipIfEqualZero available so that we don't have to manually generate
> that code in the template code.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8191987
> WebRev: http://cr.openjdk.java.net/~rasbold/8191987/webrev.03/
>
> Let me know what you think and if someone could test/verify it, that
> would be great.
>
> Thanks for your help,
> Jc


More information about the hotspot-dev mailing list