From: Kumar Kartikeya Dwivedi <[email protected]>
To: [email protected]
Cc: Alexei Starovoitov <[email protected]>,
Daniel Borkmann <[email protected]>,
Andrii Nakryiko <[email protected]>, Yonghong Song <[email protected]>,
Pavel Emelyanov <[email protected]>,
Alexander Mikhalitsyn <[email protected]>,
Andrei Vagin <[email protected]>,
[email protected], [email protected],
[email protected]
Subject: [PATCH bpf-next v2 00/10] Introduce BPF iterators for io_uring and epoll
Date: Tue, 23 Nov 2021 04:23:42 +0530 [thread overview]
Message-ID: <[email protected]> (raw)
The CRIU [0] project developers are exploring potential uses of the BPF
subsystem to do complicated tasks that are difficult to add support for in the
kernel using existing interfaces. Even if they are implemented using procfs,
or kcmp, it is difficult to make it perform well without having some kind of
programmable introspection into the kernel data structures. Moreover, for
procfs based state inspection, the output format once agreed upon is set in
stone and hard to extend, and at the same time inefficient to consume from
programs (where it is first converted from machine readable form to human
readable form, only to be converted again to machine readable form). In
addition to this, kcmp based file set matching algorithm performs poorly since
each file in one set needs to be compared to each file in another set, to
determine struct file equivalence.
This set adds a io_uring file iterator (for registered files), a io_uring ubuf
iterator (for registered buffers), and a epoll iterator (for registered items
(files, registered using EPOLL_CTL_ADD)) to overcome these limitations. Using
existing task, task_file, task_vma iterators, all of these can be combined
together to significantly enhance and speed up the task dumping procedure.
The two immediate use cases are io_uring checkpoint/restore support and epoll
checkpoint/restore support. The first is unimplemented, and the second is being
expedited using a new epoll iterator. In the future, more stages of the
checkpointing sequence can be offloaded to eBPF programs to reduce process
downtime, e.g. in pre-dump stage, before task is seized.
The io_uring file iterator is even more important now due to the advent of
descriptorless files in io_uring [1], which makes dumping a task's files a lot
more harder for CRIU, since there is no visibility into these hidden
descriptors that the task depends upon for operation. Similarly, the
io_uring_ubuf iterator is useful in case original VMA used in registering a
buffer has been destroyed.
The set includes an example sample showing how these iterator(s) along with
task_file iterator can be useful to restore an io_uring instance, implementing a
simplified version of the code we are planning to adopt for CRIU. Patch 10 is
not meant for submission, only exposition. It implements all the missing
features noted in [2].
Please see the individual patches for more details.
[ Note (for Yonghong): I am still unusure what will be useful in show_fdinfo,
fill_link_info for epoll, so that has been left out. I was reminded that
io_uring now uses anon_inode_getfile_secure, which we also use in CRIU to
determine source fd of ring mapping, so this should be enough to identify
the io_uring fd in userspace, hence I implemented it for io_uring in v2. ]
[0]: https://criu.org/Main_Page
[1]: https://lwn.net/Articles/863071
[2]: https://github.com/checkpoint-restore/criu/pull/1597
Changelog:
----------
v1 -> v2:
v1: https://lore.kernel.org/bpf/[email protected]
* Add example showing how iterator is useful in C/R of io_uring (Alexei)
* Change type of index from unsigned long to u64 (Yonghong)
* Fix build error for CONFIG_IO_URING=n (Kernel Test Robot)
* Move bpf_page_to_pfn out of CONFIG_IO_URING (Yonghong)
* Add comment to bpf_iter_aux_info for map member (Yonghong)
* show_fdinfo/fill_link_info for io_uring (Yonghong)
* Fix other nits
Kumar Kartikeya Dwivedi (10):
io_uring: Implement eBPF iterator for registered buffers
bpf: Add bpf_page_to_pfn helper
io_uring: Implement eBPF iterator for registered files
epoll: Implement eBPF iterator for registered items
bpftool: Output io_uring iterator info
selftests/bpf: Add test for io_uring BPF iterators
selftests/bpf: Add test for epoll BPF iterator
selftests/bpf: Test partial reads for io_uring, epoll iterators
selftests/bpf: Fix btf_dump test for bpf_iter_link_info
samples/bpf: Add example to checkpoint/restore io_uring
fs/eventpoll.c | 196 ++++-
fs/io_uring.c | 345 +++++++++
include/linux/bpf.h | 16 +
include/uapi/linux/bpf.h | 18 +
kernel/trace/bpf_trace.c | 19 +
samples/bpf/.gitignore | 1 +
samples/bpf/Makefile | 8 +-
samples/bpf/bpf_cr.bpf.c | 185 +++++
samples/bpf/bpf_cr.c | 686 ++++++++++++++++++
samples/bpf/bpf_cr.h | 48 ++
samples/bpf/hbm_kern.h | 2 -
scripts/bpf_doc.py | 2 +
tools/bpf/bpftool/link.c | 10 +
tools/include/uapi/linux/bpf.h | 18 +
.../selftests/bpf/prog_tests/bpf_iter.c | 387 +++++++++-
.../selftests/bpf/prog_tests/btf_dump.c | 4 +-
.../selftests/bpf/progs/bpf_iter_epoll.c | 33 +
.../selftests/bpf/progs/bpf_iter_io_uring.c | 50 ++
18 files changed, 2020 insertions(+), 8 deletions(-)
create mode 100644 samples/bpf/bpf_cr.bpf.c
create mode 100644 samples/bpf/bpf_cr.c
create mode 100644 samples/bpf/bpf_cr.h
create mode 100644 tools/testing/selftests/bpf/progs/bpf_iter_epoll.c
create mode 100644 tools/testing/selftests/bpf/progs/bpf_iter_io_uring.c
--
2.34.0
next reply other threads:[~2021-11-22 22:53 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-22 22:53 Kumar Kartikeya Dwivedi [this message]
2021-11-22 22:53 ` [PATCH bpf-next v2 01/10] io_uring: Implement eBPF iterator for registered buffers Kumar Kartikeya Dwivedi
2021-11-23 4:27 ` kernel test robot
2021-11-22 22:53 ` [PATCH bpf-next v2 02/10] bpf: Add bpf_page_to_pfn helper Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH bpf-next v2 03/10] io_uring: Implement eBPF iterator for registered files Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH bpf-next v2 04/10] epoll: Implement eBPF iterator for registered items Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH bpf-next v2 05/10] bpftool: Output io_uring iterator info Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH bpf-next v2 06/10] selftests/bpf: Add test for io_uring BPF iterators Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH bpf-next v2 07/10] selftests/bpf: Add test for epoll BPF iterator Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH bpf-next v2 08/10] selftests/bpf: Test partial reads for io_uring, epoll iterators Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH bpf-next v2 09/10] selftests/bpf: Fix btf_dump test for bpf_iter_link_info Kumar Kartikeya Dwivedi
2021-11-22 22:53 ` [PATCH RFC bpf-next v2 10/10] samples/bpf: Add example to checkpoint/restore io_uring Kumar Kartikeya Dwivedi
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 \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
/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