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

tetragon: Factor the maps max entries setup #2565

Merged
merged 4 commits into from
Jun 17, 2024
Merged

Conversation

olsajiri
Copy link
Contributor

@olsajiri olsajiri commented Jun 15, 2024

Making it more straight forward to setup max entries for map shared between multiple programs.

@olsajiri olsajiri force-pushed the pr/olsajiri/max_entries branch 2 times, most recently from 47916cd to 74dd1c8 Compare June 15, 2024 21:57
@olsajiri olsajiri added the release-note/minor This PR introduces a minor user-visible change label Jun 15, 2024
@olsajiri olsajiri force-pushed the pr/olsajiri/max_entries branch from 74dd1c8 to 7e18f2c Compare June 16, 2024 07:44
olsajiri added 2 commits June 16, 2024 09:46
When we store Map object in PinMap map we will have access to
all the map details, which allows us to do easier max entries
setup coming in following changes.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Moving max entries values directly to Map object, because now
we can access the map object directly in the loader, so we do
not need to store them in program object.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
@olsajiri olsajiri force-pushed the pr/olsajiri/max_entries branch from 7e18f2c to a47c50a Compare June 16, 2024 17:47
@olsajiri olsajiri changed the title Pr/olsajiri/max entries tetragon: Factor the maps max entries setup Jun 16, 2024
@olsajiri olsajiri marked this pull request as ready for review June 16, 2024 19:37
@olsajiri olsajiri requested a review from a team as a code owner June 16, 2024 19:37
@olsajiri olsajiri requested a review from kkourt June 16, 2024 19:37
@olsajiri olsajiri self-assigned this Jun 16, 2024
@olsajiri olsajiri requested a review from mtardy June 16, 2024 19:37
olsajiri added 2 commits June 16, 2024 19:41
Making MapBuilder/Pin take variadic program arguments, so we don't
need MapBuilderPinManyProgs function.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Using multiple programs argument for CgroupRateMap and creating just
single CgroupRateMap object. Now with new set max entries interface
we can have just single map object for multiple programs.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
@olsajiri olsajiri force-pushed the pr/olsajiri/max_entries branch from a47c50a to be4de5c Compare June 16, 2024 19:41
@olsajiri olsajiri merged commit e3dd9f6 into main Jun 17, 2024
39 checks passed
@olsajiri olsajiri deleted the pr/olsajiri/max_entries branch June 17, 2024 10:36
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.

2 participants