Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240223003253.GX4163@brightrain.aerifal.cx>
Date: Thu, 22 Feb 2024 19:32:53 -0500
From: Rich Felker <dalias@...c.org>
To: Gaël PORTAY <gael.portay@...ne.fr>
Cc: musl@...ts.openwall.com
Subject: Re: [PATCH v2 1/8] bits/syscall.h: add process_madvise from
 linux v5.14

On Sun, Feb 18, 2024 at 03:26:43AM +0100, Gaël PORTAY wrote:
> see
> 
>     linux commit 7bb7f2ac24a028b20fca466b9633847b289b156a
>     arch, mm: wire up memfd_secret system call where relevant
> 
>     linux commit 1507f51255c9ff07d75909a84e7c0d7f3c4b2f49
>     mm: introduce memfd_secret system call to create "secret" memory areas
> 
>     linux commit b633896314c0f78f2b4eb7b19a530d68f2a35445
>     tools headers UAPI: Sync s390 syscall table file that wires up the memfd_secret syscall
> ---
>  arch/aarch64/bits/syscall.h.in    | 1 +
>  arch/arm/bits/syscall.h.in        | 1 +
>  arch/i386/bits/syscall.h.in       | 1 +
>  arch/m68k/bits/syscall.h.in       | 1 +
>  arch/microblaze/bits/syscall.h.in | 1 +
>  arch/mips/bits/syscall.h.in       | 1 +
>  arch/mips64/bits/syscall.h.in     | 1 +
>  arch/mipsn32/bits/syscall.h.in    | 1 +
>  arch/or1k/bits/syscall.h.in       | 1 +
>  arch/powerpc/bits/syscall.h.in    | 1 +
>  arch/powerpc64/bits/syscall.h.in  | 1 +
>  arch/riscv64/bits/syscall.h.in    | 1 +
>  arch/s390x/bits/syscall.h.in      | 1 +
>  arch/sh/bits/syscall.h.in         | 1 +
>  arch/x32/bits/syscall.h.in        | 1 +
>  arch/x86_64/bits/syscall.h.in     | 1 +
>  16 files changed, 16 insertions(+)
> 
> diff --git a/arch/aarch64/bits/syscall.h.in b/arch/aarch64/bits/syscall.h.in
> index 5f420e61..88512ec9 100644
> --- a/arch/aarch64/bits/syscall.h.in
> +++ b/arch/aarch64/bits/syscall.h.in
> @@ -299,4 +299,5 @@
>  #define __NR_landlock_create_ruleset	444
>  #define __NR_landlock_add_rule	445
>  #define __NR_landlock_restrict_self	446
> +// reserved for memfd_secret	447
> [...]

This patch seems mistitled, and should avoid introducing comments
(especially // form which are c89-incompatible) into an installed
header. I've renamed it to:

"bits/syscall.h: add memfd_secret from linux v5.14"

and removed the comments.

(No need to resubmit)

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.