<div dir="ltr">Chris, Vladimir, thank you for your reviews. <div>I followed your suggestions and added other useful statistics. The output now looks like this:<div><br></div><div><div>Method flushing statistics:</div><div>

  Total sweep time: 152.649ms</div><div>  Peak sweep time: 15.979ms</div><div>  Peak sweep fraction time: 3.209ms</div><div>  Total number of stack traversals: 25</div><div>  Total number of flushed methods: 1320</div><div>

  Total size of flushed methods: 7188Kb</div></div><div><br></div><div>I did not include the other values, because as Vladimir already said, they are removed by <span style="font-family:arial,sans-serif;font-size:13px">8020151.</span></div>

<div><br></div><div>Here is the new webrev:</div><div><a href="http://cr.openjdk.java.net/~anoll/8025277/webrev.01/" target="_blank">http://cr.openjdk.java.net/~anoll/8025277/webrev.01/</a><br></div><div><br></div><div>Thanks again for your review.</div>

<div><br></div><div>Regards,</div><div>Tobias</div><div><br></div><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/27 Vladimir Kozlov <span dir="ltr">&lt;<a href="mailto:vladimir.kozlov@oracle.com" target="_blank">vladimir.kozlov@oracle.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I agree with Chris, we are collecting those counters anyway. I also like Chris&#39;s idea about collecting freed sizes. Note, some counters will be removed by 8020151 changes:<br>

<br>
<a href="http://cr.openjdk.java.net/~anoll/8020151/webrev.03/" target="_blank">http://cr.openjdk.java.net/~<u></u>anoll/8020151/webrev.03/</a><br>
<br>
Thanks,<br>
Vladimir<div class="im"><br>
<br>
On 9/26/13 2:13 PM, Chris Plummer wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi Tobias,<br>
<br>
The one comment I would have is that the statistics you produce are all<br>
timing related, but the option name implies you would get a bit more<br>
than this. For example, maybe something like a count of the number of<br>
methods flushed (and maybe total size). So I would suggest either rename<br>
to PrintMethodFlushingTimingStati<u></u>stics (and also rename the print<br>
method), or add some other useful statistics to the output. Probably all<br>
the counters located below the &quot; Stat counters&quot; comment are good candidates.<br>
<br>
best regards,<br>
<br>
Chris<br>
<br>
On 9/25/13 12:20 AM, Tobias Hartmann wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi,<br>
<br>
Please review the following small patch.<br>
<br>
bug:�<a href="https://bugs.openjdk.java.net/browse/JDK-8025277" target="_blank">https://bugs.openjdk.<u></u>java.net/browse/JDK-8025277</a><br>
webrev:�<a href="http://cr.openjdk.java.net/~anoll/8025277/webrev.00/" target="_blank">http://cr.openjdk.<u></u>java.net/~anoll/8025277/<u></u>webrev.00/</a><br></div>
&lt;<a href="http://cr.openjdk.java.net/%7Eanoll/8025277/webrev.00/" target="_blank">http://cr.openjdk.java.net/%<u></u>7Eanoll/8025277/webrev.00/</a>&gt;<div class="im"><br>
<br>
I added the diagnostic VM option�<u></u>PrintMethodFlushingStatistics to<br>
print statistics about the NMethodSweeper (sweep times) on exit. This<br>
is useful to determine performance bottlenecks of applications that<br>
heavily compile code (e.g. nashorn).<br>
<br>
Thanks in advance!<br>
<br>
Best regards,<br>
Tobias<br>
<br>
<br>
By the way, I&#39;m Tobias Hartmann, a student from Switzerland currently<br>
working on my Master Thesis &quot;Code Cache Optimizations for Dynamically<br>
Compiled Languages&quot; on supervision of Albert Noll.<br>
<br>
<br>
</div></blockquote>
<br>
</blockquote>
</blockquote></div><br></div>