<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi all,</div><div><br></div><div>Could anyone review this logging change for oopstorage?</div><div>Webrev: <a href="http://cr.openjdk.java.net/~jcbeyler/8210724/webrev.00/">http://cr.openjdk.java.net/~jcbeyler/8210724/webrev.00/</a><br></div>Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8210724">https://bugs.openjdk.java.net/browse/JDK-8210724</a></div><div dir="ltr"><br></div><div dir="ltr">This is similar to the webrev I previously sent for <a href="https://bugs.openjdk.java.net/browse/JDK-8210562">https://bugs.openjdk.java.net/browse/JDK-8210562</a>. Kim mentioned it might be reasonable to improve the logging in oopStorage.cpp. We discussed and decided to create a separate issue for improving the logging.</div><div dir="ltr"><br></div><div dir="ltr">The patch changes the logging level for [oopstorage,ref] to trace, and consistently uses debug level for [oopstorage,blocks] and unusual events, and uses log_info() for printing statistics. I think this would make the logging for oopstorage more user-friendly.<br></div><div dir="ltr"><br></div><div>Below is comparison of lines of messages:</div><div>$ java -Xlog:oopstorage*=<level> -Xms500m -Xmx500m -jar dacapo-9.12-bach.jar tradesoap<br></div><font face="monospace, monospace">                info   | debug  | trace <br>without patch:  107934 | 112990 | 112701</font></div><div><font face="monospace, monospace">with patch:       1694 |   5247 | 112241</font></div><div dir="ltr"><div dir="ltr"><br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr">-Man</div></div></div></div></div></div></div></div>