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

Outstanding patches for Rust STD support in ESP-IDF master & V4.3 branch (IDFGH-5970) #7658

Open
ivmarkov opened this issue Oct 7, 2021 · 4 comments
Labels
Status: Opened Issue is new

Comments

@ivmarkov
Copy link
Contributor

ivmarkov commented Oct 7, 2021

@igrr asked me for a summary of what needs to be merged so that we have Rust STD support without patching the ESP-IDF SDK.

Here it goes:

Master

The only remaining ones are:

V4.3

@ivmarkov
Copy link
Contributor Author

ivmarkov commented Oct 7, 2021

Oh, and we need a pthread rwlock implementation (can't find the relevant issue in GitHub), but I guess that one should be tracked separately.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Oct 7, 2021
@github-actions github-actions bot changed the title Outstanding patches for Rust STD support in ESP-IDF master & V4.3 branch Outstanding patches for Rust STD support in ESP-IDF master & V4.3 branch (IDFGH-5970) Oct 7, 2021
@0xjakob
Copy link
Contributor

0xjakob commented Oct 8, 2021

Oh, and we need a pthread rwlock implementation (can't find the relevant issue in GitHub), but I guess that one should be tracked separately.

#7411

@0xjakob
Copy link
Contributor

0xjakob commented Oct 8, 2021

@ivmarkov We currently have a PoC for rwlock using a condition variable. We haven't done any performance tests (runtime and mem usage) yet but so far it works in unit tests. Given that it's faster than a normal mutex, we could release this as a preliminary implementation until we have a better solution.

@ivmarkov
Copy link
Contributor Author

ivmarkov commented Oct 8, 2021

Thanks for the link. Even if it is not faster than a normal mutex, we might still want to use it in Rust STD as-is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Opened Issue is new
Projects
None yet
Development

No branches or pull requests

3 participants