Skip to content

DAOS-17674 vos: add missing ABT mutex lock protection #16501

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: release/2.6
Choose a base branch
from
Open

DAOS-17674 vos: add missing ABT mutex lock protection #16501

wants to merge 2 commits into from

Conversation

wangshilong
Copy link
Contributor

ABT_cond_wait() requires the caller to hold the mutex lock.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

@wangshilong
DAOS-17674 vos: add missing ABT mutex lock protection
3f37409 
ABT_cond_wait() requires the caller to hold the mutex lock.

Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@wangshilong wangshilong requested review from a team as code owners June 11, 2025 02:55
@wangshilong wangshilong requested review from jolivier23, NiuYawei and janekmi and removed request for jolivier23 June 11, 2025 02:56
NiuYawei
NiuYawei previously approved these changes Jun 11, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link

Ticket title is 'utils/config/examples/daos_server_mdonssd.yml crashes on dmg pool create?'
Status is 'Open'
Labels: 'dmg,md_on_ssd,request_for_2.6.4'
https://daosio.atlassian.net/browse/DAOS-17674

@daosbuild3
Copy link
Collaborator

Test stage Unit Test bdev with memcheck on EL 8.8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-16501/1/display/redirect

janekmi
janekmi previously approved these changes Jun 11, 2025
View reviewed changes
Copy link
Contributor

@janekmi janekmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please just fix the copyrights.

@wangshilong
Fix copyright
e5b773d 
Signed-off-by: Wang Shilong <shilong.wang@hpe.com>
@wangshilong wangshilong dismissed stale reviews from janekmi and NiuYawei via e5b773d June 11, 2025 13:37
@daosbuild3
Copy link
Collaborator

Test stage Build RPM on EL 8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-16501/2/execution/node/331/log

@daosbuild3
Copy link
Collaborator

Test stage Build RPM on EL 9 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-16501/2/execution/node/330/log

@daosbuild3
Copy link
Collaborator

Test stage Build RPM on Leap 15.5 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-16501/2/execution/node/325/log

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@janekmi janekmi janekmi left review comments

@NiuYawei NiuYawei NiuYawei left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@wangshilong @daosbuild3 @janekmi @NiuYawei