RFR: 8215262: PPC64: FMA Vectorization on PPC64

Thu Dec 13 13:10:02 UTC 2018

Hi Martin, Gustavo,

> most of our tests (including a subset of JCK) were running over night and
I couldn’t see any problems related to this change.
Thank you so much for long-running tests.

>Pushed:
>http://hg.openjdk.java.net/jdk/jdk/rev/1c85328b7631
Thank you, Gustavo!

Best regards,
--
Michihiro,
IBM Research - Tokyo

From:	"Gustavo Romero" <gromero at linux.vnet.ibm.com>
To:	"Doerr, Martin" <martin.doerr at sap.com>, Michihiro
            Horie/Japan/IBM at IBMJP
Cc:	"hotspot compiler" <hotspot-compiler-dev at openjdk.java.net>,
            "ppc-aix-port-dev at openjdk.java.net"
            <ppc-aix-port-dev at openjdk.java.net>, "Simonis, Volker"
            <volker.simonis at sap.com>
Date:	2018/12/13 21:29
Subject:	Re: RFR: 8215262: PPC64: FMA Vectorization on PPC64

Hi Michi, Martin

On 12/13/2018 07:24 AM, Doerr, Martin wrote:
> most of our tests (including a subset of JCK) were running over night and
I couldn’t see any problems related to this change.
>
> So I think you can push it if you want to get it in jdk12.

Pushed:
http://hg.openjdk.java.net/jdk/jdk/rev/1c85328b7631

Thank you.

Best regards,
Gustavo

> Best regards,
>
> Martin
>
> *From:*Michihiro Horie <HORIE at jp.ibm.com>
> *Sent:* Donnerstag, 13. Dezember 2018 00:05
> *To:* Gustavo Romero <gromero at linux.vnet.ibm.com>
> *Cc:* hotspot compiler <hotspot-compiler-dev at openjdk.java.net>; Doerr,
Martin <martin.doerr at sap.com>; ppc-aix-port-dev at openjdk.java.net; Simonis,
Volker <volker.simonis at sap.com>
> *Subject:* Re: RFR: 8215262: PPC64: FMA Vectorization on PPC64
>
> Hi Gustavo,
>
> Thanks a lot. It's great to have ~80% reduction on POWER9 Witherspoon.
>
>
> Best regards,
> --
> Michihiro,
> IBM Research - Tokyo
>
> Inactive hide details for "Gustavo Romero" ---2018/12/13 07:39:15---Hi
Michi and Martin, On 12/12/2018 01:53 PM, Doerr, Martin "Gustavo Romero"
---2018/12/13 07:39:15---Hi Michi and Martin, On 12/12/2018 01:53 PM,
Doerr, Martin wrote:
>
> From: "Gustavo Romero" <gromero at linux.vnet.ibm.com <
mailto:gromero at linux.vnet.ibm.com>>
> To: "Doerr, Martin" <martin.doerr at sap.com <mailto:martin.doerr at sap.com>>,
Michihiro Horie/Japan/IBM at IBMJP, "ppc-aix-port-dev at openjdk.java.net <
mailto:ppc-aix-port-dev at openjdk.java.net>"
<ppc-aix-port-dev at openjdk.java.net <
mailto:ppc-aix-port-dev at openjdk.java.net>>, "hotspot compiler"
<hotspot-compiler-dev at openjdk.java.net <
mailto:hotspot-compiler-dev at openjdk.java.net>>
> Cc: "Simonis, Volker" <volker.simonis at sap.com <
mailto:volker.simonis at sap.com>>
> Date: 2018/12/13 07:39
> Subject: Re: 8215262: PPC64: FMA Vectorization on PPC64
>
>
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

>
>
>
>
> Hi Michi and Martin,
>
> On 12/12/2018 01:53 PM, Doerr, Martin wrote:
>> this looks good to me. We should be able to test it since it is for
Power8. Maybe Gustavo would like to test in addition.
>
> Michi, the change looks good to me too.
> I've tested it on POWER8 and on POWER9 and observed a reduction of ~30%
on my
> POWER8, but on my POWER9 (Witherspoon) it was ~80%, so it can be even
better on
> a Boston (POWER9 too) machine. Thus, thumbs up from my side.
>
> I can sponsor that change but I'll wait an ack. from SAP that it's ok for
> pushing. Maybe Martin wants to perform additional tests after his first
review
> or others might want to review too...  I'm wondering if it would be
reasonable
> to run JCK/TCK against it. Would it test Math.fma() since it's a 9+
method and
> needs a warmed up JVM? Unfortunately I can't run the JCK/TCK against it
myself
> if it applies here.
>
> Thank you.
>
> Best regards,
> Gustavo
>
>> Best regards,
>>
>> Martin
>>
>> *From:*Michihiro Horie <HORIE at jp.ibm.com <mailto:HORIE at jp.ibm.com>>
>> *Sent:* Mittwoch, 12. Dezember 2018 15:50
>> *To:* ppc-aix-port-dev at openjdk.java.net <
mailto:ppc-aix-port-dev at openjdk.java.net>; hotspot compiler
<hotspot-compiler-dev at openjdk.java.net <
mailto:hotspot-compiler-dev at openjdk.java.net>>
>> *Cc:* Gustavo Romero <gromero at linux.vnet.ibm.com <
mailto:gromero at linux.vnet.ibm.com>>; Doerr, Martin <martin.doerr at sap.com <
mailto:martin.doerr at sap.com>>; Simonis, Volker <volker.simonis at sap.com <
mailto:volker.simonis at sap.com>>
>> *Subject:* RFR: 8215262: PPC64: FMA Vectorization on PPC64
>>
>> Dear all,
>>
>> Would you please review the following change?
>> This change supports FMA vectorization on PPC64.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8215262
>> Webrev: http://cr.openjdk.java.net/~mhorie/8215262/webrev.00/ <
http://cr.openjdk.java.net/%7Emhorie/8215262/webrev.00/> <
http://cr.openjdk.java.net/%7Emhorie/8215262/webrev.00/>
>>
>> Microbenchmark I used is
http://cr.openjdk.java.net/~mhorie/8215262/VFma.java <
http://cr.openjdk.java.net/%7Emhorie/8215262/VFma.java> <
http://cr.openjdk.java.net/%7Emhorie/8215262/VFma.java>
>> It showed ~50% reduction in the execution time.
>>
>> I also tested with JTREG and there was no difference in the results
compared to the original.
>>
>>
>> Best regards,
>> --
>> Michihiro,
>> IBM Research - Tokyo
>>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/ppc-aix-port-dev/attachments/20181213/c2c0a6bb/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: graycol.gif
Type: image/gif
Size: 105 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/ppc-aix-port-dev/attachments/20181213/c2c0a6bb/graycol-0001.gif>