diff options
author | Christoph Lameter <cl@linux.com> | 2010-08-20 13:37:12 -0400 |
---|---|---|
committer | Pekka Enberg <penberg@kernel.org> | 2010-10-02 03:24:26 -0400 |
commit | 1537066c69bee9ebc4b998626c532ce1724d4e67 (patch) | |
tree | 914bc354a286b6e14cf8f69f831f4656d9a49636 /mm/slub.c | |
parent | 899611ee7d373e5eeda08e9a8632684e1ebbbf00 (diff) |
slub: Force no inlining of debug functions
Compiler folds the debgging functions into the critical paths.
Avoid that by adding noinline to the functions that check for
problems.
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'mm/slub.c')
-rw-r--r-- | mm/slub.c | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -862,7 +862,7 @@ static void setup_object_debug(struct kmem_cache *s, struct page *page, | |||
862 | init_tracking(s, object); | 862 | init_tracking(s, object); |
863 | } | 863 | } |
864 | 864 | ||
865 | static int alloc_debug_processing(struct kmem_cache *s, struct page *page, | 865 | static noinline int alloc_debug_processing(struct kmem_cache *s, struct page *page, |
866 | void *object, unsigned long addr) | 866 | void *object, unsigned long addr) |
867 | { | 867 | { |
868 | if (!check_slab(s, page)) | 868 | if (!check_slab(s, page)) |
@@ -902,8 +902,8 @@ bad: | |||
902 | return 0; | 902 | return 0; |
903 | } | 903 | } |
904 | 904 | ||
905 | static int free_debug_processing(struct kmem_cache *s, struct page *page, | 905 | static noinline int free_debug_processing(struct kmem_cache *s, |
906 | void *object, unsigned long addr) | 906 | struct page *page, void *object, unsigned long addr) |
907 | { | 907 | { |
908 | if (!check_slab(s, page)) | 908 | if (!check_slab(s, page)) |
909 | goto fail; | 909 | goto fail; |