From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751522AbdG0Fgb (ORCPT ); Thu, 27 Jul 2017 01:36:31 -0400 Received: from szxga01-in.huawei.com ([45.249.212.187]:10286 "EHLO szxga01-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750859AbdG0Fga (ORCPT ); Thu, 27 Jul 2017 01:36:30 -0400 Subject: Re: [f2fs-dev] [PATCH 1/2] f2fs: expose /sys/fs/f2fs/features To: Jaegeuk Kim , Chao Yu CC: , , References: <20170726012921.53002-1-jaegeuk@kernel.org> <765ab60d-b0fc-7233-83b5-86a617e4134f@kernel.org> <20170726214326.GC66153@jaegeuk-macbookpro.roam.corp.google.com> From: Chao Yu Message-ID: <3e7ed741-8aee-02a3-a04a-b62a72beb8ef@huawei.com> Date: Thu, 27 Jul 2017 13:36:06 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20170726214326.GC66153@jaegeuk-macbookpro.roam.corp.google.com> Content-Type: text/plain; charset="windows-1252" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.134.22.195] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020201.59797BCB.006E,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 8d33d3ed1ebf27f32de94261dbc05ab3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2017/7/27 5:43, Jaegeuk Kim wrote: > On 07/26, Chao Yu wrote: >> On 2017/7/26 9:29, Jaegeuk Kim wrote: >>> This patch exposes what features are supported by current f2fs build to sysfs >>> entry. >>> >>> Signed-off-by: Jaegeuk Kim >> >> Reviewed-by: Chao Yu >> >> Minor thing, can you exchange below function name to follow ext4: >> f2fs_init_sysfs <-> f2fs_register_sysfs >> f2fs_exit_sysfs <-> f2fs_unregister_sysfs >> >>>From 4cf8a3138f3ee171750ae279fefd4de672a0ee29 Mon Sep 17 00:00:00 2001 > From: Jaegeuk Kim > Date: Wed, 26 Jul 2017 11:24:13 -0700 > Subject: [PATCH] f2fs: avoid naming confusion of sysfs init > > This patch changes the function names of sysfs init to follow ext4. > > f2fs_init_sysfs <-> f2fs_register_sysfs > f2fs_exit_sysfs <-> f2fs_unregister_sysfs > > Suggested-by: Chao Yu > Signed-off-by: Jaegeuk Kim Reivewed-by: Chao Yu Thanks, > --- > fs/f2fs/f2fs.h | 8 ++++---- > fs/f2fs/super.c | 10 +++++----- > fs/f2fs/sysfs.c | 8 ++++---- > 3 files changed, 13 insertions(+), 13 deletions(-) > > diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h > index d486539086b6..a1bfd0b9be44 100644 > --- a/fs/f2fs/f2fs.h > +++ b/fs/f2fs/f2fs.h > @@ -2820,10 +2820,10 @@ void destroy_extent_cache(void); > /* > * sysfs.c > */ > -int __init f2fs_register_sysfs(void); > -void f2fs_unregister_sysfs(void); > -int f2fs_init_sysfs(struct f2fs_sb_info *sbi); > -void f2fs_exit_sysfs(struct f2fs_sb_info *sbi); > +int __init f2fs_init_sysfs(void); > +void f2fs_exit_sysfs(void); > +int f2fs_register_sysfs(struct f2fs_sb_info *sbi); > +void f2fs_unregister_sysfs(struct f2fs_sb_info *sbi); > > /* > * crypto support > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c > index 1241739f4d54..12513caabe6e 100644 > --- a/fs/f2fs/super.c > +++ b/fs/f2fs/super.c > @@ -649,7 +649,7 @@ static void f2fs_put_super(struct super_block *sb) > > kfree(sbi->ckpt); > > - f2fs_exit_sysfs(sbi); > + f2fs_unregister_sysfs(sbi); > > sb->s_fs_info = NULL; > if (sbi->s_chksum_driver) > @@ -2145,7 +2145,7 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > goto free_root_inode; > } > > - err = f2fs_init_sysfs(sbi); > + err = f2fs_register_sysfs(sbi); > if (err) > goto free_root_inode; > > @@ -2216,7 +2216,7 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent) > > free_sysfs: > f2fs_sync_inode_meta(sbi); > - f2fs_exit_sysfs(sbi); > + f2fs_unregister_sysfs(sbi); > free_root_inode: > dput(sb->s_root); > sb->s_root = NULL; > @@ -2334,7 +2334,7 @@ static int __init init_f2fs_fs(void) > err = create_extent_cache(); > if (err) > goto free_checkpoint_caches; > - err = f2fs_register_sysfs(); > + err = f2fs_init_sysfs(); > if (err) > goto free_extent_cache; > err = register_shrinker(&f2fs_shrinker_info); > @@ -2373,7 +2373,7 @@ static void __exit exit_f2fs_fs(void) > f2fs_destroy_root_stats(); > unregister_filesystem(&f2fs_fs_type); > unregister_shrinker(&f2fs_shrinker_info); > - f2fs_unregister_sysfs(); > + f2fs_exit_sysfs(); > destroy_extent_cache(); > destroy_checkpoint_caches(); > destroy_segment_manager_caches(); > diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c > index 71191d89917d..5a78b9af92ef 100644 > --- a/fs/f2fs/sysfs.c > +++ b/fs/f2fs/sysfs.c > @@ -304,7 +304,7 @@ static const struct file_operations f2fs_seq_##_name##_fops = { \ > F2FS_PROC_FILE_DEF(segment_info); > F2FS_PROC_FILE_DEF(segment_bits); > > -int __init f2fs_register_sysfs(void) > +int __init f2fs_init_sysfs(void) > { > f2fs_proc_root = proc_mkdir("fs/f2fs", NULL); > > @@ -314,13 +314,13 @@ int __init f2fs_register_sysfs(void) > return 0; > } > > -void f2fs_unregister_sysfs(void) > +void f2fs_exit_sysfs(void) > { > kset_unregister(f2fs_kset); > remove_proc_entry("fs/f2fs", NULL); > } > > -int f2fs_init_sysfs(struct f2fs_sb_info *sbi) > +int f2fs_register_sysfs(struct f2fs_sb_info *sbi) > { > struct super_block *sb = sbi->sb; > int err; > @@ -351,7 +351,7 @@ int f2fs_init_sysfs(struct f2fs_sb_info *sbi) > return err; > } > > -void f2fs_exit_sysfs(struct f2fs_sb_info *sbi) > +void f2fs_unregister_sysfs(struct f2fs_sb_info *sbi) > { > kobject_del(&sbi->s_kobj); > kobject_put(&sbi->s_kobj); >