-
Notifications
You must be signed in to change notification settings - Fork 2k
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
cpu/sam0_common: flashpage: disable interrupts while writing #15965
cpu/sam0_common: flashpage: disable interrupts while writing #15965
Conversation
What's the state here? Could someone take a look? |
Do we have a way to reproduce this ? I don't think the flashpage driver is faulty here, maybe the problem is related to highspeed bus. |
So far this only happened on SAM D5x/E5x but it's pretty reliable. |
@benpicco is this still happening? |
Yes, we've been including this in our downstream branch ever since. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested through deployment according to @benpicco
Always happy when I get to shorten this list 😃 |
bb9d5cf
to
a21acf0
Compare
Contribution description
This disabled interrupts while write operations on the internal flash are ongoing.
@firas-hamdi discovered that firmware updates over 802.15.4 with 1024 byte CoAP block size would sometimes (~10% of all updates) trigger a hard fault caused by an Imprecise data access (
IMPRECISERR
) on samd51.I failed to re-create this on same54-xpro over Ethernet, so I was wondering if the increased interrupt load due to 6lo fragmentation might have something to do with it.
Testing procedure
I'll try to come up with a setup that triggers this via GPIO interrupts.
Issues/PRs references