mm/slub.c: remove an unused addr argument

Qian Cai · torvalds · commit 278d7756dff0 · 2019-03-05T21:07:14.000-08:00
"addr" function argument is not used in alloc_consistency_checks() at all, so remove it. Link: http://lkml.kernel.org/r/20190211123214.35592-1-cai@lca.pw Fixes: becfda6 ("slub: convert SLAB_DEBUG_FREE to SLAB_CONSISTENCY_CHECKS") Signed-off-by: Qian Cai <cai@lca.pw> Reviewed-by: Andrew Morton <akpm@linux-foundation.org> Acked-by: David Rientjes <rientjes@google.com> Cc: Christoph Lameter <cl@linux.com> Cc: Pekka Enberg <penberg@kernel.org> Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
diff --git a/mm/slub.c b/mm/slub.c
@@ -1093,8 +1093,7 @@ static void setup_page_debug(struct kmem_cache *s, void *addr, int order)
 }
 
 static inline int alloc_consistency_checks(struct kmem_cache *s,
-					struct page *page,
-					void *object, unsigned long addr)
+					struct page *page, void *object)
 {
 	if (!check_slab(s, page))
 		return 0;
@@ -1115,7 +1114,7 @@ static noinline int alloc_debug_processing(struct kmem_cache *s,
 					void *object, unsigned long addr)
 {
 	if (s->flags & SLAB_CONSISTENCY_CHECKS) {
-		if (!alloc_consistency_checks(s, page, object, addr))
+		if (!alloc_consistency_checks(s, page, object))
 			goto bad;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1093,8 +1093,7 @@ static void setup_page_debug(struct kmem_cache s, void addr, int order)`
`1093`	`1093`	`}`
`1094`	`1094`
`1095`	`1095`	`static inline int alloc_consistency_checks(struct kmem_cache *s,`
`1096`		`- struct page *page,`
`1097`		`- void *object, unsigned long addr)`
	`1096`	`+ struct page page, void object)`
`1098`	`1097`	`{`
`1099`	`1098`	`if (!check_slab(s, page))`
`1100`	`1099`	`return 0;`
`@@ -1115,7 +1114,7 @@ static noinline int alloc_debug_processing(struct kmem_cache *s,`
`1115`	`1114`	`void *object, unsigned long addr)`
`1116`	`1115`	`{`
`1117`	`1116`	`if (s->flags & SLAB_CONSISTENCY_CHECKS) {`
`1118`		`- if (!alloc_consistency_checks(s, page, object, addr))`
	`1117`	`+ if (!alloc_consistency_checks(s, page, object))`
`1119`	`1118`	`goto bad;`
`1120`	`1119`	`}`
`1121`	`1120`