diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-09-15 16:46:08 -0700 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-09-23 11:10:16 -0700 |
| commit | 441ac5cb323a47b0a665f77f7cd6f76aacbdf21c (patch) | |
| tree | b8aca7fbc39d37a06123c491438ad5b82a241af0 /init | |
| parent | 88bd02c9472a166b706284a34a84f1243322d782 (diff) | |
| download | op-kernel-dev-441ac5cb323a47b0a665f77f7cd6f76aacbdf21c.zip op-kernel-dev-441ac5cb323a47b0a665f77f7cd6f76aacbdf21c.tar.gz | |
f2fs: fix roll-forward missing scenarios
We can summarize the roll forward recovery scenarios as follows.
[Term] F: fsync_mark, D: dentry_mark
1. inode(x) | CP | inode(x) | dnode(F)
-> Update the latest inode(x).
2. inode(x) | CP | inode(F) | dnode(F)
-> No problem.
3. inode(x) | CP | dnode(F) | inode(x)
-> Recover to the latest dnode(F), and drop the last inode(x)
4. inode(x) | CP | dnode(F) | inode(F)
-> No problem.
5. CP | inode(x) | dnode(F)
-> The inode(DF) was missing. Should drop this dnode(F).
6. CP | inode(DF) | dnode(F)
-> No problem.
7. CP | dnode(F) | inode(DF)
-> If f2fs_iget fails, then goto next to find inode(DF).
8. CP | dnode(F) | inode(x)
-> If f2fs_iget fails, then goto next to find inode(DF).
But it will fail due to no inode(DF).
So, this patch adds some missing points such as #1, #5, #7, and #8.
Signed-off-by: Huang Ying <ying.huang@intel.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'init')
0 files changed, 0 insertions, 0 deletions
