diff options
| author | Michael Sevakis <jethead71@rockbox.org> | 2017-01-21 14:18:37 -0500 |
|---|---|---|
| committer | Michael Sevakis <jethead71@rockbox.org> | 2017-01-21 14:25:52 -0500 |
| commit | 3e738661108fb2a8e3474c6336c2a5c2668f82b6 (patch) | |
| tree | c3c7bc51c4329ab6652fd2550a241a56bbafda1f /firmware/kernel/queue.c | |
| parent | da464572318a34f4791ccb419acbf0a470b810e6 (diff) | |
| download | rockbox-3e738661108fb2a8e3474c6336c2a5c2668f82b6.zip rockbox-3e738661108fb2a8e3474c6336c2a5c2668f82b6.tar.gz rockbox-3e738661108fb2a8e3474c6336c2a5c2668f82b6.tar.bz2 rockbox-3e738661108fb2a8e3474c6336c2a5c2668f82b6.tar.xz | |
Add CPU mode asserts to kernel on blocking functions.
This scourge finds it's way back in far too often.
Right now, only defined for ARM.
Have fun!
Change-Id: Ib21be09ebf71dec10dc652a7a664779251f49644
Diffstat (limited to 'firmware/kernel/queue.c')
| -rw-r--r-- | firmware/kernel/queue.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/firmware/kernel/queue.c b/firmware/kernel/queue.c index 927e552..70dba46 100644 --- a/firmware/kernel/queue.c +++ b/firmware/kernel/queue.c @@ -283,6 +283,9 @@ void queue_wait(struct event_queue *q, struct queue_event *ev) #endif oldlevel = disable_irq_save(); + + ASSERT_CPU_MODE(CPU_MODE_THREAD_CONTEXT, oldlevel); + corelock_lock(&q->cl); #ifdef HAVE_EXTENDED_MESSAGING_AND_NAME @@ -335,6 +338,10 @@ void queue_wait_w_tmo(struct event_queue *q, struct queue_event *ev, int ticks) #endif oldlevel = disable_irq_save(); + + if (ticks != TIMEOUT_NOBLOCK) + ASSERT_CPU_MODE(CPU_MODE_THREAD_CONTEXT, oldlevel); + corelock_lock(&q->cl); #ifdef HAVE_EXTENDED_MESSAGING_AND_NAME @@ -421,6 +428,9 @@ intptr_t queue_send(struct event_queue *q, long id, intptr_t data) unsigned int wr; oldlevel = disable_irq_save(); + + ASSERT_CPU_MODE(CPU_MODE_THREAD_CONTEXT, oldlevel); + corelock_lock(&q->cl); wr = q->write++ & QUEUE_LENGTH_MASK; |