diff options
author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-05-31 14:26:07 +0300 |
---|---|---|
committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-06-03 18:12:31 +0300 |
commit | 837072377034d0a0b18b851d1ab95676b245cc0a (patch) | |
tree | 7020b369ac3373f292d59fe5fd470f73ecb69897 /fs/nfsctl.c | |
parent | 812eb258311f89bcd664a34a620f249d54a2cd83 (diff) | |
download | lwn-837072377034d0a0b18b851d1ab95676b245cc0a.tar.gz lwn-837072377034d0a0b18b851d1ab95676b245cc0a.zip |
UBIFS: fix clean znode counter corruption in error cases
UBIFS maintains per-filesystem and global clean znode counters
('c->clean_zn_cnt' and 'ubifs_clean_zn_cnt'). It is important to maintain
correct values there since the shrinker relies on 'ubifs_clean_zn_cnt'.
However, in case of failures during commit the counters were corrupted. E.g.,
if a failure happens in the middle of 'write_index()', then some nodes in the
commit list ('c->cnext') are marked as clean, and some are marked as dirty. And
the 'ubifs_destroy_tnc_subtree()' frees does not retrun correct count, and we
end up with non-zero 'c->clean_zn_cnt' when unmounting. This means that if we
have 2 file-sytem and one of them fails, and we unmount it,
'ubifs_clean_zn_cnt' stays incorrect and confuses the shrinker.
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Diffstat (limited to 'fs/nfsctl.c')
0 files changed, 0 insertions, 0 deletions