Message ID | 1579592059-86386-1-git-send-email-alex.shi@linux.alibaba.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | btrfs: remove unused actions | expand |
Hi Alex, I love your patch! Perhaps something to improve: [auto build test WARNING on v5.5-rc7] [also build test WARNING on next-20200123] [cannot apply to btrfs/next] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Alex-Shi/btrfs-remove-unused-actions/20200124-043621 base: def9d2780727cec3313ed3522d0123158d87224d config: powerpc-defconfig (attached as .config) compiler: powerpc64-linux-gcc (GCC) 7.5.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=7.5.0 make.cross ARCH=powerpc If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All warnings (new ones prefixed by >>): fs//btrfs/sysfs.c: In function 'btrfs_sysfs_feature_update': >> fs//btrfs/sysfs.c:1168:2: warning: ignoring return value of 'sysfs_create_group', declared with attribute warn_unused_result [-Wunused-result] sysfs_create_group(fsid_kobj, &btrfs_feature_attr_group); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ vim +/sysfs_create_group +1168 fs//btrfs/sysfs.c 1140 1141 1142 /* 1143 * Change per-fs features in /sys/fs/btrfs/UUID/features to match current 1144 * values in superblock. Call after any changes to incompat/compat_ro flags 1145 */ 1146 void btrfs_sysfs_feature_update(struct btrfs_fs_info *fs_info, 1147 u64 bit, enum btrfs_feature_set set) 1148 { 1149 struct btrfs_fs_devices *fs_devs; 1150 struct kobject *fsid_kobj; 1151 1152 if (!fs_info) 1153 return; 1154 1155 ASSERT(bit & supported_feature_masks[set]); 1156 1157 fs_devs = fs_info->fs_devices; 1158 fsid_kobj = &fs_devs->fsid_kobj; 1159 1160 if (!fsid_kobj->state_initialized) 1161 return; 1162 1163 /* 1164 * FIXME: this is too heavy to update just one value, ideally we'd like 1165 * to use sysfs_update_group but some refactoring is needed first. 1166 */ 1167 sysfs_remove_group(fsid_kobj, &btrfs_feature_attr_group); > 1168 sysfs_create_group(fsid_kobj, &btrfs_feature_attr_group); 1169 } 1170 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation
diff --git a/fs/btrfs/sysfs.c b/fs/btrfs/sysfs.c index 5ebbe8a5ee76..93f870727aa7 100644 --- a/fs/btrfs/sysfs.c +++ b/fs/btrfs/sysfs.c @@ -1148,13 +1148,10 @@ void btrfs_sysfs_feature_update(struct btrfs_fs_info *fs_info, { struct btrfs_fs_devices *fs_devs; struct kobject *fsid_kobj; - u64 features; - int ret; if (!fs_info) return; - features = get_features(fs_info, set); ASSERT(bit & supported_feature_masks[set]); fs_devs = fs_info->fs_devices; @@ -1168,7 +1165,7 @@ void btrfs_sysfs_feature_update(struct btrfs_fs_info *fs_info, * to use sysfs_update_group but some refactoring is needed first. */ sysfs_remove_group(fsid_kobj, &btrfs_feature_attr_group); - ret = sysfs_create_group(fsid_kobj, &btrfs_feature_attr_group); + sysfs_create_group(fsid_kobj, &btrfs_feature_attr_group); } int __init btrfs_init_sysfs(void)
Seems no one care ret and features in func btrfs_sysfs_feature_update, so better to remove them. Signed-off-by: Alex Shi <alex.shi@linux.alibaba.com> Cc: Chris Mason <clm@fb.com> Cc: Josef Bacik <josef@toxicpanda.com> Cc: David Sterba <dsterba@suse.com> Cc: linux-btrfs@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- fs/btrfs/sysfs.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-)