From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christian Brauner Subject: Re: [RFC 0/3] seccomp trap to userspace Date: Thu, 15 Mar 2018 17:09:25 +0100 Message-ID: <20180315160924.GA12744__19244.0013088801$1521130059$gmane$org@gmail.com> References: <20180204104946.25559-1-tycho@tycho.ws> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20180204104946.25559-1-tycho-E0fblnxP3wo@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Tycho Andersen Cc: Kees Cook , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, Akihiro Suda , Oleg Nesterov , Andy Lutomirski , "Eric W . Biederman" , Christian Brauner , Tyler Hicks List-Id: containers.vger.kernel.org On Sun, Feb 04, 2018 at 11:49:43AM +0100, Tycho Andersen wrote: > Several months ago at Linux Plumber's, we had a discussion about adding a > feature to seccomp which would allow seccomp to trigger a notification for some > other process. Here's a draft of that feature. > > Patch 1 contains the bulk of it, patches 2 & 3 offer an alternative way to > acquire the fd that receives notifications via ptrace (the method in patch 1 > poses some problems). Other suggestions for how to acquire one of these fds > would be welcome. > > Take a close look at the synchronization. I think I've got it right, but I > probably don't :) > > Thanks! > > Tycho Andersen (3): > seccomp: add a return code to trap to userspace > seccomp: hoist out filter resolving logic > seccomp: add a way to get a listener fd from ptrace > > arch/Kconfig | 7 + > include/linux/seccomp.h | 14 +- > include/uapi/linux/ptrace.h | 1 + > include/uapi/linux/seccomp.h | 18 +- > kernel/ptrace.c | 4 + > kernel/seccomp.c | 467 ++++++++++++++++++++++++-- > tools/testing/selftests/seccomp/seccomp_bpf.c | 180 +++++++++- > 7 files changed, 653 insertions(+), 38 deletions(-) Hey, So, I've been following the discussion silently in the background and I see that it got sidetracked into seccomp + ebpf. While I can see that there is value in adding epbf support to seccomp I'd really like to see this decoupled from this patchset. Afaict, this patchset would just work fine without the ebpf portion (but I might be just have missed the point). So if possible I would like to see a second version of this with the comments accounted for and - if possible - have this up for merging independent of the ebpf patchset that's floating around. Christian