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

mbed_rtx_idle: uVisor: Don't attempt to sleep #5107

Merged
merged 1 commit into from Sep 15, 2017

Conversation

Projects
None yet
5 participants
@Patater
Contributor

Patater commented Sep 14, 2017

Description

mbed_rtx_idle: uVisor: Don't attempt to sleep

When uVisor is enabled, don't attempt to sleep. Attempting to sleep will fail, as per ARMmbed/uvisor#420.

Fixes OOB issue ARMmbed/mbed-os-example-uvisor#55

Status

READY

@Patater Patater requested review from c1728p9 and 0xc0170 Sep 14, 2017

@Patater Patater force-pushed the Patater:uvisor-tickless-slumber branch Sep 14, 2017

mbed_rtx_idle: uVisor: Don't attempt to sleep
When uVisor is enabled, don't attempt to sleep. Attempting to sleep will
fail, as per <ARMmbed/uvisor#420>.

@Patater Patater force-pushed the Patater:uvisor-tickless-slumber branch to 0d34c36 Sep 14, 2017

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

@c1728p9 Could you take a look?

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

@Patater What CI does this need?

@Patater

This comment has been minimized.

Contributor

Patater commented Sep 15, 2017

@theotherjimmy Whatever we normally run for changes coming in between rc1 and rc2

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 15, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 15, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1316

All builds and test passed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 15, 2017

@c1728p9 Could you give this a once over so that I can merge it right now?

@theotherjimmy theotherjimmy merged commit 8569c09 into ARMmbed:master Sep 15, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment