Skip to content

No std mutex#6058

Open
Person-93 wants to merge 11 commits into
PyO3:mainfrom
Person-93:no_std_mutex
Open

No std mutex#6058
Person-93 wants to merge 11 commits into
PyO3:mainfrom
Person-93:no_std_mutex

Conversation

@Person-93
Copy link
Copy Markdown
Contributor

This PR is to prepare for no_std support.

I've added non_poison::Mutex and Once to the platform mod.

There was once place that relied on poisoning for correctness: err_state.rs, but that didn't really need a mutex because it was guarding a Copy type, I've replaced it with a Cell.

@Person-93 Person-93 force-pushed the no_std_mutex branch 2 times, most recently from b7b3dae to 7d53f79 Compare May 18, 2026 23:54
@Person-93 Person-93 marked this pull request as draft May 18, 2026 23:55
@Person-93 Person-93 force-pushed the no_std_mutex branch 2 times, most recently from 9bb808d to 465c958 Compare May 19, 2026 00:15
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 19, 2026

Merging this PR will improve performance by 30.65%

⚡ 1 improved benchmark
✅ 124 untouched benchmarks

Performance Changes

Benchmark BASE HEAD Efficiency
drop_many_objects_without_gil 48.2 µs 36.9 µs +30.65%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing Person-93:no_std_mutex (e2e9ad0) with main (6d0c1ee)

Open in CodSpeed

@Person-93 Person-93 marked this pull request as ready for review May 19, 2026 01:22
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.

1 participant