public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Richard Palethorpe <rpalethorpe@suse.de>
To: Petr Vorel <pvorel@suse.cz>
Cc: ltp@lists.linux.it
Subject: Re: [LTP] [PATCH 1/2] syscall: System call numbers are word sized
Date: Thu, 12 Jan 2023 10:06:11 +0000	[thread overview]
Message-ID: <87358gjc0o.fsf@suse.de> (raw)
In-Reply-To: <Y729aGTHRwHoPX42@pevik>

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


-- 
Thank you,
Richard.

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

      reply	other threads:[~2023-01-12 10:14 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-10 14:16 [LTP] [PATCH 1/2] syscall: System call numbers are word sized Richard Palethorpe via ltp
2023-01-10 14:16 ` [LTP] [PATCH 2/2] fcntl{34, 36}: Always use 64-bit flock struct to avoid EINVAL Richard Palethorpe via ltp
2023-01-10 19:14   ` Petr Vorel
2023-01-12  8:54     ` Richard Palethorpe
2023-01-10 19:32 ` [LTP] [PATCH 1/2] syscall: System call numbers are word sized Petr Vorel
2023-01-12 10:06   ` Richard Palethorpe [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87358gjc0o.fsf@suse.de \
    --to=rpalethorpe@suse.de \
    --cc=ltp@lists.linux.it \
    --cc=pvorel@suse.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox