Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

open_posix_testsuite: fix pthread_cond_destroy test error. #1097

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qinyuntan
Copy link

According to the description of the pthread_cond_destroy function in glibc 2.32, it is advised to ensure that all threads blocked on the condition variable have been successfully awakened before calling this function, otherwise unpredictable situations may occur.

In the glibc 2.32 version, this testcase will hang. Therefore, it is necessary to ensure that all threads have been successfully awakened before calling pthread_cond_destroy to destroy the condition variable.

[ type description here; PLEASE REMOVE THIS LINE AND THE LINES BELOW BEFORE SUBMITTING THIS PULL REQUEST ]

According to the description of the pthread_cond_destroy function in glibc 2.32,
it is advised to ensure that all threads blocked on the condition variable have
been successfully awakened before calling this function, otherwise unpredictable
situations may occur.

In the glibc 2.32 version, this testcase will hang. Therefore, it is necessary
to ensure that all threads have been successfully awakened before calling
pthread_cond_destroy to destroy the condition variable.

Signed-off-by: tanqinyun.tqy <tanqinyun.tqy@alibaba-inc.com>
@qinyuntan
Copy link
Author

The logs below indicate a failure in the test.

./interfaces/pthread_cond_destroy/pthread_cond_destroy_2-1.run-test

[14:25:57]Test starting
[14:25:57]System abilities:
[14:25:57] TPS : 200809
[14:25:57] CS : 200809
[14:25:57] MON : 200809
[14:25:57] MF : 200809
[14:27:57]Test ../../../conformance/interfaces/pthread_cond_destroy/2-1.c FAILED: Test failed (hang)

@richiejp richiejp added the posix POSIX conformance test suite label Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
posix POSIX conformance test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants