On Mon, Jan 25, 2010 at 05:15:45PM -0500, Trond Myklebust wrote: > int nfs_wb_nocommit(struct inode *inode) > { > - return nfs_write_mapping(inode->i_mapping, FLUSH_NOCOMMIT); > + return filemap_write_and_wait(inode->i_mapping); Any point in keeping this as a wrapper for a single well-documented caller? Also taking i_mutex around it seems a bit questionable these days given that filemap_write_and_wait avoids lifelocks with writing applications okay and we use it without i_mutex all over the place. -- 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