<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 31, 2016 at 9:31 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi!<br>
<span class="">>  testcases/kernel/mem/include/mem.h |  3 +++<br>
>  testcases/kernel/mem/ksm/ksm01.c   |  4 ++++<br>
>  testcases/kernel/mem/ksm/ksm02.c   |  3 +++<br>
>  testcases/kernel/mem/ksm/ksm03.c   |  4 ++++<br>
>  testcases/kernel/mem/ksm/ksm04.c   |  4 ++++<br>
>  testcases/kernel/mem/ksm/ksm06.c   |  4 ++++<br>
>  testcases/kernel/mem/lib/mem.c     | 22 +++++++++++++++++++++-<br>
>  7 files changed, 43 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/testcases/kernel/mem/include/mem.h b/testcases/kernel/mem/include/mem.h<br>
> index 43988fe..69d08dd 100644<br>
> --- a/testcases/kernel/mem/include/mem.h<br>
> +++ b/testcases/kernel/mem/include/mem.h<br>
> @@ -43,7 +43,10 @@ void testoom(int mempolicy, int lite, int retcode, int allow_sigkill);<br>
>  /* KSM */<br>
><br>
>  #define PATH_KSM             "/sys/kernel/mm/ksm/"<br>
> +int max_page_sharing;<br>
<br>
</span>This value is not modified outside the library. It makes no sense to<br>
declare it in the header.<br>
<br>
As a matter of fact it would be far better to declare it as static int<br>
inside the library so that there is no chance that it will be modified<br>
by the testcases.<br></blockquote><div><br></div><div>sounds good, could you help to fix that, or need I format a patch next version? <br></div></div><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>