Re: [PATCH] process_madvise.2: SYNOPSIS: Fix prototype parameter type

["Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>]

Hi Alex,

On 2/11/21 6:44 PM, Alejandro Colomar wrote:
> The type of vlen is 'size_t', and not 'unsigned int'.  Fix it.
> 
> ......
> 
> $ grep_syscall process_madvise
> mm/madvise.c:1161:
> SYSCALL_DEFINE5(process_madvise, int, pidfd, const struct iovec __user *, vec,
> 		size_t, vlen, int, behavior, unsigned int, flags)
> include/linux/syscalls.h:887:
> asmlinkage long sys_process_madvise(int pidfd, const struct iovec __user *vec,
> 			size_t vlen, int behavior, unsigned int flags);
> 
> function grep_syscall()
> {
> 	if ! [ -v 1 ]; then
> 		>&2 echo "Usage: ${FUNCNAME[0]} <syscall>";
> 		return ${EX_USAGE};
> 	fi
> 
> 	find * -type f \
> 	|grep '\.c$' \
> 	|sort -V \
> 	|xargs pcregrep -Mn "(?s)^\w*SYSCALL_DEFINE.\(${1},.*?\)" \
> 	|sed -E 's/^[^:]+:[0-9]+:/&\n/';
> 
> 	find * -type f \
> 	|grep '\.[ch]$' \
> 	|sort -V \
> 	|xargs pcregrep -Mn "(?s)^asmlinkage\s+[\w\s]+\**sys_${1}\s*\(.*?\)" \
> 	|sed -E 's/^[^:]+:[0-9]+:/&\n/';
> }
> 
> Signed-off-by: Alejandro Colomar <alx.manpages@gmail.com>

Thanks. Patch applied.

Cheers,

Michael


> ---
>  man2/process_madvise.2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/man2/process_madvise.2 b/man2/process_madvise.2
> index 9dc90ced1..b00ceb6fc 100644
> --- a/man2/process_madvise.2
> +++ b/man2/process_madvise.2
> @@ -33,7 +33,7 @@ process_madvise \- give advice about use of memory to a process
>  .B #include <sys/uio.h>
>  .PP
>  .BI "ssize_t process_madvise(int " pidfd ", const struct iovec *" iovec ,
> -.BI "                        unsigned long " vlen ", int " advice ,
> +.BI "                        size_t " vlen ", int " advice ,
>  .BI "                        unsigned int " flags ");"
>  .fi
>  .PP
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/