| 
		
			| [PATCH] cgroup: prefer [kv]zalloc over [kv]malloc+memset in memory controller code. [message #41962] | Mon, 01 November 2010 19:40 |  
			| 
				
				
					|  Jesper Juhl Messages: 7
 Registered: October 2010
 | Junior Member |  |  |  
	| 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
 
 _______________________________________________
 Containers mailing list
 Containers@lists.linux-foundation.org
 https://lists.linux-foundation.org/mailman/listinfo/containe rs
 |  
	|  |  |