Hi, On Sat, 2013-12-07 at 18:55 +0800, Zheng Liu wrote: > From: Zheng Liu <wenqing.lz@xxxxxxxxxx> > > Currently we check i_size in buffered read path after we know the page > is update. But it could return some zero-filled pages to the userspace > when we do some append dio writes. We could use the following code > snippet to reproduce this problem in a ext2/3/4 filesystem. > If the page is not uptodate, then neither (potentially) is i_size, since the underlying fs has not had a chance to get its own locks and update the inode size. I suspect that the correct fix would be to implement ->launder_page to avoid the race that you've identified here, if I've understood it correctly, Steve. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
