A little modification to error message (original) (raw)

Chris Hegarty chris.hegarty at oracle.com
Mon Jul 2 00:57:04 PDT 2012


Thanks Sean, looks fine to me.

-Chris

Sean Chou <zhouyx at linux.vnet.ibm.com> wrote:

Hello,

Any one would like to take a look again ? ---------- Forwarded message ---------- From: Sean Chou <zhouyx at linux.vnet.ibm.com> Date: Tue, Jun 26, 2012 at 1:03 PM Subject: Re: A little modification to error message To: Chris Hegarty <chris.hegarty at oracle.com> Cc: net-dev at openjdk.java.net

Hi Chris, They are not all native OOMs, I made a new one which includes all of them. Please take a look. webrev: http://cr.openjdk.java.net/~zhouyx/OJDK-528/webrev.01/ On Thu, Jun 21, 2012 at 5:23 PM, Chris Hegarty <chris.hegarty at oracle.com> wrote: Sean,

The updated error messages look fine to me. Are these all the possibly native OOM throw sites in the networking area, or just some you came across? -Chris.

On 21/06/2012 06:41, Sean Chou wrote: Hi all, We did some modification in these error messages because of some user feedback “It is confusing to realize if the OOM is from java heap or native heap”. So these error messages are modified from "heap allocation failed" to "Native heap allocation failed". And in some places a little more to help locate to functions like "send buffer heap allocation failed". Would any one like to take a look? webrev: http://cr.openjdk.java.net/**zhouyx/OJDK-528/webrev.00/<http://cr.openjdk.java.net/zhouyx/OJDK-528/webrev.00/> . -- Best Regards, Sean Chou

-- Best Regards, Sean Chou -- Best Regards, Sean Chou



More information about the net-dev mailing list