<div dir="ltr">Hi all,<div><br></div><div>Here is the webrev with all the reviewers added:</div><div><a href="http://cr.openjdk.java.net/~jcbeyler/8063110/webrev.03/">http://cr.openjdk.java.net/~jcbeyler/8063110/webrev.03/</a><br></div><div><br></div><div>Let me know if I can/should do anything else for this webrev.</div><div><br></div><div>Thanks for all your help,</div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 13, 2018 at 11:04 PM Per Liden <<a href="mailto:per.liden@oracle.com">per.liden@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
On 08/13/2018 06:22 PM, JC Beyler wrote:<br>
> Hi all,<br>
> <br>
> I added Per's fix into the webrev:<br>
> <a href="http://cr.openjdk.java.net/~jcbeyler/8063110/webrev.02/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8063110/webrev.02/</a><br>
> <br>
> (I also put you Thomas as reviewer, I was not sure Per wanted to be put <br>
> as reviewer, let me know).<br>
<br>
Sure, you can put me as reviewer.<br>
<br>
/Per<br>
<br>
> <br>
> Not sure if you wanted to wait or not to push it but at least this <br>
> version would be the version that does not run with ZGC.<br>
> <br>
> Thanks for your help,<br>
> Jc<br>
> <br>
> On Mon, Aug 13, 2018 at 5:54 AM Per Liden <<a href="mailto:per.liden@oracle.com" target="_blank">per.liden@oracle.com</a> <br>
> <mailto:<a href="mailto:per.liden@oracle.com" target="_blank">per.liden@oracle.com</a>>> wrote:<br>
> <br>
>     On 08/13/2018 11:41 AM, Thomas Schatzl wrote:<br>
>      > Hi,<br>
>      ><br>
>      >    while testing this patch before push it seems like the problem<br>
>     is ZGC<br>
>      > which does not fall back to using small pages (which I did not know).<br>
>      ><br>
>      >> [5.048s][error][gc,init] Not enough space available on the backing<br>
>      > filesystem to hold the current max<br>
>      >> [5.048s][error][gc,init] Java heap size (128M). Forcefully lowering<br>
>      > max Java heap size to 0M (0%).<br>
>      >> Error occurred during initialization of VM<br>
>      >> java.lang.OutOfMemoryError: Java heap too small<br>
>      ><br>
>      > I will hold off pushing this before this issue has been resolved<br>
>     in one<br>
>      > way or the other.<br>
> <br>
>     UseNUMA is enabled by default in ZGC, so it's tested frequently anyway.<br>
>     I guess the easiest solution is to just not run this test when ZGC is<br>
>     enabled, something like this:<br>
> <br>
>     diff --git a/test/hotspot/jtreg/gc/TestNUMAPageSize.java<br>
>     b/test/hotspot/jtreg/gc/TestNUMAPageSize.java<br>
>     --- a/test/hotspot/jtreg/gc/TestNUMAPageSize.java<br>
>     +++ b/test/hotspot/jtreg/gc/TestNUMAPageSize.java<br>
>     @@ -27,7 +27,8 @@<br>
>         * @summary Make sure that start up with NUMA support does not cause<br>
>     problems.<br>
>         * @bug 8061467<br>
>         * @requires vm.opt.AggressiveOpts != true<br>
>     - * @run main/othervm -Xmx128m -XX:+UseNUMA TestNUMAPageSize<br>
>     + * @requires vm.gc != "Z"<br>
>     + * @run main/othervm -Xmx128m -XX:+UseNUMA -XX:+UseLargePages<br>
>     TestNUMAPageSize<br>
>         */<br>
> <br>
>        public class TestNUMAPageSize {<br>
> <br>
> <br>
>     cheers,<br>
>     Per<br>
> <br>
> <br>
>      ><br>
>      > Thanks,<br>
>      >    Thomas<br>
>      ><br>
>      > On Mon, 2018-08-13 at 10:23 +0200, Thomas Schatzl wrote:<br>
>      >> Hi,<br>
>      >><br>
>      >> On Mon, 2018-08-13 at 09:58 +0200, Per Liden wrote:<br>
>      >>> Hmm, I suspect this might fail on many Linux systems, since it<br>
>      >>> might require configuring the huge page pool with enough pages.<br>
>      >><br>
>      >>    from my understanding -XX:+UseLargePages will just give some<br>
>      >> warnings<br>
>      >>   at startup if it can not allocate pages. It will not fail.<br>
>      >><br>
>      >> Assuming that the intent is to check whether when trying to use<br>
>     large<br>
>      >> pages (and the machine is properly configured), does not in any way<br>
>      >> make the VM crash, the warnings can be ignored.<br>
>      >><br>
>      >> I am not aware there is a way to run tests on machines only that<br>
>     have<br>
>      >> the property "properly configured large pages" at this time.<br>
>      >><br>
>      >> Thanks,<br>
>      >>    Thomas<br>
>      >><br>
>      ><br>
> <br>
> <br>
> <br>
> -- <br>
> <br>
> Thanks,<br>
> Jc<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>