<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, Apr 29, 2019 at 7:59 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</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">Hi!<br>
> > > So as usually please go ahead and point out anything that should go in<br>
> > > before the release.<br>
> ><br>
> > Only Peter replied so far, so I suppose that we are ready to freeze git<br>
> > starting next week and start the pre-release testing.<br>
> ><br>
> > Or is there anything that has to be considered to be included before we<br>
> > do that?<br>
> ><br>
> <br>
> Here is a simple fix for hugepage test broken with memory shortage.<br>
> Consider merging?<br>
> <a href="http://lists.linux.it/pipermail/ltp/2019-April/011714.html" rel="noreferrer" target="_blank">http://lists.linux.it/pipermail/ltp/2019-April/011714.html</a><br>
<br>
The basic idea looks OK however:<br>
<br>
* the function name (reset_hugepages) is a bit confusing<br>
  what about "cap_hugepages" or "limit_hugepages"<br></blockquote><div><br></div><div>"limit_hugepages" sounds good to me.<br></div><div class="gmail_default" 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>
* The MemFree: may be close to zero on long running system because of<br>
  cached files, AFAIK MemAvailable: would be a better choice, see:<br>
<br>
  <a href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=34e431b0ae398fc54ea69ff85ec700722c9da773" rel="noreferrer" target="_blank">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=34e431b0ae398fc54ea69ff85ec700722c9da773</a><br>
<br>
  Unfortunately that one is available since 3.14 kernel, if we wanted to<br>
  run on older kenrnels I would propose to fall back to dropping caches<br>
  and using MemFree: on kernel that does not have MemAvailable: in<br>
  meminfo.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Good proposal. I will improve like this in patch V4. Thanks!</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>