diff options
author | Sage Weil <sage@newdream.net> | 2011-08-24 23:07:01 +0200 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2011-10-26 01:10:16 +0200 |
commit | 83eaea22bdfc9e1cec88f81be5b64f30f6c37e8b (patch) | |
tree | f63f415ce4241c12639ef78f4920ed6b99915760 /fs/file.c | |
parent | ceph: replace leading spaces with tabs (diff) | |
download | linux-83eaea22bdfc9e1cec88f81be5b64f30f6c37e8b.tar.xz linux-83eaea22bdfc9e1cec88f81be5b64f30f6c37e8b.zip |
Revert "ceph: don't truncate dirty pages in invalidate work thread"
This reverts commit c9af9fb68e01eb2c2165e1bc45cfeeed510c64e6.
We need to block and truncate all pages in order to reliably invalidate
them. Otherwise, we could:
- have some uptodate pages in the cache
- queue an invalidate
- write(2) locks some pages
- invalidate_work skips them
- write(2) only overwrites part of the page
- page now dirty and uptodate
-> partial leakage of invalidated data
It's not entirely clear why we started skipping locked pages in the first
place. I just ran this through fsx and didn't see any problems.
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/file.c')
0 files changed, 0 insertions, 0 deletions