<div dir="ltr">Hi,<br><br>thanks for quick response.<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 26, 2016 at 7:11 PM, Cyril Hrubis <span dir="ltr"><<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
<span class="">> Sometime thp05.c fails like:<br>
> ---<br>
> thp05       0  TINFO  :  Stop all children...<br>
> thp05       0  TINFO  :  Start to scan all transparent hugepages...<br>
> thp05       0  TINFO  :  khugepaged daemon takes 10s to scan all thp pages<br>
> thp05       0  TINFO  :  Start to verify transparent hugepage size...<br>
> thp05       1  TFAIL  :  mem.c:754: child[27099] got 43008KB thps - expect 40960KB thps<br>
> thp05       2  TFAIL  :  mem.c:754: child[27099] got 43008KB thps - expect 40960KB thps<br>
> thp05       3  TFAIL  :  mem.c:754: child[27099] got 43008KB thps - expect 40960KB thps<br>
> thp05       4  TFAIL  :  mem.c:754: child[27099] got 43008KB thps - expect 40960KB thps<br>
<br>
</span>Hmm shouldn't the TCONF message be included in the output here?<br>
(otherwise the if wasn't triggered at all)<br></blockquote><div><br>yes, it should be triggered first and then print the msg here.<br><br></div><div>I have realized the TFAIL problem is not simple like that, and I had already added some comments in the following maillist. Hope you could take a consideration when you get a chance.<br><br>Of course I'm ok if this issue couldn't be solved in the next LTP released, since the time is short.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class=""><br>
> ---<br>
><br>
> Since the system has done much testing work so far as it comes here, it is very easy to<br>
> hit the fails if the memory is fragmented, the following tests should better be skiped<br>
> while system dose not have enough memory for THP testing.<br>
><br>
> Signed-off-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>><br>
> ---<br>
>  testcases/kernel/mem/lib/mem.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/testcases/kernel/mem/lib/mem.c b/testcases/kernel/mem/lib/mem.c<br>
> index cee4e91..6715c35 100644<br>
> --- a/testcases/kernel/mem/lib/mem.c<br>
> +++ b/testcases/kernel/mem/lib/mem.c<br>
> @@ -773,7 +773,7 @@ void test_transparent_hugepage(int nr_children, int nr_thps,<br>
>       memfree = read_meminfo("MemFree:");<br>
>       tst_resm(TINFO, "The current MemFree is %luMB", memfree / KB);<br>
>       if (memfree < MB)<br>
> -             tst_resm(TCONF, "Not enough memory for testing");<br>
> +             tst_brkm(TCONF, NULL, "Not enough memory for testing");<br>
<br>
</span>This looks like obvious typo, but we should pass the cleanup to the<br>
tst_brkm() here as the rest of the tst_brkm() in this fuction does.<br>
<div class=""><div class="h5"><br>
>       hugepagesize = read_meminfo("Hugepagesize:");<br>
>       tst_resm(TINFO, "The current Hugepagesize is %luMB", hugepagesize / KB);<br>
<br>
--<br>
</div></div><span class=""><font color="#888888">Cyril Hrubis<br>
<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Regards,<br></div>Li Wang<br></div><div>Email: <a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a><br></div></div></div></div></div></div>
</div></div>