Hi Ronnie, Thank you for the patch! Yet something to improve: [auto build test ERROR on cifs/for-next] [also build test ERROR on v5.14-rc6 next-20210816] [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/Ronnie-Sahlberg/cifs-only-compil... base: git://git.samba.org/sfrench/cifs-2.6.git for-next config: x86_64-randconfig-m001-20210816 (attached as .config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=1 build): # https://github.com/0day-ci/linux/commit/baf555d169cb4e93600a23119e9a489aa... git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Ronnie-Sahlberg/cifs-only-compile-in-smb1ops-c-if-we-configure-CIFS_ALLOW_INSECURE_LEGACY/20210817-132616 git checkout baf555d169cb4e93600a23119e9a489aa60edb93 # save the attached .config to linux build tree mkdir build_dir make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash fs/cifs/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All errors (new ones prefixed by >>): 8 | #include <ctype.h> | ^~~~~~~~~ compilation terminated. vim +8 fs/cifs/smb1ops.c 9 #include <linux/pagemap.h> 10 #include <linux/vfs.h> 11 #include "cifsglob.h" 12 #include "cifsproto.h" 13 #include "cifs_debug.h" 14 #include "cifspdu.h" 15 #include "cifs_unicode.h" 16 #include "fs_context.h" 17 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org