summary refs log tree commit diff
diff options
context:
space:
mode:
authorye xingchen <ye.xingchen@zte.com.cn>2022-08-26 07:19:06 +0000
committerAndrew Morton <akpm@linux-foundation.org>2022-09-11 20:26:02 -0700
commit3083da7bcf56a4922b996ea3551847488a43a8b6 (patch)
tree6391fe2c725edda005e5730d8f5a654df94bc922
parentc4f20f1479c456d9dd1c1e6d8bf956a25de742dc (diff)
downloadlinux-3083da7bcf56a4922b996ea3551847488a43a8b6.tar.gz
mm: backing-dev: Remove the unneeded result variable
Return the value cgwb_bdi_init() directly instead of storing it in another
redundant variable.

Link: https://lkml.kernel.org/r/20220826071906.252419-1-ye.xingchen@zte.com.cn
Signed-off-by: ye xingchen <ye.xingchen@zte.com.cn>
Reported-by: Zeal Robot <zealci@zte.com.cn>
Reviewed-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
-rw-r--r--mm/backing-dev.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/mm/backing-dev.c b/mm/backing-dev.c
index de65cb1e5f76..c30419a5e119 100644
--- a/mm/backing-dev.c
+++ b/mm/backing-dev.c
@@ -776,8 +776,6 @@ static void cgwb_remove_from_bdi_list(struct bdi_writeback *wb)
 
 int bdi_init(struct backing_dev_info *bdi)
 {
-	int ret;
-
 	bdi->dev = NULL;
 
 	kref_init(&bdi->refcnt);
@@ -788,9 +786,7 @@ int bdi_init(struct backing_dev_info *bdi)
 	INIT_LIST_HEAD(&bdi->wb_list);
 	init_waitqueue_head(&bdi->wb_waitq);
 
-	ret = cgwb_bdi_init(bdi);
-
-	return ret;
+	return cgwb_bdi_init(bdi);
 }
 
 struct backing_dev_info *bdi_alloc(int node_id)