]> git.itanic.dy.fi Git - linux-stable/commit
nilfs2: fix data loss in mmap page write for hole blocks
authorRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Sun, 27 Mar 2011 13:50:49 +0000 (22:50 +0900)
committerAndi Kleen <ak@linux.intel.com>
Thu, 28 Apr 2011 15:20:51 +0000 (08:20 -0700)
commit61b4a6c0ab467bb30931018fc620e93c5602c96b
treef02676a0d6dc8d78f52e27296a3ef577879ead08
parent0b03d0985f309d5f65498220a01d7c2b4bb77661
nilfs2: fix data loss in mmap page write for hole blocks

commit 34094537943113467faee98fe67c8a3d3f9a0a8b upstream.

From the result of a function test of mmap, mmap write to shared pages
turned out to be broken for hole blocks.  It doesn't write out filled
blocks and the data will be lost after umount.  This is due to a bug
that the target file is not queued for log writer when filling hole
blocks.

Also, nilfs_page_mkwrite function exits normal code path even after
successfully filled hole blocks due to a change of block_page_mkwrite
function; just after nilfs was merged into the mainline,
block_page_mkwrite() started to return VM_FAULT_LOCKED instead of zero
by the patch "mm: close page_mkwrite races" (commit:
b827e496c893de0c).  The current nilfs_page_mkwrite() is not handling
this value properly.

This corrects nilfs_page_mkwrite() and will resolve the data loss
problem in mmap write.

[This should be applied to every kernel since 2.6.30 but a fix is
 needed for 2.6.37 and prior kernels]

Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Tested-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/nilfs2/file.c