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: modified ltp,rwlock need init #1066

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

Conversation

extinguish
Copy link

@extinguish extinguish commented Aug 31, 2023

open_posix: the rwlock need to init, to add init procedure to the rwlock before use

Signed-off-by: anjiahao anjiahao@xiaomi.com

Signed-off-by: anjiahao <anjiahao@xiaomi.com>
@metan-ucw
Copy link
Member

Again the description should be rewritten, among other things it has to explain why the change is needed.

@extinguish extinguish changed the title modified ltp,rwlock need init open_posix: modified ltp,rwlock need init Sep 1, 2023
@extinguish
Copy link
Author

@metan-ucw please help to check if the descibe info meet requirements, if not, I will modify them immediately. thank you

@richiejp richiejp added the posix POSIX conformance test suite label Nov 24, 2023
@metan-ucw
Copy link
Member

Looking at the test closely the point of the test is to figure out what happens when we attempt to lock an uninitialized lock and the end result is not well defined, hence it's in the speculative directory. What exactly does happen on NuttX when this test is executed?

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.

4 participants