[7u60] Request for approval for CR 8021771 - warning: stat64 is deprecated - when building on OSX 10.7.5

2013-12-09 Thread Andreas Eriksson
Hi, Requesting approval for a backport of JDK-8021771 from JDK8 to 7u60. This problem can prevent building on newer OSX systems. Bug: https://bugs.openjdk.java.net/browse/JDK-8021771 Webrev: http://cr.openjdk.java.net/~aeriksso/8021771/webrev/ JDK8 Changeset: http://hg.openjdk.java.net/hsx/hsx2

[7u80] Request for approval for CR 7142035 - assert in j.l.instrument agents during shutdown when daemon thread is running

2014-04-03 Thread Andreas Eriksson
Hi, Requesting approval for a backport of JDK-7142035 from JDK8 to 7u80. If you use a java.lang.instrument agent trying to instrument a java application with a daemon thread, you could trigger an assert inside the JPLIS code. The changeset imported cleanly from jdk8. Bug: https://bugs.openjdk

[7u-dev] Request for approval for CR 8033696 - "assert(thread != NULL) failed: just checking" due to Thread::current() and JNI pthread interaction

2014-04-09 Thread Andreas Eriksson
Hi, Requesting approval for a backport of JDK-8033696 from JDK8 to 7u-dev. The changeset imports cleanly from jdk8. Bug: https://bugs.openjdk.java.net/browse/JDK-8033696 JDK8 Changeset: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/21dd1c827123 Regards, Andreas

Re: RFR: 8038925: Java with G1 crashes in dump_instance_fields using jmap or jcmd without fullgc

2014-05-22 Thread Andreas Eriksson
ndreas On 2014-05-22 10:14, Andreas Eriksson wrote: OK, I'll remove it. Thanks, Andreas On 2014-05-22 10:02, Bengt Rutisson wrote: Hi Andreas, On 5/21/14 2:05 PM, Andreas Eriksson wrote: A new webrev is up: http://cr.openjdk.java.net/~aeriksso/8038925/webrev.02/ The test now verifies that

Re: RFR: 8038925: Java with G1 crashes in dump_instance_fields using jmap or jcmd without fullgc

2014-05-22 Thread Andreas Eriksson
in the hsx project (http://openjdk.java.net/census#brutisso) isn't that enough to review this change? Anyway, last webrev looks good. Thanks for fixing the test! Bengt On 5/22/14 10:43 AM, Andreas Eriksson wrote: Hi, Adding jdk7u-dev. Could I have a jdk7u Reviewer look at this as well p

Re: RFR: 8038925: Java with G1 crashes in dump_instance_fields using jmap or jcmd without fullgc

2014-05-23 Thread Andreas Eriksson
Great, thanks. /Andreas On 2014-05-23 12:37, Mikael Gerdin wrote: On Thursday 22 May 2014 13.39.13 Andreas Eriksson wrote: Hi, Bengt: Right, that should be enough, thanks. Mikael: Can I use you as a reviewer for this latest version as well? Yes, this looks fine. /Mikael Regards

[7u-dev] Request for approval for CR 8038925 - Java with G1 crashes in dump_instance_fields using jmap or jcmd without fullgc

2014-05-23 Thread Andreas Eriksson
Hi, Requesting push approval for JDK-8038925 , a jdk7 only fix. It has been reviewed by Bengt Rutisson and Mikael Gerdin. Bug: https://bugs.openjdk.java.net/browse/JDK-8038925 Webrev: http://cr.openjdk.java.net/~aeriksso/8038925/webrev.03/ Reg

RFR: 8041980 + 6461635

2014-08-28 Thread Andreas Eriksson
Summary: This is a backport of a change that fixes problems with jvmstat, where it can't find local JVM processes. Also backported is a change that improves tests related to this area. I could not run the regression tests on solaris_sparcv9 or solaris_x64 due to a JPRT limitation. All other pl

Re: RFR: 8041980 + 6461635

2014-08-29 Thread Andreas Eriksson
affan Larsen wrote: Looks good! (not a Reviewer for jdk7u) Thanks, /Staffan On 28 aug 2014, at 16:43, Andreas Eriksson mailto:andreas.eriks...@oracle.com>> wrote: Summary: This is a backport of a change that fixes problems with jvmstat, where it can't find local JVM processes. Al

Re: RFR: 8041980 + 6461635

2014-09-10 Thread Andreas Eriksson
Any jdk7u Reviewers out there that could take a look at this? Regards, Andreas On 2014-08-29 13:20, Andreas Eriksson wrote: Thanks Staffan! Could I get a Reviewer too look at this as well please? I got a question why the changesets didn't apply cleanly to jdk7, here is a summary:

Re: RFR: 8041980 + 6461635

2014-10-02 Thread Andreas Eriksson
Thanks Alan, it's greatly appreciated. Regards, Andreas On 2014-10-02 05:48, Alan Bateman wrote: On 29/08/2014 04:20, Andreas Eriksson wrote: Thanks Staffan! Could I get a Reviewer too look at this as well please? I got a question why the changesets didn't apply cleanly to jdk7

[7u-dev] Request for approval for CR 8041979 + 8041980 + 6461635

2014-10-02 Thread Andreas Eriksson
Hi, Requesting push approval for backports of three fixes. These fixes add support for the jdk to get which directory the JVM is using as its temporary directory. This is needed for example so that jvmstat and related tools can find local JVM processes. The changes were reviewed by Staffan Lar

[7u-dev] Request for approval for CR 8041979 + 8041980 + 6461635

2014-10-09 Thread Andreas Eriksson
Hi, Could someone please take a look at this? Regards, Andreas On 2014-10-02 17:49, Andreas Eriksson wrote: Hi, Requesting push approval for backports of three fixes. These fixes add support for the jdk to get which directory the JVM is using as its temporary directory. This is needed for

RFR: JDK7 backport of 8020675 - invalid jar file in the bootclasspath could lead to jvm fatal error

2014-11-06 Thread Andreas Eriksson
Hi, Could someone please review this jdk7 backport of JDK-8020675 please. Summary: invalid jar file in the bootclasspath could lead to jvm fatal error removed offending EXCEPTION_MARK calls and code cleanup One code change necessary for the ba

[7u-dev] Request for approval for CR 8020675 - invalid jar file in the bootclasspath could lead to jvm fatal error

2014-11-10 Thread Andreas Eriksson
Hi, I'd like approval for backporting JDK-8020675 - invalid jar file in the bootclasspath could lead to jvm fatal error. Backport reviewed by Calvin Cheung. Webrev: http://cr.openjdk.java.net/~aeriksso/8020675/webrev.01/ Review thread: http:

Re: [7u-dev] Request for approval for CR 8020675 - invalid jar file in the bootclasspath could lead to jvm fatal error

2014-11-10 Thread Andreas Eriksson
13:04, Andreas Eriksson wrote: Hi, I'd like approval for backporting JDK-8020675 <https://bugs.openjdk.java.net/browse/JDK-8020675> - invalid jar file in the bootclasspath could lead to jvm fatal error. Backport reviewed by Calvin Cheung. Webrev: http://cr.openjdk.java.net/~aeriksso/8

[7u-dev] Request for approval for CR 8058583: Remove CompilationRepeat

2014-11-14 Thread Andreas Eriksson
Hi, I'd like approval for a jdk7 backport of CR 8058583: Remove CompilationRepeat Backport reviewed by Nils Eliasson. Webrev: http://cr.openjdk.java.net/~aeriksso/8058583/webrev.00/ Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-November/016281.html Regards,