From: Ming Lei <tom.leiming@gmail.com>
To: Pavel Begunkov <asml.silence@gmail.com>
Cc: Alexei Starovoitov <alexei.starovoitov@gmail.com>,
io-uring <io-uring@vger.kernel.org>, bpf <bpf@vger.kernel.org>,
Jens Axboe <axboe@kernel.dk>, Ming Lei <tom.leiming@gmail.com>
Subject: Re: [PATCH v8 5/5] selftests/io_uring: add a bpf io_uring selftest
Date: Mon, 23 Feb 2026 23:23:47 +0800 [thread overview]
Message-ID: <CACVXFVPx5AcC9y9xVPCaahDeumNGm4TSkkfhsJ8w+wmW52JQ4w@mail.gmail.com> (raw)
In-Reply-To: <591a7f0e-7b78-42f1-9486-163249f5e306@gmail.com>
On Sat, Feb 21, 2026 at 6:35 AM Pavel Begunkov <asml.silence@gmail.com> wrote:
>
> On 2/20/26 17:45, Alexei Starovoitov wrote:
> > On Fri, Feb 20, 2026 at 3:41 AM Pavel Begunkov <asml.silence@gmail.com> wrote:
> >>
> >> I had such examples, but selftests is not the best place for that.
> >> It can use abstractions, and I want to make them reusable instead
> >> of people copy-pasting from selftests.
> >
> > Sure, but please still post them as extra patches so it's easier
> > to see what's the end result.
> >
> > Also please reply to that thread:
> > https://lore.kernel.org/bpf/CALTww28QMg=YXqKWpWLZrLO+xiqOe3LGyput8dx68-dnQsxg=g@mail.gmail.com/
> >
> > It's not clear to me whether your io_uring+bpf setup will work
> > for Xiao's use case.
> > I don't think we need 2 ways of doing it.
>
> We discussed this with Ming on the list before, that's one of the use
> cases I target as well, there is no reason why it shouldn't work. The
> difference is that this approach gives a flexible framework for
> extensibility and covers a good bunch of other needs, which is exactly
> the reason I moved from a BPF opcode approach, while Ming's proposal is
> more specific but argued to be a way easier to plug into ublk servers.
> If you ask me, we need a solution that covers a broader spectrum of
> use cases, but I guess it all can be argued in either way.
One big drawback of Pavel's approach is that it needs a totally
different userspace
implementation for using BPF, which is just hard to use in existing
io_uring applications.
But BPF opcode approach is seamless with existing io_uring applications, because
it simply uses existing io_uring interface.
Thanks,
Ming Lei
next prev parent reply other threads:[~2026-02-23 15:23 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-17 11:33 [PATCH v8 0/5] BPF controlled io_uring Pavel Begunkov
2026-02-17 11:33 ` [PATCH v8 1/5] io_uring: introduce callback driven main loop Pavel Begunkov
2026-02-17 11:33 ` [PATCH v8 2/5] io_uring/bpf-ops: implement loop_step with BPF struct_ops Pavel Begunkov
2026-02-17 11:33 ` [PATCH v8 3/5] io_uring/bpf-ops: add kfunc helpers Pavel Begunkov
2026-02-17 11:33 ` [PATCH v8 4/5] io_uring/bpf-ops: implement bpf ops registration Pavel Begunkov
2026-02-17 11:33 ` [PATCH v8 5/5] selftests/io_uring: add a bpf io_uring selftest Pavel Begunkov
2026-02-19 19:01 ` Alexei Starovoitov
2026-02-20 11:41 ` Pavel Begunkov
2026-02-20 17:45 ` Alexei Starovoitov
2026-02-20 22:35 ` Pavel Begunkov
2026-02-23 15:23 ` Ming Lei [this message]
2026-02-23 16:26 ` Pavel Begunkov
2026-02-23 14:37 ` Pavel Begunkov
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=CACVXFVPx5AcC9y9xVPCaahDeumNGm4TSkkfhsJ8w+wmW52JQ4w@mail.gmail.com \
--to=tom.leiming@gmail.com \
--cc=alexei.starovoitov@gmail.com \
--cc=asml.silence@gmail.com \
--cc=axboe@kernel.dk \
--cc=bpf@vger.kernel.org \
--cc=io-uring@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox