Re: [PATCH v2 2/3] sysctl: add support for drop_caches for individual filesystem

[kernel test robot <lkp@intel.com>]

Hi Ye,

kernel test robot noticed the following build errors:

[auto build test ERROR on viro-vfs/for-next]
[also build test ERROR on linus/master brauner-vfs/vfs.all v6.18-rc6 next-20251117]
[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#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Ye-Bin/vfs-introduce-reclaim_icache_sb-and-reclaim_dcache_sb-helper/20251117-193502
base:   https://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git for-next
patch link:    https://lore.kernel.org/r/20251117112735.4170831-3-yebin%40huaweicloud.com
patch subject: [PATCH v2 2/3] sysctl: add support for drop_caches for individual filesystem
config: alpha-allnoconfig (https://download.01.org/0day-ci/archive/20251117/202511172139.326qNHOk-lkp@intel.com/config)
compiler: alpha-linux-gcc (GCC) 15.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251117/202511172139.326qNHOk-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202511172139.326qNHOk-lkp@intel.com/

All errors (new ones prefixed by >>):

   fs/drop_caches.c: In function 'drop_fs_caches':
>> fs/drop_caches.c:107:25: error: implicit declaration of function 'syscall_set_return_value'; did you mean 'syscall_get_return_value'? [-Wimplicit-function-declaration]
     107 |                         syscall_set_return_value(current,
         |                         ^~~~~~~~~~~~~~~~~~~~~~~~
         |                         syscall_get_return_value


vim +107 fs/drop_caches.c

    91	
    92	static void drop_fs_caches(struct callback_head *twork)
    93	{
    94		int ret;
    95		struct super_block *sb;
    96		static bool suppress;
    97		struct drop_fs_caches_work *work = container_of(twork,
    98				struct drop_fs_caches_work, task_work);
    99		unsigned int ctl = work->ctl;
   100		dev_t dev = work->dev;
   101	
   102		if (work->path) {
   103			struct path path;
   104	
   105			ret = kern_path(work->path, LOOKUP_FOLLOW, &path);
   106			if (ret) {
 > 107				syscall_set_return_value(current,
   108							 current_pt_regs(),
   109							 0, ret);
   110				goto out;
   111			}
   112			dev = path.dentry->d_sb->s_dev;
   113			/* Make this file's dentry and inode recyclable */
   114			path_put(&path);
   115		}
   116	
   117		sb = user_get_super(dev, false);
   118		if (!sb) {
   119			syscall_set_return_value(current, current_pt_regs(), 0,
   120						 -EINVAL);
   121			goto out;
   122		}
   123	
   124		if (ctl & BIT(0)) {
   125			lru_add_drain_all();
   126			drop_pagecache_sb(sb, NULL);
   127			count_vm_event(DROP_PAGECACHE);
   128		}
   129	
   130		if (ctl & BIT(1)) {
   131			reclaim_dcache_sb(sb);
   132			reclaim_icache_sb(sb);
   133			count_vm_event(DROP_SLAB);
   134		}
   135	
   136		if (!READ_ONCE(suppress)) {
   137			pr_info("%s (%d): %s: %d %u:%u\n", current->comm,
   138				task_pid_nr(current), __func__, ctl,
   139				MAJOR(sb->s_dev), MINOR(sb->s_dev));
   140	
   141			if (ctl & BIT(2))
   142				WRITE_ONCE(suppress, true);
   143		}
   144	
   145		drop_super(sb);
   146	out:
   147		kfree(work->path);
   148		kfree(work);
   149	}
   150	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki