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

[Christian Tornqvist]

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