Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-16 Thread Andrew Haley
On 09/12/2013 10:15 PM, Alex Kasko wrote: > On 09/12/2013 09:20 PM, Andrew Hughes wrote: >> >> >> - Original Message - >>> On 09/12/2013 10:06 AM, Andrew Hughes wrote: Enjoy: http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ >>> >>> I tried to apply the patch linked in the

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-15 Thread Alex Kasko
On 09/14/2013 11:37 PM, Mark Wielaard wrote: On Fri, Sep 13, 2013 at 02:24:26PM -0400, Andrew Hughes wrote: - Original Message - Maybe a build server for windows builds will help? If there exists a spare server (e.g. at redhat) I can setup automated windows builds there (or write detail

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-14 Thread Mark Wielaard
On Fri, Sep 13, 2013 at 02:24:26PM -0400, Andrew Hughes wrote: > - Original Message - > > Maybe a build server for windows builds will help? If there exists a > > spare server (e.g. at redhat) I can setup automated windows builds there > > (or write detailed instructions how to do it). It w

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-13 Thread Andrew Hughes
- Original Message - > On 09/13/2013 02:19 AM, Andrew Hughes wrote: > > - Original Message - > >> On 09/12/2013 09:20 PM, Andrew Hughes wrote: > >>> > >>> > >>> - Original Message - > On 09/12/2013 10:06 AM, Andrew Hughes wrote: > > Enjoy: > > > > http://c

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-13 Thread Omair Majid
On 09/12/2013 02:44 PM, Omair Majid wrote: > One concern, though (not that it needs to be resolved right now, but I > figured I would point it out anyway): > > In current hotspot at jdk6, src/share/vm/runtime/globals.hpp contains > this bit: > > experimental(bool, EnableInvokeDynamic, false, >

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-13 Thread Alex Kasko
On 09/13/2013 02:19 AM, Andrew Hughes wrote: - Original Message - On 09/12/2013 09:20 PM, Andrew Hughes wrote: - Original Message - On 09/12/2013 10:06 AM, Andrew Hughes wrote: Enjoy: http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ I tried to apply the patch linked in

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-13 Thread Alex Kasko
On 09/13/2013 12:49 PM, Andrew Haley wrote: On 09/12/2013 10:15 PM, Alex Kasko wrote: On 09/12/2013 09:20 PM, Andrew Hughes wrote: Tried it on windows-i586 - I propose this patch [1] (alt link [2], description [3]) to be included. I'm not sure whether it will be enough (patched build is in pro

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-13 Thread Andrew Haley
On 09/12/2013 10:15 PM, Alex Kasko wrote: > On 09/12/2013 09:20 PM, Andrew Hughes wrote: > > Tried it on windows-i586 - I propose this patch [1] (alt link [2], > description [3]) to be included. > > I'm not sure whether it will be enough (patched build is in progress > now), but windows build wi

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > Hi, > > In case it was not clear, please go ahead with the push. > added 1587 changesets with 6604 changes to 1453 files :-) > On 09/12/2013 04:01 PM, Andrew Hughes wrote: > >> One concern, though (not that it needs to be resolved right now, but I > >> figured

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > Hi, > > In case it was not clear, please go ahead with the push. > > On 09/12/2013 04:01 PM, Andrew Hughes wrote: > >> One concern, though (not that it needs to be resolved right now, but I > >> figured I would point it out anyway): > > (snip) > > > Maybe, but e

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > On 09/12/2013 09:20 PM, Andrew Hughes wrote: > > > > > > - Original Message - > >> On 09/12/2013 10:06 AM, Andrew Hughes wrote: > >>> Enjoy: > >>> > >>> http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > >> > >> I tried to apply the patch linked in the webr

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > On 09/12/2013 10:06 AM, Andrew Hughes wrote: > > Enjoy: > > > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > > > > Differences that remain between this and jdk7u25-b15 [0]: > > Thanks for the patch and explaining the differences. All the changes > looks se

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Omair Majid
Hi, In case it was not clear, please go ahead with the push. On 09/12/2013 04:01 PM, Andrew Hughes wrote: >> One concern, though (not that it needs to be resolved right now, but I >> figured I would point it out anyway): (snip) > Maybe, but either way, that's a patch to apply on top, not someth

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Alex Kasko
On 09/12/2013 09:20 PM, Andrew Hughes wrote: - Original Message - On 09/12/2013 10:06 AM, Andrew Hughes wrote: Enjoy: http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ I tried to apply the patch linked in the webrev, but it fails to apply. The patch contains changes like [1] part

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Omair Majid
On 09/12/2013 10:06 AM, Andrew Hughes wrote: > Enjoy: > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > > Differences that remain between this and jdk7u25-b15 [0]: Thanks for the patch and explaining the differences. All the changes looks sensible to me. > * src/os/linux/vm/os_linux.cpp

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
- Original Message - > On 09/12/2013 10:06 AM, Andrew Hughes wrote: > > Enjoy: > > > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ > > I tried to apply the patch linked in the webrev, but it fails to apply. > The patch contains changes like [1] part of which are already present

Re: [MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Omair Majid
On 09/12/2013 10:06 AM, Andrew Hughes wrote: > Enjoy: > > http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ I tried to apply the patch linked in the webrev, but it fails to apply. The patch contains changes like [1] part of which are already present in jdk6/hotspot [2]. I am not sure if this is

[MERGE FOR REVIEW] HotSpot 23.25-b01 Merge

2013-09-12 Thread Andrew Hughes
Enjoy: http://cr.openjdk.java.net/~andrew/jdk6-hs23-merge/ Differences that remain between this and jdk7u25-b15 [0]: * Formatting correction in agent/test/jdi/sagtest.java * ^webrev/ in .hgignore * .jcheck/conf settings * hotspot/make/hotspot_version has JDK_MINOR_VER=6 * hotspot/make/jprt.prope