On Fri, Mar 22, 2013 at 11:18:24AM -0700, Kent Overstreet wrote: > vfs_writev() updates the offset argument - but the code then passes the > offset to vfs_fsync_range(). Since offset now points to the offset after > what was just written, this is probably not what was intended Agreed, the original code does look fishy and this fix right to me. Reviewed-by: Zach Brown <zab@xxxxxxxxxx> - z -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html