From 9afcd342829d91dbd448c46a42861ce1796c4756 Mon Sep 17 00:00:00 2001
From: chao an <anchao@lixiang.com>
Date: Mon, 23 Sep 2024 08:51:29 +0800
Subject: [PATCH] Revert "pthread_mutex:add deadlock assert"

EDEADLK alreay checked in:
https://github.com/apache/nuttx/blob/master/sched/pthread/pthread_mutextimedlock.c#L112-L127
https://github.com/apache/nuttx/blob/master/sched/pthread/pthread_mutex.c#L195-L201

This reverts commit 112cc083d30047b160fbeda753a26b88362cdf70.

Signed-off-by: chao an <anchao@lixiang.com>
---
 sched/pthread/pthread_mutex.c | 41 +++--------------------------------
 1 file changed, 3 insertions(+), 38 deletions(-)

diff --git a/sched/pthread/pthread_mutex.c b/sched/pthread/pthread_mutex.c
index 833f4f4acf1d0..79223d88a46e0 100644
--- a/sched/pthread/pthread_mutex.c
+++ b/sched/pthread/pthread_mutex.c
@@ -71,41 +71,6 @@ static void pthread_mutex_add(FAR struct pthread_mutex_s *mutex)
   leave_critical_section(flags);
 }
 
-/****************************************************************************
- * Name: pthread_mutex_check
- *
- * Description:
- *   Verify that the mutex is not in the list of mutexes held by
- *   this pthread.
- *
- * Input Parameters:
- *  mutex - The mutex to be locked
- *
- * Returned Value:
- *   None
- *
- ****************************************************************************/
-
-#ifdef CONFIG_DEBUG_ASSERTIONS
-static void pthread_mutex_check(FAR struct pthread_mutex_s *mutex)
-{
-  FAR struct tcb_s *tcb = this_task();
-  irqstate_t flags = enter_critical_section();
-  FAR struct pthread_mutex_s *cur;
-
-  DEBUGASSERT(mutex != NULL);
-  for (cur = tcb->mhead; cur != NULL; cur = cur->flink)
-    {
-      /* The mutex should not be in the list of mutexes held by this task */
-
-      DEBUGASSERT(cur != mutex);
-    }
-
-  leave_critical_section(flags);
-}
-
-#endif
-
 /****************************************************************************
  * Name: pthread_mutex_remove
  *
@@ -180,6 +145,9 @@ int pthread_mutex_take(FAR struct pthread_mutex_s *mutex,
 {
   int ret = EINVAL;
 
+  /* Verify input parameters */
+
+  DEBUGASSERT(mutex != NULL);
   if (mutex != NULL)
     {
       /* Make sure that no unexpected context switches occur */
@@ -229,9 +197,6 @@ int pthread_mutex_take(FAR struct pthread_mutex_s *mutex,
 
               else if (!mutex_is_recursive(&mutex->mutex))
                 {
-#ifdef CONFIG_DEBUG_ASSERTIONS
-                  pthread_mutex_check(mutex);
-#endif
                   pthread_mutex_add(mutex);
                 }
             }