[tip:timers/cleanups 5/10] kernel/time/timer_list.c:49:77: error: too few arguments provided to function-like macro invocation

[tip:timers/cleanups 5/10] kernel/time/timer_list.c:49:77: error: too few arguments provided to function-like macro invocation

[kernel test robot]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/cleanups
head:   5c4da3a96bf484f965057c281f1ef48ac46987bc
commit: 9453228aa82f20c07670d22c3d54f1be6c4244b4 [5/10] hrtimers: Make callback function pointer private
config: hexagon-randconfig-002-20250404 (https://download.01.org/0day-ci/archive/20250405/202504050045.fyyfMGEQ-lkp@intel.com/config)
compiler: clang version 21.0.0git (https://github.com/llvm/llvm-project 92c93f5286b9ff33f27ff694d2dc33da1c07afdd)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250405/202504050045.fyyfMGEQ-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/202504050045.fyyfMGEQ-lkp@intel.com/

All errors (new ones prefixed by >>):

>> kernel/time/timer_list.c:49:77: error: too few arguments provided to function-like macro invocation
      49 |         SEQ_printf(m, " #%d: <%p>, %ps", idx, taddr, ACCESS_PRIVATE(timer->function));
         |                                                                                    ^
   include/linux/compiler_types.h:78:10: note: macro 'ACCESS_PRIVATE' defined here
      78 | # define ACCESS_PRIVATE(p, member) ((p)->member)
         |          ^
>> kernel/time/timer_list.c:49:47: error: use of undeclared identifier 'ACCESS_PRIVATE'
      49 |         SEQ_printf(m, " #%d: <%p>, %ps", idx, taddr, ACCESS_PRIVATE(timer->function));
         |                                                      ^
   2 errors generated.


vim +49 kernel/time/timer_list.c

    44	
    45	static void
    46	print_timer(struct seq_file *m, struct hrtimer *taddr, struct hrtimer *timer,
    47		    int idx, u64 now)
    48	{
  > 49		SEQ_printf(m, " #%d: <%p>, %ps", idx, taddr, ACCESS_PRIVATE(timer->function));
    50		SEQ_printf(m, ", S:%02x", timer->state);
    51		SEQ_printf(m, "\n");
    52		SEQ_printf(m, " # expires at %Lu-%Lu nsecs [in %Ld to %Ld nsecs]\n",
    53			(unsigned long long)ktime_to_ns(hrtimer_get_softexpires(timer)),
    54			(unsigned long long)ktime_to_ns(hrtimer_get_expires(timer)),
    55			(long long)(ktime_to_ns(hrtimer_get_softexpires(timer)) - now),
    56			(long long)(ktime_to_ns(hrtimer_get_expires(timer)) - now));
    57	}
    58	

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

Re: [tip:timers/cleanups 5/10] kernel/time/timer_list.c:49:77: error: too few arguments provided to function-like macro invocation

[Nam Cao]

On Sat, Apr 05, 2025 at 01:04:04AM +0800, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/cleanups
> head:   5c4da3a96bf484f965057c281f1ef48ac46987bc
> commit: 9453228aa82f20c07670d22c3d54f1be6c4244b4 [5/10] hrtimers: Make callback function pointer private
...
> 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/202504050045.fyyfMGEQ-lkp@intel.com/
> 
> All errors (new ones prefixed by >>):
> 
> >> kernel/time/timer_list.c:49:77: error: too few arguments provided to function-like macro invocation
>       49 |         SEQ_printf(m, " #%d: <%p>, %ps", idx, taddr, ACCESS_PRIVATE(timer->function));
>          |                                                                                    ^
>    include/linux/compiler_types.h:78:10: note: macro 'ACCESS_PRIVATE' defined here
>       78 | # define ACCESS_PRIVATE(p, member) ((p)->member)
>          |          ^
> >> kernel/time/timer_list.c:49:47: error: use of undeclared identifier 'ACCESS_PRIVATE'
>       49 |         SEQ_printf(m, " #%d: <%p>, %ps", idx, taddr, ACCESS_PRIVATE(timer->function));
>          |                                                      ^
>    2 errors generated.

Not sure how did this even get pass by build-test..

Patch below cures it. Thomas, can you stuff it into tip tree, or what do
you prefer?

Best regards,
Nam

diff --git a/kernel/time/timer_list.c b/kernel/time/timer_list.c
index 51fa7ae5e2dd..b03d0ada6469 100644
--- a/kernel/time/timer_list.c
+++ b/kernel/time/timer_list.c
@@ -46,7 +46,7 @@ static void
 print_timer(struct seq_file *m, struct hrtimer *taddr, struct hrtimer *timer,
 	    int idx, u64 now)
 {
-	SEQ_printf(m, " #%d: <%p>, %ps", idx, taddr, ACCESS_PRIVATE(timer->function));
+	SEQ_printf(m, " #%d: <%p>, %ps", idx, taddr, ACCESS_PRIVATE(timer, function));
 	SEQ_printf(m, ", S:%02x", timer->state);
 	SEQ_printf(m, "\n");
 	SEQ_printf(m, " # expires at %Lu-%Lu nsecs [in %Ld to %Ld nsecs]\n",