public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP]  [PATCH] mm/tunable: add a new testcase min_free_kbytes
@ 2012-03-22  6:35 Zhouping Liu
  2012-03-22 13:32 ` Wanlong Gao
  0 siblings, 1 reply; 3+ messages in thread
From: Zhouping Liu @ 2012-03-22  6:35 UTC (permalink / raw)
  To: Caspar Zhang; +Cc: LTP list

The case is designed to test min_free_kbytes tunable file.

The tune is used to control free memory, and system always
reserve min_free_kbytes memory at least.

Since the tune is not too large or too little, which will
lead to the system hang, so I choose two cases, and test them
on all overcommit_memory policy, at the same time, compare
the current free memory with the tunable value repeatedly.

a) default min_free_kbytes with all overcommit memory policy
b) half of mem_free with all overcommit memory policy

Signed-off-by: Zhouping Liu <zliu@redhat.com>
---
 runtest/mm                                     |    2 +
 testcases/kernel/mem/tunable/min_free_kbytes.c |  222 ++++++++++++++++++++++++
 2 files changed, 224 insertions(+), 0 deletions(-)
 create mode 100644 testcases/kernel/mem/tunable/min_free_kbytes.c

I have tested on RHEL6.2, it's fine, log like this:
<<<test_start>>>
tag=min_free_kbytes stime=1332315788
cmdline="min_free_kbytes"
contacts=""
analysis=exit
<<<test_output>>>
min_free_kbytes    0  TINFO  :  set overcommit_memory to 2
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
min_free_kbytes    0  TINFO  :  MemFree is 15397328 kB before eatup mem
min_free_kbytes    0  TINFO  :  MemFree is 151964 kB after eatup mem
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 7999962
min_free_kbytes    0  TINFO  :  MemFree is 15999924 kB before eatup mem
min_free_kbytes    0  TINFO  :  MemFree is 8044308 kB after eatup mem
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 7999962
min_free_kbytes    0  TINFO  :  set overcommit_memory to 0
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
Child[26301] is killed by SIGKILL signal.
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 8002878
Child[27570] is killed by SIGKILL signal.
min_free_kbytes    0  TINFO  :  set overcommit_memory to 1
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
Child[28782] is killed by SIGKILL signal.
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 8003788
Child[30056] is killed by SIGKILL signal.
min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
min_free_kbytes    0  TINFO  :  set overcommit_memory to 0

diff --git a/runtest/mm b/runtest/mm
index be3b342..2dd66f0 100644
--- a/runtest/mm
+++ b/runtest/mm
@@ -98,3 +98,5 @@ overcommit_memory overcommit_memory -R 100
 overcommit_memory overcommit_memory -R 200
 
 max_map_count max_map_count -i 10
+
+min_free_kbytes min_free_kbytes
diff --git a/testcases/kernel/mem/tunable/min_free_kbytes.c b/testcases/kernel/mem/tunable/min_free_kbytes.c
new file mode 100644
index 0000000..722a56a
--- /dev/null
+++ b/testcases/kernel/mem/tunable/min_free_kbytes.c
@@ -0,0 +1,222 @@
+/*
+ * The case is designed to test min_free_kbytes tunable.
+ *
+ * The tune is used to control free memory, and system always
+ * reserve min_free_kbytes memory at least.
+ *
+ * Since the tune is not too large or too little, which will
+ * lead to the system hang, so I choose two cases, and test them
+ * on all overcommit_memory policy, at the same time, compare
+ * the current free memory with the tunable value repeatedly.
+ *
+ * a) default min_free_kbytes with all overcommit memory policy
+ * b) half of mem_free with all overcommit memory policy
+ *
+ ********************************************************************
+ * Copyright (C) 2012 Red Hat, Inc.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of version 2 of the GNU General Public
+ * License as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it would be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
+ *
+ * Further, this software is distributed without any warranty that it
+ * is free of the rightful claim of any third person regarding
+ * infringement or the like.  Any license provided herein, whether
+ * implied or otherwise, applies only to this software file.  Patent
+ * licenses, if any, provided herein do not apply to combinations of
+ * this program with other software, or any other product whatsoever.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+ * 02110-1301, USA.
+ *
+ * ********************************************************************
+ */
+
+#include <sys/types.h>
+#include <sys/mman.h>
+#include <sys/wait.h>
+#include <errno.h>
+#include <fcntl.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include "test.h"
+#include "usctest.h"
+#include "../include/mem.h"
+
+#define MAP_SIZE (1UL<<20)
+
+char *TCID = "min_free_kbytes";
+int TST_TOTAL = 1;
+static unsigned long default_tune;
+static unsigned long orig_overcommit;
+static unsigned long total_mem;
+
+static void test_tune(unsigned long overcommit_policy);
+static void eatup_mem();
+static void check_monitor(void);
+
+int main(int argc, char *argv[])
+{
+	char *msg;
+	int lc;
+
+	msg = parse_opts(argc, argv, NULL, NULL);
+	if (msg != NULL)
+		tst_brkm(TBROK, NULL, "OPTION PARSING ERROR -s %s", msg);
+	setup();
+
+	for (lc = 0; TEST_LOOPING(lc); lc++) {
+		Tst_count = 0;
+
+		test_tune(2);
+		test_tune(0);
+		test_tune(1);
+	}
+
+	cleanup();
+	tst_exit();
+}
+
+static void test_tune(unsigned long overcommit_policy)
+{
+	int status;
+	int pid[2];
+	unsigned long tune, memfree;
+
+	set_sys_tune("overcommit_memory", overcommit_policy, 1);
+
+	fflush(stdout);
+	switch (pid[0] = fork()) {
+	case -1:
+		tst_brkm(TBROK|TERRNO, cleanup, "fork");
+
+	case 0:
+		/* startup the check monitor */
+		check_monitor();
+	}
+
+	fflush(stdout);
+	switch (pid[1] = fork()) {
+	case -1:
+		tst_brkm(TBROK|TERRNO, cleanup, "fork");
+	case 0:
+
+		/* case1 */
+		set_sys_tune("min_free_kbytes", default_tune, 1);
+		eatup_mem();
+
+		/* case2 */
+		memfree = read_meminfo("MemFree:");
+		tune = memfree / 2;
+		set_sys_tune("min_free_kbytes", tune, 1);
+		eatup_mem();
+
+		exit(0);
+	}
+
+	if (waitpid(pid[1], &status, 0) == -1)
+		tst_brkm(TBROK|TERRNO, cleanup, "waitpid");
+
+	/* send SIGKILL to end up check_monitor thread */
+	if (kill(pid[0], SIGKILL) == -1)
+		tst_brkm(TBROK|TERRNO, cleanup, "kill %d", pid[0]);
+	if (waitpid(pid[0], &status, 0) == -1)
+		tst_brkm(TBROK|TERRNO, cleanup,
+		    "check_monitor thread didn't end up");
+}
+
+static void eatup_mem()
+{
+	int map_count, i;
+	int status, pid;
+	unsigned long memfree;
+	void **addrs;
+
+	map_count = total_mem * KB / MAP_SIZE;
+	addrs = (void **)malloc(map_count * sizeof(void *));
+
+	switch (pid = fork()) {
+	case -1:
+		tst_brkm(TBROK | TERRNO, cleanup, "fork");
+	case 0:
+		memfree = read_meminfo("MemFree:");
+		tst_resm(TINFO, "MemFree is %lu kB before eatup mem", memfree);
+
+		for (i = 0; i < map_count; i++) {
+			addrs[i] = mmap(NULL, MAP_SIZE, PROT_READ|PROT_WRITE,
+			    MAP_ANONYMOUS|MAP_PRIVATE, -1, 0);
+			if (addrs[i] == MAP_FAILED) {
+				map_count = i - 1;
+				break;
+			}
+
+			memset(addrs[i], i, MAP_SIZE);
+		}
+
+		memfree = read_meminfo("MemFree:");
+		tst_resm(TINFO, "MemFree is %lu kB after eatup mem", memfree);
+
+		for (i = 0; i < map_count; i++)
+			if (munmap(addrs[i], MAP_SIZE) != 0)
+				tst_brkm(TBROK|TERRNO, cleanup, "munmap");
+		free(addrs);
+		exit(0);
+	default:
+		if (waitpid(pid, &status, 0) == -1)
+			tst_brkm(TBROK|TERRNO, cleanup, "waitpid");
+		if (!WIFEXITED(status) || WEXITSTATUS(status) != 0) {
+			if (WIFSIGNALED(status)
+			    && WTERMSIG(status) == SIGKILL)
+				printf("Child[%d] is killed by "
+				    "SIGKILL signal.\n", pid);
+			else
+				tst_resm(TFAIL, "Child[%d] "
+				    "failed unexpectedly", pid);
+		}
+	}
+}
+
+static void check_monitor(void)
+{
+	unsigned long tune;
+	unsigned long memfree;
+
+	while (1) {
+		memfree = read_meminfo("MemFree:");
+		tune = get_sys_tune("min_free_kbytes");
+
+		if (memfree < tune) {
+			tst_resm(TINFO, "MemFree is %lu kB, "
+			    "min_free_kbytes is %lu kB", memfree, tune);
+			tst_resm(TFAIL, "MemFree < min_free_kbytes");
+		}
+
+		sleep(2);
+	}
+}
+
+void setup(void)
+{
+	tst_require_root(NULL);
+	tst_sig(FORK, DEF_HANDLER, cleanup);
+	TEST_PAUSE;
+
+	total_mem = read_meminfo("MemTotal:") + read_meminfo("SwapTotal:");
+
+	default_tune = get_sys_tune("min_free_kbytes");
+	orig_overcommit = get_sys_tune("overcommit_memory");
+}
+
+void cleanup()
+{
+	set_sys_tune("min_free_kbytes", default_tune, 0);
+	set_sys_tune("overcommit_memory", orig_overcommit, 0);
+
+	TEST_CLEANUP;
+}
-- 
1.7.7.6


------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [LTP] [PATCH] mm/tunable: add a new testcase min_free_kbytes
  2012-03-22  6:35 [LTP] [PATCH] mm/tunable: add a new testcase min_free_kbytes Zhouping Liu
@ 2012-03-22 13:32 ` Wanlong Gao
  2012-03-23 14:16   ` ZhoupingLiu
  0 siblings, 1 reply; 3+ messages in thread
From: Wanlong Gao @ 2012-03-22 13:32 UTC (permalink / raw)
  To: Zhouping Liu; +Cc: LTP list

On 03/22/2012 02:35 PM, Zhouping Liu wrote:

> The case is designed to test min_free_kbytes tunable file.
> 
> The tune is used to control free memory, and system always
> reserve min_free_kbytes memory at least.
> 
> Since the tune is not too large or too little, which will
> lead to the system hang, so I choose two cases, and test them
> on all overcommit_memory policy, at the same time, compare
> the current free memory with the tunable value repeatedly.
> 
> a) default min_free_kbytes with all overcommit memory policy
> b) half of mem_free with all overcommit memory policy


Looks good and test OK to me. thanks,

Reviewed-by: Wanlong Gao <gaowanlong@cn.fujitsu.com>


> 
> Signed-off-by: Zhouping Liu <zliu@redhat.com>
> ---
>  runtest/mm                                     |    2 +
>  testcases/kernel/mem/tunable/min_free_kbytes.c |  222 ++++++++++++++++++++++++
>  2 files changed, 224 insertions(+), 0 deletions(-)
>  create mode 100644 testcases/kernel/mem/tunable/min_free_kbytes.c
> 
> I have tested on RHEL6.2, it's fine, log like this:
> <<<test_start>>>
> tag=min_free_kbytes stime=1332315788
> cmdline="min_free_kbytes"
> contacts=""
> analysis=exit
> <<<test_output>>>
> min_free_kbytes    0  TINFO  :  set overcommit_memory to 2
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
> min_free_kbytes    0  TINFO  :  MemFree is 15397328 kB before eatup mem
> min_free_kbytes    0  TINFO  :  MemFree is 151964 kB after eatup mem
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 7999962
> min_free_kbytes    0  TINFO  :  MemFree is 15999924 kB before eatup mem
> min_free_kbytes    0  TINFO  :  MemFree is 8044308 kB after eatup mem
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 7999962
> min_free_kbytes    0  TINFO  :  set overcommit_memory to 0
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
> Child[26301] is killed by SIGKILL signal.
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 8002878
> Child[27570] is killed by SIGKILL signal.
> min_free_kbytes    0  TINFO  :  set overcommit_memory to 1
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
> Child[28782] is killed by SIGKILL signal.
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 8003788
> Child[30056] is killed by SIGKILL signal.
> min_free_kbytes    0  TINFO  :  set min_free_kbytes to 90112
> min_free_kbytes    0  TINFO  :  set overcommit_memory to 0
> 
> diff --git a/runtest/mm b/runtest/mm
> index be3b342..2dd66f0 100644
> --- a/runtest/mm
> +++ b/runtest/mm
> @@ -98,3 +98,5 @@ overcommit_memory overcommit_memory -R 100
>  overcommit_memory overcommit_memory -R 200
>  
>  max_map_count max_map_count -i 10
> +
> +min_free_kbytes min_free_kbytes
> diff --git a/testcases/kernel/mem/tunable/min_free_kbytes.c b/testcases/kernel/mem/tunable/min_free_kbytes.c
> new file mode 100644
> index 0000000..722a56a
> --- /dev/null
> +++ b/testcases/kernel/mem/tunable/min_free_kbytes.c
> @@ -0,0 +1,222 @@
> +/*
> + * The case is designed to test min_free_kbytes tunable.
> + *
> + * The tune is used to control free memory, and system always
> + * reserve min_free_kbytes memory at least.
> + *
> + * Since the tune is not too large or too little, which will
> + * lead to the system hang, so I choose two cases, and test them
> + * on all overcommit_memory policy, at the same time, compare
> + * the current free memory with the tunable value repeatedly.
> + *
> + * a) default min_free_kbytes with all overcommit memory policy
> + * b) half of mem_free with all overcommit memory policy
> + *
> + ********************************************************************
> + * Copyright (C) 2012 Red Hat, Inc.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of version 2 of the GNU General Public
> + * License as published by the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it would be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> + *
> + * Further, this software is distributed without any warranty that it
> + * is free of the rightful claim of any third person regarding
> + * infringement or the like.  Any license provided herein, whether
> + * implied or otherwise, applies only to this software file.  Patent
> + * licenses, if any, provided herein do not apply to combinations of
> + * this program with other software, or any other product whatsoever.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
> + * 02110-1301, USA.
> + *
> + * ********************************************************************
> + */
> +
> +#include <sys/types.h>
> +#include <sys/mman.h>
> +#include <sys/wait.h>
> +#include <errno.h>
> +#include <fcntl.h>
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include "test.h"
> +#include "usctest.h"
> +#include "../include/mem.h"
> +
> +#define MAP_SIZE (1UL<<20)
> +
> +char *TCID = "min_free_kbytes";
> +int TST_TOTAL = 1;
> +static unsigned long default_tune;
> +static unsigned long orig_overcommit;
> +static unsigned long total_mem;
> +
> +static void test_tune(unsigned long overcommit_policy);
> +static void eatup_mem();
> +static void check_monitor(void);
> +
> +int main(int argc, char *argv[])
> +{
> +	char *msg;
> +	int lc;
> +
> +	msg = parse_opts(argc, argv, NULL, NULL);
> +	if (msg != NULL)
> +		tst_brkm(TBROK, NULL, "OPTION PARSING ERROR -s %s", msg);
> +	setup();
> +
> +	for (lc = 0; TEST_LOOPING(lc); lc++) {
> +		Tst_count = 0;
> +
> +		test_tune(2);
> +		test_tune(0);
> +		test_tune(1);
> +	}
> +
> +	cleanup();
> +	tst_exit();
> +}
> +
> +static void test_tune(unsigned long overcommit_policy)
> +{
> +	int status;
> +	int pid[2];
> +	unsigned long tune, memfree;
> +
> +	set_sys_tune("overcommit_memory", overcommit_policy, 1);
> +
> +	fflush(stdout);
> +	switch (pid[0] = fork()) {
> +	case -1:
> +		tst_brkm(TBROK|TERRNO, cleanup, "fork");
> +
> +	case 0:
> +		/* startup the check monitor */
> +		check_monitor();
> +	}
> +
> +	fflush(stdout);
> +	switch (pid[1] = fork()) {
> +	case -1:
> +		tst_brkm(TBROK|TERRNO, cleanup, "fork");
> +	case 0:
> +
> +		/* case1 */
> +		set_sys_tune("min_free_kbytes", default_tune, 1);
> +		eatup_mem();
> +
> +		/* case2 */
> +		memfree = read_meminfo("MemFree:");
> +		tune = memfree / 2;
> +		set_sys_tune("min_free_kbytes", tune, 1);
> +		eatup_mem();
> +
> +		exit(0);
> +	}
> +
> +	if (waitpid(pid[1], &status, 0) == -1)
> +		tst_brkm(TBROK|TERRNO, cleanup, "waitpid");
> +
> +	/* send SIGKILL to end up check_monitor thread */
> +	if (kill(pid[0], SIGKILL) == -1)
> +		tst_brkm(TBROK|TERRNO, cleanup, "kill %d", pid[0]);
> +	if (waitpid(pid[0], &status, 0) == -1)
> +		tst_brkm(TBROK|TERRNO, cleanup,
> +		    "check_monitor thread didn't end up");
> +}
> +
> +static void eatup_mem()
> +{
> +	int map_count, i;
> +	int status, pid;
> +	unsigned long memfree;
> +	void **addrs;
> +
> +	map_count = total_mem * KB / MAP_SIZE;
> +	addrs = (void **)malloc(map_count * sizeof(void *));
> +
> +	switch (pid = fork()) {
> +	case -1:
> +		tst_brkm(TBROK | TERRNO, cleanup, "fork");
> +	case 0:
> +		memfree = read_meminfo("MemFree:");
> +		tst_resm(TINFO, "MemFree is %lu kB before eatup mem", memfree);
> +
> +		for (i = 0; i < map_count; i++) {
> +			addrs[i] = mmap(NULL, MAP_SIZE, PROT_READ|PROT_WRITE,
> +			    MAP_ANONYMOUS|MAP_PRIVATE, -1, 0);
> +			if (addrs[i] == MAP_FAILED) {
> +				map_count = i - 1;
> +				break;
> +			}
> +
> +			memset(addrs[i], i, MAP_SIZE);
> +		}
> +
> +		memfree = read_meminfo("MemFree:");
> +		tst_resm(TINFO, "MemFree is %lu kB after eatup mem", memfree);
> +
> +		for (i = 0; i < map_count; i++)
> +			if (munmap(addrs[i], MAP_SIZE) != 0)
> +				tst_brkm(TBROK|TERRNO, cleanup, "munmap");
> +		free(addrs);
> +		exit(0);
> +	default:
> +		if (waitpid(pid, &status, 0) == -1)
> +			tst_brkm(TBROK|TERRNO, cleanup, "waitpid");
> +		if (!WIFEXITED(status) || WEXITSTATUS(status) != 0) {
> +			if (WIFSIGNALED(status)
> +			    && WTERMSIG(status) == SIGKILL)
> +				printf("Child[%d] is killed by "
> +				    "SIGKILL signal.\n", pid);
> +			else
> +				tst_resm(TFAIL, "Child[%d] "
> +				    "failed unexpectedly", pid);
> +		}
> +	}
> +}
> +
> +static void check_monitor(void)
> +{
> +	unsigned long tune;
> +	unsigned long memfree;
> +
> +	while (1) {
> +		memfree = read_meminfo("MemFree:");
> +		tune = get_sys_tune("min_free_kbytes");
> +
> +		if (memfree < tune) {
> +			tst_resm(TINFO, "MemFree is %lu kB, "
> +			    "min_free_kbytes is %lu kB", memfree, tune);
> +			tst_resm(TFAIL, "MemFree < min_free_kbytes");
> +		}
> +
> +		sleep(2);
> +	}
> +}
> +
> +void setup(void)
> +{
> +	tst_require_root(NULL);
> +	tst_sig(FORK, DEF_HANDLER, cleanup);
> +	TEST_PAUSE;
> +
> +	total_mem = read_meminfo("MemTotal:") + read_meminfo("SwapTotal:");
> +
> +	default_tune = get_sys_tune("min_free_kbytes");
> +	orig_overcommit = get_sys_tune("overcommit_memory");
> +}
> +
> +void cleanup()
> +{
> +	set_sys_tune("min_free_kbytes", default_tune, 0);
> +	set_sys_tune("overcommit_memory", orig_overcommit, 0);
> +
> +	TEST_CLEANUP;
> +}



------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [LTP] [PATCH] mm/tunable: add a new testcase min_free_kbytes
  2012-03-22 13:32 ` Wanlong Gao
@ 2012-03-23 14:16   ` ZhoupingLiu
  0 siblings, 0 replies; 3+ messages in thread
From: ZhoupingLiu @ 2012-03-23 14:16 UTC (permalink / raw)
  To: gaowanlong; +Cc: LTP list

On 03/22/2012 09:32 PM, Wanlong Gao wrote:
> On 03/22/2012 02:35 PM, Zhouping Liu wrote:
>
>> The case is designed to test min_free_kbytes tunable file.
>>
>> The tune is used to control free memory, and system always
>> reserve min_free_kbytes memory at least.
>>
>> Since the tune is not too large or too little, which will
>> lead to the system hang, so I choose two cases, and test them
>> on all overcommit_memory policy, at the same time, compare
>> the current free memory with the tunable value repeatedly.
>>
>> a) default min_free_kbytes with all overcommit memory policy
>> b) half of mem_free with all overcommit memory policy
>
> Looks good and test OK to me. thanks,
>
> Reviewed-by: Wanlong Gao<gaowanlong@cn.fujitsu.com>
>

hi, Wanlong
Thanks for your reviewing :)
I sent a new version, please review it again.

Thanks,
Zhouping

------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-03-23 14:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-22  6:35 [LTP] [PATCH] mm/tunable: add a new testcase min_free_kbytes Zhouping Liu
2012-03-22 13:32 ` Wanlong Gao
2012-03-23 14:16   ` ZhoupingLiu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox