mm/slab_common: Remove the unneeded result variable

ye xingchen · tehcaster · commit 610f9c00ce6e · 2022-08-23T16:03:05.000+02:00
Return the value from __kmem_cache_shrink() directly instead of storing it
 in another redundant variable.

Reported-by: Zeal Robot &lt;zealci@zte.com.cn&gt;
Signed-off-by: ye xingchen &lt;ye.xingchen@zte.com.cn&gt;
Acked-by: Hyeonggon Yoo &lt;42.hyeyoo@gmail.com&gt;
Signed-off-by: Vlastimil Babka &lt;vbabka@suse.cz&gt;
diff --git a/mm/slab_common.c b/mm/slab_common.c
@@ -495,13 +495,9 @@ EXPORT_SYMBOL(kmem_cache_destroy);
  */
 int kmem_cache_shrink(struct kmem_cache *cachep)
 {
-	int ret;
-
-
 	kasan_cache_shrink(cachep);
-	ret = __kmem_cache_shrink(cachep);
 
-	return ret;
+	return __kmem_cache_shrink(cachep);
 }
 EXPORT_SYMBOL(kmem_cache_shrink);
 

Original file line number	Diff line number	Diff line change
`@@ -495,13 +495,9 @@ EXPORT_SYMBOL(kmem_cache_destroy);`
`495`	`495`	`*/`
`496`	`496`	`int kmem_cache_shrink(struct kmem_cache *cachep)`
`497`	`497`	`{`
`498`		`- int ret;`
`499`		`-`
`500`		`-`
`501`	`498`	`kasan_cache_shrink(cachep);`
`502`		`- ret = __kmem_cache_shrink(cachep);`
`503`	`499`
`504`		`- return ret;`
	`500`	`+ return __kmem_cache_shrink(cachep);`
`505`	`501`	`}`
`506`	`502`	`EXPORT_SYMBOL(kmem_cache_shrink);`
`507`	`503`