Skip to content

Allow non-pollable woken futures to make progress#71

Merged
pchickey merged 5 commits intomainfrom
pch/fix_69
Jul 3, 2025
Merged

Allow non-pollable woken futures to make progress#71
pchickey merged 5 commits intomainfrom
pch/fix_69

Conversation

@pchickey
Copy link
Contributor

Fixes #69

Alternative to #70 that tracks the state of the root task's waker, and keeps polling if that waker is awake.

pchickey and others added 2 commits June 23, 2025 11:29
This test demonstrates behavior for a future which does not pend on a
pollable, but does wake the root task when pending. As long as the
root task has had wake() called, the runtime should keep polling it.

This bug was reported in #69
and this test was derived from one written by @SilverMira in
https://github.com/SilverMira/wstd/tree/allow_independent_futures

Co-Authored-By: SilverMira <66930495+SilverMira@users.noreply.github.com>
@pchickey pchickey requested a review from yoshuawuyts June 23, 2025 18:41
@pchickey pchickey merged commit 90bb588 into main Jul 3, 2025
4 checks passed
@pchickey pchickey deleted the pch/fix_69 branch July 3, 2025 17:14
@pchickey pchickey mentioned this pull request Sep 4, 2025
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.

Futures that don't depend on Pollable while returning Poll::Pending crashes the runtime

2 participants