Revert "os init_state: add new state OSINIT_IDLELOOP"

This reverts commit 051bb32010.
This commit is contained in:
Masayuki Ishikawa
2022-02-22 17:33:14 +09:00
committed by Petro Karashchenko
parent db260593f1
commit fccdcf7011
4 changed files with 3 additions and 13 deletions
+1 -3
View File
@@ -41,7 +41,6 @@
#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)
/**************************************************************************** /****************************************************************************
@@ -66,9 +65,8 @@ 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 */
}; };
/**************************************************************************** /****************************************************************************
-4
View File
@@ -776,10 +776,6 @@ 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();
+1 -3
View File
@@ -29,7 +29,6 @@
#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>
@@ -70,10 +69,9 @@ 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 & idleloop */ /* This API should not be called from interrupt handlers */
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)
{ {
+1 -3
View File
@@ -28,7 +28,6 @@
#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>
@@ -74,10 +73,9 @@ 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 & idleloop */ /* This API should not be called from interrupt handlers */
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