[Bug bpf/24758] New: stapbpf-next: more robust and generous userspace interpreter

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[Bug bpf/24758] New: stapbpf-next: more robust and generous userspace interpreter

fweimer at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=24758

            Bug ID: 24758
           Summary: stapbpf-next: more robust and generous userspace
                    interpreter
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: bpf
          Assignee: systemtap at sourceware dot org
          Reporter: me at serhei dot io
  Target Milestone: ---

The BPF interpreter (in stapbpf/bpfinterp.cxx) is still a cobbled-together
thing with some holes and probable bugs lurking. It needs a pass over the code
to be brought up to standard bytecode-interpreter 'best practice'. No need for
fancy performance tricks, but:
- It could be more robust (runtime errors handled properly, memory management
with fewer pitfalls).
- It could be more generous (some of the kernel bpf restrictions are still
duplicated in userspace where there's no need for them -- pr24528 allows us to
generate different code for userspace probes).
- It could maybe use fewer hacky C++ interfaces.
- It could maybe do with refactoring to include some code from the lkm runtime
where functionality is being duplicated. (We already use lkm code in-userspace
for the dyninst backend, so reusing it here is not totally shocking.)

The only thing that's needed for certain is 'more robust' so I am filing just
one PR for now to track this work.

--
You are receiving this mail because:
You are the assignee for the bug.