Github/littlefs
1
Fork 0
Code Actions Activity

Small cleanup of inlined compaction update conditions

Browse Source 
This deduplicates quite a bit of logic which is very satisfying.

It could be even better if the block field was located in the same place
for both sprouts and shrubs...
This commit is contained in:
Christopher Haster
2023-09-27 14:04:10 -05:00
parent a6357e8a5c
commit 69993da7e1
1 changed files with 74 additions and 81 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lfs.c
+20 -27
View File
@@ -5548,23 +5548,20 @@ static int lfsr_mdir_compact__(lfs_t *lfs, lfsr_mdir_t *mdir_,
opened; opened;
opened = opened->next) { opened = opened->next) {
lfsr_file_t *file = (lfsr_file_t*)opened; lfsr_file_t *file = (lfsr_file_t*)opened;
// no inlined data? we don't write a tag for these but we DO need to // belongs to our mdir?
// update them, they may have been clobbered in a failed commit if (lfsr_file_isunsynced(file)
if (lfsr_inlined_hasnull(&file->inlined) && lfsr_inlined_block(&file->inlined) == mdir->u.rbyd.block
&& lfsr_file_isunsynced(file)
&& file->inlined.u.data.u.disk.block == mdir->u.rbyd.block
&& (file->m.mdir.mid & lfsr_midrmask(lfs)) >= start_rid && (file->m.mdir.mid & lfsr_midrmask(lfs)) >= start_rid
&& (lfs_size_t)(file->m.mdir.mid & lfsr_midrmask(lfs)) && (lfs_size_t)(file->m.mdir.mid & lfsr_midrmask(lfs))
< (lfs_size_t)end_rid) { < (lfs_size_t)end_rid) {
file->inlined_.u.data = LFSR_DATA_DISK(mdir_->u.rbyd.block, 0, 0); // inlined null? we don't write a tag for these but we do need to
// update them, they may have been clobbered in a failed commit
if (lfsr_inlined_hasnull(&file->inlined)) {
file->inlined_.u.data = LFSR_DATA_DISK(
mdir_->u.rbyd.block, 0, 0);
// inlined data? // inlined data?
} else if (lfsr_inlined_hassprout(&file->inlined) } else if (lfsr_inlined_hassprout(&file->inlined)) {
&& lfsr_file_isunsynced(file)
&& file->inlined.u.data.u.disk.block == mdir->u.rbyd.block
&& (file->m.mdir.mid & lfsr_midrmask(lfs)) >= start_rid
&& (lfs_size_t)(file->m.mdir.mid & lfsr_midrmask(lfs))
< (lfs_size_t)end_rid) {
// write the data as a shrub tag // write the data as a shrub tag
err = lfsr_rbyd_appendcompactattr(lfs, &mdir_->u.rbyd, err = lfsr_rbyd_appendcompactattr(lfs, &mdir_->u.rbyd,
LFSR_TAG_SHRUB(INLINED), 0, file->inlined.u.data); LFSR_TAG_SHRUB(INLINED), 0, file->inlined.u.data);
@@ -5582,12 +5579,7 @@ static int lfsr_mdir_compact__(lfs_t *lfs, lfsr_mdir_t *mdir_,
lfsr_data_size(&file->inlined.u.data)); lfsr_data_size(&file->inlined.u.data));
// inlined tree? // inlined tree?
} else if (lfsr_inlined_hasshrub(&file->inlined) } else if (lfsr_inlined_hasshrub(&file->inlined)) {
&& lfsr_file_isunsynced(file)
&& file->inlined.u.rbyd.block == mdir->u.rbyd.block
&& (file->m.mdir.mid & lfsr_midrmask(lfs)) >= start_rid
&& (lfs_size_t)(file->m.mdir.mid & lfsr_midrmask(lfs))
< (lfs_size_t)end_rid) {
// save our current off/trunk/weight // save our current off/trunk/weight
lfs_size_t off = mdir_->u.rbyd.eoff; lfs_size_t off = mdir_->u.rbyd.eoff;
lfs_size_t trunk = mdir_->u.rbyd.trunk; lfs_size_t trunk = mdir_->u.rbyd.trunk;
@@ -5618,6 +5610,7 @@ static int lfsr_mdir_compact__(lfs_t *lfs, lfsr_mdir_t *mdir_,
mdir_->u.rbyd.weight = weight; mdir_->u.rbyd.weight = weight;
} }
} }
}
return 0; return 0;
} }
@@ -5682,18 +5675,16 @@ static lfs_ssize_t lfsr_mdir_estimate_(lfs_t *lfs, const lfsr_mdir_t *mdir,
opened; opened;
opened = opened->next) { opened = opened->next) {
lfsr_file_t *file = (lfsr_file_t*)opened; lfsr_file_t *file = (lfsr_file_t*)opened;
// inlined data? // belongs to our mdir?
if (lfsr_inlined_hassprout(&file->inlined) if (lfsr_file_isunsynced(file)
&& lfsr_file_isunsynced(file) && lfsr_inlined_block(&file->inlined) == mdir->u.rbyd.block
&& file->inlined.u.data.u.disk.block == mdir->u.rbyd.block
&& (file->m.mdir.mid & lfsr_midrmask(lfs)) == rid) { && (file->m.mdir.mid & lfsr_midrmask(lfs)) == rid) {
// inlined data?
if (lfsr_inlined_hassprout(&file->inlined)) {
dsize += lfsr_data_estimate(&file->inlined.u.data); dsize += lfsr_data_estimate(&file->inlined.u.data);
// inlined tree? // inlined tree?
} else if (lfsr_inlined_hasshrub(&file->inlined) } else if (lfsr_inlined_hasshrub(&file->inlined)) {
&& lfsr_file_isunsynced(file)
&& file->inlined.u.rbyd.block == mdir->u.rbyd.block
&& (file->m.mdir.mid & lfsr_midrmask(lfs)) == rid) {
lfs_ssize_t dsize_ = lfsr_rbyd_estimate(lfs, lfs_ssize_t dsize_ = lfsr_rbyd_estimate(lfs,
&file->inlined.u.rbyd, -1, -1, NULL); &file->inlined.u.rbyd, -1, -1, NULL);
if (dsize_ < 0) { if (dsize_ < 0) {
@@ -5701,10 +5692,12 @@ static lfs_ssize_t lfsr_mdir_estimate_(lfs_t *lfs, const lfsr_mdir_t *mdir,
} }
// make sure to include the actual tag cost // make sure to include the actual tag cost
dsize += lfsr_data_estimate(&LFSR_DATA_BUF(NULL, LFSR_TRUNK_DSIZE)) dsize += lfsr_data_estimate(
&LFSR_DATA_BUF(NULL, LFSR_TRUNK_DSIZE))
+ dsize_; + dsize_;
} }
} }
}
return dsize; return dsize;
} }