microsoft / microsoft/ebpf-for-windows

Unify ebpf_execution_type_t and ebpf_code_type_t into one enum.

Open
#866 0 comments 0 reactions 0 assignees View on GitHub
cleanup good first issue triaged
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.