-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change libphoenix initialization #313
Conversation
8265311
to
8381f1f
Compare
8381f1f
to
26abc73
Compare
7c71601
to
bdcf9c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but add yourself to authors list
bdcf9c3
to
b158221
Compare
b158221
to
3c0d767
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@agkaminski Is the CI failling on imx1060 a known issue or sth new created by this pr? |
I don't think I've seen it, looks like a user stack overflow. Might be caused by this PR I guess - maybe |
Locally it doesn't inline _libc_init, also seems to be too undeterministic for that to be the case |
3c0d767
to
cda3903
Compare
This change prepares libphoenix to be used in dynamic loader, where crt0 is not included. Moved constructor and destrutor calling into crt0. Wrapped libphoenix initialization into a single function to reduce crt0 and libphoenix intertwine. JIRA: RTOS-694
cda3903
to
a103773
Compare
@agkaminski I added noinlien to _init_array as it got inlined after rewrite and increased stack size of |
Through the night I run the CI 2 more times still no errors |
This change prepares libphoenix to be used in dynamic loader, where crt0 is not included.
Moved constructor and destrutor calling into crt0. Wrapped libphoenix initialization into a single function to reduce crt0 and libphoenix intertwine.
JIRA: RTOS-694
Description
[RTOS-694] #done
Motivation and Context
Types of changes
How Has This Been Tested?
Checklist:
Special treatment