diff options
| author | Gregory Nutt <gnutt@nuttx.org> | 2014-11-20 06:58:19 -0600 |
|---|---|---|
| committer | Gregory Nutt <gnutt@nuttx.org> | 2014-11-20 07:10:48 -0600 |
| commit | e04da2894b67d6f13cd1fc3b754e90f62e5df35d (patch) | |
| tree | 54ae2573a4b6cca23b5425b9cc845d6ae5d33d4b | |
| parent | 7d79fc44e134dbfe542b9c500f71cee2ddc85845 (diff) | |
| download | nuttx-e04da2894b67d6f13cd1fc3b754e90f62e5df35d.tar.gz nuttx-e04da2894b67d6f13cd1fc3b754e90f62e5df35d.tar.bz2 nuttx-e04da2894b67d6f13cd1fc3b754e90f62e5df35d.zip | |
Cosmetic chnages to comments
| -rw-r--r-- | nuttx/sched/init/os_start.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/nuttx/sched/init/os_start.c b/nuttx/sched/init/os_start.c index 59929e0b5..f48bea318 100644 --- a/nuttx/sched/init/os_start.c +++ b/nuttx/sched/init/os_start.c @@ -534,6 +534,12 @@ void os_start(void) * BUT the idle task cannot wait on a semaphore. So we only do * the cleanup now if we can get the semaphore -- this should be * possible because if the IDLE thread is running, no other task is! + * + * WARNING: This logic could have undesirable side-effects if priority + * inheritance is enabled. Imaginee the possible issues if the + * priority of the IDLE thread were to get boosted! Moral: If you + * use priority inheritance, then you should also enable the work + * queue so that is done in a safer context. */ if (kmm_trysemaphore() == 0) |