gh-121621: Move asyncio freelist to thread state - #121915
Fidget-Spinner wants to merge 13 commits into
Conversation
There was a problem hiding this comment.
Does anyone with better freelist internals knowledge know why this happens:
When I remove this function's body, the refleak tests pass. Otherwise they currently fail.
There was a problem hiding this comment.
As mentioned by Sam we should try to find a way to avoid this.
There was a problem hiding this comment.
If we go with #121934, then that will make the asyncio freelist implementation a lot easier and will avoid a lot of these warts.
Maybe, in the meantime, we should just disable the freelist in the free-threaded build? That would be a much smaller change and wouldn't introduce duplicate types.
There was a problem hiding this comment.
We could backport the disabling of freelist in the free-threaded build to 3.13 to fix the race conditions.
2 participants
Uh oh!
There was an error while loading. Please reload this page.