<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Quick question:  why is is called <b style="color: blue;">WB_GetThreadFullStackSize</b>?  What defines “full”?<div><br><div><div>On May 21, 2014, at 7:28 AM, Kirill Shirokov <<a href="mailto:kirill.shirokov@oracle.com">kirill.shirokov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hi Vladimir,<br><br>Thanks for reviewing. Please see the fixed diff:<br><a href="http://cr.openjdk.java.net/~iignatyev/kshirokov/8032970/webrev.04/">http://cr.openjdk.java.net/~iignatyev/kshirokov/8032970/webrev.04/</a><br><br>I verified it with -Xint, -Xcomp and -Xmixed on all platforms (32- and 64-bit).<br><br>Best regards,<br>Kirill<br><br>5/14/2014 10:56 PM, Vladimir Kozlov пишет:<br><blockquote type="cite">Thank you explanation, Kirill<br><br>WBStackSize.java: please, use more descriptive names for variables:<br><br>fss < tss * 0.9 || fss > tss<br><br>You may want to run WBStackSize with -Xint. With compiled code stack consumption is different. jtreg test could be run with -Xcomp by harness (during Nightly), please, test such configurations.<br><br>Thanks,<br>Vladimir<br><br>On 5/14/14 9:29 AM, Kirill Shirokov wrote:<br><blockquote type="cite">Thanks to all reviewers!<br><br>Could I ask to reivew the fixed patch:<br>http://cr.openjdk.java.net/~iignatyev/kshirokov/8032970/webrev.03/<br><br>Bug report:<br>https://bugs.openjdk.java.net/browse/JDK-8032970<br><br>The new functions will be used in the existing JSR292 tests -- we see a<br>lot of failures due to StackOverflowError.<br><br>Best regards,<br>Kirill<br><br>On 02/04/2014 10:03 PM, Vladimir Kozlov wrote:<br><blockquote type="cite">Kirill,<br><br>You should include link to the bug report and add more explanation<br>what this changes for. Will new functions be used in new or existing<br>jsr292 tests?<br><br>test/testlibrary_tests directory is used for classes used by other<br>tests. I think you need to move new test WBStackSize.java into<br>test/runtime directory.<br><br>Thanks,<br>Vladimir<br><br>On 2/4/14 6:43 AM, Kirill Shirokov wrote:<br><blockquote type="cite">Hi,<br><br>Please review following webrev, which adds stack size checking methods<br>to WhiteBox API:<br><br>http://cr.openjdk.java.net/~iignatyev/kshirokov/8032970/webrev.00/<br><http://cr.openjdk.java.net/%7Eiignatyev/kshirokov/8032970/webrev.00/><br><br>Thank you,<br>Kirill<br><br><br><br></blockquote></blockquote><br></blockquote></blockquote><br></blockquote></div><br></div></body></html>