kernel: threads: Do not use string compare instead of bit ops
Remove converting bit to string and comparing the string instead of
ready helpers. The "Check if thread is in use" seems to check only
that parameters state_buf and sizeof(state_buf) not zero.
Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
diff --git a/kernel/dynamic.c b/kernel/dynamic.c
index d30dba8..df94aaa 100644
--- a/kernel/dynamic.c
+++ b/kernel/dynamic.c
@@ -7,6 +7,7 @@
#include "kernel_internal.h"
#include <zephyr/kernel.h>
+#include <ksched.h>
#include <zephyr/kernel/thread_stack.h>
#include <zephyr/logging/log.h>
#include <zephyr/sys/bitarray.h>
@@ -120,20 +121,14 @@
int z_impl_k_thread_stack_free(k_thread_stack_t *stack)
{
- char state_buf[16] = {0};
struct dyn_cb_data data = {.stack = stack};
/* Get a possible tid associated with stack */
k_thread_foreach(dyn_cb, &data);
if (data.tid != NULL) {
- /* Check if thread is in use */
- if (k_thread_state_str(data.tid, state_buf, sizeof(state_buf)) != state_buf) {
- LOG_ERR("tid %p is invalid!", data.tid);
- return -EINVAL;
- }
-
- if (!(strcmp("dummy", state_buf) == 0) || (strcmp("dead", state_buf) == 0)) {
+ if (!(z_is_thread_state_set(data.tid, _THREAD_DUMMY) ||
+ z_is_thread_state_set(data.tid, _THREAD_DEAD))) {
LOG_ERR("tid %p is in use!", data.tid);
return -EBUSY;
}
