microsoft / microsoft/ebpf-for-windows
Unify ebpf_execution_type_t and ebpf_code_type_t into one enum.
- Dominant language
- C
- Stars
- 3.6k
- Forks
- 311
- Avg merge
- 6d 10h
- Merged PRs (30d)
- 21
Description
### Describe the feature you'd like supported
Currently there are 2 enums: `ebpf_execution_type_t` and `ebpf_code_type_t`, one in user mode and one in EC.
They should be ideally merged into 1 enum.
The proposed solution is in this discussion:
https://github.com/microsoft/ebpf-for-windows/pull/811#discussion_r836641519
### Proposed solution
The proposed solution is in this discussion:
https://github.com/microsoft/ebpf-for-windows/pull/811#discussion_r836641519
### Additional context
_No response_
Contributor guide
Research direction
Start by reading the definitions and uses of ebpf_execution_type_t and ebpf_code_type_t in the user-mode and EC portions of the repository. Review the proposed solution in the linked discussion on pull request #811, then trace callers and related tests to identify the compatibility impact. Done means the two enums are represented consistently without breaking user-mode or EC behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- operating-systems
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100