Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-06-07 Thread Leonid Mesnik
Thank you for review. Leonid On 07.06.2016 12:32, David Holmes wrote: On 7/06/2016 7:03 PM, Leonid Mesnik wrote: Hi The testset hotspot is not affected by this change. It run c2 only on windows/linux-x86. Right -sorry - I forgot we already ripped out client testing in 8153071. All good.

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-06-07 Thread David Holmes
On 7/06/2016 7:03 PM, Leonid Mesnik wrote: Hi The testset hotspot is not affected by this change. It run c2 only on windows/linux-x86. Right -sorry - I forgot we already ripped out client testing in 8153071. All good. Thanks, David I verified that default target and pit has same results

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-06-07 Thread Leonid Mesnik
Hi The testset hotspot is not affected by this change. It run c2 only on windows/linux-x86. I verified that default target and pit has same results and similar time as before my fix. Leonid On 27.05.2016 01:09, David Holmes wrote: Hi Leonid, On 27/05/2016 5:00 AM, Leonid Mesnik wrote:

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-05-26 Thread David Holmes
Hi Leonid, On 27/05/2016 5:00 AM, Leonid Mesnik wrote: Dear All Please find updated webrev for this fix here: http://cr.openjdk.java.net/~lmesnik/8154209/webrev.00/hotspot/

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-05-26 Thread Leonid Mesnik
Dear All Please find updated webrev for this fix here: http://cr.openjdk.java.net/~lmesnik/8154209/webrev.00/hotspot/ http://cr.openjdk.java.net/~lmesnik/8154209/webrev.00/root/ The client jvm is removed from linux-x86 and

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-21 Thread David Holmes
On 21/04/2016 7:41 AM, Mikael Vidstedt wrote: Good catch. Updated webrevs here: top: http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.02/webrev/ Minor nit - jib-profiles.js: ! configure_args_32bit: ["--with-target-bits=32", "--with-jvm-variants=server"], I think server

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-21 Thread Leonid Mesnik
Mikael On 21.04.2016 00:41, Mikael Vidstedt wrote: Good catch. Updated webrevs here: top: http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.02/webrev/ http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.02/webrev/common/conf/jib-profiles.js.udiff.html

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-20 Thread Mikael Vidstedt
Good catch. Updated webrevs here: top: http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.02/webrev/ hotspot: http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.02/hotspot/webrev/ Incremental webrevs (from webrev.01): top:

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-20 Thread Leonid Mesnik
Hi Shouldn't be jprt targets in jprt.properties updates to stop using client also? http://hg.openjdk.java.net/jdk9/hs/file/645c48292130/make/jprt.properties line 206 - 214 # Test target list (no fastdebug & limited c2 testing) my.test.target.set= \

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-15 Thread Erik Joelsson
On 2016-04-14 19:20, Christian Thalinger wrote: On Apr 14, 2016, at 6:41 AM, Mikael Vidstedt wrote: Please review the following change which removes the "client" VM from the default JIB build profile Is there some public documentation about JIB? A quick search

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-15 Thread Erik Joelsson
Looks good to me. /Erik On 2016-04-14 18:41, Mikael Vidstedt wrote: Please review the following change which removes the "client" VM from the default JIB build profile on windows-x86 and linux-x86: Bug: https://bugs.openjdk.java.net/browse/JDK-8154209 Webrev (top):

Re: RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-14 Thread Christian Thalinger
> On Apr 14, 2016, at 6:41 AM, Mikael Vidstedt > wrote: > > > Please review the following change which removes the "client" VM from the > default JIB build profile Is there some public documentation about JIB? A quick search only showed a few JBS bugs that

RFR(S): 8154209: Remove client VM from default JIB profile on windows-x86 and linux-x86

2016-04-14 Thread Mikael Vidstedt
Please review the following change which removes the "client" VM from the default JIB build profile on windows-x86 and linux-x86: Bug: https://bugs.openjdk.java.net/browse/JDK-8154209 Webrev (top): http://cr.openjdk.java.net/~mikael/webrevs/8154209/webrev.01/ Webrev (hotspot):