From: Haibo Liu <haiboliu6@gmail.com>
To: djwong@kernel.org, linux-ext4@vger.kernel.org
Cc: haiboliu6@gmail.com
Subject: [v2] ext4/super.c : Fix a goto label
Date: Tue, 29 Aug 2023 15:52:22 +0800 [thread overview]
Message-ID: <20230829075222.50962-1-haiboliu6@gmail.com> (raw)
In-Reply-To: <20230828092726.19400-1-haiboliu6@gmail.com>
Thank you for Darrick J. Wong's suggestions :).
I wrote a new patch and renamed these 9 labels.
Original labels -> New labels:
out -> out_unregister_ext23_and_dentry_cache
out05 -> out_inodecache
out1 -> out_mballoc
out2 -> out_sysfs
out3 -> out_system_zone
out4 -> out_pageio
out5 -> out_post_read_processing
out6 -> out_pending
out7 -> out_es
v1->v2:
Followed Darrick J. Wong's suggestions, renamed these 9 goto labels.
Signed-off-by: Haibo Liu <haiboliu6@gmail.com>
---
fs/ext4/super.c | 36 ++++++++++++++++++------------------
1 file changed, 18 insertions(+), 18 deletions(-)
diff --git a/fs/ext4/super.c b/fs/ext4/super.c
index 91f20afa1d71..11cffb5a05a4 100644
--- a/fs/ext4/super.c
+++ b/fs/ext4/super.c
@@ -7347,61 +7347,61 @@ static int __init ext4_init_fs(void)
err = ext4_init_pending();
if (err)
- goto out7;
+ goto out_es;
err = ext4_init_post_read_processing();
if (err)
- goto out6;
+ goto out_pending;
err = ext4_init_pageio();
if (err)
- goto out5;
+ goto out_post_read_processing;
err = ext4_init_system_zone();
if (err)
- goto out4;
+ goto out_pageio;
err = ext4_init_sysfs();
if (err)
- goto out3;
+ goto out_system_zone;
err = ext4_init_mballoc();
if (err)
- goto out2;
+ goto out_sysfs;
err = init_inodecache();
if (err)
- goto out1;
+ goto out_mballoc;
err = ext4_fc_init_dentry_cache();
if (err)
- goto out05;
+ goto out_inodecache;
register_as_ext3();
register_as_ext2();
err = register_filesystem(&ext4_fs_type);
if (err)
- goto out;
+ goto out_unregister_ext23_and_dentry_cache;
return 0;
-out:
+out_unregister_ext23_and_dentry_cache:
unregister_as_ext2();
unregister_as_ext3();
ext4_fc_destroy_dentry_cache();
-out05:
+out_inodecache:
destroy_inodecache();
-out1:
+out_mballoc:
ext4_exit_mballoc();
-out2:
+out_sysfs:
ext4_exit_sysfs();
-out3:
+out_system_zone:
ext4_exit_system_zone();
-out4:
+out_pageio:
ext4_exit_pageio();
-out5:
+out_post_read_processing:
ext4_exit_post_read_processing();
-out6:
+out_pending:
ext4_exit_pending();
-out7:
+out_es:
ext4_exit_es();
return err;
--
2.34.1
next prev parent reply other threads:[~2023-08-29 7:55 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-28 9:27 ext4/super.c : Fix a goto label Haibo Liu
2023-08-28 16:18 ` Darrick J. Wong
2023-08-29 7:52 ` Haibo Liu [this message]
2024-01-02 14:22 ` [v2] " Jan Kara
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230829075222.50962-1-haiboliu6@gmail.com \
--to=haiboliu6@gmail.com \
--cc=djwong@kernel.org \
--cc=linux-ext4@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox