Skip to content
This repository has been archived by the owner on Jan 15, 2021. It is now read-only.

Comment out assertions in critical.c #98

Merged
merged 1 commit into from
Mar 4, 2016
Merged

Comment out assertions in critical.c #98

merged 1 commit into from
Mar 4, 2016

Conversation

AlessandroA
Copy link

For the moment assertions need to be commented out as this code might
run when the libc initialization has not completed yet. In those cases,
an uninitialized stderr pointer will trigger a bus fault / uVisor access
fault as it points to flash (__sf_fake_stderr).

For reference:
ARMmbed/mbed-drivers#176 Printing to stderr attempts a write into flash

@bogdanm @0xc0170 @adbridge

For the moment assertions need to be commented out as this code might
run when the libc initialization has not completed yet. In those cases,
an uninitialized stderr pointer will trigger a bus fault / uVisor access
fault as it points to flash (`__sf_fake_stderr`).

For reference:
ARMmbed/mbed-drivers#176 Printing to stderr attempts a write into flash
@0xc0170
Copy link
Contributor

0xc0170 commented Mar 4, 2016

I would like to find out the real cause of this, if we can fix it in our code base or not. Is this critical , as to be fixed within today or can wait until Monday to have a quick look at stdio initialization ?

I am happy with fixing this until we get real cause. If this gets in, a tracking issue would be good to have.

@AlessandroA
Copy link
Author

This is a workaround that unblocks uvisor-helloworld.

If OOB testing starts on Monday I'd like this to be merged, as uvisor-helloworld cannot work otherwise. We can create an issue and track this here in core-util, to remind us to revert the changes when we fix the real problem.

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 4, 2016

If OOB testing starts on Monday I'd like this to be merged, as uvisor-helloworld cannot work otherwise. We can create an issue and track this here in core-util, to remind us to revert the changes when we fix the real problem.

@bogdanm agree?

@bogdanm
Copy link
Contributor

bogdanm commented Mar 4, 2016

Yes, we have to do this for now. +1.

0xc0170 added a commit that referenced this pull request Mar 4, 2016
Comment out assertions in critical.c
@0xc0170 0xc0170 merged commit 0ef4e59 into ARMmbed:master Mar 4, 2016
@bogdanm
Copy link
Contributor

bogdanm commented Mar 4, 2016

Please don't forget to open the tracking issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants