ARMmbed / ARMmbed/uvisor

Replace page_allocator.c_inc mechanism with header file

Open
#239 1 comment 0 reactions 0 assignees View on GitHub
future uvisor RTOS support mirrored
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

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.