From: "Dmitry V. Levin" <ldv@altlinux.org>
To: Paul Moore <paul@paul-moore.com>, Richard Guy Briggs <rgb@redhat.com>
Cc: Elvira Khabirova <lineprinter@altlinux.org>,
Eugene Syromyatnikov <esyr@redhat.com>,
Oleg Nesterov <oleg@redhat.com>,
Andy Lutomirski <luto@kernel.org>,
linux-audit@redhat.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/14] Prepare syscall_get_arch for PTRACE_GET_SYSCALL_INFO
Date: Wed, 27 Feb 2019 17:13:00 +0300 [thread overview]
Message-ID: <20190227141259.GA22352@altlinux.org> (raw)
In-Reply-To: <20190208222219.GA12864@altlinux.org>
[-- Attachment #1: Type: text/plain, Size: 3155 bytes --]
On Sat, Feb 09, 2019 at 01:22:19AM +0300, Dmitry V. Levin wrote:
> On Thu, Jan 17, 2019 at 03:34:44PM -0500, Richard Guy Briggs wrote:
> > On 2019-01-09 15:40, Dmitry V. Levin wrote:
> > > syscall_get_arch() is required to be implemented on all architectures in order
> > > to extend the generic ptrace API with PTRACE_GET_SYSCALL_INFO request:
> > > syscall_get_arch() is going to be called from ptrace_request() along with
> > > syscall_get_nr(), syscall_get_arguments(), syscall_get_error(), and
> > > syscall_get_return_value() functions with a tracee as their argument.
> > >
> > > The primary intent is that the triple (audit_arch, syscall_nr, arg1..arg6)
> > > should describe what system call is being called and what its arguments are.
> > >
> > > This patchset began as a series called "Prepare for PTRACE_GET_SYSCALL_INFO",
> > > then I merged it into a series called "ptrace: add PTRACE_GET_SYSCALL_INFO request"
> > > that also contains ptrace-specific changes.
> > >
> > > The ptrace-specific part, however, needs more attention to workaround problems
> > > on niche architectures like alpha, while the syscall_get_arch() part is
> > > straightforward, so I decided to split it out into a separate patchset that
> > > just prepares syscall_get_arch() for PTRACE_GET_SYSCALL_INFO: it adds
> > > syscall_get_arch() to those architectures that haven't implemented it yet,
> > > and then adds "struct task_struct *" argument to syscall_get_arch()
> > > on all architectures.
> >
> > Glad to see syscall_get_arch() added to the remaining arches. As Paul
> > said, it gets us closer to auditing syscalls on those remaining
> > unsupported arches and getting rid of the extra CONFIG_AUDITSYSCALL.
> > A little ironic that Eric (Paris) and I purged task_struct from
> > syscall_get_arch() 5 years ago since everything could use current.
> >
> > > All patches from this patchset have been already reviewed, so it's ready
> > > to be merged without waiting for the ptrace-specific part. As it's all
> > > about syscall_get_arch(), it should probably go via audit tree.
> >
> > ACK.
> >
> > Thanks Dmitry.
>
> Thanks.
> Please let me know if some action related to this patch series is expected from me.
>
> > > Dmitry V. Levin (14):
> > > Move EM_ARCOMPACT and EM_ARCV2 to uapi/linux/elf-em.h
> > > arc: define syscall_get_arch()
> > > c6x: define syscall_get_arch()
> > > h8300: define syscall_get_arch()
> > > Move EM_HEXAGON to uapi/linux/elf-em.h
> > > hexagon: define syscall_get_arch()
> > > m68k: define syscall_get_arch()
> > > Move EM_NDS32 to uapi/linux/elf-em.h
> > > nds32: define syscall_get_arch()
> > > nios2: define syscall_get_arch()
> > > riscv: define syscall_get_arch()
> > > Move EM_UNICORE to uapi/linux/elf-em.h
> > > unicore32: define syscall_get_arch()
> > > syscall_get_arch: add "struct task_struct *" argument
This is just a gentle ping of the series which is still applicable
to v5.0-rc8 with one exception: "riscv: define syscall_get_arch()"
is no longer needed as riscv already has syscall_get_arch() now.
--
ldv
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
next prev parent reply other threads:[~2019-02-27 14:13 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-09 12:40 [PATCH 00/14] Prepare syscall_get_arch for PTRACE_GET_SYSCALL_INFO Dmitry V. Levin
2019-01-09 12:42 ` [PATCH 01/14] Move EM_ARCOMPACT and EM_ARCV2 to uapi/linux/elf-em.h Dmitry V. Levin
2019-01-09 12:42 ` [PATCH 02/14] arc: define syscall_get_arch() Dmitry V. Levin
2019-01-09 12:42 ` [PATCH 03/14] c6x: " Dmitry V. Levin
2019-01-09 12:42 ` [PATCH 04/14] h8300: " Dmitry V. Levin
2019-02-27 15:29 ` Dmitry V. Levin
2019-01-09 12:43 ` [PATCH 05/14] Move EM_HEXAGON to uapi/linux/elf-em.h Dmitry V. Levin
2019-02-27 15:29 ` Dmitry V. Levin
2019-01-09 12:43 ` [PATCH 06/14] hexagon: define syscall_get_arch() Dmitry V. Levin
2019-02-27 15:30 ` Dmitry V. Levin
2019-01-09 12:43 ` [PATCH 07/14] m68k: " Dmitry V. Levin
2019-01-09 12:43 ` [PATCH 08/14] Move EM_NDS32 to uapi/linux/elf-em.h Dmitry V. Levin
2019-02-27 15:30 ` Dmitry V. Levin
2019-03-05 0:26 ` Vincent Chen
2019-03-06 6:27 ` Greentime Hu
2019-01-09 12:43 ` [PATCH 09/14] nds32: define syscall_get_arch() Dmitry V. Levin
2019-02-27 15:30 ` Dmitry V. Levin
2019-03-04 23:49 ` Vincent Chen
2019-03-06 6:26 ` Greentime Hu
2019-01-09 12:43 ` [PATCH 10/14] nios2: " Dmitry V. Levin
2019-02-27 15:31 ` Dmitry V. Levin
2019-03-01 7:30 ` Ley Foon Tan
2019-01-09 12:43 ` [PATCH 11/14] riscv: " Dmitry V. Levin
2019-01-09 12:43 ` [PATCH 12/14] Move EM_UNICORE to uapi/linux/elf-em.h Dmitry V. Levin
2019-02-27 15:31 ` Dmitry V. Levin
2019-01-09 12:43 ` [PATCH 13/14] unicore32: define syscall_get_arch() Dmitry V. Levin
2019-02-27 15:31 ` Dmitry V. Levin
2019-01-09 12:43 ` [PATCH 14/14] syscall_get_arch: add "struct task_struct *" argument Dmitry V. Levin
2019-01-17 20:34 ` [PATCH 00/14] Prepare syscall_get_arch for PTRACE_GET_SYSCALL_INFO Richard Guy Briggs
2019-02-08 22:22 ` Dmitry V. Levin
2019-02-27 14:13 ` Dmitry V. Levin [this message]
2019-02-27 14:57 ` Paul Moore
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=20190227141259.GA22352@altlinux.org \
--to=ldv@altlinux.org \
--cc=esyr@redhat.com \
--cc=lineprinter@altlinux.org \
--cc=linux-audit@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@kernel.org \
--cc=oleg@redhat.com \
--cc=paul@paul-moore.com \
--cc=rgb@redhat.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.
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).