Re: RFR 9: 8087286 Need a way to handle control-C and possibly some other signals

2016-02-05 Thread David Holmes
On 5/02/2016 2:24 PM, Mandy Chung wrote: On Feb 4, 2016, at 5:05 PM, David Holmes wrote: Java hasn't needed a public signal handling API for the last 20 years. It's only getting one now because modularity is forcing us to cut-off sun.misc.Signal from the few use-cases that do need it. And w

Re: RFR JDK-8141491: Unaligned memory access in Bits.c

2016-02-05 Thread Mikael Vidstedt
I fully agree that moving the arguments checking up to Java makes more sense, and I've prepared new webrevs which do exactly that, including changes to address the other feedback from David, John and others: hotspot: http://cr.openjdk.java.net/~mikael/webrevs/8141491/webrev.04/hotspot/webrev

Re: RFR: JDK-8031767 Support system or alternative implementations of zlib

2016-02-05 Thread Alan Bateman
On 05/02/2016 18:55, Xueming Shen wrote: Hi Please help codereview the change to build the jdk9 runtime to use the system zlib on Solaris and Linux platforms by default. Issue: https://bugs.openjdk.java.net/browse/JDK-8031767 Webrev: http://cr.openjdk.java.net/~sherman/8031767/webrev/ I'm hap

RFR: JDK-8031767 Support system or alternative implementations of zlib

2016-02-05 Thread Xueming Shen
Hi Please help codereview the change to build the jdk9 runtime to use the system zlib on Solaris and Linux platforms by default. Issue: https://bugs.openjdk.java.net/browse/JDK-8031767 Webrev: http://cr.openjdk.java.net/~sherman/8031767/webrev/ Background info: Compression is heavily used in

RFR: JDK-8135259: InetAddress.getAllByName only reports "unknown error" instead of actual cause

2016-02-05 Thread Ramanand Patil
Hi all Please review the fix for bug: https://bugs.openjdk.java.net/browse/JDK-8135259 Bug Description: Attempts to resolve a host unknown to the DNS server cause an UnknownHostException stating "unknown error" instead of "Name or service not known". Webrev: http://cr.openjdk.java.net/

Re: [9] RFR (S): 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization

2016-02-05 Thread Vladimir Ivanov
Thanks, John. On 2/4/16 9:52 PM, John Rose wrote: Looks good. Except I don't recall how is_inline interacts, if at all, with customization. It's not clear what is being gated, and which optimizations get let through. Inlining through MH::invokeBasic doesn't happen when there's @DontInline on

Re: RFR: 8071368 Use more concrete types for NamedFunction constants in the code

2016-02-05 Thread Vladimir Ivanov
Proposed fix looks good. Quoting John: "The use of erased types (any ref => Object) in the MH runtime is an artifact of bootstrapping difficulties, early in the project. I hope it is not necessary any more." Best regards, Vladimir Ivanov On 2/5/16 4:29 PM, Paul Sandoz wrote: Hi Shilpi, In

Re: RFR: JDK-8144144 - ORB destroy() leaks filedescriptors after unsuccessful connection

2016-02-05 Thread Seán Coffey
Looks fine to me Mark. Regards, Sean. On 29/01/16 01:10, Mark Sheppard wrote: Hi, based on feedback from reviews the current version of this fix can be found at http://cr.openjdk.java.net/~msheppar/8144144/webrev.05/ please oblige and review regards Mark On 22/01/2016 11:14, Seán Coffe

Re: RFR: 8071368 Use more concrete types for NamedFunction constants in the code

2016-02-05 Thread Paul Sandoz
Hi Shilpi, In principle this change is ok, however these methods are kind of special because they are called by code generated from LambdaForms with erased types. IIUC the casts can serve as belts-and-braces checks just in case that generated code passes something that is not of the correct typ

Re: RFR 8138578:MethodHandles.Lookup.findSpecial() Javadoc fails to consider static methods

2016-02-05 Thread Paul Sandoz
> On 29 Jan 2016, at 12:43, shilpi rastogi wrote: > > Hi All, > > Please review the updated patch- > > http://cr.openjdk.java.net/~srastogi/8138578/webrev.01/ > > I verified Lookup.unreflectSpecial() also throws > java.lang.IllegalAccessException so updated the JavaDoc. > +1 Paul.

Re: RFR JDK-8141491: Unaligned memory access in Bits.c

2016-02-05 Thread Paul Sandoz
Hi, Nice use of C++ templates :-) Overall looks good. I too would prefer if we could move the argument checking out, perhaps even to the point of requiring callers do that rather than providing another method, for example for Buffer i think the arguments are known to be valid? I think in eith

Re: RFR 9: 8087286 Need a way to handle control-C and possibly some other signals

2016-02-05 Thread Thomas Stüfe
On Fri, Feb 5, 2016 at 3:00 AM, Stuart Marks wrote: > > > On 2/4/16 5:05 PM, David Holmes wrote: > >> The initial task was to provide a replacement for sun.misc.Signal. >>> Sun.misc.Signal provides to Java the same set of >>> signals the VM is able to handle and exposes via the JVM_findSignal, >>