tetragon: Add uprobe multi support #1914
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
olsajiri
added
the
release-note/minor
olsajiri
force-pushed
the
pr/olsajiri/uprobe_multi
branch
4 times, most recently
from
496f82b
to
5c1ff8c
Compare
✅ Deploy Preview for tetragon ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
olsajiri
force-pushed
the
pr/olsajiri/uprobe_multi
branch
2 times, most recently
from
503d8fc
to
9c509ad
Compare
olsajiri
force-pushed
the
pr/olsajiri/uprobe_multi
branch
4 times, most recently
from
338d415
to
4fc0d21
Compare
olsajiri
force-pushed
the
pr/olsajiri/uprobe_multi
branch
from
4fc0d21
to
370051e
Compare
olsajiri
force-pushed
the
pr/olsajiri/uprobe_multi
branch
from
370051e
to
1113839
Compare
olsajiri
force-pushed
the
pr/olsajiri/uprobe_multi
branch
3 times, most recently
from
7992c37
to
6d1da2b
Compare
kkourt
approved these changes
Apr 1, 2024
pkg/bpf/detect.go
Outdated
| return false | ||
| } | ||
|
|
||
| // need cilium/ebp fix, can't pass just addresses without symbol |
There was a problem hiding this comment.
Is there an issue for this on cilium/ebpf? Would be nice to have a reference to check when this is fixed and update the code.
nit: typo -> cilium/ebp
There was a problem hiding this comment.
ah, the comment is no longer valid, it got fixed as preparation for uprobe multi support in ebpf/cilium lib
now it is:
func (ex *Executable) addresses(symbols []string, addresses, offsets []uint64) ([]uint64, error) {
n := len(symbols)
if n == 0 {
n = len(addresses)
}
will remove the comment and fix the typo, thanks
jrfastab
approved these changes
Apr 1, 2024
|
Looks good fix small typo and merge imo. |
The MultiRelinkUnloader unloader keeps and unloads multiple links, which will be needed for multi uprobes. Signed-off-by: Jiri Olsa <[email protected]>
To detect uprobe multi link. Signed-off-by: Jiri Olsa <[email protected]>
Compile new bpf_multi_uprobe_v60.o object with __MULTI_KPROBE defined. It will be used for uprobe multi sensor. Signed-off-by: Jiri Olsa <[email protected]>
Adding LoadMultiUprobeProgram function to load uprobe multi programs. Signed-off-by: Jiri Olsa <[email protected]>
Adding support to load multiple uprobes via uprobes multi interface. Signed-off-by: Jiri Olsa <[email protected]>
Adding support to process options passed in spec for uprobe sensor. At the moment the only supported option is to disable uprobe multi. Signed-off-by: Jiri Olsa <[email protected]>
Signed-off-by: Jiri Olsa <[email protected]>
olsajiri
force-pushed
the
pr/olsajiri/uprobe_multi
branch
from
6d1da2b
to
2de8e75
Compare
kkourt
added
release-note/major
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
cilium/ebpf changes cilium/ebpf#1269