From mboxrd@z Thu Jan 1 00:00:00 1970 Message-Id: <20071107004710.642423857@sgi.com> References: <20071107004357.233417373@sgi.com> Date: Tue, 06 Nov 2007 16:43:58 -0800 From: clameter@sgi.com Subject: [patch 1/2] x86_64: Clean up stack allocation and free Content-Disposition: inline; filename=x86_64_stack_cleanup Sender: owner-linux-mm@kvack.org Return-Path: To: akpm@linux-foundation.org Cc: linux-mm@kvack.org, ak@suse.de, travis@sgi.com List-ID: Cleanup the allocation and freeing of stacks a bit by using a __GFP_ZERO flag instead of memset. Cc: ak@suse.de Cc: travis@sgi.com Signed-off-by: Christoph Lameter --- include/asm-x86/thread_info_64.h | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) Index: linux-2.6/include/asm-x86/thread_info_64.h =================================================================== --- linux-2.6.orig/include/asm-x86/thread_info_64.h 2007-10-12 12:41:32.000000000 -0700 +++ linux-2.6/include/asm-x86/thread_info_64.h 2007-11-06 15:38:22.000000000 -0800 @@ -74,20 +74,14 @@ static inline struct thread_info *stack_ /* thread information allocation */ #ifdef CONFIG_DEBUG_STACK_USAGE -#define alloc_thread_info(tsk) \ - ({ \ - struct thread_info *ret; \ - \ - ret = ((struct thread_info *) __get_free_pages(GFP_KERNEL,THREAD_ORDER)); \ - if (ret) \ - memset(ret, 0, THREAD_SIZE); \ - ret; \ - }) +#define THREAD_FLAGS (GFP_KERNEL | __GFP_ZERO) #else -#define alloc_thread_info(tsk) \ - ((struct thread_info *) __get_free_pages(GFP_KERNEL,THREAD_ORDER)) +#define THREAD_FLAGS GFP_KERNEL #endif +#define alloc_thread_info(tsk) \ + ((struct thread_info *) __get_free_pages(THREAD_FLAGS, THREAD_ORDER)) + #define free_thread_info(ti) free_pages((unsigned long) (ti), THREAD_ORDER) #else /* !__ASSEMBLY__ */ -- -- 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/ . Don't email: email@kvack.org