Code review request: 7190089: NMT ON: NMT failed assertion on thread's stack base address

Karen Kinnear karen.kinnear at oracle.com
Thu Aug 30 10:41:24 PDT 2012


Zhengyu.

Code looks good - and this looks like a clean solution. Also, thank you for the additional
assertions and the renaming to stack_low_addr.

Ship it!
Karen

On Aug 30, 2012, at 1:11 PM, Zhengyu Gu wrote:

> That will be great!
> 
> Thanks,
> 
> -Zhengyu
> 
> On 8/30/2012 12:53 PM, Vladimir Kozlov wrote:
>> Looks good. If you want, you can push it into hotspot-comp since runtime repo is closed already as I understand.
>> 
>> Thanks,
>> Vladimir
>> 
>> Zhengyu Gu wrote:
>>> This is a Solaris only bug. When stack size limit is set to unlimited (ulimit -s unlimited), primordial thread's stack size has to be further adjusted before native memory tracker can record its base and size.
>>> 
>>> CR: http://bugs.sun.com/view_bug.do?bug_id=7190089
>>> Webrev: http://cr.openjdk.java.net/~zgu/7190089/webrev.00/
>>> 
>>> 
>>> Thanks,
>>> 
>>> -Zhengyu
>>> 



More information about the hotspot-dev mailing list