sergiotarxz/nuttx-apps
sergiotarxz/nuttx-apps
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Revert "testing/ostest/cancel: joining a detached/canceled thread should return EINVAL, not ESRCH"

Browse Source 
This reverts commit 69e497f68127cad345fd13786c5342822c58173f.

In order to ensure the detached thread obtain the correct return
value from pthread_join()/pthread_cancel(), the detached thread
will create joininfo to save the detached status after thread
destroyed.  If there are too many of detached threads in the
process group, the joininfo will consume too much memory.
This is not friendly to embedded MCU devices.
This commit keep the semantics as #11898 was introduced,
will no longer save joininfo for detached threads to avoid wasting memory.

Refer PR:
https://github.com/apache/nuttx/pull/12106

Signed-off-by: chao an <anchao@lixiang.com>
This commit is contained in:
chao an 2024-04-09 12:24:26 +08:00 committed by Xiang Xiao
parent 92d922a108
commit 78a8dd27c9
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
testing/ostest/cancel.c
View File

@ -607,7 +607,7 @@ void cancel_test(void)
printf("cancel_test: ERROR pthread_join succeeded\n");
ASSERT(false);
}
else if (status != EINVAL)
else if (status != ESRCH)
{
printf("cancel_test:"
" ERROR pthread_join failed but with wrong status=%d\n",
@ -616,7 +616,7 @@ void cancel_test(void)
}
else
{
printf("cancel_test: PASS pthread_join failed with status=EINVAL\n");
printf("cancel_test: PASS pthread_join failed with status=ESRCH\n");
}
/* Test 4: Non-cancelable threads *****************************************/