<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<font face="Times New Roman, Times, serif">Derek,<br>
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~drwhite/8149405/webrev.01/src/share/vm/oops/methodData.cpp.udiff.html">http://cr.openjdk.java.net/~drwhite/8149405/webrev.01/src/share/vm/oops/methodData.cpp.udiff.html</a><br>
</font><br>
<blockquote type="cite">
<pre> void MethodData::clean_method_data(BoolObjectClosure* is_alive) {
<span class="new">+ ResourceMark rm;</span>
for (ProfileData* data = first_data();
is_valid(data);
data = next_data(data)) {
data->clean_weak_klass_links(is_alive);
}
</pre>
</blockquote>
<font face="Times New Roman, Times, serif"><br>
The clean_weak_klass_links() above has a ResourceMark in it.<br>
<br>
<blockquote type="cite">void
DataLayout::clean_weak_klass_links(BoolObjectClosure* cl) {<br>
ResourceMark m;<br>
data_in()->clean_weak_klass_links(cl);<br>
}</blockquote>
<br>
It's not clear to me where space is being allocated in clean_method_data()
so<br>
that the ResourceMark helps. My understanding of ResourceMarks is
thin<br>
so don't assume there is anything deep about this question.<br>
<br>
Jon<br>
<br>
<br>
</font><br>
<div class="moz-cite-prefix">On 03/23/2016 08:02 AM, Derek White
wrote:<br>
</div>
<blockquote cite="mid:56F2B016.30304@oracle.com" type="cite">This is
a very small fix that adds ResourceMarks to
MethodData::clean_method_data() (and two similar functions
nearby).
<br>
<br>
Basically an iteration over all classes in all methods in all
classes was occurring in one ResourceMark during a full gc, so we
occasionally ran out of malloc space.
<br>
<br>
Once again, x86 builds running on Win64 are the "canary in the
coal mine" for these kinds of temp. memory leaks, so it's a great
test case even if not a very realistic one!
<br>
<br>
BUG: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8149405">https://bugs.openjdk.java.net/browse/JDK-8149405</a>
<br>
WEBREV: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~drwhite/8149405/webrev.01/">http://cr.openjdk.java.net/~drwhite/8149405/webrev.01/</a>
<br>
TESTS: jprt
<br>
<br>
Thanks for looking,
<br>
<br>
- Derek
<br>
</blockquote>
<br>
</body>
</html>