Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pr/olsajiri/footprint 1 #1804

Closed
wants to merge 4 commits into from
Closed

Pr/olsajiri/footprint 1 #1804

wants to merge 4 commits into from

Conversation

olsajiri
Copy link
Contributor

wip

The pass in msg_selector_data is treated as bool value,
make it one then.

Signed-off-by: Jiri Olsa <[email protected]>
There's no need to keep separate kprobe programs for each
filter_read_argX call, we can just tail call into single
program with increased index.

The number of loaded programs gets reduced, so we need to
fix TestLoad* tests as well.

Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri added the release-note/minor This PR introduces a minor user-visible change label Nov 27, 2023
@olsajiri olsajiri force-pushed the pr/olsajiri/footprint_1 branch from f11eaa2 to 92e4c99 Compare November 27, 2023 15:21
There's no need to keep separate kprobe programs for each
generic_kprobe_process_event4X call, we can just tail call
into single program with increased index.

The number of loaded programs gets reduced, so we need to
fix TestLoad* tests as well.

I had to add several &= masks to make it verifiable under 5.4,
can't find better solution yet.

Signed-off-by: Jiri Olsa <[email protected]>
Adding TAIL_CALL_* enum values to identify tail calls for generic
probes and make it more human friendly.

Suggested-by: Djalal Harouni <[email protected]>
Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri force-pushed the pr/olsajiri/footprint_1 branch from 92e4c99 to 8f15a30 Compare November 27, 2023 16:59
@olsajiri olsajiri closed this Nov 28, 2023
@olsajiri olsajiri deleted the pr/olsajiri/footprint_1 branch November 28, 2023 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/minor This PR introduces a minor user-visible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant