RFR(xs): 8205390: jtreg: Fix failing TestRTMSpinLoopCount on PPC64
Hi, Could the following change be reviewed please? bug : https://bugs.openjdk.java.net/browse/JDK-8205390 webrev: http://cr.openjdk.java.net/~gromero/8205390/v1/ It adds a new throttling sequence for PPC64 because the last value on current sequence does not fit on PPC64. By using the new sequence the following test is fixed: +Passed: compiler/rtm/locking/TestRTMSpinLoopCount.java Thank you and best regards, Gustavo
Good. Thanks, Vladimir On 6/25/18 1:31 AM, Gustavo Romero wrote:
Hi,
Could the following change be reviewed please?
bug : https://bugs.openjdk.java.net/browse/JDK-8205390 webrev: http://cr.openjdk.java.net/~gromero/8205390/v1/
It adds a new throttling sequence for PPC64 because the last value on current sequence does not fit on PPC64.
By using the new sequence the following test is fixed:
+Passed: compiler/rtm/locking/TestRTMSpinLoopCount.java
Thank you and best regards, Gustavo
On 06/25/2018 02:21 PM, Vladimir Kozlov wrote:
Good.
Thanks for reviewing it Vladimir! Regards, Gustavo
Thanks, Vladimir
On 6/25/18 1:31 AM, Gustavo Romero wrote:
Hi,
Could the following change be reviewed please?
bug : https://bugs.openjdk.java.net/browse/JDK-8205390 webrev: http://cr.openjdk.java.net/~gromero/8205390/v1/
It adds a new throttling sequence for PPC64 because the last value on current sequence does not fit on PPC64.
By using the new sequence the following test is fixed:
+Passed: compiler/rtm/locking/TestRTMSpinLoopCount.java
Thank you and best regards, Gustavo
+1 -- Igor
On Jun 25, 2018, at 10:21 AM, Vladimir Kozlov <vladimir.kozlov@oracle.com> wrote:
Good.
Thanks, Vladimir
On 6/25/18 1:31 AM, Gustavo Romero wrote:
Hi, Could the following change be reviewed please? bug : https://bugs.openjdk.java.net/browse/JDK-8205390 webrev: http://cr.openjdk.java.net/~gromero/8205390/v1/ It adds a new throttling sequence for PPC64 because the last value on current sequence does not fit on PPC64. By using the new sequence the following test is fixed: +Passed: compiler/rtm/locking/TestRTMSpinLoopCount.java Thank you and best regards, Gustavo
Hi, I'm going to push it to jdk/jdk11 after running the tests if there are no objections. Thanks, Gustavo On 06/26/2018 05:56 PM, Igor Ignatyev wrote:
+1
-- Igor
On Jun 25, 2018, at 10:21 AM, Vladimir Kozlov <vladimir.kozlov@oracle.com> wrote:
Good.
Thanks, Vladimir
On 6/25/18 1:31 AM, Gustavo Romero wrote:
Hi, Could the following change be reviewed please? bug : https://bugs.openjdk.java.net/browse/JDK-8205390 webrev: http://cr.openjdk.java.net/~gromero/8205390/v1/ It adds a new throttling sequence for PPC64 because the last value on current sequence does not fit on PPC64. By using the new sequence the following test is fixed: +Passed: compiler/rtm/locking/TestRTMSpinLoopCount.java Thank you and best regards, Gustavo
On 07/17/2018 04:10 AM, Gustavo Romero wrote:
I'm going to push it to jdk/jdk11 after running the tests if there are no objections.
All tests passed: mach5-one-gromero-JDK-8205390-20180718-1940-31577: Build tasks PASSED. Test tasks SUCCESSFUL. Pushing it. Thanks, Gustavo
Thanks, Gustavo
On 06/26/2018 05:56 PM, Igor Ignatyev wrote:
+1
-- Igor
On Jun 25, 2018, at 10:21 AM, Vladimir Kozlov <vladimir.kozlov@oracle.com> wrote:
Good.
Thanks, Vladimir
On 6/25/18 1:31 AM, Gustavo Romero wrote:
Hi, Could the following change be reviewed please? bug : https://bugs.openjdk.java.net/browse/JDK-8205390 webrev: http://cr.openjdk.java.net/~gromero/8205390/v1/ It adds a new throttling sequence for PPC64 because the last value on current sequence does not fit on PPC64. By using the new sequence the following test is fixed: +Passed: compiler/rtm/locking/TestRTMSpinLoopCount.java Thank you and best regards, Gustavo
participants (3)
-
Gustavo Romero
-
Igor Ignatyev
-
Vladimir Kozlov