Skip to content

forbid sending interrupt tokens across interrupts #27

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

Merged
merged 1 commit into from
May 8, 2017
Merged

Conversation

japaric
Copy link
Member

@japaric japaric commented May 8, 2017

which would break the ctxt::Local abstraction by making Mutex Sync only if
the protected data is Send. See the CHANGELOG for details. To fully fix the
memory unsafety, svd2rust needs to be updated to mark interrupt tokens as
!Send

cc @whitequark

which would break the `ctxt::Local` abstraction by making `Mutex` `Sync` only if
the protected data is `Send`. See the CHANGELOG for details. To fully fix the
memory unsafety, svd2rust needs to be updated to mark interrupt tokens as
`!Send`
@japaric
Copy link
Member Author

japaric commented May 8, 2017

svd2rust needs to be updated to mark interrupt tokens as !Send

rust-embedded/svd2rust#86

@japaric
Copy link
Member Author

japaric commented May 8, 2017

@homunkulus r+

@homunkulus
Copy link
Contributor

📌 Commit 42968fc has been approved by japaric

@homunkulus
Copy link
Contributor

⌛ Testing commit 42968fc with merge 42968fc...

@homunkulus
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: japaric
Pushing 42968fc to master...

@homunkulus homunkulus merged commit 42968fc into master May 8, 2017
@japaric japaric deleted the send branch May 8, 2017 18:15
adamgreig pushed a commit that referenced this pull request Jan 12, 2022
define handlers as weak aliases of DEFAULT_HANDLER
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.

2 participants