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

[Gen 3] LittleFS truncate fix (https://github.com/littlefs-project/littlefs/issues/268) #2385

Merged
merged 3 commits into from
Dec 23, 2021

Conversation

avtolstoy
Copy link
Member

Problem

See littlefs-project/littlefs#268

Solution

  1. Use patch from the above PR.
  2. Add stress test to wiring/filesystem test suite. It fails without the patch on checking the integrity of the data after a truncate, it works fine with the patch.
  3. (Minor and unrelated) Use .balign instead of .align in inline assembly, as .align X on ARM platforms semantically means 2^X alignment (and on other platforms it's just X alignment). .balign always means X alignment.

Steps to Test

  • wiring/filesystem built out of this branch with 3.2.0-rc.1, wiring/filesystem with system-part built out of this branch

Example App

N/A

References

  • [SC93531]

Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

@avtolstoy avtolstoy merged commit 3cb05eb into develop Dec 23, 2021
@avtolstoy avtolstoy deleted the fix/littlefs-truncate branch December 23, 2021 19:04
@keeramis keeramis mentioned this pull request Jan 13, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants