Bug 1401060 - Poor StrictMath performance due to non-optimized compilation
Summary: Poor StrictMath performance due to non-optimized compilation
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: java-1.8.0-openjdk
Version: 7.3
Hardware: ppc64le
OS: Linux
unspecified
medium
Target Milestone: rc
: ---
Assignee: Andrew John Hughes
QA Contact: Lukáš Zachar
URL:
Whiteboard:
Depends On:
Blocks: 1390370
TreeView+ depends on / blocked
 
Reported: 2016-12-02 16:27 UTC by Gustavo Romero
Modified: 2017-08-01 08:46 UTC (History)
4 users (show)

Fixed In Version: java-1.8.0-openjdk-1.8.0.111-4.b18.el7
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-08-01 08:46:49 UTC
Target Upstream Version:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2017:1831 0 normal SHIPPED_LIVE java-1.8.0-openjdk bug fix and enhancement update 2017-08-01 12:39:07 UTC
openjdk bug system JDK-8170153 0 None None None 2017-01-17 22:55:07 UTC

Description Gustavo Romero 2016-12-02 16:27:04 UTC
Description of problem:

Currently fdlibm optimization is disabled for PPC64, as a consequence StrictMath methods like, but not limited to, sin(), cos(), and tan() perform poorly. 

JBS bug id comprising full discussion and references:
https://bugs.openjdk.java.net/browse/JDK-8170153

How reproducible:
Please refer to "Initial discussion on ML" link for a full discussion and how to reproduce the issue.

Comment 1 Gustavo Romero 2016-12-02 16:29:02 UTC
Dear maintainer, upstream patch for OpenJDK 9 is reviewed and awaiting Oracle's approval to be merged. Backport to OpenJDK 8 will be requested in the due time but is trivial.

Thanks.

Comment 2 Gustavo Romero 2016-12-19 20:05:19 UTC
Hi Andrew,

Please, find the patches in question already backported to 8u upstream:

http://hg.openjdk.java.net/jdk8u/jdk8u-dev/rev/fddd627ea372
http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/80927e772b9a

Thanks.

Comment 3 Andrew John Hughes 2017-01-09 18:10:16 UTC
Looks like they're already in the aarch64/jdk8u repo too:

http://hg.openjdk.java.net/aarch64-port/jdk8u/rev/7dc91fd23728
http://hg.openjdk.java.net/aarch64-port/jdk8u/jdk/rev/7c54c63ba667

So we'll pick them up anyway.

Comment 7 errata-xmlrpc 2017-08-01 08:46:49 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:1831


Note You need to log in before you can comment on or make changes to this bug.