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

[Calvin Cheung]

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