Re: hardcoding -m32/-m64 is more harmful than using the toolchain defaults

2015-10-20 Thread David Holmes
On 20/10/2015 10:14 PM, Matthias Klose wrote: On 20.10.2015 09:36, David Holmes wrote: On 20/10/2015 6:50 AM, Matthias Klose wrote: I'm working around some build failures for zero targets, which fail to build because the configury in openjdk tries to set -m32/-m64 on it's own. I assume this be

RFR (XXS): 8139935: Bootcycle builds are broken on jdk9/hs due to JVMCI changes

2015-10-20 Thread Christian Thalinger
https://bugs.openjdk.java.net/browse/JDK-8139935 The reason this “works” for a non-bootcycle build is because a JDK 9 javac won't enforce the “hidden packages” rules when running on a non-JDK 9 platform. diff -r a8a8604f890f make/gensrc/Gensrc-jdk.vm.ci.gmk --- a/make/gensrc/Gensrc-jdk.vm.ci.gmk

Re: RFR: JDK-8139951 Do not store debug level in OPT part of Verona string

2015-10-20 Thread Tim Bell
Looks good to me as well. /Tim On 10/20/15 02:45, Erik Joelsson wrote: Looks good to me. Nice to see all that nastiness disappear. /Erik On 2015-10-20 11:11, Magnus Ihse Bursie wrote: After discussion, it was decided that debug level information should not be a part of the new version string

Re: [8u] RFR: JDK-8139813: Base heap size on type of boot jdk, not architecture of build machine

2015-10-20 Thread Tim Bell
On 10/20/15 05:59, Magnus Ihse Bursie wrote: On 2015-10-20 10:20, Erik Joelsson wrote: Hello, New webrevs implementing the suggestion by Magnus. Webrev JDK 9: http://cr.openjdk.java.net/~erikj/8139813/webrev.jdk9.02/ Webrev JDK 8u: http://cr.openjdk.java.net/~erikj/8139813/webrev.jdk8u.02/ Lo

Re: [patch] correctly detect 32bit platforms

2015-10-20 Thread Magnus Ihse Bursie
On 2015-10-19 22:16, Matthias Klose wrote: Sorry if this is the wrong ML, but this results in hotspot build failures on various 32bit zero targets. As you're probably aware by now, build questions should be directed to [email protected]. :-) common/autoconf/boot-jdk.m4 uses BUILD_NUM_

Re: RFR: JDK-8139969 Fix unzip in compare.sh broken by JDK-8136813

2015-10-20 Thread Erik Joelsson
Looks good to me. /Erik On 2015-10-20 15:57, Magnus Ihse Bursie wrote: Unfortunately the logging of compare.sh introduced in JDK-8136813 broke unzip. In JDK-8136813 we started exportin the shell variable UNZIP, which contains the path to UNZIP. Unfortunately, UNZIP is interpreted by UNZIP a

Re: RFR: 8139271: Add top-level Makefile target to run hotspots jtreg tests

2015-10-20 Thread Erik Helin
On 2015-10-09, Magnus Ihse Bursie wrote: > Looks good to me. Thanks! Erik > /Magnus > > > 9 okt 2015 kl. 14:23 skrev Erik Helin : > > > > Hi all, > > > > this patch adds a new top-level Makefile to run the hotspot jtreg tests. > > This is already possible today by running: > > $ make test TES

Re: RFR: 8139271: Add top-level Makefile target to run hotspots jtreg tests

2015-10-20 Thread Erik Helin
On 2015-10-09, Erik Joelsson wrote: > Looks good to me. Thanks! Erik > /Erik > > On 2015-10-09 14:23, Erik Helin wrote: > >Hi all, > > > >this patch adds a new top-level Makefile to run the hotspot jtreg tests. > >This is already possible today by running: > >$ make test TEST="hotspot_all" > > >

RFR: JDK-8139969 Fix unzip in compare.sh broken by JDK-8136813

2015-10-20 Thread Magnus Ihse Bursie
Unfortunately the logging of compare.sh introduced in JDK-8136813 broke unzip. In JDK-8136813 we started exportin the shell variable UNZIP, which contains the path to UNZIP. Unfortunately, UNZIP is interpreted by UNZIP as a variable containing extra options to add to the command line. Bug: h

Re: RFR: 8139272: Add configure variable to set concurrency for jtreg tests

2015-10-20 Thread Erik Helin
On 2015-10-14, Magnus Ihse Bursie wrote: > On 2015-10-09 14:09, Erik Helin wrote: > >Hi all, > > > >this patch adds a new configure variable: --with-test-jobs. The new > >variable configures how many tests jobs we run concurrently (aka the > >-concurrency flag to JTReg). Today --with-jobs is passed

Re: hardcoding -m32/-m64 is more harmful than using the toolchain defaults

2015-10-20 Thread Andrew Haley
On 10/20/2015 01:57 PM, Magnus Ihse Bursie wrote: > What does x86_64-linux-gnux32 imply? Some sort of mix between 32 and 64-bit? I think that's a 32-bit ABI running on a 64-bit processor. https://en.wikipedia.org/wiki/X32_ABI If so, I'm surprised it works at all. Andrew.

Re: [8u] RFR: JDK-8139813: Base heap size on type of boot jdk, not architecture of build machine

2015-10-20 Thread Magnus Ihse Bursie
On 2015-10-20 10:20, Erik Joelsson wrote: Hello, New webrevs implementing the suggestion by Magnus. Webrev JDK 9: http://cr.openjdk.java.net/~erikj/8139813/webrev.jdk9.02/ Webrev JDK 8u: http://cr.openjdk.java.net/~erikj/8139813/webrev.jdk8u.02/ Looks good. This version was even better than I i

Re: hardcoding -m32/-m64 is more harmful than using the toolchain defaults

2015-10-20 Thread Magnus Ihse Bursie
On 2015-10-20 14:14, Matthias Klose wrote: On 20.10.2015 09:36, David Holmes wrote: On 20/10/2015 6:50 AM, Matthias Klose wrote: I'm working around some build failures for zero targets, which fail to build because the configury in openjdk tries to set -m32/-m64 on it's own. I assume this behav

Re: hardcoding -m32/-m64 is more harmful than using the toolchain defaults

2015-10-20 Thread Matthias Klose
On 20.10.2015 09:36, David Holmes wrote: On 20/10/2015 6:50 AM, Matthias Klose wrote: I'm working around some build failures for zero targets, which fail to build because the configury in openjdk tries to set -m32/-m64 on it's own. I assume this behaviour was added for sun/oracle product builds

Re: [patch] fix the build with a toolchain with a linker defaulting to ld --as-needed

2015-10-20 Thread Matthias Klose
On 20.10.2015 10:28, Erik Joelsson wrote: Hello Matthias, I think this looks good and can sponsor the push for you. Have you signed the OCA? see http://www.oracle.com/technetwork/community/oca-486395.html#k /Erik On 2015-10-19 22:06, Matthias Klose wrote: Toolchains for some Linux distri

Re: RFR: JDK-8139951 Do not store debug level in OPT part of Verona string

2015-10-20 Thread Erik Joelsson
Looks good to me. Nice to see all that nastiness disappear. /Erik On 2015-10-20 11:11, Magnus Ihse Bursie wrote: After discussion, it was decided that debug level information should not be a part of the new version string as implemented in Verona. Instead, this information should be propagated

RFR: JDK-8139951 Do not store debug level in OPT part of Verona string

2015-10-20 Thread Magnus Ihse Bursie
After discussion, it was decided that debug level information should not be a part of the new version string as implemented in Verona. Instead, this information should be propagated using other means. (Follow-up bugs will be filed for this). This bug removes the debug-level from the opt part o

Re: [patch] fix the build with a toolchain with a linker defaulting to ld --as-needed

2015-10-20 Thread Erik Joelsson
Hello Matthias, I think this looks good and can sponsor the push for you. Have you signed the OCA? /Erik On 2015-10-19 22:06, Matthias Klose wrote: Toolchains for some Linux distributions (e.g. Ubuntu, OpenSuse) are configured to pass --as-needed to the linker by default, only linking with

Re: [8u] RFR: JDK-8139813: Base heap size on type of boot jdk, not architecture of build machine

2015-10-20 Thread Erik Joelsson
Hello, New webrevs implementing the suggestion by Magnus. Webrev JDK 9: http://cr.openjdk.java.net/~erikj/8139813/webrev.jdk9.02/ Webrev JDK 8u: http://cr.openjdk.java.net/~erikj/8139813/webrev.jdk8u.02/ /Erik On 2015-10-20 08:51, Magnus Ihse Bursie wrote: On 2015-10-19 13:54, Erik Joelsson w

Re: hardcoding -m32/-m64 is more harmful than using the toolchain defaults

2015-10-20 Thread David Holmes
On 20/10/2015 6:50 AM, Matthias Klose wrote: I'm working around some build failures for zero targets, which fail to build because the configury in openjdk tries to set -m32/-m64 on it's own. I assume this behaviour was added for sun/oracle product builds to build x86 and x86_64 targets on a x86_

Re: hardcoding -m32/-m64 is more harmful than using the toolchain defaults

2015-10-20 Thread David Holmes
Hi Matthias, Please direct all these build related issues to build-dev (cc'd). Thanks, David On 20/10/2015 6:50 AM, Matthias Klose wrote: I'm working around some build failures for zero targets, which fail to build because the configury in openjdk tries to set -m32/-m64 on it's own. I assume