| Message ID | 20181226053744.74740-1-kjlu@umn.edu (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2] btrfs: add a check for sysfs_create_group | expand |
On 2018/12/26 下午1:37, Kangjie Lu wrote: > In case sysfs_create_group fails, let's check its return value and > issues an error message. > > Signed-off-by: Kangjie Lu <kjlu@umn.edu> > --- > fs/btrfs/sysfs.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/btrfs/sysfs.c b/fs/btrfs/sysfs.c > index 3717c864ba23..24ef416e700b 100644 > --- a/fs/btrfs/sysfs.c > +++ b/fs/btrfs/sysfs.c > @@ -889,6 +889,8 @@ void btrfs_sysfs_feature_update(struct btrfs_fs_info *fs_info, > */ > sysfs_remove_group(fsid_kobj, &btrfs_feature_attr_group); > ret = sysfs_create_group(fsid_kobj, &btrfs_feature_attr_group); > + if (ret) > + btrfs_err(fs_info, "failed to create btrfs_feature_attr_group.\n"); Forgot to mention, for btrfs_* infrastructure, no need for the ending '\n'. Despite that, looks good. Reviewed-by: Qu Wenruo <wqu@suse.com> Thanks, Qu > } > > static int btrfs_init_debugfs(void) >
On 12/26/18 1:37 PM, Kangjie Lu wrote: > In case sysfs_create_group fails, let's check its return value and > issues an error message. > > Signed-off-by: Kangjie Lu <kjlu@umn.edu> > --- > fs/btrfs/sysfs.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/btrfs/sysfs.c b/fs/btrfs/sysfs.c > index 3717c864ba23..24ef416e700b 100644 > --- a/fs/btrfs/sysfs.c > +++ b/fs/btrfs/sysfs.c > @@ -889,6 +889,8 @@ void btrfs_sysfs_feature_update(struct btrfs_fs_info *fs_info, > */ > sysfs_remove_group(fsid_kobj, &btrfs_feature_attr_group); > ret = sysfs_create_group(fsid_kobj, &btrfs_feature_attr_group); > + if (ret) > + btrfs_err(fs_info, "failed to create btrfs_feature_attr_group.\n"); NIT: ".\n" is unnecessary. --- Su > } > > static int btrfs_init_debugfs(void) >
On Tue, Dec 25, 2018 at 11:37:44PM -0600, Kangjie Lu wrote: > In case sysfs_create_group fails, let's check its return value and > issues an error message. It's in a function that's not currently used and the sysfs code needs to be reworked. The return code should be passed to the caller, printing error message in this case may not be desired but I can't say right now.
diff --git a/fs/btrfs/sysfs.c b/fs/btrfs/sysfs.c index 3717c864ba23..24ef416e700b 100644 --- a/fs/btrfs/sysfs.c +++ b/fs/btrfs/sysfs.c @@ -889,6 +889,8 @@ void btrfs_sysfs_feature_update(struct btrfs_fs_info *fs_info, */ sysfs_remove_group(fsid_kobj, &btrfs_feature_attr_group); ret = sysfs_create_group(fsid_kobj, &btrfs_feature_attr_group); + if (ret) + btrfs_err(fs_info, "failed to create btrfs_feature_attr_group.\n"); } static int btrfs_init_debugfs(void)
In case sysfs_create_group fails, let's check its return value and issues an error message. Signed-off-by: Kangjie Lu <kjlu@umn.edu> --- fs/btrfs/sysfs.c | 2 ++ 1 file changed, 2 insertions(+)