RTM/HTM jtreg test regression

[Lindenmaier, Goetz]

Hi Gustave, 

Yes, this seems necessary.

Best regards,
  Goetz

> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf
> Of Gustavo Romero
> Sent: Montag, 12. Dezember 2016 22:51
> To: hotspot-dev at openjdk.java.net; ppc-aix-port-dev at openjdk.java.net
> Subject: RTM/HTM jtreg test regression
> 
> Hi,
> 
> After 8160851 [1] removed the -XaddExports option it deprecated the use of
> -XaddExports in the RTM jtreg tests introduced in 8164987 [2].
> 
> As a consequence all tests on POWER8 (and possible on x64 with RTM support)
> under compiler/rtm/locking/* are failing as reported in [2].
> 
> The following simple change is proposed to fix that issue:
> 
> diff -r 91ef517cb13e test/compiler/testlibrary/rtm/RTMTestBase.java
> --- a/test/compiler/testlibrary/rtm/RTMTestBase.java    Mon Dec 12 21:18:54
> 2016 +0300
> +++ b/test/compiler/testlibrary/rtm/RTMTestBase.java    Mon Dec 12 16:42:27
> 2016 -0500
> @@ -241,7 +241,7 @@
>                  CommandLineOptionTest.UNLOCK_DIAGNOSTIC_VM_OPTIONS,
>                  CommandLineOptionTest.UNLOCK_EXPERIMENTAL_VM_OPTIONS,
>                  "-Xbootclasspath/a:.", "-XX:+WhiteBoxAPI",
> -                "-XaddExports:java.base/jdk.internal.misc=ALL-UNNAMED");
> +                "--add-exports", "java.base/jdk.internal.misc=ALL-UNNAMED");
> 
>          if (test != null) {
>              for (String method : test.getMethodsToCompileNames()) {
> 
> 
> Could I open a bug to fix that on 9?
> 
> Thank you.
> 
> 
> Regards,
> Gustavo
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8160851
> [2] https://bugs.openjdk.java.net/browse/JDK-8164987