<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">[Cc' LTP mailling list] To respect the ML:).</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 3, 2019 at 11:12 PM Xiao Liang <<a href="mailto:xiliang@redhat.com">xiliang@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"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 3, 2019 at 8:19 PM Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@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"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="font-size:small">Hi Xiao,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 3, 2019 at 7:56 PM Xiao Liang <<a href="mailto:xiliang@redhat.com" target="_blank">xiliang@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">When the system's Hugepagesize is large(eg. 512MiB), the test may<br>
break as cannot set nr_hugepages to default 128.<br></blockquote><div><br></div><div style="font-size:small">Yes, that's true. But each of ltp/hugemmap test has a parameter(hugemmap01 -s #nr_hpage#) to reset the nr_huagepges number before testing. What we have done to avoid that is to customize the hugetlb runtest file via a wrapper script.</div></div></div></div></div></div></blockquote><div> Yes, I tried '-s' option too, but runtest file does not specify it. Although default 128 works in most situations, I feel it is not very convenient for users to specify it or need double check it in each run in different platforms.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div class="gmail_quote"><div style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
This patch checks the memory available and Hugepagesize registered<br>
to set a reasonable num.<br></blockquote><div><br></div><div><div>I have no objection on this way. But a better way maybe to achieve a new function in the hugetlb library to reset nr_hugepages dynamically for the whole hugepage tests.</div></div></div></div></div></div></div></blockquote><div>Yes. I prefer this way than hard coding it with 128. I think we can add a new function into library as other cases use it too.<br></div></div></div></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Ok, It'd be great if you can help to draft a new patch for this.</div></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>