<div dir="ltr">Hi all,<div><br></div><div>I added Per's fix into the webrev:</div><div><a href="http://cr.openjdk.java.net/~jcbeyler/8063110/webrev.02/">http://cr.openjdk.java.net/~jcbeyler/8063110/webrev.02/</a><br></div><div><br></div><div>(I also put you Thomas as reviewer, I was not sure Per wanted to be put as reviewer, let me know).</div><div><br></div><div>Not sure if you wanted to wait or not to push it but at least this version would be the version that does not run with ZGC.</div><div><br></div><div>Thanks for your help,<br></div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 13, 2018 at 5:54 AM 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">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 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 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 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 have<br>
>> the property "properly configured large pages" at this time.<br>
>><br>
>> Thanks,<br>
>>    Thomas<br>
>><br>
> <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>