Code review request: three native memory tracking related bugs (original) (raw)
Zhengyu Gu zhengyu.gu at oracle.com
Mon Jul 16 07:52:34 PDT 2012
- Previous message: Code review request: three native memory tracking related bugs
- Next message: Code review request: three native memory tracking related bugs
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Vladmire,
Thanks for reviewing.
7182543: NMT ON: Aggregate a few NMT related bugs CRhttp://bugs.sun.com/bugdatabase/viewbug.do?bugid=7182543 Webrev: http://cr.openjdk.java.net/~zgu/7182543/webrev.00/ I think the next should be guarantee() instead of check since all MemRecorders should be deleted in preceding code: if (MemRecorder::instancecount == 0) { Not necessary true, it depends on when "shutdown_pending" flag is visible to a particular thread, that thread can still generate pending recorders when they are overflown.
Thanks,
-Zhengyu
Vladimir
- Fixed generationsinused calculation - Wait MemRecorder instance count to drop to zero before completely shutdown NMT - Added assertion for JavaThread in threadblocked state.
Thanks, -Zhengyu
- Previous message: Code review request: three native memory tracking related bugs
- Next message: Code review request: three native memory tracking related bugs
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]