michaelh/zephyr
1
0
Fork 0
Code Releases Activity

kernel: work_q: Add an init function

Browse source 
k_work_queue_start receives a struct that is expected to be
uninitialized (zeroed). Otherwise the behavior is undefined.

Following the Zephyr semantics, this pr introduce a new init function
for this struct.

Fixes #36865

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
This commit is contained in:
Flavio Ceolin 2021-08-23 14:33:40 -07:00 • committed by Anas Nashif
commit d9aa414831
2 changed files with 22 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

11
include/kernel.h
View file

@ -3045,6 +3045,17 @@ int k_work_cancel(struct k_work *work);
*/
bool k_work_cancel_sync(struct k_work *work, struct k_work_sync *sync);
/** @brief Initialize a work queue structure.
*
* This must be invoked before starting a work queue structure for the first time.
* It need not be invoked again on the same work queue structure.
*
* @funcprops \isr_ok
*
* @param queue the queue structure to be initialized.
*/
void k_work_queue_init(struct k_work_q *queue);
/** @brief Initialize a work queue.
*
* This configures the work queue thread and starts it running. The function

11
kernel/work.c
View file

@ -676,6 +676,17 @@ static void work_queue_main(void *workq_ptr, void *p2, void *p3)
}
}
void k_work_queue_init(struct k_work_q *queue)
{
__ASSERT_NO_MSG(queue != NULL);
*queue = (struct k_work_q) {
.flags = 0,
};
SYS_PORT_TRACING_OBJ_INIT(k_work_queue, queue);
}
void k_work_queue_start(struct k_work_q *queue,
k_thread_stack_t *stack,
size_t stack_size,