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: Keep map setup in doLoadProgram #2803

Closed
wants to merge 11 commits into from
Closed

Conversation

olsajiri
Copy link
Contributor

wip

@olsajiri olsajiri added the release-note/minor This PR introduces a minor user-visible change label Aug 15, 2024
@olsajiri olsajiri force-pushed the pr/olsajiri/maps branch 15 times, most recently from 699b282 to be45657 Compare August 21, 2024 11:15
Copy link

netlify bot commented Aug 21, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 699b282
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/66c5cc47e31dcb0008cf1a6c
😎 Deploy Preview https://deploy-preview-2803--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Aug 21, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 02d88da
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/6792a50c97f79c0008490d86
😎 Deploy Preview https://deploy-preview-2803--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@olsajiri olsajiri force-pushed the pr/olsajiri/maps branch 3 times, most recently from e7d3cfc to acd4931 Compare August 30, 2024 07:54
Moving override action code in do_override_action so we can
define it only for kprobe and lsm program (GENERIC_KPROBE and
GENERIC_LSM) where it belongs. Now it's also part of kretprobe.

Signed-off-by: Jiri Olsa <[email protected]>
Moving SizeWithSuffix to strutils package, so we can
use it from other places.

Signed-off-by: Jiri Olsa <[email protected]>
It's possible to omit programs when we create map in which
case we explode. Adding check for that.

Signed-off-by: Jiri Olsa <[email protected]>
Adding TAIL_CALL_SETUP tail call index, which was missing
in the TAIL_CALL_* enum values.

Signed-off-by: Jiri Olsa <[email protected]>
Switch 'enter' check to '!enter' and decrease indent of the
following code, which makes the function more readable and
will ease up following changes.

There's no functional change.

Signed-off-by: Jiri Olsa <[email protected]>
Passing execve_map_value directly to match_binaries to
eliminate superfluous event_find_curr in it.

Signed-off-by: Jiri Olsa <[email protected]>
Adding support to allow to setup execve_map max entries,
so we can control the size of this map.

Signed-off-by: Jiri Olsa <[email protected]>
Adding execve-map-entries option to setup entries of execve_map map.

Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri closed this Jan 23, 2025
@olsajiri olsajiri deleted the pr/olsajiri/maps branch January 23, 2025 20:23
@olsajiri olsajiri restored the pr/olsajiri/maps branch January 23, 2025 20:23
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