RE: [PATCH] mm: compaction: fix echo 1 > compact_memory returnerror issue

From: Liu Hui-R64343
Date: Sun Jan 06 2013 - 04:22:38 EST

>-----Original Message-----
>From: Simon Jeons [mailto:simon.jeons@xxxxxxxxx]
>Sent: Sunday, January 06, 2013 5:20 PM
>To: Liu Hui-R64343
>Cc: Wanpeng Li; linux-kernel@xxxxxxxxxxxxxxx; mgorman@xxxxxxx;
>akpm@xxxxxxxxxxxxxxxxxxxx; riel@xxxxxxxxxx; minchan@xxxxxxxxxx;
>kamezawa.hiroyu@xxxxxxxxxxxxxx; linux-mm@xxxxxxxxx
>Subject: Re: [PATCH] mm: compaction: fix echo 1 > compact_memory return
>error issue
>
>On Sun, 2013-01-06 at 08:48 +0000, Liu Hui-R64343 wrote:
>> >-----Original Message-----
>> >From: Wanpeng Li [mailto:liwanp@xxxxxxxxxxxxxxxxxx]
>> >Sent: Sunday, January 06, 2013 4:46 PM
>> >To: Liu Hui-R64343
>> >Cc: linux-kernel@xxxxxxxxxxxxxxx; mgorman@xxxxxxx; akpm@linux-
>> >foundation.org; riel@xxxxxxxxxx; minchan@xxxxxxxxxx;
>> >kamezawa.hiroyu@xxxxxxxxxxxxxx; linux-mm@xxxxxxxxx
>> >Subject: Re: [PATCH] mm: compaction: fix echo 1 > compact_memory
>> >return error issue
>> >
>> >On Sun, Jan 06, 2013 at 08:11:58AM +0000, Liu Hui-R64343 wrote:
>> >>>-----Original Message-----
>> >>>From: Wanpeng Li [mailto:liwanp@xxxxxxxxxxxxxxxxxx]
>> >>>Sent: Sunday, January 06, 2013 4:00 PM
>> >>>To: Liu Hui-R64343
>> >>>Cc: linux-kernel@xxxxxxxxxxxxxxx; mgorman@xxxxxxx; akpm@linux-
>> >>>foundation.org; riel@xxxxxxxxxx; minchan@xxxxxxxxxx;
>> >>>kamezawa.hiroyu@xxxxxxxxxxxxxx; linux-mm@xxxxxxxxx
>> >>>Subject: Re: [PATCH] mm: compaction: fix echo 1 > compact_memory
>> >>>return error issue
>> >>>
>> >>>On Sun, Jan 06, 2013 at 03:44:33PM +0800, Jason Liu wrote:
>> >>>
>> >>>Hi Jason,
>> >>>
>> >>>>when run the folloing command under shell, it will return error
>> >>>>sh/$ echo 1 > /proc/sys/vm/compact_memory sh/$ sh: write error:
>> >>>>Bad address
>> >>>>
>> >>>
>> >>>How can you modify the value through none privileged user since the
>> >>>mode == 0200?
>> >>
>> >>I write it through privileged user(root). I'm using the GNOME_Mobile
>rootfs.
>> >>
>> >>>
>> >>>>After strace, I found the following log:
>> >>>>...
>> >>>>write(1, "1\n", 2) = 3
>> >>>>write(1, "", 4294967295) = -1 EFAULT (Bad address)
>> >>>>write(2, "echo: write error: Bad address\n", 31echo: write error:
>> >>>>Bad address
>> >>>>) = 31
>> >>>>
>> >>>>This tells system return 3(COMPACT_COMPLETE) after write data to
>> >>>compact_memory.
>> >>>>
>> >>>>The fix is to make the system just return 0 instead
>> >>>>3(COMPACT_COMPLETE) from sysctl_compaction_handler after
>> >compaction_nodes finished.
>> >>>
>> >>>What's the special scenario you are in? I couldn't figure out the
>> >>>similar error against latest 3.8-rc2, how could you reproduce it?
>> >>
>> >>I'm using the BusyBox v1.20.2 () built-in shell (ash), it reproduces the
>issue:
>> >100%.
>> >>
>> >>root@freescale /$ sh
>> >>
>> >>
>> >>BusyBox v1.20.2 () built-in shell (ash) Enter 'help' for a list of
>> >>built-in commands.
>> >>
>> >>Could you run strace and see the log: strace echo 1 >
>> >>/proc/sys/vm/compact_memory
>> >>
>> >
>> >I test it on my desktop against latest 3.8-rc2, can't repoduce it. :)
>> >
>> >write(1, "1\n", 2) = 3
>>
>> Here it tells it.
>
>Why this value trouble you?

root@freescale /$strace echo 1 > /proc/sys/vm/compact_memory

write(1, "1\n", 2) = 3
write(1, "", 4294967295) = -1 EFAULT (Bad address)
write(2, "echo: write error: Bad address\n", 31echo: write error: Bad address
) = 31

root@freescale /$ sh


BusyBox v1.20.2 () built-in shell (ash)
Enter 'help' for a list of built-in commands.

root@freescale /$ echo 1 > /proc/sys/vm/compact_memory
sh: write error: Bad address

>
>>
>> >close(1) = 0
>> >munmap(0xb779c000, 4096) = 0
>> >close(2) = 0
>> >exit_group(0) = ?
>> >+++ exited with 0 +++
>> >
>> >Regards,
>> >Wanpeng Li
>> >
>> >>>
>> >>>Regards,
>> >>>Wanpeng Li
>> >>>
>> >>>>
>> >>>>Suggested-by:David Rientjes <rientjes@xxxxxxxxxx> Cc:Mel Gorman
>> >>>><mgorman@xxxxxxx> Cc:Andrew Morton <akpm@linux-
>foundation.org>
>> >>>Cc:Rik
>> >>>>van Riel <riel@xxxxxxxxxx> Cc:Minchan Kim <minchan@xxxxxxxxxx>
>> >>>>Cc:KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx>
>> >>>>Signed-off-by: Jason Liu <r64343@xxxxxxxxxxxxx>
>> >>>>---
>> >>>> mm/compaction.c | 6 ++----
>> >>>> 1 files changed, 2 insertions(+), 4 deletions(-)
>> >>>>
>> >>>>diff --git a/mm/compaction.c b/mm/compaction.c index
>> >>>>6b807e4..f8f5c11
>> >>>>100644
>> >>>>--- a/mm/compaction.c
>> >>>>+++ b/mm/compaction.c
>> >>>>@@ -1210,7 +1210,7 @@ static int compact_node(int nid) }
>> >>>>
>> >>>> /* Compact all nodes in the system */ -static int
>> >>>>compact_nodes(void)
>> >>>>+static void compact_nodes(void)
>> >>>> {
>> >>>> int nid;
>> >>>>
>> >>>>@@ -1219,8 +1219,6 @@ static int compact_nodes(void)
>> >>>>
>> >>>> for_each_online_node(nid)
>> >>>> compact_node(nid);
>> >>>>-
>> >>>>- return COMPACT_COMPLETE;
>> >>>> }
>> >>>>
>> >>>> /* The written value is actually unused, all memory is compacted
>> >>>>*/ @@
>> >>>>-1231,7 +1229,7 @@ int sysctl_compaction_handler(struct ctl_table
>> >>>>*table,
>> >>>int write,
>> >>>> void __user *buffer, size_t *length, loff_t *ppos)
>{
>> >>>> if (write)
>> >>>>- return compact_nodes();
>> >>>>+ compact_nodes();
>> >>>>
>> >>>> return 0;
>> >>>> }
>> >>>>--
>> >>>>1.7.5.4
>> >>>>
>> >>>>
>> >>>>--
>> >>>>To unsubscribe, send a message with 'unsubscribe linux-mm' in the
>> >>>>body to majordomo@xxxxxxxxxx For more info on Linux MM,
>> >>>>see: http://www.linux-mm.org/ .
>> >>>>Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
>> >>>
>> >>
>> >
>>
>>
>> --
>> To unsubscribe, send a message with 'unsubscribe linux-mm' in the body
>> to majordomo@xxxxxxxxxx For more info on Linux MM,
>> see: http://www.linux-mm.org/ .
>> Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
>
>

¢éì®&Þ~º&¶¬–+-±éÝ¥Šw®žË±Êâmébžìdz¹Þ)í…æèw*jg¬±¨¶‰šŽŠÝj/êäz¹ÞŠà2ŠÞ¨è­Ú&¢)ß«a¶Úþø®G«éh®æj:+v‰¨Šwè†Ù>Wš±êÞiÛaxPjØm¶Ÿÿà -»+ƒùdš_