On Tue, Jul 14, 2009 at 05:29:26PM +0800, Wu Fengguang wrote: >Here set_page_dirty() can be moved into the page lock. > >CC: Hugh Dickins <hugh@xxxxxxxxxxx> >Signed-off-by: Wu Fengguang <fengguang.wu@xxxxxxxxx> Reviewed-by: WANG Cong <xiyou.wangcong@xxxxxxxxx> >--- > mm/shmem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >--- linux.orig/mm/shmem.c >+++ linux/mm/shmem.c >@@ -1630,8 +1630,8 @@ shmem_write_end(struct file *file, struc > if (pos + copied > inode->i_size) > i_size_write(inode, pos + copied); > >- unlock_page(page); > set_page_dirty(page); >+ unlock_page(page); > page_cache_release(page); > > return copied; >-- >To unsubscribe from this list: send the line "unsubscribe linux-kernel" in >the body of a message to majordomo@xxxxxxxxxxxxxxx >More majordomo info at http://vger.kernel.org/majordomo-info.html >Please read the FAQ at http://www.tux.org/lkml/ -- Live like a child, think like the god. -- 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