Skip to content
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

tests/kernel/sched/deadline: Disable CONFIG_BT #9855

Merged
merged 1 commit into from
Sep 8, 2018

Conversation

andyross
Copy link
Contributor

@andyross andyross commented Sep 7, 2018

The BT threads are interfering with the main thread priority selection
and the test fails semi-spuriously on NRF5x boards with "threads ran
too soon" (i.e. not an EDF failure per se, but the fact that the new
threads at K_LOWEST_APPLICATION_PRIO are running instead of the test
thread). This is a reasonable workaround for testing the
SCHED_DEADLINE ordering behavior, though.

Fixes #9843

Signed-off-by: Andy Ross [email protected]

The BT threads are interfering with the main thread priority selection
and the test fails semi-spuriously on NRF5x boards with "threads ran
too soon" (i.e. not an EDF failure per se, but the fact that the new
threads at K_LOWEST_APPLICATION_PRIO are running instead of the test
thread).  This is a reasonable workaround for testing the
SCHED_DEADLINE ordering behavior, though.

Fixes zephyrproject-rtos#9843

Signed-off-by: Andy Ross <[email protected]>
@codecov-io
Copy link

Codecov Report

Merging #9855 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9855   +/-   ##
=======================================
  Coverage   52.27%   52.27%           
=======================================
  Files         213      213           
  Lines       25962    25962           
  Branches     5596     5596           
=======================================
  Hits        13571    13571           
  Misses      10154    10154           
  Partials     2237     2237

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cdc7395...4888da3. Read the comment docs.

@nashif nashif merged commit a7e5d2f into zephyrproject-rtos:master Sep 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants