hg: hsx/hotspot-rt/hotspot: 8016131: nsk/sysdict/vm/stress/chain tests crash the VM in 'entry_frame_is_first()'

2013-07-17 Thread rickard . backman
Changeset: e619a2766bcc Author:rbackman Date: 2013-06-12 11:17 +0200 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/e619a2766bcc 8016131: nsk/sysdict/vm/stress/chain tests crash the VM in 'entry_frame_is_first()' Reviewed-by: jrose, kvn, mgronlun !

RE: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Markus Grönlund
Hi again,   Trying again, could I please have a couple of reviews for this? Many thanks Markus   From: Markus Grönlund Sent: den 16 juli 2013 17:48 To: serviceability-dev@openjdk.java.net; hotspot-runtime-...@openjdk.java.net Subject: RFR(XS): 8020547 : Event based tracing needs a UNICODE

Re: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Bengt Rutisson
Hi Markus, Looks reasonable. Bengt On 7/17/13 12:25 PM, Markus Grönlund wrote: Hi again, Trying again, could I please have a couple of reviews for this? Many thanks Markus From: Markus Grönlund Sent: den 16 juli 2013 17:48 To: serviceability-dev@openjdk.java.net;

Re: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Erik Gahlin
Looks good! (not a Reviewer) /Erik Markus Grönlund skrev 7/16/13 5:48 PM: Greetings, Kindly asking for reviews for the following change in hsx24: Bugid: http://bugs.sun.com/view_bug.do?bug_id=8020547 Webrev: http://cr.openjdk.java.net/~mgronlun/8020547/webrev01/

Re: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Rickard Bäckman
Looks good to me. /R On Jul 17, 2013, at 12:25 PM, Markus Grönlund wrote: Hi again, Trying again, could I please have a couple of reviews for this? Many thanks Markus From: Markus Grönlund Sent: den 16 juli 2013 17:48 To: serviceability-dev@openjdk.java.net;

RFR(S): 8020107: Avoid crashes in WatcherThread

2013-07-17 Thread Rickard Bäckman
Hi all, can I please have reviews for the following change? We are adding a mechanism for avoiding some crashes in the WatcherThread using different OS-specific methods. Webrev: http://cr.openjdk.java.net/~rbackman/8020701/webrev/ Thanks /R

Re: RFR(S): 8020107: Avoid crashes in WatcherThread

2013-07-17 Thread Rickard Bäckman
Sorry everyone, wrong bugnr in the topic. It should be 8020701. /R On Jul 17, 2013, at 1:58 PM, Rickard Bäckman wrote: Hi all, can I please have reviews for the following change? We are adding a mechanism for avoiding some crashes in the WatcherThread using different OS-specific

Re: RFR(S): 8020107: Avoid crashes in WatcherThread

2013-07-17 Thread Rickard Bäckman
Thank you David. /R On Jul 17, 2013, at 2:56 PM, David Simms wrote: Looks good to me. On 17/07/2013 1:58 p.m., Rickard Bäckman wrote: Hi all, can I please have reviews for the following change? We are adding a mechanism for avoiding some crashes in the WatcherThread using

Re: RFR(S): 8020107: Avoid crashes in WatcherThread

2013-07-17 Thread David Simms
Looks good to me. On 17/07/2013 1:58 p.m., Rickard Bäckman wrote: Hi all, can I please have reviews for the following change? We are adding a mechanism for avoiding some crashes in the WatcherThread using different OS-specific methods. Webrev:

Re: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Daniel D. Daugherty
On 7/16/13 9:48 AM, Markus Grönlund wrote: Greetings, Kindly asking for reviews for the following change in hsx24: Bugid: http://bugs.sun.com/view_bug.do?bug_id=8020547 Webrev: http://cr.openjdk.java.net/~mgronlun/8020547/webrev01/ http://cr.openjdk.java.net/%7Emgronlun/8020547/webrev01/

hg: jdk8/tl/langtools: 10 new changesets

2013-07-17 Thread maurizio . cimadamore
Changeset: 44e27378f523 Author:mcimadamore Date: 2013-07-17 14:04 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/44e27378f523 8012242: Lambda compatibility and checked exceptions Summary: Inference variables in 'throws' clause with no constraints should be inferred as

hg: jdk8/tl/langtools: 8020586: Warning produced for an incorrect file

2013-07-17 Thread maurizio . cimadamore
Changeset: 10711bd8bb2d Author:jlahoda Date: 2013-07-17 15:08 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/10711bd8bb2d 8020586: Warning produced for an incorrect file Summary: Always using DeferredLintHandler.immediateHandler when processing import classes

Re: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Karen Kinnear
Markus, Looks good. (Ok, I miss the comment about j.l.Thread :-) ship it! Karen On Jul 16, 2013, at 11:48 AM, Markus Grönlund wrote: Greetings, Kindly asking for reviews for the following change in hsx24: Bugid: http://bugs.sun.com/view_bug.do?bug_id=8020547 Webrev:

Re: RFR(S): 8020107: Avoid crashes in WatcherThread

2013-07-17 Thread Karen Kinnear
Code looks good. Thank you for the assertions and comments. Cleanly done, as usual for your changes Rickard. thanks, Karen On Jul 17, 2013, at 8:22 AM, Rickard Bäckman wrote: Sorry everyone, wrong bugnr in the topic. It should be 8020701. /R On Jul 17, 2013, at 1:58 PM, Rickard Bäckman

Re: RFR(S): 8020107: Avoid crashes in WatcherThread

2013-07-17 Thread Rickard Bäckman
Thank you for the review Karen! /R On Jul 17, 2013, at 5:00 PM, Karen Kinnear wrote: Code looks good. Thank you for the assertions and comments. Cleanly done, as usual for your changes Rickard. thanks, Karen On Jul 17, 2013, at 8:22 AM, Rickard Bäckman wrote: Sorry everyone, wrong

Re: Attach API on Linux and Root

2013-07-17 Thread Andrew Hughes
- Original Message - Thanks for the patch, I think it sounds like a good idea. For consistency, however, we need to do the same modifications to the other platforms (solaris, windows) and preferably add tests for this functionality as well. The patch makes sense to me. While a very

RE: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Markus Grönlund
Thanks Dan!   No technical reason for the *(star) placements in this file, it might be more readable for a human with a space added or something, I don't know, one of those human aspects again called preferences or something.   Cheers Markus   From: Daniel D. Daugherty Sent: den 17 juli

RE: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)

2013-07-17 Thread Markus Grönlund
Many thanks for your reviews!   Markus   From: Karen Kinnear Sent: den 17 juli 2013 17:06 To: Markus Grönlund Cc: serviceability-dev@openjdk.java.net; hotspot-runtime-...@openjdk.java.net Subject: Re: RFR(XS): 8020547 : Event based tracing needs a UNICODE string type (hsx24)   Markus,  

Re: RFR(S): 8020701: Avoid crashes in WatcherThread

2013-07-17 Thread Daniel D. Daugherty
On 7/17/13 5:58 AM, Rickard Bäckman wrote: Hi all, can I please have reviews for the following change? We are adding a mechanism for avoiding some crashes in the WatcherThread using different OS-specific methods. Webrev: http://cr.openjdk.java.net/~rbackman/8020701/webrev/ Thanks /R

Re: RFR(S): 8020701: Avoid crashes in WatcherThread

2013-07-17 Thread Rickard Bäckman
On Jul 17, 2013, at 7:03 PM, Daniel D. Daugherty wrote: On 7/17/13 5:58 AM, Rickard Bäckman wrote: Hi all, can I please have reviews for the following change? We are adding a mechanism for avoiding some crashes in the WatcherThread using different OS-specific methods. Webrev:

hg: jdk8/tl/nashorn: 2 new changesets

2013-07-17 Thread lana . steuck
Changeset: 10a1ab9e20a4 Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/10a1ab9e20a4 Added tag jdk8-b98 for changeset 542b7803f038 ! .hgtags Changeset: 81cbb18d558a Author:lana Date: 2013-07-17 00:36 -0700 URL:

hg: jdk8/tl/jaxws: Added tag jdk8-b98 for changeset b1fb4612a2ca

2013-07-17 Thread lana . steuck
Changeset: 8ef83d4b23c9 Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/8ef83d4b23c9 Added tag jdk8-b98 for changeset b1fb4612a2ca ! .hgtags

hg: jdk8/tl/jdk: 5 new changesets

2013-07-17 Thread lana . steuck
Changeset: 2c26ccf0a85b Author:tbell Date: 2013-07-08 07:20 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2c26ccf0a85b 8012925: [parfait] Missing return value in jdk/src/macosx/native/sun/awt/AWTEvent.m Reviewed-by: katleman, leonidr Contributed-by: petr.pche...@oracle.com

hg: jdk8/tl/corba: Added tag jdk8-b98 for changeset 3370fb6146e4

2013-07-17 Thread lana . steuck
Changeset: 3f67804ab613 Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/3f67804ab613 Added tag jdk8-b98 for changeset 3370fb6146e4 ! .hgtags

hg: jdk8/tl/hotspot: 69 new changesets

2013-07-17 Thread lana . steuck
Changeset: 8c4424890028 Author:amurillo Date: 2013-06-28 02:33 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/hotspot/rev/8c4424890028 8019302: new hotspot build - hs25-b40 Reviewed-by: jcoomes ! make/hotspot_version Changeset: 8cff1de240de Author:zgu Date: 2013-06-25

hg: jdk8/tl/jaxp: 2 new changesets

2013-07-17 Thread lana . steuck
Changeset: adf49c3ef83c Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/adf49c3ef83c Added tag jdk8-b98 for changeset 15e5bb51bc0c ! .hgtags Changeset: 74ec7b48e3be Author:lana Date: 2013-07-17 00:34 -0700 URL:

hg: jdk8/tl: 2 new changesets

2013-07-17 Thread lana . steuck
Changeset: 0d0c983a817b Author:tbell Date: 2013-07-09 08:35 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/rev/0d0c983a817b 8009315: F# on PATH breaks Cygwin tools (mkdir, echo, mktemp ...) Reviewed-by: erikj ! common/autoconf/generated-configure.sh !

hg: jdk8/tl/langtools: 3 new changesets

2013-07-17 Thread lana . steuck
Changeset: bdeef606be8e Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/bdeef606be8e Added tag jdk8-b98 for changeset ce5a90df517b ! .hgtags Changeset: 39ec5d8a691b Author:mcimadamore Date: 2013-07-11 14:07 +0100 URL:

hg: hsx/hotspot-rt/hotspot: 8017498: JVM crashes when native code calls sigaction(sig) where sig=0x20

2013-07-17 Thread yumin . qi
Changeset: 732af649bc3a Author:ccheung Date: 2013-07-17 12:22 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/732af649bc3a 8017498: JVM crashes when native code calls sigaction(sig) where sig=0x20 Summary: Added (sig MAXSIGNUM) check in jsig.c Reviewed-by:

hg: jdk8/tl/langtools: 8014636: TestLiteralCodeInPre fails on windows

2013-07-17 Thread jonathan . gibbons
Changeset: 80e75aa6a707 Author:jjg Date: 2013-07-17 18:18 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/80e75aa6a707 8014636: TestLiteralCodeInPre fails on windows Reviewed-by: ksrini ! src/share/classes/com/sun/tools/doclets/formats/html/HtmlDocletWriter.java !

hg: jdk8/tl/langtools: 8020664: doclint gives incorrect warnings on normal package statements

2013-07-17 Thread jonathan . gibbons
Changeset: 1476d54fdc61 Author:jjg Date: 2013-07-17 19:16 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/1476d54fdc61 8020664: doclint gives incorrect warnings on normal package statements Reviewed-by: mcimadamore ! src/share/classes/com/sun/tools/doclint/DocLint.java

hg: jdk8/tl/langtools: 8020313: doclint doesn't reset HTML anchors correctly

2013-07-17 Thread jonathan . gibbons
Changeset: 1e533c1bfb01 Author:jjg Date: 2013-07-17 19:12 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/1e533c1bfb01 8020313: doclint doesn't reset HTML anchors correctly Reviewed-by: mcimadamore ! src/share/classes/com/sun/tools/doclint/Checker.java +

hg: hsx/hotspot-rt/hotspot: 2 new changesets

2013-07-17 Thread jiangli . zhou
Changeset: 825e6cb66923 Author:jiangli Date: 2013-07-17 18:06 -0400 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/825e6cb66923 8020309: Eliminate InstanceKlass::_cached_class_file_len. Summary: Use JvmtiCachedClassFileData. Reviewed-by: iklam, sspitsyn, dcubed !