remove usage of SYS_INIT in the kernel #74219
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
nashif
commented
Jun 13, 2024
- init: move cpu initialization out of init
- kernel: initialize object cores directly
- init: move system work to main init process
- kernel: remove SYS_INIT usage for userspace
- init: remove SYS_INIT usage for memory domains
- init: initialize kheap statics in the main init process
- kernel: do not use SYS_INIT for mailbox initialization
- kernel: rename k_sys_work_q_init -> init_system_workqueue
12d0147 to
b8117b1
Compare
19a7650 to
e8e9f80
Compare
There is no need for the cpu and idle thread initialization to be crammed into init.c. Move out to new/existing files for readability. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Do not use SYS_INIT for initialization object cores and statsitics. Those are expected to be initialized in the same spot and their dependencies are fixed. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Do not use SYS_INIT for start the system workqueue. It does have fixed dependencies and can be directly started from the main initialization process. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Directly initialize memory for userspace in the main init process. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Directly initialize memory domains in the main init process. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Do not use SYS_INIT for kheap statics, directly invoke code in the main init process. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Directly call initialization function instead of SYS_INIT. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
Avoid confusion with API using same prefix. This is no public API. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
obj_core and object initialization were done all under obj_core which is not working as expected. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
e8e9f80 to
044ebe8
Compare
|
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
|
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.