Re: [PATCH v4 04/12] kernel: ftrace: export ftrace_sync_ipi

[Alexandre Ghiti <alex@ghiti.fr>]

Hi Andy,

On 09/04/2025 00:31, kernel test robot wrote:
> Hi Andy,
>
> kernel test robot noticed the following build warnings:
>
> [auto build test WARNING on linus/master]
> [also build test WARNING on v6.15-rc1 next-20250408]
> [cannot apply to trace/for-next]
> [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/Andy-Chiu/riscv-ftrace-factor-out-code-defined-by-WITH_ARG/20250408-025114
> base:   linus/master
> patch link:    https://lore.kernel.org/r/20250407180838.42877-4-andybnac%40gmail.com
> patch subject: [PATCH v4 04/12] kernel: ftrace: export ftrace_sync_ipi
> config: xtensa-randconfig-001-20250409 (https://download.01.org/0day-ci/archive/20250409/202504090657.5fiH4UIS-lkp@intel.com/config)
> compiler: xtensa-linux-gcc (GCC) 11.5.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250409/202504090657.5fiH4UIS-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/202504090657.5fiH4UIS-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
>>> kernel/trace/ftrace.c:191:6: warning: no previous prototype for 'ftrace_sync_ipi' [-Wmissing-prototypes]
>       191 | void ftrace_sync_ipi(void *data)
>           |      ^~~~~~~~~~~~~~~
>
>
> vim +/ftrace_sync_ipi +191 kernel/trace/ftrace.c
>
>     190	
>   > 191	void ftrace_sync_ipi(void *data)
>     192	{
>     193		/* Probably not needed, but do it anyway */
>     194		smp_rmb();
>     195	}
>     196	
>

I fixed this with the following that I squashed into your patch, no need 
to resend a new series for this.

diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h
index 0d4eec5747074..30374478cb077 100644
--- a/include/linux/ftrace.h
+++ b/include/linux/ftrace.h
@@ -635,6 +635,8 @@ enum {
  #define ftrace_get_symaddr(fentry_ip) (0)
  #endif

+void ftrace_sync_ipi(void *data);
+
  #ifdef CONFIG_DYNAMIC_FTRACE

  void ftrace_arch_code_modify_prepare(void);
@@ -807,7 +809,6 @@ extern void ftrace_call(void);
  extern void ftrace_regs_call(void);
  extern void mcount_call(void);

-void ftrace_sync_ipi(void *data);
  void ftrace_modify_all_code(int command);

  #ifndef FTRACE_ADDR


Thanks,

Alex