RFR: 8011661: Insufficient memory message says "malloc" when sometimes it should say "mmap"

[David Holmes]

Hi Calvin,

This would be far less disruptive if you used a default parameter so 
that the majority of the call sites did not need to be changed. Also the 
use of an enum seems excessive unless you think we will discover other 
OOM reasons ?

David

On 30/04/2013 11:37 AM, Calvin Cheung wrote:
> Zhengyu,
>
> Thanks for your review.
> Since the fix for 8011773 (Some tests on Interned String crashed JVM
> with OOM) has been pushed today, I needed to update the fix to include
> the change in oop.cpp. Webrev is at the same location:
> http://cr.openjdk.java.net/~ccheung/8011661/webrev/
>
> Also, I need one more reviewer so that this fix can be pushed.
>
> I ran the vm.quick.testlist on linux_x64, there's no new failures
> compared with the promoted build.
>
> thanks,
> Calvin
>
> On 4/29/2013 7:01 AM, Zhengyu Gu wrote:
>> Good to me.
>>
>> Thanks,
>>
>> -Zhengyu
>>
>> On Apr 26, 2013, at 6:32 PM, Calvin Cheung wrote:
>>
>>> Problem: needs a way to differentiate the 2 OOM error cases - malloc
>>> or mmap
>>>
>>> Fix:
>>> Adding an enum parameter to the vm_exit_out_of_memory macro and the
>>> report_vm_out_of_memory
>>> function so that the caller can indicate whether it's a malloc or
>>> mmap error.
>>> Main changes are in debug.[cpp | hpp] and vmError.[cpp | hpp].
>>> Other changes are mainly for passing in the additional enum when
>>> calling vm_exit_out_of_memory.
>>>
>>> Test coverage:
>>>     jprt
>>>
>>> webrev: http://cr.openjdk.java.net/~ccheung/8011661/webrev/
>>> jbs: https://jbs.oracle.com/bugs/browse/JDK-8011661
>>> bugs.sun.com: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8011661
>>>
>>> thanks,
>>> Calvin
>