Skip to content

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

@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%)

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
@sosthene-nitrokey sosthene-nitrokey force-pushed the disable-block-count-check branch from f738038 to bfc30d2 Compare May 15, 2025 07:39
@geky-bot
Copy link
Collaborator

Tests passed ✓, Code: 17132 B (+0.2%), Stack: 1448 B (+0.0%), Structs: 816 B (+0.5%)
Code Stack Structs Coverage
Default 17132 B (+0.2%) 1448 B (+0.0%) 816 B (+0.5%) Lines 2437/2598 lines (+0.0%)
Readonly 6246 B (+0.3%) 448 B (+0.0%) 816 B (+0.5%) Branches 1288/1622 branches (+0.0%)
Threadsafe 17976 B (+0.1%) 1448 B (+0.0%) 824 B (+0.5%) Benchmarks
Multiversion 17204 B (+0.2%) 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 17948 B (+0.2%) 1440 B (+0.0%) 816 B (+0.5%) Erased 1568888832 B (+0.0%)

bouttier added a commit to bouttier/littlefs2 that referenced this pull request Aug 23, 2025
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