libs/rt-thread-official
libs/rt-thread-official
4
0
Fork 0
mirror of https://github.com/RT-Thread/rt-thread.git synced 2025-02-28 02:47:06 +08:00
Code Issues Releases Wiki Activity

optimize code

Browse Source
This commit is contained in:
Henson 2022-01-26 21:39:06 +08:00
parent 9b44535740
commit 500d26c257
5 changed files with 26 additions and 59 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
components/drivers/ipc/completion.c
View File

@ -58,10 +58,7 @@ rt_err_t rt_completion_wait(struct rt_completion *completion,
RT_ASSERT(completion != RT_NULL); RT_ASSERT(completion != RT_NULL);
/* current context checking */ /* current context checking */
if (timeout != 0) RT_DEBUG_SCHEDULER_AVAILABLE(timeout != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
result = RT_EOK; result = RT_EOK;
thread = rt_thread_self(); thread = rt_thread_self();
@ -88,9 +85,6 @@ rt_err_t rt_completion_wait(struct rt_completion *completion,
rt_list_insert_before(&(completion->suspended_list), rt_list_insert_before(&(completion->suspended_list),
&(thread->tlist)); &(thread->tlist));
/* current context checking */
RT_DEBUG_NOT_IN_INTERRUPT;
/* start timer */ /* start timer */
if (timeout > 0) if (timeout > 0)
{ {

10
components/drivers/ipc/dataqueue.c
View File

@ -99,10 +99,7 @@ rt_err_t rt_data_queue_push(struct rt_data_queue *queue,
RT_ASSERT(queue->magic == DATAQUEUE_MAGIC); RT_ASSERT(queue->magic == DATAQUEUE_MAGIC);
/* current context checking */ /* current context checking */
if (timeout != 0) RT_DEBUG_SCHEDULER_AVAILABLE(timeout != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
result = RT_EOK; result = RT_EOK;
thread = rt_thread_self(); thread = rt_thread_self();
@ -221,10 +218,7 @@ rt_err_t rt_data_queue_pop(struct rt_data_queue *queue,
RT_ASSERT(size != RT_NULL); RT_ASSERT(size != RT_NULL);
/* current context checking */ /* current context checking */
if (timeout != 0) RT_DEBUG_SCHEDULER_AVAILABLE(timeout != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
result = RT_EOK; result = RT_EOK;
thread = rt_thread_self(); thread = rt_thread_self();

2
components/drivers/ipc/waitqueue.c
View File

@ -129,7 +129,7 @@ int rt_wqueue_wait(rt_wqueue_t *queue, int condition, int msec)
rt_base_t level; rt_base_t level;
/* current context checking */ /* current context checking */
RT_DEBUG_SCHEDULER_AVAILABLE; RT_DEBUG_SCHEDULER_AVAILABLE(RT_TRUE);
tick = rt_tick_from_millisecond(msec); tick = rt_tick_from_millisecond(msec);

9
include/rtdebug.h
View File

@ -116,9 +116,11 @@ while (0)
* 2) not in interrupt context. * 2) not in interrupt context.
* 3) scheduler is not locked. * 3) scheduler is not locked.
*/ */
#define RT_DEBUG_SCHEDULER_AVAILABLE \ #define RT_DEBUG_SCHEDULER_AVAILABLE(need_check) \
do \ do \
{ \ { \
if (need_check) \
{ \
rt_base_t level; \ rt_base_t level; \
level = rt_hw_interrupt_disable(); \ level = rt_hw_interrupt_disable(); \
if (rt_critical_level() != 0) \ if (rt_critical_level() != 0) \
@ -129,12 +131,13 @@ do \
} \ } \
RT_DEBUG_IN_THREAD_CONTEXT; \ RT_DEBUG_IN_THREAD_CONTEXT; \
rt_hw_interrupt_enable(level); \ rt_hw_interrupt_enable(level); \
} \
} \ } \
while (0) while (0)
#else #else
#define RT_DEBUG_NOT_IN_INTERRUPT #define RT_DEBUG_NOT_IN_INTERRUPT
#define RT_DEBUG_IN_THREAD_CONTEXT #define RT_DEBUG_IN_THREAD_CONTEXT
#define RT_DEBUG_SCHEDULER_AVAILABLE #define RT_DEBUG_SCHEDULER_AVAILABLE(need_check)
#endif #endif
#else /* RT_DEBUG */ #else /* RT_DEBUG */
@ -143,7 +146,7 @@ while (0)
#define RT_DEBUG_LOG(type, message) #define RT_DEBUG_LOG(type, message)
#define RT_DEBUG_NOT_IN_INTERRUPT #define RT_DEBUG_NOT_IN_INTERRUPT
#define RT_DEBUG_IN_THREAD_CONTEXT #define RT_DEBUG_IN_THREAD_CONTEXT
#define RT_DEBUG_SCHEDULER_AVAILABLE #define RT_DEBUG_SCHEDULER_AVAILABLE(need_check)
#endif /* RT_DEBUG */ #endif /* RT_DEBUG */

40
src/ipc.c
View File

@ -484,11 +484,9 @@ rt_err_t rt_sem_take(rt_sem_t sem, rt_int32_t time)
/* parameter check */ /* parameter check */
RT_ASSERT(sem != RT_NULL); RT_ASSERT(sem != RT_NULL);
RT_ASSERT(rt_object_get_type(&sem->parent.parent) == RT_Object_Class_Semaphore); RT_ASSERT(rt_object_get_type(&sem->parent.parent) == RT_Object_Class_Semaphore);
/* current context checking */ /* current context checking */
if (time != 0) RT_DEBUG_SCHEDULER_AVAILABLE(time != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
RT_OBJECT_HOOK_CALL(rt_object_trytake_hook, (&(sem->parent.parent))); RT_OBJECT_HOOK_CALL(rt_object_trytake_hook, (&(sem->parent.parent)));
@ -914,13 +912,8 @@ rt_err_t rt_mutex_take(rt_mutex_t mutex, rt_int32_t time)
struct rt_thread *thread; struct rt_thread *thread;
/* this function must not be used in interrupt even if time = 0 */ /* this function must not be used in interrupt even if time = 0 */
RT_DEBUG_IN_THREAD_CONTEXT;
/* current context checking */ /* current context checking */
if (time != 0) RT_DEBUG_SCHEDULER_AVAILABLE(RT_TRUE);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
/* parameter check */ /* parameter check */
RT_ASSERT(mutex != RT_NULL); RT_ASSERT(mutex != RT_NULL);
@ -1579,12 +1572,7 @@ rt_err_t rt_event_recv(rt_event_t event,
RT_ASSERT(rt_object_get_type(&event->parent.parent) == RT_Object_Class_Event); RT_ASSERT(rt_object_get_type(&event->parent.parent) == RT_Object_Class_Event);
/* current context checking */ /* current context checking */
RT_DEBUG_IN_THREAD_CONTEXT; RT_DEBUG_SCHEDULER_AVAILABLE(RT_TRUE);
if (timeout != 0)
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
if (set == 0) if (set == 0)
return -RT_ERROR; return -RT_ERROR;
@ -2008,10 +1996,7 @@ rt_err_t rt_mb_send_wait(rt_mailbox_t mb,
RT_ASSERT(rt_object_get_type(&mb->parent.parent) == RT_Object_Class_MailBox); RT_ASSERT(rt_object_get_type(&mb->parent.parent) == RT_Object_Class_MailBox);
/* current context checking */ /* current context checking */
if (timeout != 0) RT_DEBUG_SCHEDULER_AVAILABLE(timeout != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
/* initialize delta tick */ /* initialize delta tick */
tick_delta = 0; tick_delta = 0;
@ -2256,10 +2241,7 @@ rt_err_t rt_mb_recv(rt_mailbox_t mb, rt_ubase_t *value, rt_int32_t timeout)
RT_ASSERT(rt_object_get_type(&mb->parent.parent) == RT_Object_Class_MailBox); RT_ASSERT(rt_object_get_type(&mb->parent.parent) == RT_Object_Class_MailBox);
/* current context checking */ /* current context checking */
if (timeout != 0) RT_DEBUG_SCHEDULER_AVAILABLE(timeout != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
/* initialize delta tick */ /* initialize delta tick */
tick_delta = 0; tick_delta = 0;
@ -2769,10 +2751,7 @@ rt_err_t rt_mq_send_wait(rt_mq_t mq,
RT_ASSERT(size != 0); RT_ASSERT(size != 0);
/* current context checking */ /* current context checking */
if (timeout != 0) RT_DEBUG_SCHEDULER_AVAILABLE(timeout != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
/* greater than one message size */ /* greater than one message size */
if (size > mq->msg_size) if (size > mq->msg_size)
@ -3084,10 +3063,7 @@ rt_err_t rt_mq_recv(rt_mq_t mq,
RT_ASSERT(size != 0); RT_ASSERT(size != 0);
/* current context checking */ /* current context checking */
if (timeout != 0) RT_DEBUG_SCHEDULER_AVAILABLE(timeout != 0);
{
RT_DEBUG_SCHEDULER_AVAILABLE;
}
/* initialize delta tick */ /* initialize delta tick */
tick_delta = 0; tick_delta = 0;