RFR: 8013120 NMT: Kitchensink crashes with assert(next_region == NULL || !next_region->is_committed_region()) failed: Sanity check

[Zhengyu Gu]

OK, I will move the test before check in.

Thanks,

-Zhengyu

On 4/25/2013 4:31 PM, Christian Tornqvist wrote:
> Hi Zhengyu,
>
> The test should be placed in test/runtime/NMT together with the other NMT tests :)
>
> Thanks,
> Christian
>
> -----Original Message-----
> From: Zhengyu Gu [mailto:zhengyu.gu at oracle.com]
> Sent: den 25 april 2013 16:23
> To: hotspot-dev at openjdk.java.net
> Cc: Christian Törnqvist
> Subject: Re: RFR: 8013120 NMT: Kitchensink crashes with assert(next_region == NULL || !next_region->is_committed_region()) failed: Sanity check
>
>
> Updated webrev to include regression test:
> http://cr.openjdk.java.net/~zgu/8013120/webrev.01/
> <http://cr.openjdk.java.net/%7Ezgu/8013120/webrev.01/>
>
> Thanks Christian for writing the test.
>
> -Zhengyu
>
>
> On 4/25/2013 2:18 PM, Zhengyu Gu wrote:
>> NMT expects all committed regions are uncommitted before the whole
>> reserved region can be released. It was the case until 8011268.
>>
>> This change allows NMT to handle the scenario that virtual memory is
>> being released while there are committed segments within the reserved
>> region.
>>
>> It also fixed a missing virtual memory tag when ArrayAllocator
>> allocates from virtual memory.
>>
>> Bug: http://bugs.sun.com/view_bug.do?bug_id=8013120
>> Webrev: http://cr.openjdk.java.net/~zgu/8013120/webrev.00/
>> <http://cr.openjdk.java.net/%7Ezgu/8013120/webrev.00/>
>>
>> Thanks,
>>
>> -Zhengyu
>