Messages in this thread | | | Date | Fri, 15 Apr 2022 18:36:12 +0800 | From | kernel test robot <> | Subject | [ammarfaizi2-block:palmer/linux/riscv-compat 20/20] arch/riscv/kernel/compat_signal.c:198:5: warning: no previous prototype for function 'compat_setup_rt_frame' |
| |
tree: https://github.com/ammarfaizi2/linux-block palmer/linux/riscv-compat head: d84810b12dce65023c222aea22d30e2b22e3016a commit: d84810b12dce65023c222aea22d30e2b22e3016a [20/20] riscv: compat: Add COMPAT Kbuild skeletal support config: riscv-randconfig-r015-20220414 (https://download.01.org/0day-ci/archive/20220415/202204151803.TXKhZhma-lkp@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 6b7e6ea489f6dd45a9b0da9ac20871560917b9b0) 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 # install riscv cross compiling tool for clang build # apt-get install binutils-riscv64-linux-gnu # https://github.com/ammarfaizi2/linux-block/commit/d84810b12dce65023c222aea22d30e2b22e3016a git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block git fetch --no-tags ammarfaizi2-block palmer/linux/riscv-compat git checkout d84810b12dce65023c222aea22d30e2b22e3016a # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash arch/riscv/kernel/
If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
>> arch/riscv/kernel/compat_signal.c:198:5: warning: no previous prototype for function 'compat_setup_rt_frame' [-Wmissing-prototypes] int compat_setup_rt_frame(struct ksignal *ksig, sigset_t *set, ^ arch/riscv/kernel/compat_signal.c:198:1: note: declare 'static' if the function is not intended to be used outside of this translation unit int compat_setup_rt_frame(struct ksignal *ksig, sigset_t *set, ^ static 1 warning generated.
vim +/compat_setup_rt_frame +198 arch/riscv/kernel/compat_signal.c
7b07d6662e1218 Guo Ren 2022-04-05 197 7b07d6662e1218 Guo Ren 2022-04-05 @198 int compat_setup_rt_frame(struct ksignal *ksig, sigset_t *set, 7b07d6662e1218 Guo Ren 2022-04-05 199 struct pt_regs *regs) 7b07d6662e1218 Guo Ren 2022-04-05 200 { 7b07d6662e1218 Guo Ren 2022-04-05 201 struct compat_rt_sigframe __user *frame; 7b07d6662e1218 Guo Ren 2022-04-05 202 long err = 0; 7b07d6662e1218 Guo Ren 2022-04-05 203 7b07d6662e1218 Guo Ren 2022-04-05 204 frame = compat_get_sigframe(ksig, regs, sizeof(*frame)); 7b07d6662e1218 Guo Ren 2022-04-05 205 if (!access_ok(frame, sizeof(*frame))) 7b07d6662e1218 Guo Ren 2022-04-05 206 return -EFAULT; 7b07d6662e1218 Guo Ren 2022-04-05 207 7b07d6662e1218 Guo Ren 2022-04-05 208 err |= copy_siginfo_to_user32(&frame->info, &ksig->info); 7b07d6662e1218 Guo Ren 2022-04-05 209 7b07d6662e1218 Guo Ren 2022-04-05 210 /* Create the ucontext. */ 7b07d6662e1218 Guo Ren 2022-04-05 211 err |= __put_user(0, &frame->uc.uc_flags); 7b07d6662e1218 Guo Ren 2022-04-05 212 err |= __put_user(NULL, &frame->uc.uc_link); 7b07d6662e1218 Guo Ren 2022-04-05 213 err |= __compat_save_altstack(&frame->uc.uc_stack, regs->sp); 7b07d6662e1218 Guo Ren 2022-04-05 214 err |= compat_setup_sigcontext(frame, regs); 7b07d6662e1218 Guo Ren 2022-04-05 215 err |= __copy_to_user(&frame->uc.uc_sigmask, set, sizeof(*set)); 7b07d6662e1218 Guo Ren 2022-04-05 216 if (err) 7b07d6662e1218 Guo Ren 2022-04-05 217 return -EFAULT; 7b07d6662e1218 Guo Ren 2022-04-05 218 7b07d6662e1218 Guo Ren 2022-04-05 219 regs->ra = (unsigned long)COMPAT_VDSO_SYMBOL( 7b07d6662e1218 Guo Ren 2022-04-05 220 current->mm->context.vdso, rt_sigreturn); 7b07d6662e1218 Guo Ren 2022-04-05 221 7b07d6662e1218 Guo Ren 2022-04-05 222 /* 7b07d6662e1218 Guo Ren 2022-04-05 223 * Set up registers for signal handler. 7b07d6662e1218 Guo Ren 2022-04-05 224 * Registers that we don't modify keep the value they had from 7b07d6662e1218 Guo Ren 2022-04-05 225 * user-space at the time we took the signal. 7b07d6662e1218 Guo Ren 2022-04-05 226 * We always pass siginfo and mcontext, regardless of SA_SIGINFO, 7b07d6662e1218 Guo Ren 2022-04-05 227 * since some things rely on this (e.g. glibc's debug/segfault.c). 7b07d6662e1218 Guo Ren 2022-04-05 228 */ 7b07d6662e1218 Guo Ren 2022-04-05 229 regs->epc = (unsigned long)ksig->ka.sa.sa_handler; 7b07d6662e1218 Guo Ren 2022-04-05 230 regs->sp = (unsigned long)frame; 7b07d6662e1218 Guo Ren 2022-04-05 231 regs->a0 = ksig->sig; /* a0: signal number */ 7b07d6662e1218 Guo Ren 2022-04-05 232 regs->a1 = (unsigned long)(&frame->info); /* a1: siginfo pointer */ 7b07d6662e1218 Guo Ren 2022-04-05 233 regs->a2 = (unsigned long)(&frame->uc); /* a2: ucontext pointer */ 7b07d6662e1218 Guo Ren 2022-04-05 234
:::::: The code at line 198 was first introduced by commit :::::: 7b07d6662e1218841c2b618ab9a9255440ccee85 riscv: compat: signal: Add rt_frame implementation
:::::: TO: Guo Ren <guoren@linux.alibaba.com> :::::: CC: Palmer Dabbelt <palmer@rivosinc.com>
-- 0-DAY CI Kernel Test Service https://01.org/lkp
| |