Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2026-06-06 00:14:22 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

os init_state: add new state OSINIT_IDLELOOP

Browse Source 
This is for distinguish we are in init idle or idle loop.
Use for assertion for sem_trywait & sem_wait

Signed-off-by: ligd <liguiding1@xiaomi.com>
This commit is contained in:
ligd
2022-02-21 18:14:10 +08:00
committed by Petro Karashchenko
parent 8785673a70
commit 051bb32010
4 changed files with 13 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
include/nuttx/init.h
+3 -1
View File
@@ -41,6 +41,7 @@
#define OSINIT_MM_READY() (g_nx_initstate >= OSINIT_MEMORY) #define OSINIT_MM_READY() (g_nx_initstate >= OSINIT_MEMORY)
#define OSINIT_HW_READY() (g_nx_initstate >= OSINIT_HARDWARE) #define OSINIT_HW_READY() (g_nx_initstate >= OSINIT_HARDWARE)
#define OSINIT_OS_READY() (g_nx_initstate >= OSINIT_OSREADY) #define OSINIT_OS_READY() (g_nx_initstate >= OSINIT_OSREADY)
#define OSINIT_IDLELOOP() (g_nx_initstate >= OSINIT_IDLELOOP)
#define OSINIT_OS_INITIALIZING() (g_nx_initstate < OSINIT_OSREADY) #define OSINIT_OS_INITIALIZING() (g_nx_initstate < OSINIT_OSREADY)
/**************************************************************************** /****************************************************************************
@@ -65,8 +66,9 @@ enum nx_initstate_e
* to support the hardware are also available but * to support the hardware are also available but
* the OS has not yet completed its full * the OS has not yet completed its full
* initialization. */ * initialization. */
OSINIT_OSREADY = 5 /* The OS is fully initialized and multi-tasking is OSINIT_OSREADY = 5, /* The OS is fully initialized and multi-tasking is
* active. */ * active. */
OSINIT_IDLELOOP = 6 /* The OS enter idle loop */
}; };
/**************************************************************************** /****************************************************************************
sched/init/nx_start.c
+4
View File
@@ -776,6 +776,10 @@ void nx_start(void)
DEBUGVERIFY(nx_bringup()); DEBUGVERIFY(nx_bringup());
/* Enter to idleloop */
g_nx_initstate = OSINIT_IDLELOOP;
/* Let other threads have access to the memory manager */ /* Let other threads have access to the memory manager */
sched_unlock(); sched_unlock();
sched/semaphore/sem_trywait.c
+3 -1
View File
@@ -29,6 +29,7 @@
#include <assert.h> #include <assert.h>
#include <errno.h> #include <errno.h>
#include <nuttx/init.h>
#include <nuttx/irq.h> #include <nuttx/irq.h>
#include <nuttx/arch.h> #include <nuttx/arch.h>
@@ -69,9 +70,10 @@ int nxsem_trywait(FAR sem_t *sem)
irqstate_t flags; irqstate_t flags;
int ret; int ret;
/* This API should not be called from interrupt handlers */ /* This API should not be called from interrupt handlers & idleloop */
DEBUGASSERT(sem != NULL && up_interrupt_context() == false); DEBUGASSERT(sem != NULL && up_interrupt_context() == false);
DEBUGASSERT(OSINIT_IDLELOOP() && !sched_idletask());
if (sem != NULL) if (sem != NULL)
{ {
sched/semaphore/sem_wait.c
+3 -1
View File
@@ -28,6 +28,7 @@
#include <errno.h> #include <errno.h>
#include <assert.h> #include <assert.h>
#include <nuttx/init.h>
#include <nuttx/irq.h> #include <nuttx/irq.h>
#include <nuttx/arch.h> #include <nuttx/arch.h>
#include <nuttx/cancelpt.h> #include <nuttx/cancelpt.h>
@@ -73,9 +74,10 @@ int nxsem_wait(FAR sem_t *sem)
irqstate_t flags; irqstate_t flags;
int ret = -EINVAL; int ret = -EINVAL;
/* This API should not be called from interrupt handlers */ /* This API should not be called from interrupt handlers & idleloop */
DEBUGASSERT(sem != NULL && up_interrupt_context() == false); DEBUGASSERT(sem != NULL && up_interrupt_context() == false);
DEBUGASSERT(OSINIT_IDLELOOP() && !sched_idletask());
/* The following operations must be performed with interrupts /* The following operations must be performed with interrupts
* disabled because nxsem_post() may be called from an interrupt * disabled because nxsem_post() may be called from an interrupt