From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] asm-generic: add memfd_create system call to unistd.h
Date: Tue, 12 Aug 2014 12:10:30 +0100 [thread overview]
Message-ID: <20140812111030.GK29013@arm.com> (raw)
In-Reply-To: <20140812102734.GG29013@arm.com>
On Tue, Aug 12, 2014 at 11:27:34AM +0100, Will Deacon wrote:
> On Mon, Aug 11, 2014 at 08:37:41PM +0100, Geert Uytterhoeven wrote:
> > On Mon, Aug 11, 2014 at 8:57 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> > > On Monday 11 August 2014, Will Deacon wrote:
> > >> On Mon, Aug 11, 2014 at 05:54:08PM +0100, David Herrmann wrote:
> > >> > On Mon, Aug 11, 2014 at 3:37 PM, Will Deacon <will.deacon@arm.com> wrote:
> > >> > > Commit 9183df25fe7b ("shm: add memfd_create() syscall") added a new
> > >> > > system call (memfd_create) but didn't update the asm-generic unistd
> > >> > > header.
> > >> > >
> > >> > > This patch adds the new system call to the asm-generic version of
> > >> > > unistd.h so that it can be used by architectures such as arm64.
> > >> > >
> > >> > > Cc: Arnd Bergmann <arnd@arndb.de>
> > >> > > Cc: David Herrmann <dh.herrmann@gmail.com>
> > >> > > Signed-off-by: Will Deacon <will.deacon@arm.com>
> > >> >
> > >> > Reviewed-by: David Herrmann <dh.herrmann@gmail.com>
> > >>
> > >> Thanks, David.
> > >>
> > >> Arnd: shall I take this via the arm64 tree, or do you already have some
> > >> asm-generic patches queued?
> > >
> > > Please take it for arm64, I don't have any other patches queued at the moment
> > > and am currently on vacation.
> >
> > Thanks, I was just going to look into adding it...
> >
> > BTW, another one sneaked in: kexec_file_load. I say "sneaked", as
> > scripts/checksyscalls.sh doesn't catch it, due to it being wired up on
> > amd64 only.
>
> Oh, well spotted!
>
> > Will, you can please add that one, too, while you're at it?
>
> Sure, I'll add an extra patch and repost.
Hmm, so whilst I can easily wire-up the new syscall, it's pretty useless for
anybody other than x86 at the moment. There are a bunch of arch helpers:
arch_kexec_kernel_image_probe
arch_kexec_kernel_verify_sig
arch_kexec_kernel_image_load
arch_kimage_file_post_load_cleanup
which are only implemented for x86 (arch/x86/kernel/machine_kexec_64.c),
even though I don't really see what makes them arch-specific as opposed to
file format specific.
So this syscall will always fail with -ENOEXEC at the moment. Is it still
worth wiring it up?
Will
WARNING: multiple messages have this Message-ID (diff)
From: Will Deacon <will.deacon@arm.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Arnd Bergmann <arnd@arndb.de>,
David Herrmann <dh.herrmann@gmail.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
linux-kernel <linux-kernel@vger.kernel.org>,
Vivek Goyal <vgoyal@redhat.com>
Subject: Re: [PATCH 2/2] asm-generic: add memfd_create system call to unistd.h
Date: Tue, 12 Aug 2014 12:10:30 +0100 [thread overview]
Message-ID: <20140812111030.GK29013@arm.com> (raw)
In-Reply-To: <20140812102734.GG29013@arm.com>
On Tue, Aug 12, 2014 at 11:27:34AM +0100, Will Deacon wrote:
> On Mon, Aug 11, 2014 at 08:37:41PM +0100, Geert Uytterhoeven wrote:
> > On Mon, Aug 11, 2014 at 8:57 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> > > On Monday 11 August 2014, Will Deacon wrote:
> > >> On Mon, Aug 11, 2014 at 05:54:08PM +0100, David Herrmann wrote:
> > >> > On Mon, Aug 11, 2014 at 3:37 PM, Will Deacon <will.deacon@arm.com> wrote:
> > >> > > Commit 9183df25fe7b ("shm: add memfd_create() syscall") added a new
> > >> > > system call (memfd_create) but didn't update the asm-generic unistd
> > >> > > header.
> > >> > >
> > >> > > This patch adds the new system call to the asm-generic version of
> > >> > > unistd.h so that it can be used by architectures such as arm64.
> > >> > >
> > >> > > Cc: Arnd Bergmann <arnd@arndb.de>
> > >> > > Cc: David Herrmann <dh.herrmann@gmail.com>
> > >> > > Signed-off-by: Will Deacon <will.deacon@arm.com>
> > >> >
> > >> > Reviewed-by: David Herrmann <dh.herrmann@gmail.com>
> > >>
> > >> Thanks, David.
> > >>
> > >> Arnd: shall I take this via the arm64 tree, or do you already have some
> > >> asm-generic patches queued?
> > >
> > > Please take it for arm64, I don't have any other patches queued at the moment
> > > and am currently on vacation.
> >
> > Thanks, I was just going to look into adding it...
> >
> > BTW, another one sneaked in: kexec_file_load. I say "sneaked", as
> > scripts/checksyscalls.sh doesn't catch it, due to it being wired up on
> > amd64 only.
>
> Oh, well spotted!
>
> > Will, you can please add that one, too, while you're at it?
>
> Sure, I'll add an extra patch and repost.
Hmm, so whilst I can easily wire-up the new syscall, it's pretty useless for
anybody other than x86 at the moment. There are a bunch of arch helpers:
arch_kexec_kernel_image_probe
arch_kexec_kernel_verify_sig
arch_kexec_kernel_image_load
arch_kimage_file_post_load_cleanup
which are only implemented for x86 (arch/x86/kernel/machine_kexec_64.c),
even though I don't really see what makes them arch-specific as opposed to
file format specific.
So this syscall will always fail with -ENOEXEC at the moment. Is it still
worth wiring it up?
Will
next prev parent reply other threads:[~2014-08-12 11:10 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-11 13:37 [PATCH 1/2] arm64: compat: wire up memfd_create and getrandom syscalls for aarch32 Will Deacon
2014-08-11 13:37 ` Will Deacon
2014-08-11 13:37 ` [PATCH 2/2] asm-generic: add memfd_create system call to unistd.h Will Deacon
2014-08-11 13:37 ` Will Deacon
2014-08-11 16:54 ` David Herrmann
2014-08-11 16:54 ` David Herrmann
2014-08-11 17:15 ` Will Deacon
2014-08-11 17:15 ` Will Deacon
2014-08-11 18:57 ` Arnd Bergmann
2014-08-11 18:57 ` Arnd Bergmann
2014-08-11 19:37 ` Geert Uytterhoeven
2014-08-11 19:37 ` Geert Uytterhoeven
2014-08-12 10:27 ` Will Deacon
2014-08-12 10:27 ` Will Deacon
2014-08-12 11:10 ` Will Deacon [this message]
2014-08-12 11:10 ` Will Deacon
2014-08-12 12:37 ` Vivek Goyal
2014-08-12 12:37 ` Vivek Goyal
2014-08-15 13:55 ` Will Deacon
2014-08-15 13:55 ` Will Deacon
2014-08-18 17:15 ` Vivek Goyal
2014-08-18 17:15 ` Vivek Goyal
2014-08-18 17:40 ` Russell King - ARM Linux
2014-08-18 17:40 ` Russell King - ARM Linux
2014-08-18 21:04 ` Vivek Goyal
2014-08-18 21:04 ` Vivek Goyal
2014-08-12 10:28 ` Will Deacon
2014-08-12 10:28 ` Will Deacon
2014-08-11 16:55 ` [PATCH 1/2] arm64: compat: wire up memfd_create and getrandom syscalls for aarch32 David Herrmann
2014-08-11 16:55 ` David Herrmann
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=20140812111030.GK29013@arm.com \
--to=will.deacon@arm.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.