Re: Fail to build on windows 'g1ParScanThreadState.cpp'

2015-11-30 Thread Maurizio Cimadamore
On 30/11/15 07:59, Volker Simonis wrote: If it's not there, you should just sync valhalla with the latest jdk9 or use VS2013 or newer. Valhalla has been synced last week to jdk9 b93, so it should be fairly recent. But it seems like this particular fix is very recent and hasn't made it to b93.

Re: Fail to build on windows 'g1ParScanThreadState.cpp'

2015-11-30 Thread Maurizio Cimadamore
I've pushed the fix indicated by Volker (thanks!). Let me know if you see some improvements. Maurizio On 30/11/15 10:43, Maurizio Cimadamore wrote: On 30/11/15 07:59, Volker Simonis wrote: If it's not there, you should just sync valhalla with the latest jdk9 or use VS2013 or newer. Valhalla

Re: RFR: JDK-8036003: Add --with-debug-symbols=[none|internal|external|zipped]

2015-11-30 Thread Magnus Ihse Bursie
On 2015-11-26 15:19, Yasumasa Suenaga wrote: I uploaded a new webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8036003/webrev.04/ I implemented --with-native-debug-symbols and --enable-java-debug-symbols in it. And I deprecated --enable-debug-symbols and --enable-zip-debug-info . Hi Yasu

Re: RFR: 8073139 PPC64: User-visible arch directory and os.arch value on ppc64le cause issues with Java tooling

2015-11-30 Thread Magnus Ihse Bursie
On 2015-11-30 05:23, David Holmes wrote: Hi Sasha, Trying to trace through this is somewhat complex :) So ... At the top level if we see ppc64le then we set VAR_CPU to ppc64le instead of ppc64. However, once we get into hotspot build we want ARCH to be ppc64 again (in hotspot-spec.gmk.in)

Re: RFR 8043138: Attach API should not require jvmstat rmi protocol

2015-11-30 Thread Magnus Ihse Bursie
On 2015-11-25 17:53, Jaroslav Bachorik wrote: On 25.11.2015 11:36, Magnus Ihse Bursie wrote: On 2015-11-25 10:39, Alan Bateman wrote: On 25/11/2015 09:25, Jaroslav Bachorik wrote: I don't think we can just repackage these interfaces - they are remote API and we would break compatibility (

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

2015-11-30 Thread Magnus Ihse Bursie
On 2015-10-20 16:05, Erik Helin wrote: 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).

Re: RFR: JDK-8036003: Add --with-debug-symbols=[none|internal|external|zipped]

2015-11-30 Thread Yasumasa Suenaga
Hi Magnus, I've uploaded new webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8036003/webrev.05/ If we do implement this flag, it should work as expected. If you believe this is too hard to get right, I'm alright with fixing this as a separate bug. Okay, I removed --enable-java-debug-symbo

Re: Fail to build on windows 'g1ParScanThreadState.cpp'

2015-11-30 Thread Boaz Nahum
Thank you every one. Now it works. If you are interested, here is the crash report after attempting a simple compilation(Worked in the past) ===Begin here === Information:11/30/2015 7:52 PM - Compilation completed with 1 error and 0 warnings in 9ms E

Re: Fail to build on windows 'g1ParScanThreadState.cpp'

2015-11-30 Thread Boaz Nahum
SORRY !!! forget to undo g1ParScanThreadState.cpp(46) with _scanner(g1h, *NULL*), On Mon, Nov 30, 2015 at 7:58 PM Boaz Nahum wrote: > Thank you every one. Now it works. > > If you are interested, here is the crash report after attempting a simple > compilation(Worked in the past) > ===Begin her