diff options
| author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2025-09-15 15:55:08 +0200 |
|---|---|---|
| committer | Vlastimil Babka <vbabka@suse.cz> | 2025-09-15 16:10:10 +0200 |
| commit | f4930de03dcf4a3f2515546f5b3c04bac016685d (patch) | |
| tree | 17b549428ca7ae9b070c292e86078831f1276306 | |
| parent | 41534d499e50e23571d6b9960498777d93f817ce (diff) | |
slab: Remove dead code in free_consistency_checks()
We already know that slab is a valid slab as that's checked by the
caller. In the future, we won't be able to get to a slab pointer
from a non-slab page.
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Reviewed-by: Harry Yoo <harry.yoo@oracle.com>
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
| -rw-r--r-- | mm/slub.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/mm/slub.c b/mm/slub.c index 3062f56bf498..56143bfd1ae3 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -1684,10 +1684,7 @@ static inline int free_consistency_checks(struct kmem_cache *s, return 0; if (unlikely(s != slab->slab_cache)) { - if (!folio_test_slab(slab_folio(slab))) { - slab_err(s, slab, "Attempt to free object(0x%p) outside of slab", - object); - } else if (!slab->slab_cache) { + if (!slab->slab_cache) { slab_err(NULL, slab, "No slab cache for object 0x%p", object); } else { |