summaryrefslogtreecommitdiffstats
path: root/mm/util.c
diff options
context:
space:
mode:
authorZhen Lei <thunder.leizhen@huawei.com>2023-08-05 11:17:25 +0800
committerFrederic Weisbecker <frederic@kernel.org>2023-09-13 22:28:59 +0200
commit6e284c55fc0bef7d25fd34d29db11f483da60ea4 (patch)
treeae98df125e1e383ad0b4e86fa16fa28ee9d39ebb /mm/util.c
parentb93c5fe16e4aa177cd072c1c4652cbe1b19a7812 (diff)
downloadlinux-6e284c55fc0bef7d25fd34d29db11f483da60ea4.tar.gz
linux-6e284c55fc0bef7d25fd34d29db11f483da60ea4.tar.bz2
linux-6e284c55fc0bef7d25fd34d29db11f483da60ea4.zip
mm: Remove kmem_valid_obj()
Function kmem_dump_obj() will splat if passed a pointer to a non-slab object. So nothing calls it directly, instead calling kmem_valid_obj() first to determine whether the passed pointer to a valid slab object. This means that merging kmem_valid_obj() into kmem_dump_obj() will make the code more concise. Therefore, convert kmem_dump_obj() to work the same way as vmalloc_dump_obj(), removing the need for the kmem_dump_obj() caller to check kmem_valid_obj(). After this, there are no remaining calls to kmem_valid_obj() anymore, and it can be safely removed. Suggested-by: Matthew Wilcox <willy@infradead.org> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org> Acked-by: Vlastimil Babka <vbabka@suse.cz> Signed-off-by: Paul E. McKenney <paulmck@kernel.org> Signed-off-by: Frederic Weisbecker <frederic@kernel.org>
Diffstat (limited to 'mm/util.c')
-rw-r--r--mm/util.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/mm/util.c b/mm/util.c
index 8cbbfd3a3d59..6eddd891198e 100644
--- a/mm/util.c
+++ b/mm/util.c
@@ -1060,10 +1060,8 @@ void mem_dump_obj(void *object)
{
const char *type;
- if (kmem_valid_obj(object)) {
- kmem_dump_obj(object);
+ if (kmem_dump_obj(object))
return;
- }
if (vmalloc_dump_obj(object))
return;