Hi Aurelien,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on cifs/for-next]
[also build test ERROR on v5.6 next-20200331]
[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/Aurelien-Aptel/cifs-ignore-cache...
base:
git://git.samba.org/sfrench/cifs-2.6.git for-next
config: x86_64-randconfig-s1-20200401 (attached as .config)
compiler: gcc-6 (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
fs/cifs/smb2ops.c: In function 'smb2_close_cached_fid':
> fs/cifs/smb2ops.c:611:3: error: implicit declaration of function
'SMB2_close_flags' [-Werror=implicit-function-declaration]
SMB2_close_flags(0, cfid->tcon, cfid->fid->persistent_fid,
^~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/SMB2_close_flags +611 fs/cifs/smb2ops.c
601
602 static void
603 smb2_close_cached_fid(struct kref *ref)
604 {
605 struct cached_fid *cfid = container_of(ref, struct cached_fid,
606 refcount);
607
608 if (cfid->is_valid) {
609 cifs_dbg(FYI, "clear cached root file handle\n");
610 /* ignore errors here & do not retry in worker thread */
611 SMB2_close_flags(0, cfid->tcon,
cfid->fid->persistent_fid,
612 cfid->fid->volatile_fid, 0);
613 cfid->is_valid = false;
614 cfid->file_all_info_is_valid = false;
615 cfid->has_lease = false;
616 }
617 }
618
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org