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

bpf: Fix perf_event_output_metric on 4.19 #2868

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Conversation

lambdanis
Copy link
Contributor

@lambdanis lambdanis commented Sep 2, 2024

Using this function on 4.19 I hit:

loadInstance: opening collection '/host/bpf/objs/bpf_execve_event.o' failed:
program event_execve: load program: argument list too long

A switch seems more verifier friendly on 4.19 🤷

Using this function on 4.19 I hit:
```
loadInstance: opening collection '/host/bpf/objs/bpf_execve_event.o' failed:
program event_execve: load program: argument list too long
```

A switch seems more verifier friednly on 4.19 🤷

Signed-off-by: Anna Kapuscinska <[email protected]>
@lambdanis lambdanis force-pushed the pr/lambdanis/missedevents branch from e3c4fcd to dd84825 Compare September 3, 2024 09:45
@lambdanis lambdanis marked this pull request as ready for review September 3, 2024 09:45
@lambdanis lambdanis requested a review from a team as a code owner September 3, 2024 09:45
@lambdanis lambdanis requested a review from jrfastab September 3, 2024 09:45
@lambdanis lambdanis added the release-note/misc This PR makes changes that have no direct user impact. label Sep 3, 2024
@lambdanis lambdanis merged commit 4360b28 into main Sep 3, 2024
45 of 47 checks passed
@lambdanis lambdanis deleted the pr/lambdanis/missedevents branch September 3, 2024 11:12
@lambdanis lambdanis changed the title bpf: Fix perf_event_output_metric bpf: Fix perf_event_output_metric on 4.19 Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants