The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: "Farid Zakaria" <farid.m.zakaria@gmail.com>
To: "Christian Brauner" <brauner@kernel.org>
Cc: "Farid Zakaria" <farid.m.zakaria@gmail.com>,
	<david.laight.linux@gmail.com>, <jack@suse.cz>,
	<jannh@google.com>, <kees@kernel.org>,
	<linux-fsdevel@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-kselftest@vger.kernel.org>, <linux-mm@kvack.org>,
	<mail@johnericson.me>, <shuah@kernel.org>,
	<viro@zeniv.linux.org.uk>, "Gemini" <assistant@google.com>
Subject: Re: [RFC PATCH] fs: binfmt_misc: introduce eBPF-based matching and interpreter selection
Date: Wed, 08 Jul 2026 21:07:55 -0700	[thread overview]
Message-ID: <DJTQTPG7QZSH.1YIDYKL3O985U@gmail.com> (raw)
In-Reply-To: <20260708-kordel-laben-kopfhaar-1c0dd234aae5@brauner>

On Wed Jul 8, 2026 at 9:47 AM PDT, Christian Brauner wrote:
> On 2026-07-08 11:14 +0200, Christian Brauner wrote:
>> On 2026-07-07 15:44 -0700, Farid Zakaria wrote:
>> > On Tue Jul 7, 2026 at 12:45 PM PDT, Christian Brauner wrote:
>> > > On 2026-07-07 13:34 +0200, Christian Brauner wrote:
>> > >> On 2026-07-06 09:47:53-07:00, Farid Zakaria wrote:
>> > >> > On Mon, Jul 6, 2026 at 9:01 AM Christian Brauner <brauner@kernel.org> wrote:
>> > >> > 
>> > >> > > Ignoring the blatant bpf abuse here I think this is quite workable. So
>> > >> > > this can be turned into an actual design and patch imho...
>> > >> > 
>> > >> > Glad to see you think this proposal is in the right direction.
>> > >> > Just for my edification, when you say "balatant bpf abuse", you mean
>> > >> > how it abuses the socket bpf type?
>> > >> > If so, do you have any guidance on a new type or should I just take a
>> > >> > stab at it.
>> > >
>> > > I drafted a POC for you that should be closer to how this should look
>> > > like with the bpf people in Cc. I'll let you lead from now on. I just
>> > > wanted to help you out and illustrate a more viable direction. Might be
>> > > that the bpf people want it done differentely. Let's see.
>> > 
>> > Cool. I just saw the POC.
>> > (I'm trying out aerc to view the patch series. I will also try b4, I've
>> > never set it up before)
>> > I will rework my qemu demo to use the struct_ops and go through my own
>> > testing to see it work.
>> > 
>> > That looks very similar to what I was researching with an LLM. I was
>> > just going over the idea of struct_ops bpf (a new concept to me) and
>> > using traditional bpftool load, a new bpf program type and kfuncs.
>> > 
>> > You put out a surprising amount of polished code -- thank you.
>> 
>> I would not call this polished and it also lacks any tests... I've not
>> given the locking much thought and maybe there's easy wins on the table
>> I haven't been considering.
>> 
>> > I would have had to lean on an LLM (heavily) and from the patches I've
>> > put up so far they still have quite the gap to cover in terms of polish.
>> 
>> This isn't my first bpf ride in both kernel and userspace.
>> 
>> > What does "take it from here" mean in practicality?
>> 
>> Bring it over the finish line so I can merge it into my tree. :)
>> If the bpf people have no comments or worries than we're done. If they
>> want changes I'd appreciate if you could make them and slap you
>> Co-developed-by onto the whole thing as necessary.
>> 
>> Also, please ensure that this works for all the cases you're
>> considering.
>
> I have another cleanup series for binfmt_misc btw. That code is quite
> old. There might be a few additional series coming out of this work that
> we base the bpf bits on.

Sounds good. I just saw the patch series.
My plan at the least by the weekend is to:
* apply your binfmt_misc cleanup series
  * I will try b4 with aerc. Here  is my config if anyone wants to crib it [1].
* apply your POC series for the bpf to validate
* make any changes if necessary or just add selftests if it works as is

[1]: https://github.com/fzakaria/nix-home/blob/aaee67c3a6d035d06a62cb75392657563de58f4a/users/fmzakari/aerc.nix 

  reply	other threads:[~2026-07-09  4:07 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-22  4:39 [PATCH 0/2] fs: support $ORIGIN in ELF interpreter paths Farid Zakaria
2026-06-22  4:39 ` [PATCH 1/2] " Farid Zakaria
2026-06-22  9:53   ` Jori Koolstra
2026-06-23 20:14   ` Kees Cook
2026-06-23 20:35     ` Farid Zakaria
2026-06-22  4:39 ` [PATCH 2/2] selftests/exec: add test suites for $ORIGIN interpreter resolution Farid Zakaria
2026-06-22 10:39 ` [PATCH 0/2] fs: support $ORIGIN in ELF interpreter paths Jan Kara
2026-06-22 17:15   ` Farid Zakaria
2026-06-22 21:08     ` John Ericson
2026-06-25  8:50       ` Christian Brauner
2026-06-25 19:34         ` John Ericson
2026-06-26 12:39         ` Jann Horn
2026-06-26 13:26           ` David Laight
2026-06-26 13:34             ` Jann Horn
2026-06-26 16:28               ` David Laight
2026-06-28 12:36           ` Christian Brauner
2026-06-28 13:20             ` Christian Brauner
2026-06-28 18:44               ` David Laight
2026-06-30 16:58               ` Farid Zakaria
2026-07-02  6:47                 ` Christian Brauner
2026-06-30 17:59               ` David Laight
2026-07-02 21:42                 ` [RFC PATCH] fs: introduce pluggable ELF interpreter loader registry Farid Zakaria
2026-07-03  9:22                   ` Christian Brauner
2026-07-03 18:32                     ` Farid Zakaria
2026-07-04  8:15                       ` Christian Brauner
2026-07-04 21:14                         ` [RFC PATCH] fs: binfmt_misc: introduce eBPF-based matching and interpreter selection Farid Zakaria
2026-07-06 16:01                           ` Christian Brauner
2026-07-06 16:47                             ` Farid Zakaria
2026-07-07 11:34                               ` Christian Brauner
2026-07-07 19:45                                 ` Christian Brauner
2026-07-07 22:44                                   ` Farid Zakaria
2026-07-08  9:14                                     ` Christian Brauner
2026-07-08 16:47                                       ` Christian Brauner
2026-07-09  4:07                                         ` Farid Zakaria [this message]
2026-07-09  8:46                                           ` Christian Brauner

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=DJTQTPG7QZSH.1YIDYKL3O985U@gmail.com \
    --to=farid.m.zakaria@gmail.com \
    --cc=assistant@google.com \
    --cc=brauner@kernel.org \
    --cc=david.laight.linux@gmail.com \
    --cc=jack@suse.cz \
    --cc=jannh@google.com \
    --cc=kees@kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mail@johnericson.me \
    --cc=shuah@kernel.org \
    --cc=viro@zeniv.linux.org.uk \
    /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