Hi Milind,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on ceph-client/for-linus]
[also build test ERROR on v5.17-rc4 next-20220211]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url:
https://github.com/0day-ci/linux/commits/Milind-Changire/ceph-add-getvxat...
base:
https://github.com/ceph/ceph-client.git for-linus
config: arc-randconfig-r043-20220213
(
https://download.01.org/0day-ci/archive/20220214/202202140832.lVpTfiK0-lk...)
compiler: arc-elf-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
#
https://github.com/0day-ci/linux/commit/a3c4729dbc82d1234f75f72b863709bda...
git remote add linux-review
https://github.com/0day-ci/linux
git fetch --no-tags linux-review
Milind-Changire/ceph-add-getvxattr-support/20220211-201353
git checkout a3c4729dbc82d1234f75f72b863709bdab51ce99
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir
ARCH=arc SHELL=/bin/bash fs/ceph/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
fs/ceph/mds_client.c: In function 'parse_reply_info_getvxattr':
> fs/ceph/mds_client.c:563:13: error: variable 'struct_len'
set but not used [-Werror=unused-but-set-variable]
563 | u32
struct_len;
| ^~~~~~~~~~
> fs/ceph/mds_client.c:562:22: error: variable
'struct_compat' set but not used [-Werror=unused-but-set-variable]
562
| u8 struct_v, struct_compat;
| ^~~~~~~~~~~~~
> fs/ceph/mds_client.c:562:12: error: variable 'struct_v'
set but not used [-Werror=unused-but-set-variable]
562 | u8 struct_v,
struct_compat;
| ^~~~~~~~
cc1: all warnings being treated as errors
vim +/struct_len +563 fs/ceph/mds_client.c
557
558 static int parse_reply_info_getvxattr(void **p, void *end,
559 struct ceph_mds_reply_info_parsed *info,
560 u64 features)
561 {
562 u8 struct_v, struct_compat;
563 u32 struct_len;
564 u32 value_len;
565
566 ceph_decode_8_safe(p, end, struct_v, bad);
567 ceph_decode_8_safe(p, end, struct_compat, bad);
568 ceph_decode_32_safe(p, end, struct_len, bad);
569 ceph_decode_32_safe(p, end, value_len, bad);
570
571 if (value_len == end - *p) {
572 info->xattr_info.xattr_value = *p;
573 info->xattr_info.xattr_value_len = value_len;
574 *p = end;
575 return value_len;
576 }
577 bad:
578 return -EIO;
579 }
580
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org