-
Notifications
You must be signed in to change notification settings - Fork 176
feat!: drop some syscalls enter events gen, testing and parsing code (part 10) #2608
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
Conversation
As the `io_uring_*` exit events contain all the information needed, drop `io_uring_*` enter events generation from all drivers, and all related testing and parsing code. BREAKING CHANGE: drop `io_uring_*` enter evts gen and parsing Signed-off-by: Leonardo Di Giovanna <[email protected]>
As the `*chown*` exit events contain all the information needed, drop `*chown*` enter events generation from all drivers, and all related testing and parsing code. BREAKING CHANGE: drop `*chown*` enter evts gen and parsing Signed-off-by: Leonardo Di Giovanna <[email protected]>
As the `pidfd_{getfd,open}` exit events contain all the information
needed, drop `pidfd_{getfd,open}` enter events generation from all
drivers, and all related testing and parsing code.
BREAKING CHANGE: drop `pidfd_{getfd,open}` enter evts gen and parsing
Signed-off-by: Leonardo Di Giovanna <[email protected]>
As the `{,f}init_module` exit events contain all the information
needed, drop `{,f}init_module` enter events generation from all
drivers, and all related testing and parsing code.
BREAKING CHANGE: drop `{,f}init_module` enter evts gen and parsing
Signed-off-by: Leonardo Di Giovanna <[email protected]>
As the `umount2` exit events contain all the information needed, drop `umount2` enter events generation from all drivers, and all related testing and parsing code. BREAKING CHANGE: drop `umount2` enter evts gen and parsing Signed-off-by: Leonardo Di Giovanna <[email protected]>
As the `setre{uid,gid}` exit events contain all the information
needed, drop `setre{uid,gid}` enter events generation from all
drivers, and all related testing and parsing code.
BREAKING CHANGE: drop `setre{uid,gid}` enter evts gen and parsing
Signed-off-by: Leonardo Di Giovanna <[email protected]>
|
Please double check driver/SCHEMA_VERSION file. See versioning. /hold |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ekoops The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2608 +/- ##
==========================================
- Coverage 78.10% 78.09% -0.01%
==========================================
Files 298 298
Lines 32023 32014 -9
Branches 4696 4696
==========================================
- Hits 25011 25002 -9
Misses 7012 7012
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
X64 kernel testing matrix
ARM64 kernel testing matrix
|
|
/hold cancel |
What type of PR is this?
/kind cleanup
/kind test
/kind feature
Any specific area of the project related to this PR?
/area driver-kmod
/area driver-bpf
/area driver-modern-bpf
/area libpman
/area libsinsp
/area tests
Does this PR require a change in the driver versions?
What this PR does / why we need it:
This PR is part of #2588.
Specifically, this addresses the following system calls:
io_uring_setupio_uring_enterio_uring_registerchownlchownfchownfchownatumount2pidfd_getfdpidfd_openinit_modulefinit_modulesetreuidsetregidWhich issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Not bumping the driver schema version is currently a practice we are following as we will bump it in a single shot once we are done with the #2068 proposal.
/milestone 0.22.0
Does this PR introduce a user-facing change?: