Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make RTIC 2 not require nightly by using main's stack as an allocator #888

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

korken89
Copy link
Collaborator

@korken89 korken89 commented Jan 29, 2024

With this PR RTIC v2 will work on stable from 1.75 and forward.

This is done by instead of using Type Alias Impl Trait to do static allocations of async executors (causing us to require nightly), we are now using the stack of main (before user init is called) to allocate the async executors there. Basically doing a manual alloca.

From testing this seems to not have any performance impact compared to TAIT other than startup taking a few extra cycles to move the pointer to the global storage.

@korken89 korken89 marked this pull request as draft January 29, 2024 20:51
@korken89 korken89 added skip-changelog Sometimes changes are not significant enough for a changelog entry and removed skip-changelog Sometimes changes are not significant enough for a changelog entry labels Jan 29, 2024
@korken89 korken89 force-pushed the feature/stable-support branch from 519d337 to 22ecaf6 Compare January 30, 2024 06:56
@korken89 korken89 marked this pull request as ready for review January 30, 2024 06:56
@korken89 korken89 force-pushed the feature/stable-support branch 2 times, most recently from 34c8efe to 808dc14 Compare January 30, 2024 07:01
Copy link
Contributor

@glaeqen glaeqen left a comment

Choose a reason for hiding this comment

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

Stable yoooo ♥️

@korken89 korken89 force-pushed the feature/stable-support branch from 0d2d5ce to d78da46 Compare January 31, 2024 19:05
AfoHT
AfoHT previously approved these changes Jan 31, 2024
Copy link
Contributor

@AfoHT AfoHT left a comment

Choose a reason for hiding this comment

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

Exciting! One tiny thing

@korken89 korken89 force-pushed the feature/stable-support branch 2 times, most recently from 457ec3e to dcea309 Compare January 31, 2024 20:38
@jordens jordens mentioned this pull request Feb 12, 2024
1 task
@korken89 korken89 force-pushed the feature/stable-support branch from bbac2ee to de41e9a Compare February 27, 2024 11:00
@korken89 korken89 force-pushed the feature/stable-support branch from de41e9a to 4152983 Compare February 27, 2024 11:04
@korken89 korken89 merged commit 2798500 into master Feb 27, 2024
50 checks passed
@korken89 korken89 deleted the feature/stable-support branch February 27, 2024 11: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.

4 participants