From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cliff Wickman Date: Tue, 25 Apr 2006 15:47:48 +0000 Subject: [PATCH 1/1] ia64: enable dumps to capture second page of kernel stack Message-Id: <20060425154748.GA29493@sgi.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-ia64@vger.kernel.org In SLES10 (2.6.16) crash dumping (in my experience, LKCD) is unable to capture the second page of the 2-page task/stack allocation. This is particularly troublesome for dump analysis, as the stack traceback cannot be done. (A similar convention is probably needed throughout the kernel to make kernel multi-page allocations detectable for dumping) Multi-page kernel allocations are represented by the single page structure associated with the first page of the allocation. The page structures associated with the other pages are unintialized. If the dumper is selecting only kernel pages it has no way to identify any but the first page of the allocation. The fix is to make the task/stack allocation a compound page. Diffed against 2.6.16 Signed-off-by: Cliff Wickman --- --- include/asm-ia64/thread_info.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux/include/asm-ia64/thread_info.h =================================--- linux.orig/include/asm-ia64/thread_info.h +++ linux/include/asm-ia64/thread_info.h @@ -74,7 +74,7 @@ struct thread_info { #define end_of_stack(p) (unsigned long *)((void *)(p) + IA64_RBS_OFFSET) #define __HAVE_ARCH_TASK_STRUCT_ALLOCATOR -#define alloc_task_struct() ((task_t *)__get_free_pages(GFP_KERNEL, KERNEL_STACK_SIZE_ORDER)) +#define alloc_task_struct() ((task_t *)__get_free_pages(GFP_KERNEL | __GFP_COMP, KERNEL_STACK_SIZE_ORDER)) #define free_task_struct(tsk) free_pages((unsigned long) (tsk), KERNEL_STACK_SIZE_ORDER) #endif /* !__ASSEMBLY */ --------- Nick, You had copied me on a reply on this subject earlier: Date: Sat, 22 Apr 2006 13:56:12 +1000 From: Nick Piggin Yeah, we're moving toward compound pages for all these types of things (including nommu). So making those pages compound pages should be the right thing to do. Still agree?