From: Randy Dunlap Localize poison values into one header file for better documentation and easier/quicker debugging and so that the same values won't be used for multiple purposes. Use these constants in core arch., mm, driver, and fs code. Signed-off-by: Randy Dunlap Acked-by: Matt Mackall Cc: Paul Mackerras Cc: Benjamin Herrenschmidt Cc: "David S. Miller" Cc: Andi Kleen Signed-off-by: Andrew Morton --- arch/i386/mm/init.c | 3 +- arch/powerpc/mm/init_64.c | 3 +- arch/sparc64/mm/init.c | 3 +- arch/x86_64/mm/init.c | 7 ++++- drivers/base/dmapool.c | 3 -- fs/jbd/journal.c | 3 +- include/linux/list.h | 9 ------- include/linux/poison.h | 45 ++++++++++++++++++++++++++++++++++++ mm/slab.c | 12 --------- 9 files changed, 61 insertions(+), 27 deletions(-) diff -puN arch/i386/mm/init.c~add-poisonh-and-patch-primary-users arch/i386/mm/init.c --- devel/arch/i386/mm/init.c~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/arch/i386/mm/init.c 2006-06-09 15:22:06.000000000 -0700 @@ -23,6 +23,7 @@ #include #include #include +#include #include #include #include @@ -753,7 +754,7 @@ void free_init_pages(char *what, unsigne for (addr = begin; addr < end; addr += PAGE_SIZE) { ClearPageReserved(virt_to_page(addr)); init_page_count(virt_to_page(addr)); - memset((void *)addr, 0xcc, PAGE_SIZE); + memset((void *)addr, POISON_FREE_INITMEM, PAGE_SIZE); free_page(addr); totalram_pages++; } diff -puN arch/powerpc/mm/init_64.c~add-poisonh-and-patch-primary-users arch/powerpc/mm/init_64.c --- devel/arch/powerpc/mm/init_64.c~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/arch/powerpc/mm/init_64.c 2006-06-09 15:22:06.000000000 -0700 @@ -41,6 +41,7 @@ #include #include #include +#include #include #include @@ -90,7 +91,7 @@ void free_initmem(void) addr = (unsigned long)__init_begin; for (; addr < (unsigned long)__init_end; addr += PAGE_SIZE) { - memset((void *)addr, 0xcc, PAGE_SIZE); + memset((void *)addr, POISON_FREE_INITMEM, PAGE_SIZE); ClearPageReserved(virt_to_page(addr)); init_page_count(virt_to_page(addr)); free_page(addr); diff -puN arch/sparc64/mm/init.c~add-poisonh-and-patch-primary-users arch/sparc64/mm/init.c --- devel/arch/sparc64/mm/init.c~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/arch/sparc64/mm/init.c 2006-06-09 15:22:06.000000000 -0700 @@ -18,6 +18,7 @@ #include #include #include +#include #include #include #include @@ -1474,7 +1475,7 @@ void free_initmem(void) page = (addr + ((unsigned long) __va(kern_base)) - ((unsigned long) KERNBASE)); - memset((void *)addr, 0xcc, PAGE_SIZE); + memset((void *)addr, POISON_FREE_INITMEM, PAGE_SIZE); p = virt_to_page(page); ClearPageReserved(p); diff -puN arch/x86_64/mm/init.c~add-poisonh-and-patch-primary-users arch/x86_64/mm/init.c --- devel/arch/x86_64/mm/init.c~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/arch/x86_64/mm/init.c 2006-06-09 15:22:06.000000000 -0700 @@ -23,6 +23,7 @@ #include #include #include +#include #include #include #include @@ -660,7 +661,8 @@ void free_init_pages(char *what, unsigne for (addr = begin; addr < end; addr += PAGE_SIZE) { ClearPageReserved(virt_to_page(addr)); init_page_count(virt_to_page(addr)); - memset((void *)(addr & ~(PAGE_SIZE-1)), 0xcc, PAGE_SIZE); + memset((void *)(addr & ~(PAGE_SIZE-1)), + POISON_FREE_INITMEM, PAGE_SIZE); free_page(addr); totalram_pages++; } @@ -668,7 +670,8 @@ void free_init_pages(char *what, unsigne void free_initmem(void) { - memset(__initdata_begin, 0xba, __initdata_end - __initdata_begin); + memset(__initdata_begin, POISON_FREE_INITDATA, + __initdata_end - __initdata_begin); free_init_pages("unused kernel memory", (unsigned long)(&__init_begin), (unsigned long)(&__init_end)); diff -puN drivers/base/dmapool.c~add-poisonh-and-patch-primary-users drivers/base/dmapool.c --- devel/drivers/base/dmapool.c~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/drivers/base/dmapool.c 2006-06-09 15:22:06.000000000 -0700 @@ -7,6 +7,7 @@ #include #include #include +#include /* * Pool allocator ... wraps the dma_alloc_coherent page allocator, so @@ -35,8 +36,6 @@ struct dma_page { /* cacheable header fo }; #define POOL_TIMEOUT_JIFFIES ((100 /* msec */ * HZ) / 1000) -#define POOL_POISON_FREED 0xa7 /* !inuse */ -#define POOL_POISON_ALLOCATED 0xa9 /* !initted */ static DECLARE_MUTEX (pools_lock); diff -puN fs/jbd/journal.c~add-poisonh-and-patch-primary-users fs/jbd/journal.c --- devel/fs/jbd/journal.c~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/fs/jbd/journal.c 2006-06-09 15:22:06.000000000 -0700 @@ -34,6 +34,7 @@ #include #include #include +#include #include #include @@ -1675,7 +1676,7 @@ static void journal_free_journal_head(st { #ifdef CONFIG_JBD_DEBUG atomic_dec(&nr_journal_heads); - memset(jh, 0x5b, sizeof(*jh)); + memset(jh, JBD_POISON_FREE, sizeof(*jh)); #endif kmem_cache_free(journal_head_cache, jh); } diff -puN include/linux/list.h~add-poisonh-and-patch-primary-users include/linux/list.h --- devel/include/linux/list.h~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/include/linux/list.h 2006-06-09 15:22:06.000000000 -0700 @@ -4,18 +4,11 @@ #ifdef __KERNEL__ #include +#include #include #include /* - * These are non-NULL pointers that will result in page faults - * under normal circumstances, used to verify that nobody uses - * non-initialized list entries. - */ -#define LIST_POISON1 ((void *) 0x00100100) -#define LIST_POISON2 ((void *) 0x00200200) - -/* * Simple doubly linked list implementation. * * Some of the internal functions ("__xxx") are useful when diff -puN /dev/null include/linux/poison.h --- /dev/null 2006-06-03 22:34:36.282200750 -0700 +++ devel-akpm/include/linux/poison.h 2006-06-09 15:22:06.000000000 -0700 @@ -0,0 +1,45 @@ +#ifndef _LINUX_POISON_H +#define _LINUX_POISON_H + +/********** include/linux/list.h **********/ +/* + * These are non-NULL pointers that will result in page faults + * under normal circumstances, used to verify that nobody uses + * non-initialized list entries. + */ +#define LIST_POISON1 ((void *) 0x00100100) +#define LIST_POISON2 ((void *) 0x00200200) + +/********** mm/slab.c **********/ +/* + * Magic nums for obj red zoning. + * Placed in the first word before and the first word after an obj. + */ +#define RED_INACTIVE 0x5A2CF071UL /* when obj is inactive */ +#define RED_ACTIVE 0x170FC2A5UL /* when obj is active */ + +/* ...and for poisoning */ +#define POISON_INUSE 0x5a /* for use-uninitialised poisoning */ +#define POISON_FREE 0x6b /* for use-after-free poisoning */ +#define POISON_END 0xa5 /* end-byte of poisoning */ + +/********** arch/$ARCH/mm/init.c **********/ +#define POISON_FREE_INITMEM 0xcc + +/********** arch/x86_64/mm/init.c **********/ +#define POISON_FREE_INITDATA 0xba + +/********** arch/ia64/hp/common/sba_iommu.c **********/ +/* + * arch/ia64/hp/common/sba_iommu.c uses a 16-byte poison string with a + * value of "SBAIOMMU POISON\0" for spill-over poisoning. + */ + +/********** fs/jbd/journal.c **********/ +#define JBD_POISON_FREE 0x5b + +/********** drivers/base/dmapool.c **********/ +#define POOL_POISON_FREED 0xa7 /* !inuse */ +#define POOL_POISON_ALLOCATED 0xa9 /* !initted */ + +#endif diff -puN mm/slab.c~add-poisonh-and-patch-primary-users mm/slab.c --- devel/mm/slab.c~add-poisonh-and-patch-primary-users 2006-06-09 15:22:06.000000000 -0700 +++ devel-akpm/mm/slab.c 2006-06-09 15:22:06.000000000 -0700 @@ -89,6 +89,7 @@ #include #include #include +#include #include #include #include @@ -492,17 +493,6 @@ struct kmem_cache { #endif #if DEBUG -/* - * Magic nums for obj red zoning. - * Placed in the first word before and the first word after an obj. - */ -#define RED_INACTIVE 0x5A2CF071UL /* when obj is inactive */ -#define RED_ACTIVE 0x170FC2A5UL /* when obj is active */ - -/* ...and for poisoning */ -#define POISON_INUSE 0x5a /* for use-uninitialised poisoning */ -#define POISON_FREE 0x6b /* for use-after-free poisoning */ -#define POISON_END 0xa5 /* end-byte of poisoning */ /* * memory layout of objects: _