Skip to content

Add sched_yield calls in thread_ptrace#1127

Merged
sporksmith merged 2 commits intoshadow:devfrom
sporksmith:ptrace-sched-yield
Feb 27, 2021
Merged

Add sched_yield calls in thread_ptrace#1127
sporksmith merged 2 commits intoshadow:devfrom
sporksmith:ptrace-sched-yield

Conversation

@sporksmith
Copy link
Copy Markdown
Contributor

This gives the plugin a chance to run before checking whether it's
stopped again. This prevents deadlock in ptrace's "hybrid spin" when
using --pin-cpus and --sched-fifo together, and helps performance in
other cases.

sporksmith added a commit to sporksmith/shadow that referenced this pull request Feb 26, 2021
This gives the plugin a chance to run before checking whether it's
stopped again. This prevents deadlock in ptrace's "hybrid spin" when
using --pin-cpus and --sched-fifo together, and helps performance in
other cases.

Progress on shadow#1127
This gives the plugin a chance to run before checking whether it's
stopped again. This prevents deadlock in ptrace's "hybrid spin" when
using --pin-cpus and --sched-fifo together, and helps performance in
other cases.

Progress on shadow#1041
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 26, 2021

Codecov Report

Merging #1127 (b3daabb) into dev (4e90ac2) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1127      +/-   ##
==========================================
- Coverage   55.96%   55.95%   -0.01%     
==========================================
  Files         136      136              
  Lines       20075    20077       +2     
  Branches     4770     4770              
==========================================
  Hits        11235    11235              
  Misses       5995     5995              
- Partials     2845     2847       +2     
Flag Coverage Δ
tests 55.95% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/main/host/thread_ptrace.c 67.29% <100.00%> (+0.09%) ⬆️
src/main/core/scheduler/scheduler.c 77.40% <0.00%> (-0.34%) ⬇️
src/main/host/descriptor/file.c 28.53% <0.00%> (-0.24%) ⬇️

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 4e90ac2...0e863d6. Read the comment docs.

@sporksmith sporksmith merged commit f1bcb25 into shadow:dev Feb 27, 2021
@sporksmith sporksmith deleted the ptrace-sched-yield branch February 27, 2021 18:00
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.

2 participants