Skip to content

Add suppresion file for helgrind for disjointPool test #405

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

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

igchor
Copy link
Member

@igchor igchor commented Apr 3, 2024

Helgrind is complaining:
"pthread_mutex_lock with a pthread_rwlock_t* argument"

The call to pthread_mutex_lock is coming from c++ standard lib so the only thing we can do is to suppress it.

Description

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

Helgrind is complaining:
"pthread_mutex_lock with a pthread_rwlock_t* argument"

The call to pthread_mutex_lock is coming from c++ standard lib so
the only thing we can do is to suppress it.
@bratpiorka bratpiorka requested a review from ldorau April 4, 2024 06:20
@ldorau ldorau merged commit d10c392 into oneapi-src:main Apr 4, 2024
ldorau added a commit to ldorau/unified-memory-framework that referenced this pull request Apr 4, 2024
This is a continuation of oneapi-src#405.
This patch adds missing suppressions.

Ref: oneapi-src#405
Fixes: oneapi-src#338

Signed-off-by: Lukasz Dorau <[email protected]>
ldorau added a commit to ldorau/unified-memory-framework that referenced this pull request Apr 4, 2024
This is a continuation of oneapi-src#405.
This patch adds missing suppressions.

Ref: oneapi-src#405
Fixes: oneapi-src#338

Signed-off-by: Lukasz Dorau <[email protected]>
ldorau added a commit to ldorau/unified-memory-framework that referenced this pull request Apr 4, 2024
This is a continuation of oneapi-src#405.
This patch adds missing suppressions.

Ref: oneapi-src#405
Fixes: oneapi-src#338

Signed-off-by: Lukasz Dorau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants