Signed-off-by: Geert Uytterhoeven <[email protected]>
---
mm/slab.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/mm/slab.c b/mm/slab.c
index 70be5823227dcb3e..dd8c6d33f59a11d1 100644
--- a/mm/slab.c
+++ b/mm/slab.c
@@ -1569,9 +1569,6 @@ static void dump_line(char *data, int offset, int limit)
}
}
}
-#endif
-
-#if DEBUG
static void print_objinfo(struct kmem_cache *cachep, void *objp, int lines)
{
@@ -1661,9 +1658,7 @@ static void check_poison_obj(struct kmem_cache *cachep, void *objp)
}
}
}
-#endif
-#if DEBUG
static void slab_destroy_debugcheck(struct kmem_cache *cachep,
struct page *page)
{
--
2.7.4
On Thu, 7 Dec 2017, Geert Uytterhoeven wrote:
> --- a/mm/slab.c
> +++ b/mm/slab.c
> @@ -1569,9 +1569,6 @@ static void dump_line(char *data, int offset, int limit)
> }
> }
> }
> -#endif
> -
> -#if DEBUG
Hmmm... This may match at other places. Also there are a lot of #ifdef
DEBUG / #else in that section of the code. Maybe better leave as is? Or
generally rework this into a single #ifdef DEBUG section with all the
debugging code in it and an #else section with all the empty functions.