patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 7 Packages Projects Releases Wiki Activity

kernel: Explicitly check _abort_thread_timemout

Browse source 
A lot of times this API is called during some cleanup even if the
timeout was not set to make the code simpler. In these cases it's not
necessary checking the return. Adding a cast to acknowledge it.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
This commit is contained in:
Flavio Ceolin 2018-09-12 22:11:00 -07:00 committed by Anas Nashif
parent 98c64b6d92
commit 8f72f245bd
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
kernel/sched.c
View file

@ -334,7 +334,7 @@ struct k_thread *_unpend_first_thread(_wait_q_t *wait_q)
struct k_thread *t = _unpend1_no_timeout(wait_q);
if (t) {
_abort_thread_timeout(t);
(void)_abort_thread_timeout(t);
}
return t;
@ -343,7 +343,7 @@ struct k_thread *_unpend_first_thread(_wait_q_t *wait_q)
void _unpend_thread(struct k_thread *thread)
{
_unpend_thread_no_timeout(thread);
_abort_thread_timeout(thread);
(void)_abort_thread_timeout(thread);
}
/* FIXME: this API is glitchy when used in SMP. If the thread is

4
kernel/thread.c
View file

@ -479,7 +479,7 @@ int _impl_k_thread_cancel(k_tid_t tid)
return -EINVAL;
}
_abort_thread_timeout(thread);
(void)_abort_thread_timeout(thread);
_thread_monitor_exit(thread);
irq_unlock(key);
@ -553,7 +553,7 @@ void _k_thread_single_abort(struct k_thread *thread)
_unpend_thread_no_timeout(thread);
}
if (_is_thread_timeout_active(thread)) {
_abort_thread_timeout(thread);
(void)_abort_thread_timeout(thread);
}
}