From: Jesper Juhl <jj@chaosbits.net> To: linux-kernel@vger.kernel.org Cc: linux-mm@kvack.org, Balbir Singh <balbir@linux.vnet.ibm.com>, Pavel Emelianov <xemul@openvz.org>, Minchan Kim <minchan.kim@gmail.com>, Paul Menage <menage@google.com>, Li Zefan <lizf@cn.fujitsu.com>, containers@lists.linux-foundation.org Subject: [PATCH] cgroup: prefer [kv]zalloc over [kv]malloc+memset in memory controller code. Date: Mon, 1 Nov 2010 20:40:56 +0100 (CET) [thread overview] Message-ID: <alpine.LNX.2.00.1011012038490.12889@swampdragon.chaosbits.net> (raw) Hi (please CC me on replies), Apologies to those who receive this multiple times. I screwed up the To: field in my original mail :-( In mem_cgroup_alloc() we currently do either kmalloc() or vmalloc() then followed by memset() to zero the memory. This can be more efficiently achieved by using kzalloc() and vzalloc(). Signed-off-by: Jesper Juhl <jj@chaosbits.net> --- memcontrol.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index 9a99cfa..90da698 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c @@ -4199,14 +4199,13 @@ static struct mem_cgroup *mem_cgroup_alloc(void) /* Can be very big if MAX_NUMNODES is very big */ if (size < PAGE_SIZE) - mem = kmalloc(size, GFP_KERNEL); + mem = kzalloc(size, GFP_KERNEL); else - mem = vmalloc(size); + mem = vzalloc(size); if (!mem) return NULL; - memset(mem, 0, size); mem->stat = alloc_percpu(struct mem_cgroup_stat_cpu); if (!mem->stat) { if (size < PAGE_SIZE) -- Jesper Juhl <jj@chaosbits.net> http://www.chaosbits.net/ Plain text mails only, please http://www.expita.com/nomime.html Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
WARNING: multiple messages have this Message-ID (diff)
From: Jesper Juhl <jj@chaosbits.net> To: linux-kernel@vger.kernel.org Cc: linux-mm@kvack.org, Balbir Singh <balbir@linux.vnet.ibm.com>, Pavel Emelianov <xemul@openvz.org>, Minchan Kim <minchan.kim@gmail.com>, Paul Menage <menage@google.com>, Li Zefan <lizf@cn.fujitsu.com>, containers@lists.linux-foundation.org Subject: [PATCH] cgroup: prefer [kv]zalloc over [kv]malloc+memset in memory controller code. Date: Mon, 1 Nov 2010 20:40:56 +0100 (CET) [thread overview] Message-ID: <alpine.LNX.2.00.1011012038490.12889@swampdragon.chaosbits.net> (raw) Hi (please CC me on replies), Apologies to those who receive this multiple times. I screwed up the To: field in my original mail :-( In mem_cgroup_alloc() we currently do either kmalloc() or vmalloc() then followed by memset() to zero the memory. This can be more efficiently achieved by using kzalloc() and vzalloc(). Signed-off-by: Jesper Juhl <jj@chaosbits.net> --- memcontrol.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index 9a99cfa..90da698 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c @@ -4199,14 +4199,13 @@ static struct mem_cgroup *mem_cgroup_alloc(void) /* Can be very big if MAX_NUMNODES is very big */ if (size < PAGE_SIZE) - mem = kmalloc(size, GFP_KERNEL); + mem = kzalloc(size, GFP_KERNEL); else - mem = vmalloc(size); + mem = vzalloc(size); if (!mem) return NULL; - memset(mem, 0, size); mem->stat = alloc_percpu(struct mem_cgroup_stat_cpu); if (!mem->stat) { if (size < PAGE_SIZE) -- Jesper Juhl <jj@chaosbits.net> http://www.chaosbits.net/ Plain text mails only, please http://www.expita.com/nomime.html Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next reply other threads:[~2010-11-01 19:51 UTC|newest] Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top 2010-11-01 19:40 Jesper Juhl [this message] 2010-11-01 19:40 ` [PATCH] cgroup: prefer [kv]zalloc over [kv]malloc+memset in memory controller code Jesper Juhl 2010-11-01 19:58 ` Hiroyuki Kamezawa 2010-11-01 19:58 ` Hiroyuki Kamezawa 2010-11-01 20:01 ` Johannes Weiner 2010-11-01 20:01 ` Johannes Weiner 2010-11-01 19:59 ` Jesper Juhl 2010-11-01 19:59 ` Jesper Juhl 2010-11-01 23:21 ` Minchan Kim 2010-11-01 23:21 ` Minchan Kim [not found] ` <alpine.LNX.2.00.1011012056250.12889-h2p7t3/P30RzeRGmFJ5qR7ZzlVVXadcDXqFh9Ls21Oc@public.gmane.org> 2010-11-01 23:21 ` Minchan Kim 2010-11-02 12:24 ` Johannes Weiner 2010-11-03 14:15 ` Wu Fengguang 2010-11-02 12:24 ` Johannes Weiner 2010-11-02 12:24 ` Johannes Weiner 2010-11-03 14:15 ` Wu Fengguang 2010-11-03 14:15 ` Wu Fengguang [not found] ` <20101101200122.GH840-druUgvl0LCNAfugRpC6u6w@public.gmane.org> 2010-11-01 19:59 ` Jesper Juhl 2010-11-02 5:07 ` Balbir Singh 2010-11-02 5:07 ` Balbir Singh [not found] ` <alpine.LNX.2.00.1011012038490.12889-h2p7t3/P30RzeRGmFJ5qR7ZzlVVXadcDXqFh9Ls21Oc@public.gmane.org> 2010-11-01 19:58 ` Hiroyuki Kamezawa 2010-11-01 20:01 ` Johannes Weiner 2010-11-02 5:07 ` Balbir Singh -- strict thread matches above, loose matches on Subject: below -- 2010-11-01 19:40 Jesper Juhl 2010-11-01 19:35 Jesper Juhl 2010-11-01 23:11 ` Minchan Kim [not found] ` <alpine.LNX.2.00.1011012030150.12889-h2p7t3/P30RzeRGmFJ5qR7ZzlVVXadcDXqFh9Ls21Oc@public.gmane.org> 2010-11-01 23:11 ` Minchan Kim 2010-11-01 19:35 Jesper Juhl
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=alpine.LNX.2.00.1011012038490.12889@swampdragon.chaosbits.net \ --to=jj@chaosbits.net \ --cc=balbir@linux.vnet.ibm.com \ --cc=containers@lists.linux-foundation.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=lizf@cn.fujitsu.com \ --cc=menage@google.com \ --cc=minchan.kim@gmail.com \ --cc=xemul@openvz.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.