Skip to content

Add config flag to disable block count check on mount #1096

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sosthene-nitrokey
Copy link
Contributor

When the block count check was added to lfs_mount in #584 it was mentionned to have a mount flag to disable this check in the future.

Therefore, this PR:

  • Adds a new flags field to lfs_config
  • Defines a new lfs_fs_flags enum that contains these flags
  • Defines one flag LFS_CFG_DISABLE_BLOCK_COUNT_CHECK that disables the check inserted in #584

When the block count check was added to `lfs_mount` in [littlefs-project#584][]
it was mentionned to have a mount flag to disable this check in the future.

Therefore, this PR:

- Adds a new `flags` field to lfs_config
- Defines a new `lfs_fs_flags` enum that contains these flags
- Defines one flag `LFS_CFG_DISABLE_BLOCK_COUNT_CHECK` that disables
  the check inserted in [littlefs-project#584][]

[littlefs-project#584]: littlefs-project#584
@geky-bot
Copy link
Collaborator

Tests passed ✓, Code: 17132 B (+0.1%), Stack: 1448 B (+0.0%), Structs: 816 B (+0.5%)
Code Stack Structs Coverage
Default 17132 B (+0.1%) 1448 B (+0.0%) 816 B (+0.5%) Lines 2435/2597 lines (+0.0%)
Readonly 6246 B (+0.3%) 448 B (+0.0%) 816 B (+0.5%) Branches 1283/1614 branches (+0.1%)
Threadsafe 17980 B (+0.1%) 1448 B (+0.0%) 824 B (+0.5%) Benchmarks
Multiversion 17204 B (+0.1%) 1448 B (+0.0%) 820 B (+0.5%) Readed 29369693876 B (+0.0%)
Migrate 18796 B (+0.1%) 1752 B (+0.0%) 820 B (+0.5%) Proged 1482874766 B (+0.0%)
Error-asserts 17912 B (+0.1%) 1440 B (+0.0%) 816 B (+0.5%) Erased 1568888832 B (+0.0%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement needs minor version new functionality only allowed in minor versions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants