| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sun, 06 Jan 2013 03:19:40 -0600
From: Simon Jeons <simon.jeons@...il.com>
To: Liu Hui-R64343 <r64343@...escale.com>
Cc: Wanpeng Li <liwanp@...ux.vnet.ibm.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"mgorman@...e.de" <mgorman@...e.de>,
"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"riel@...hat.com" <riel@...hat.com>,
"minchan@...nel.org" <minchan@...nel.org>,
"kamezawa.hiroyu@...fujitsu.com" <kamezawa.hiroyu@...fujitsu.com>,
"linux-mm@...ck.org" <linux-mm@...ck.org>
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@...ux.vnet.ibm.com]
> >Sent: Sunday, January 06, 2013 4:46 PM
> >To: Liu Hui-R64343
> >Cc: linux-kernel@...r.kernel.org; mgorman@...e.de; akpm@...ux-
> >foundation.org; riel@...hat.com; minchan@...nel.org;
> >kamezawa.hiroyu@...fujitsu.com; linux-mm@...ck.org
> >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@...ux.vnet.ibm.com]
> >>>Sent: Sunday, January 06, 2013 4:00 PM
> >>>To: Liu Hui-R64343
> >>>Cc: linux-kernel@...r.kernel.org; mgorman@...e.de; akpm@...ux-
> >>>foundation.org; riel@...hat.com; minchan@...nel.org;
> >>>kamezawa.hiroyu@...fujitsu.com; linux-mm@...ck.org
> >>>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@...escale /$ 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?
>
> >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@...gle.com> Cc:Mel Gorman
> >>>><mgorman@...e.de> Cc:Andrew Morton <akpm@...ux-foundation.org>
> >>>Cc:Rik
> >>>>van Riel <riel@...hat.com> Cc:Minchan Kim <minchan@...nel.org>
> >>>>Cc:KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
> >>>>Signed-off-by: Jason Liu <r64343@...escale.com>
> >>>>---
> >>>> 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@...ck.org. For more info on Linux MM,
> >>>>see: http://www.linux-mm.org/ .
> >>>>Don't email: <a href=mailto:"dont@...ck.org"> email@...ck.org </a>
> >>>
> >>
> >
>
>
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@...ck.org. For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@...ck.org"> email@...ck.org </a>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists