diff options
author | Bjoern Brandenburg <bbb@mpi-sws.org> | 2015-08-09 07:18:50 -0400 |
---|---|---|
committer | Bjoern Brandenburg <bbb@mpi-sws.org> | 2015-08-09 07:20:13 -0400 |
commit | 13e68a73175041c83713fd1905c65fe612f73862 (patch) | |
tree | be3edeaac8b1b34714dd75d54d17f1ec88ecac35 /kernel | |
parent | e82bc71470695e6a8a319962737b4abff85bf0c6 (diff) |
Augment rt_task() with is_realtime()
Whenever the kernel checks for rt_task() to avoid delaying real-time
tasks, we want it to also not delay LITMUS^RT tasks. Hence, most
calls to rt_task() should be matched by an equivalent call to
is_realtime().
Notably, this affects the implementations of select() and nanosleep(),
which use timer_slack_ns when setting up timers for non-real-time
tasks.
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/locking/mutex.c | 4 | ||||
-rw-r--r-- | kernel/locking/rwsem-xadd.c | 4 | ||||
-rw-r--r-- | kernel/time/hrtimer.c | 4 |
3 files changed, 9 insertions, 3 deletions
diff --git a/kernel/locking/mutex.c b/kernel/locking/mutex.c index 4cccea6b8934..a8546e29671e 100644 --- a/kernel/locking/mutex.c +++ b/kernel/locking/mutex.c | |||
@@ -27,6 +27,8 @@ | |||
27 | #include <linux/debug_locks.h> | 27 | #include <linux/debug_locks.h> |
28 | #include <linux/osq_lock.h> | 28 | #include <linux/osq_lock.h> |
29 | 29 | ||
30 | #include <litmus/litmus.h> /* for is_realtime() */ | ||
31 | |||
30 | /* | 32 | /* |
31 | * In the DEBUG case we are using the "NULL fastpath" for mutexes, | 33 | * In the DEBUG case we are using the "NULL fastpath" for mutexes, |
32 | * which forces all calls into the slowpath: | 34 | * which forces all calls into the slowpath: |
@@ -368,7 +370,7 @@ static bool mutex_optimistic_spin(struct mutex *lock, | |||
368 | * we're an RT task that will live-lock because we won't let | 370 | * we're an RT task that will live-lock because we won't let |
369 | * the owner complete. | 371 | * the owner complete. |
370 | */ | 372 | */ |
371 | if (!owner && (need_resched() || rt_task(task))) | 373 | if (!owner && (need_resched() || rt_task(task) || is_realtime(task))) |
372 | break; | 374 | break; |
373 | 375 | ||
374 | /* | 376 | /* |
diff --git a/kernel/locking/rwsem-xadd.c b/kernel/locking/rwsem-xadd.c index 3417d0172a5d..873ffca2321c 100644 --- a/kernel/locking/rwsem-xadd.c +++ b/kernel/locking/rwsem-xadd.c | |||
@@ -18,6 +18,8 @@ | |||
18 | 18 | ||
19 | #include "rwsem.h" | 19 | #include "rwsem.h" |
20 | 20 | ||
21 | #include <litmus/litmus.h> /* for is_realtime() */ | ||
22 | |||
21 | /* | 23 | /* |
22 | * Guide to the rw_semaphore's count field for common values. | 24 | * Guide to the rw_semaphore's count field for common values. |
23 | * (32-bit case illustrated, similar for 64-bit) | 25 | * (32-bit case illustrated, similar for 64-bit) |
@@ -392,7 +394,7 @@ static bool rwsem_optimistic_spin(struct rw_semaphore *sem) | |||
392 | * we're an RT task that will live-lock because we won't let | 394 | * we're an RT task that will live-lock because we won't let |
393 | * the owner complete. | 395 | * the owner complete. |
394 | */ | 396 | */ |
395 | if (!owner && (need_resched() || rt_task(current))) | 397 | if (!owner && (need_resched() || rt_task(current) || is_realtime(current))) |
396 | break; | 398 | break; |
397 | 399 | ||
398 | /* | 400 | /* |
diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c index 22f9156f19d2..d5a8e4db0bf9 100644 --- a/kernel/time/hrtimer.c +++ b/kernel/time/hrtimer.c | |||
@@ -58,6 +58,8 @@ | |||
58 | 58 | ||
59 | #include "tick-internal.h" | 59 | #include "tick-internal.h" |
60 | 60 | ||
61 | #include <litmus/litmus.h> /* for is_realtime() */ | ||
62 | |||
61 | /* | 63 | /* |
62 | * The timer bases: | 64 | * The timer bases: |
63 | * | 65 | * |
@@ -1659,7 +1661,7 @@ long hrtimer_nanosleep(struct timespec *rqtp, struct timespec __user *rmtp, | |||
1659 | unsigned long slack; | 1661 | unsigned long slack; |
1660 | 1662 | ||
1661 | slack = current->timer_slack_ns; | 1663 | slack = current->timer_slack_ns; |
1662 | if (dl_task(current) || rt_task(current)) | 1664 | if (dl_task(current) || rt_task(current) || is_realtime(current)) |
1663 | slack = 0; | 1665 | slack = 0; |
1664 | 1666 | ||
1665 | hrtimer_init_on_stack(&t.timer, clockid, mode); | 1667 | hrtimer_init_on_stack(&t.timer, clockid, mode); |