Re: (8u) RFR: 8152000: Java FlightRecorder does not run on Java 8 compact 3 profile since u65

2016-09-27 Thread David Holmes
Thanks Erik. David On 27/09/2016 6:08 PM, Erik Joelsson wrote: Looks good. /Erik On 2016-09-27 02:02, David Holmes wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8152000 webrev: http://cr.openjdk.java.net/~dholmes/8152000/webrev/ This is a small adjustment to the build of compact pr

Re: (8u) RFR: 8152000: Java FlightRecorder does not run on Java 8 compact 3 profile since u65

2016-09-27 Thread David Holmes
Thanks Erik. David On 27/09/2016 6:18 PM, Erik Gahlin wrote: Looks good Erik Bug: https://bugs.openjdk.java.net/browse/JDK-8152000 webrev: http://cr.openjdk.java.net/~dholmes/8152000/webrev/ This is a small adjustment to the build of compact profiles to allow a useful but unsupported config

Re: RFR: JEP draft for Linux/s3990x port

2016-09-27 Thread Vladimir Kozlov
On 9/27/16 10:49 AM, Volker Simonis wrote: Hi, can you please review and endorse the following draft JEP for the integration of the Linux/s390x port into the jkd9 master repository: https://bugs.openjdk.java.net/browse/JDK-8166730 Good. Add links to webrevs in a comment. It will help to get u

RFR: JEP draft for Linux/s3990x port

2016-09-27 Thread Volker Simonis
Hi, can you please review and endorse the following draft JEP for the integration of the Linux/s390x port into the jkd9 master repository: https://bugs.openjdk.java.net/browse/JDK-8166730 As detailed in the JEP, the Linux/s390x requires very few shared changes and we therefore don't foresee any

Re: JDK-8153362: [jigsaw] Add javac -Xlint warning to list exposed types which are not accessible

2016-09-27 Thread Jan Lahoda
On 27.9.2016 14:49, Alan Bateman wrote: On 27/09/2016 13:42, Jan Lahoda wrote: Hi, Any feedback on the top-level repository changes? http://cr.openjdk.java.net/~jlahoda/8153362/top-level.03/ This looks okay to me. Some of these should be looked at and I wonder if you have created any bugs.

Re: RFR: JDK-8164301: jib should provide a JDK to for running jtreg with

2016-09-27 Thread Tim Bell
Erik: When running tests via "jib make --profile run-test", jib does not set JT_JAVA (the JDK used to run the jtreg harness). This change makes Jib set the bootjdk as JT_JAVA. It also makes the same change in the top level makefiles when running "make test". Bug: https://bugs.openjdk.java.net/b

Re: JDK-8153362: [jigsaw] Add javac -Xlint warning to list exposed types which are not accessible

2016-09-27 Thread Alan Bateman
On 27/09/2016 13:42, Jan Lahoda wrote: Hi, Any feedback on the top-level repository changes? http://cr.openjdk.java.net/~jlahoda/8153362/top-level.03/ This looks okay to me. Some of these should be looked at and I wonder if you have created any bugs. In passing I see recipes for jdk.dev_

Re: JDK-8153362: [jigsaw] Add javac -Xlint warning to list exposed types which are not accessible

2016-09-27 Thread Jan Lahoda
Hi, Any feedback on the top-level repository changes? http://cr.openjdk.java.net/~jlahoda/8153362/top-level.03/ Thanks! Jan On 13.9.2016 16:28, Jan Lahoda wrote: Hello, I've updated the patch to the current situation. The top-level repository patch is here: http://cr.openjdk.java.net/~jlahod

Re: (8u) RFR: 8152000: Java FlightRecorder does not run on Java 8 compact 3 profile since u65

2016-09-27 Thread Erik Gahlin
Looks good Erik Bug: https://bugs.openjdk.java.net/browse/JDK-8152000 webrev: http://cr.openjdk.java.net/~dholmes/8152000/webrev/ This is a small adjustment to the build of compact profiles to allow a useful but unsupported configuration to continue to work after it stopped working in 8u60.

Re: (8u) RFR: 8152000: Java FlightRecorder does not run on Java 8 compact 3 profile since u65

2016-09-27 Thread Erik Joelsson
Looks good. /Erik On 2016-09-27 02:02, David Holmes wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8152000 webrev: http://cr.openjdk.java.net/~dholmes/8152000/webrev/ This is a small adjustment to the build of compact profiles to allow a useful but unsupported configuration to continu

Re: RFR(S): 8166738 - Enable concurrency in Hotspot jtreg testing

2016-09-27 Thread Erik Joelsson
Looks good to me. /Erik On 2016-09-26 23:07, Christian Tornqvist wrote: Hi everyone, Please review this small change that enables concurrency for the Hotspot jtreg tests when running them through the make system. Testing done so far shows very promising speed ups, both when running locally