diff mbox series

[1/2] syscall: System call numbers are word sized

Message ID 20230110141616.1449-1-rpalethorpe@suse.com
State Rejected
Headers show
Series [1/2] syscall: System call numbers are word sized | expand

Commit Message

Richard Palethorpe Jan. 10, 2023, 2:16 p.m. UTC
Signed-off-by: Richard Palethorpe <rpalethorpe@suse.com>
---
 include/lapi/syscalls/regen.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Petr Vorel Jan. 10, 2023, 7:32 p.m. UTC | #1
Hi Richie,

> Signed-off-by: Richard Palethorpe <rpalethorpe@suse.com>
> ---
>  include/lapi/syscalls/regen.sh | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

> diff --git a/include/lapi/syscalls/regen.sh b/include/lapi/syscalls/regen.sh
> index 97027e2f3..217595f34 100755
> --- a/include/lapi/syscalls/regen.sh
> +++ b/include/lapi/syscalls/regen.sh
> @@ -38,12 +38,12 @@ cat << EOF > "${output_pid}"
>  #ifdef TST_TEST_H__
>  #define TST_SYSCALL_BRK__(NR, SNR) ({ \\
>  	tst_brk(TCONF, \\
> -		"syscall(%d) " SNR " not supported on your arch", NR); \\
> +		"syscall(%ld) " SNR " not supported on your arch", NR); \\

Looks like quite old change:

36283230cb ("linux_syscall_numbers: add syscall number to output")

>  })
>  #else
>  #define TST_SYSCALL_BRK__(NR, SNR) ({ \\
>  	tst_brkm(TCONF, CLEANUP, \\
> -		"syscall(%d) " SNR " not supported on your arch", NR); \\
> +		"syscall(%ld) " SNR " not supported on your arch", NR); \\
>  })
>  #endif

I expected this would fix several warnings, but it generated several warnings on
both x86_64 and i686 due tst_syscall() being used on many places. To show just
few:

$ make autotools && ./configure && cd lib/newlib_tests && rm -f tst_expiration_timer && make tst_expiration_timer
In file included from tst_expiration_timer.c:11:
../../include/tst_timer.h: In function ‘sys_clock_getres’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:204:16: note: in expansion of macro ‘tst_syscall’
  204 |         return tst_syscall(__NR_clock_getres, clk_id, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_clock_getres64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:209:16: note: in expansion of macro ‘tst_syscall’
  209 |         return tst_syscall(__NR_clock_getres_time64, clk_id, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_clock_gettime’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:219:16: note: in expansion of macro ‘tst_syscall’
  219 |         return tst_syscall(__NR_clock_gettime, clk_id, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_clock_gettime64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:224:16: note: in expansion of macro ‘tst_syscall’
  224 |         return tst_syscall(__NR_clock_gettime64, clk_id, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_clock_settime’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:234:16: note: in expansion of macro ‘tst_syscall’
  234 |         return tst_syscall(__NR_clock_settime, clk_id, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_clock_settime64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:239:16: note: in expansion of macro ‘tst_syscall’
  239 |         return tst_syscall(__NR_clock_settime64, clk_id, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_clock_nanosleep’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:251:16: note: in expansion of macro ‘tst_syscall’
  251 |         return tst_syscall(__NR_clock_nanosleep, clk_id, flags,
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_clock_nanosleep64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:258:16: note: in expansion of macro ‘tst_syscall’
  258 |         return tst_syscall(__NR_clock_nanosleep_time64, clk_id, flags,
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_futex’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:265:16: note: in expansion of macro ‘tst_syscall’
  265 |         return tst_syscall(__NR_futex, uaddr, futex_op, val, to, uaddr2, val3);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_futex_time64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:271:16: note: in expansion of macro ‘tst_syscall’
  271 |         return tst_syscall(__NR_futex_time64, uaddr, futex_op, val, to, uaddr2, val3);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_mq_timedsend’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:283:16: note: in expansion of macro ‘tst_syscall’
  283 |         return tst_syscall(__NR_mq_timedsend, mqdes, msg_ptr, msg_len, msg_prio,
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_mq_timedsend64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:290:16: note: in expansion of macro ‘tst_syscall’
  290 |         return tst_syscall(__NR_mq_timedsend_time64, mqdes, msg_ptr, msg_len,
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_mq_timedreceive’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:303:16: note: in expansion of macro ‘tst_syscall’
  303 |         return tst_syscall(__NR_mq_timedreceive, mqdes, msg_ptr, msg_len,
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_mq_timedreceive64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:310:16: note: in expansion of macro ‘tst_syscall’
  310 |         return tst_syscall(__NR_mq_timedreceive_time64, mqdes, msg_ptr, msg_len,
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_sched_rr_get_interval’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:321:16: note: in expansion of macro ‘tst_syscall’
  321 |         return tst_syscall(__NR_sched_rr_get_interval, pid, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_sched_rr_get_interval64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:326:16: note: in expansion of macro ‘tst_syscall’
  326 |         return tst_syscall(__NR_sched_rr_get_interval_time64, pid, ts);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timer_gettime’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:331:16: note: in expansion of macro ‘tst_syscall’
  331 |         return tst_syscall(__NR_timer_gettime, timerid, its);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timer_gettime64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:336:16: note: in expansion of macro ‘tst_syscall’
  336 |         return tst_syscall(__NR_timer_gettime64, timerid, its);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timer_settime’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:342:16: note: in expansion of macro ‘tst_syscall’
  342 |         return tst_syscall(__NR_timer_settime, timerid, flags, its, old_its);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timer_settime64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:348:16: note: in expansion of macro ‘tst_syscall’
  348 |         return tst_syscall(__NR_timer_settime64, timerid, flags, its, old_its);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timerfd_gettime’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:353:16: note: in expansion of macro ‘tst_syscall’
  353 |         return tst_syscall(__NR_timerfd_gettime, fd, its);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timerfd_gettime64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:358:16: note: in expansion of macro ‘tst_syscall’
  358 |         return tst_syscall(__NR_timerfd_gettime64, fd, its);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timerfd_settime’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:364:16: note: in expansion of macro ‘tst_syscall’
  364 |         return tst_syscall(__NR_timerfd_settime, fd, flags, its, old_its);
      |                ^~~~~~~~~~~
../../include/tst_timer.h: In function ‘sys_timerfd_settime64’:
../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../include/tst_timer.h:370:16: note: in expansion of macro ‘tst_syscall’
  370 |         return tst_syscall(__NR_timerfd_settime64, fd, flags, its, old_its);
      |                ^~~~~~~~~~~
CC lib/newlib_tests/tst_expiration_timer


$ cd ../../testcases/kernel/syscalls/clone && rm -f clone09 && make clone09

../../../../include/lapi/sched.h: In function ‘clone3’:
../../../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../../../include/lapi/sched.h:61:16: note: in expansion of macro ‘tst_syscall’
   61 |         return tst_syscall(__NR_clone3, args, size);
      |                ^~~~~~~~~~~
../../../../include/lapi/sched.h: In function ‘clone3_supported_by_kernel’:
../../../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
../../../../include/lapi/sched.h:69:17: note: in expansion of macro ‘tst_syscall’
   69 |                 tst_syscall(__NR_clone3, NULL, 0);
      |                 ^~~~~~~~~~~
clone09.c: In function ‘newnet’:
../../../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
   81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
      |                                                       ^~~~~~~~~
../../../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
   23 |         tst_brk(TCONF, \
      |         ^~~~~~~
../../../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
   42 |                 TST_SYSCALL_BRK__(NR, #NR); \
      |                 ^~~~~~~~~~~~~~~~~
clone09.c:38:9: note: in expansion of macro ‘tst_syscall’
   38 |         tst_syscall(__NR_exit, 0);


Kind regards,
Petr
Richard Palethorpe Jan. 12, 2023, 10:06 a.m. UTC | #2
Hello,

Petr Vorel <pvorel@suse.cz> writes:

> Hi Richie,
>
>> Signed-off-by: Richard Palethorpe <rpalethorpe@suse.com>
>> ---
>>  include/lapi/syscalls/regen.sh | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>
>> diff --git a/include/lapi/syscalls/regen.sh b/include/lapi/syscalls/regen.sh
>> index 97027e2f3..217595f34 100755
>> --- a/include/lapi/syscalls/regen.sh
>> +++ b/include/lapi/syscalls/regen.sh
>> @@ -38,12 +38,12 @@ cat << EOF > "${output_pid}"
>>  #ifdef TST_TEST_H__
>>  #define TST_SYSCALL_BRK__(NR, SNR) ({ \\
>>  	tst_brk(TCONF, \\
>> -		"syscall(%d) " SNR " not supported on your arch", NR); \\
>> +		"syscall(%ld) " SNR " not supported on your arch", NR); \\
>
> Looks like quite old change:
>
> 36283230cb ("linux_syscall_numbers: add syscall number to output")
>
>>  })
>>  #else
>>  #define TST_SYSCALL_BRK__(NR, SNR) ({ \\
>>  	tst_brkm(TCONF, CLEANUP, \\
>> -		"syscall(%d) " SNR " not supported on your arch", NR); \\
>> +		"syscall(%ld) " SNR " not supported on your arch", NR); \\
>>  })
>>  #endif
>
> I expected this would fix several warnings, but it generated several warnings on
> both x86_64 and i686 due tst_syscall() being used on many places. To show just
> few:

Ah, OK, let's forget about it for now. I'll just change the syscall
number to int in fcntl_compat.

Possibly we would have to fix this to support x32.

>
> $ make autotools && ./configure && cd lib/newlib_tests && rm -f tst_expiration_timer && make tst_expiration_timer
> In file included from tst_expiration_timer.c:11:
> ../../include/tst_timer.h: In function ‘sys_clock_getres’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:204:16: note: in expansion of macro ‘tst_syscall’
>   204 |         return tst_syscall(__NR_clock_getres, clk_id, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_clock_getres64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:209:16: note: in expansion of macro ‘tst_syscall’
>   209 |         return tst_syscall(__NR_clock_getres_time64, clk_id, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_clock_gettime’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:219:16: note: in expansion of macro ‘tst_syscall’
>   219 |         return tst_syscall(__NR_clock_gettime, clk_id, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_clock_gettime64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:224:16: note: in expansion of macro ‘tst_syscall’
>   224 |         return tst_syscall(__NR_clock_gettime64, clk_id, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_clock_settime’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:234:16: note: in expansion of macro ‘tst_syscall’
>   234 |         return tst_syscall(__NR_clock_settime, clk_id, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_clock_settime64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:239:16: note: in expansion of macro ‘tst_syscall’
>   239 |         return tst_syscall(__NR_clock_settime64, clk_id, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_clock_nanosleep’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:251:16: note: in expansion of macro ‘tst_syscall’
>   251 |         return tst_syscall(__NR_clock_nanosleep, clk_id, flags,
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_clock_nanosleep64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:258:16: note: in expansion of macro ‘tst_syscall’
>   258 |         return tst_syscall(__NR_clock_nanosleep_time64, clk_id, flags,
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_futex’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:265:16: note: in expansion of macro ‘tst_syscall’
>   265 |         return tst_syscall(__NR_futex, uaddr, futex_op, val, to, uaddr2, val3);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_futex_time64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:271:16: note: in expansion of macro ‘tst_syscall’
>   271 |         return tst_syscall(__NR_futex_time64, uaddr, futex_op, val, to, uaddr2, val3);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_mq_timedsend’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:283:16: note: in expansion of macro ‘tst_syscall’
>   283 |         return tst_syscall(__NR_mq_timedsend, mqdes, msg_ptr, msg_len, msg_prio,
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_mq_timedsend64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:290:16: note: in expansion of macro ‘tst_syscall’
>   290 |         return tst_syscall(__NR_mq_timedsend_time64, mqdes, msg_ptr, msg_len,
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_mq_timedreceive’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:303:16: note: in expansion of macro ‘tst_syscall’
>   303 |         return tst_syscall(__NR_mq_timedreceive, mqdes, msg_ptr, msg_len,
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_mq_timedreceive64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:310:16: note: in expansion of macro ‘tst_syscall’
>   310 |         return tst_syscall(__NR_mq_timedreceive_time64, mqdes, msg_ptr, msg_len,
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_sched_rr_get_interval’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:321:16: note: in expansion of macro ‘tst_syscall’
>   321 |         return tst_syscall(__NR_sched_rr_get_interval, pid, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_sched_rr_get_interval64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:326:16: note: in expansion of macro ‘tst_syscall’
>   326 |         return tst_syscall(__NR_sched_rr_get_interval_time64, pid, ts);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timer_gettime’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:331:16: note: in expansion of macro ‘tst_syscall’
>   331 |         return tst_syscall(__NR_timer_gettime, timerid, its);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timer_gettime64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:336:16: note: in expansion of macro ‘tst_syscall’
>   336 |         return tst_syscall(__NR_timer_gettime64, timerid, its);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timer_settime’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:342:16: note: in expansion of macro ‘tst_syscall’
>   342 |         return tst_syscall(__NR_timer_settime, timerid, flags, its, old_its);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timer_settime64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:348:16: note: in expansion of macro ‘tst_syscall’
>   348 |         return tst_syscall(__NR_timer_settime64, timerid, flags, its, old_its);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timerfd_gettime’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:353:16: note: in expansion of macro ‘tst_syscall’
>   353 |         return tst_syscall(__NR_timerfd_gettime, fd, its);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timerfd_gettime64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:358:16: note: in expansion of macro ‘tst_syscall’
>   358 |         return tst_syscall(__NR_timerfd_gettime64, fd, its);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timerfd_settime’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:364:16: note: in expansion of macro ‘tst_syscall’
>   364 |         return tst_syscall(__NR_timerfd_settime, fd, flags, its, old_its);
>       |                ^~~~~~~~~~~
> ../../include/tst_timer.h: In function ‘sys_timerfd_settime64’:
> ../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../include/tst_timer.h:370:16: note: in expansion of macro ‘tst_syscall’
>   370 |         return tst_syscall(__NR_timerfd_settime64, fd, flags, its, old_its);
>       |                ^~~~~~~~~~~
> CC lib/newlib_tests/tst_expiration_timer
>
>
> $ cd ../../testcases/kernel/syscalls/clone && rm -f clone09 && make clone09
>
> ../../../../include/lapi/sched.h: In function ‘clone3’:
> ../../../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../../../include/lapi/sched.h:61:16: note: in expansion of macro ‘tst_syscall’
>    61 |         return tst_syscall(__NR_clone3, args, size);
>       |                ^~~~~~~~~~~
> ../../../../include/lapi/sched.h: In function ‘clone3_supported_by_kernel’:
> ../../../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> ../../../../include/lapi/sched.h:69:17: note: in expansion of macro ‘tst_syscall’
>    69 |                 tst_syscall(__NR_clone3, NULL, 0);
>       |                 ^~~~~~~~~~~
> clone09.c: In function ‘newnet’:
> ../../../../include/tst_test.h:81:55: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 5 has type ‘int’ [-Wformat=]
>    81 |                 tst_brk_(__FILE__, __LINE__, (ttype), (arg_fmt), ##__VA_ARGS__);\
>       |                                                       ^~~~~~~~~
> ../../../../include/lapi/syscalls.h:23:9: note: in expansion of macro ‘tst_brk’
>    23 |         tst_brk(TCONF, \
>       |         ^~~~~~~
> ../../../../include/lapi/syscalls.h:42:17: note: in expansion of macro ‘TST_SYSCALL_BRK__’
>    42 |                 TST_SYSCALL_BRK__(NR, #NR); \
>       |                 ^~~~~~~~~~~~~~~~~
> clone09.c:38:9: note: in expansion of macro ‘tst_syscall’
>    38 |         tst_syscall(__NR_exit, 0);
>
>
> Kind regards,
> Petr
diff mbox series

Patch

diff --git a/include/lapi/syscalls/regen.sh b/include/lapi/syscalls/regen.sh
index 97027e2f3..217595f34 100755
--- a/include/lapi/syscalls/regen.sh
+++ b/include/lapi/syscalls/regen.sh
@@ -38,12 +38,12 @@  cat << EOF > "${output_pid}"
 #ifdef TST_TEST_H__
 #define TST_SYSCALL_BRK__(NR, SNR) ({ \\
 	tst_brk(TCONF, \\
-		"syscall(%d) " SNR " not supported on your arch", NR); \\
+		"syscall(%ld) " SNR " not supported on your arch", NR); \\
 })
 #else
 #define TST_SYSCALL_BRK__(NR, SNR) ({ \\
 	tst_brkm(TCONF, CLEANUP, \\
-		"syscall(%d) " SNR " not supported on your arch", NR); \\
+		"syscall(%ld) " SNR " not supported on your arch", NR); \\
 })
 #endif