<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 25, 2020 at 10:15 PM Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<span class="gmail_default" style="font-size:small">...</span> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
> > >     s390x:<br>
> > >      ❌ LTP<br>
> ><br>
> > @Li, any thoughts on hugetlb TBROKs here? I thought you said<br>
> > 'request_hugepages'<br>
> > was not supposed to be hard test requirement.<br>
> ><br>
> <br>
> The hugemmap05.c doesn't goes the new 'request_hugepages' way in the huge<br>
> page reserving. it tries to reserve 128 hpages and overcommits 192 hpages<br>
> in hard coding, and probably touch the memory available uplimit on s390x.<br>
> static long size = 128, length = 384;<br>
> ...<br>
> addr = SAFE_MMAP(ADDR, (length / 2 * hugepagesize), PROTECTION, FLAGS, fd,<br>
> 0);<br>
> <br>
> We could convert this to use the new request_hugepages API and do the<br>
> requested number of huge page checking at setup() to make sure the system<br>
> only running in a reasonable situation.<br>
<br>
Sorry, I didn't make it clear I was talking about hugemmap06. The patch is OK,<br>
since it makes both use request_hugepages API. But should we also do something<br>
with this TBROK?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It seems unkind and useless to exit with TBROK when failed to reserve with 80% max_hpages again. I'm thinking if it could be directly replaced by TCONF to skip the whole test?</div><div class="gmail_default" style="font-size:small"><br></div>--- a/lib/tst_hugepage.c<br>+++ b/lib/tst_hugepage.c<br>@@ -43,7 +43,7 @@ unsigned long tst_request_hugepages(unsigned long hpages)<br>        SAFE_FILE_PRINTF(PATH_NR_HPAGES, "%lu", tst_hugepages);<br>        SAFE_FILE_SCANF(PATH_NR_HPAGES, "%lu", &val);<br>        if (val != tst_hugepages)<br>-               tst_brk(TBROK, "nr_hugepages = %lu, but expect %lu", val, tst_hugepages);<br>+               tst_brk(TCONF, "Not enough hugepages for testing.");<br> <br>        tst_res(TINFO, "%lu hugepage(s) reserved", tst_hugepages);<br> out:<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
    tst_hugepage.c:46: BROK: nr_hugepages = 171, but expect 255<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Regards,<br>Li Wang<br></div></div></div></div></div>