From: Rich Felker <dalias@libc.org>
To: "Gaël PORTAY" <gael.portay@rtone.fr>
Cc: musl@lists.openwall.com
Subject: Re: [musl] [PATCH v2 1/8] bits/syscall.h: add process_madvise from linux v5.14
Date: Thu, 22 Feb 2024 19:32:53 -0500 [thread overview]
Message-ID: <20240223003253.GX4163@brightrain.aerifal.cx> (raw)
In-Reply-To: <20240218022650.1097269-2-gael.portay@rtone.fr>
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
next prev parent reply other threads:[~2024-02-23 0:32 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-18 2:26 [musl] [PATCH v2 0/8] use new SYS_fchmodat2 syscall to implement fchmodat with flags Gaël PORTAY
2024-02-18 2:26 ` [musl] [PATCH v2 1/8] bits/syscall.h: add process_madvise from linux v5.14 Gaël PORTAY
2024-02-23 0:32 ` Rich Felker [this message]
2024-02-18 2:26 ` [musl] [PATCH v2 2/8] bits/syscall.h: add process_mrelease from linux v5.15 Gaël PORTAY
2024-02-18 2:26 ` [musl] [PATCH v2 3/8] bits/syscall.h: add futex_waitv from linux v5.16 Gaël PORTAY
2024-02-18 2:26 ` [musl] [PATCH v2 4/8] bits/syscall.h: add set_mempolicy_home_node from linux v5.17 Gaël PORTAY
2024-02-18 2:26 ` [musl] [PATCH v2 5/8] bits/syscall.h: add cachestat from linux v6.4 Gaël PORTAY
2024-02-18 2:26 ` [musl] [PATCH v2 6/8] remove flag argument from fchmodat syscall Gaël PORTAY
2024-02-18 2:26 ` [musl] [PATCH v2 7/8] bits/syscall.h: add __NR_fchmodat2 from linux v6.6 Gaël PORTAY
2024-02-18 2:26 ` [musl] [PATCH v2 8/8] use new SYS_fchmodat2 syscall to implement fchmodat with flags Gaël PORTAY
2024-02-23 0:36 ` Rich Felker
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=20240223003253.GX4163@brightrain.aerifal.cx \
--to=dalias@libc.org \
--cc=gael.portay@rtone.fr \
--cc=musl@lists.openwall.com \
/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.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/musl/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).