Commit fba5f44
committed
Bump default inline_max up to 1/4 block size
As noted by amgross, the current inline_max default (when littlefs
switches from inline files to CTZ skip-lists) does not match the
theoretical value in DESIGN.md.
The reason for this is 1/8 was chosen as a safer default during
development, due to concerns that 1/4 + mdirs splitting at 1/2 would
lead to poor distribution of large inline files in mdirs.
However, two things have happened since then:
1. Experiments have show "wasted" mdir space is less of a concern than
initially thought. Extra mdir space contributes to logging, delays
mdir compaction, and can overall lead to better performance.
2. inline_size was added as a configuration option, so if 1/4 is
problematic users can always override it.
So bumping this back up to 1/4 may make sense.1 parent 8e251dd commit fba5f44
1 file changed
+2
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4331 | 4331 | | |
4332 | 4332 | | |
4333 | 4333 | | |
4334 | | - | |
| 4334 | + | |
4335 | 4335 | | |
4336 | 4336 | | |
4337 | 4337 | | |
| |||
4342 | 4342 | | |
4343 | 4343 | | |
4344 | 4344 | | |
4345 | | - | |
| 4345 | + | |
4346 | 4346 | | |
4347 | 4347 | | |
4348 | 4348 | | |
| |||
0 commit comments