Skip to content

Remove API lock for memory management wrappers - #6775

Open
fortnern wants to merge 7 commits into
HDFGroup:developfrom
fortnern:threaded_chunks_4
Open

fortnern wants to merge 7 commits into
HDFGroup:developfrom
fortnern:threaded_chunks_4

Conversation

@fortnern

Copy link
Copy Markdown
Member

Remove API lock from H5allocae_memory(), H5resize_memory(), and H5free_memory(). This allows them to be called by plugins that otherwise don't enter the HDF5 API, and prevents deadlocks from occurring when called from external plugins during internal concurrency.

Comment thread release_docs/CHANGELOG.md
@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Review Checklist

This PR touches the following areas. Each needs a sign-off
from its listed owners before merging.

hyoklee
hyoklee previously approved these changes Oct 23, 2026
Comment thread release_docs/CHANGELOG.md
Comment thread release_docs/CHANGELOG.md Outdated
jhendersonHDF
jhendersonHDF previously approved these changes Oct 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🎉 All checklist items are signed off — @fortnern, this PR is ready to merge.

@github-actions github-actions Bot added the checklist-complete All review checklist areas are signed off label Oct 23, 2026
@fortnern
fortnern dismissed stale reviews from jhendersonHDF and hyoklee via 66c11ca October 23, 2026 20:15
@github-actions github-actions Bot removed the checklist-complete All review checklist areas are signed off label Oct 23, 2026
Comment thread release_docs/CHANGELOG.md Outdated

This branch has not been deployed

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

Labels

None yet

Projects

Status: To be triaged

Development

Successfully merging this pull request may close these issues.

3 participants