Skip to content

gh-115491: Keep some fields valid across allocations (free-threading)#115573

Merged
colesbury merged 2 commits into
python:mainfrom
colesbury:gh-115491-debug-fill-mimalloc
Feb 20, 2024
Merged

gh-115491: Keep some fields valid across allocations (free-threading)#115573
colesbury merged 2 commits into
python:mainfrom
colesbury:gh-115491-debug-fill-mimalloc

Conversation

@colesbury
Copy link
Copy Markdown
Contributor

@colesbury colesbury commented Feb 16, 2024

This avoids mimalloc filling the memory occupied by ob_tid, ob_ref_local, and ob_ref_shared with debug bytes (e.g., 0xDD) the free-threaded build.

…ading)

This avoids filling the memory occupied by ob_tid, ob_ref_local, and
ob_ref_shared with debug bytes (e.g., 0xDD) in mimalloc in the
free-threaded build.
Copy link
Copy Markdown
Contributor

@DinoV DinoV left a comment

Choose a reason for hiding this comment

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

Can you fix the warnings on Windows? Otherwise LGTM

@colesbury colesbury merged commit cc82e33 into python:main Feb 20, 2024
@colesbury colesbury deleted the gh-115491-debug-fill-mimalloc branch February 20, 2024 15:37
woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
…ading) (python#115573)

This avoids filling the memory occupied by ob_tid, ob_ref_local, and
ob_ref_shared with debug bytes (e.g., 0xDD) in mimalloc in the
free-threaded build.
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
…ading) (python#115573)

This avoids filling the memory occupied by ob_tid, ob_ref_local, and
ob_ref_shared with debug bytes (e.g., 0xDD) in mimalloc in the
free-threaded build.
LukasWoodtli pushed a commit to LukasWoodtli/cpython that referenced this pull request Jan 22, 2025
…ading) (python#115573)

This avoids filling the memory occupied by ob_tid, ob_ref_local, and
ob_ref_shared with debug bytes (e.g., 0xDD) in mimalloc in the
free-threaded build.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants