Replace page_allocator.c_inc mechanism with header file
- Dominant language
- C
- Stars
- 135
- Forks
- 72
- PR merge metrics
- No merged PRs in 30d
Description
`page_allocator.c` could have a conditional `.h` include depending on whether the `.c` is compiled in a uvisor context or in unsupported. The unsupported header file does not need to be included in the supported tree and vice versa. Potentially the header file name can be identical for bot supported and unsupported (`page_allocator_config.h`) and have different content for supported/unsupported.
Requires support from build system `UVISOR_UNSUPPORTED` folder.
Contributor guide
Research direction
Start with page_allocator.c and the UVISOR_UNSUPPORTED folder, then trace how the current page_allocator.c_inc mechanism is included in supported and unsupported builds. Done means replacing that mechanism with conditional page_allocator_config.h headers and adding the required build-system support for UVISOR_UNSUPPORTED.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- build-system
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100