[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-hotspot-compiler-dev
Subject: RE: RFR: 8221092: UseAVX=3 has performance degredation on Skylake (X7) processors
From: "Deshpande, Vivek R" <vivek.r.deshpande () intel ! com>
Date: 2019-09-26 17:51:39
Message-ID: 53E8E64DB2403849AFD89B7D4DAC8B2AAB7C4D1A () fmsmsx121 ! amr ! corp ! intel ! com
[Download RAW message or body]
Thanks you Vladimir for the review.
I will work on the adding the comments and changes to the bug report.
Yes the threshold is for architectures after skylake which support AVX512.
With this threshold( value = 4096 bytes found experimentally), AVX512 will be used if \
the array size is bigger than that.
Regards,
Vivek
-----Original Message-----
From: Vladimir Kozlov [mailto:vladimir.kozlov@oracle.com]
Sent: Wednesday, September 25, 2019 12:41 PM
To: Deshpande, Vivek R <vivek.r.deshpande@intel.com>; \
hotspot-compiler-dev@openjdk.java.net
Cc: Scott Oaks <scott.oaks@oracle.com>; eric.caspole <eric.caspole@oracle.com>; \
Robert Strout <robert.strout@oracle.com>
Subject: Re: RFR: 8221092: UseAVX=3 has performance degredation on Skylake (X7) \
processors
Thank you, Vivek
I see you did several changes including intrinsics code. Would be nice if you list \
changes you did in bug report. I see you removed _evex instructions variants in .ad \
file, replaced evex instructions in stubs and set UseAVX to 2 for Skylake. It is easy \
to understand.
But what about array limit AVX3Threshold? I assume it is for non-Skylake CPUs with \
AVX512. Right? What number 4096 is based on. It seems AVX3Threshold == 0 has special \
meaning - add line in globals_x86.hpp explaining it. I would need more time to look \
on related changes.
Thanks,
Vladimir
On 9/3/19 5:02 PM, Deshpande, Vivek R wrote:
> Hi All
>
> I have created a patch which sets AVX2 for Skylake and selectively
> uses EVEX instructions based on threshold for platforms after Skylake.
> I don't observe the regressions for SPECjvm2008 on Skylake with this patch.
> JBS id:
> https://bugs.openjdk.java.net/browse/JDK-8221092
> Webrev:
> http://cr.openjdk.java.net/~vdeshpande/8221092/webrev.00/
> Could you all please review the patch.
>
> Regards,
> Vivek
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic