Message ID | 1443634014-3026-6-git-send-email-Anna.Schumaker@Netapp.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Needs to be folded. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
I would have folded this and patch 4 earlier if I had written patch 1, but I didn't feel comfortable modifying Zach's work too much. I can make that change if it's not really a problem. Anna On 10/11/2015 10:22 AM, Christoph Hellwig wrote: > Needs to be folded. > -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Oct 14, 2015 at 01:37:13PM -0400, Anna Schumaker wrote: > I would have folded this and patch 4 earlier if I had written patch 1, > but I didn't feel comfortable modifying Zach's work too much. I can > make that change if it's not really a problem. Folding the changes is perfectly fine, just make it clear you changed it, e.g. Signed-off-by: Original Author <original@auhor.info> [anna: fixed foo & bar, rewrote changelog] Signed-off-by: Anna Schumaker <Anna.Schumaker@netapp.com> -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 10/14/2015 02:25 PM, Christoph Hellwig wrote: > On Wed, Oct 14, 2015 at 01:37:13PM -0400, Anna Schumaker wrote: >> I would have folded this and patch 4 earlier if I had written patch 1, >> but I didn't feel comfortable modifying Zach's work too much. I can >> make that change if it's not really a problem. > > Folding the changes is perfectly fine, just make it clear you changed > it, e.g. > > Signed-off-by: Original Author <original@auhor.info> > [anna: fixed foo & bar, rewrote changelog] > Signed-off-by: Anna Schumaker <Anna.Schumaker@netapp.com> > Okay, I'll do that. Thanks! Anna -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/fs/read_write.c b/fs/read_write.c index f3d6c48..8e7cb33 100644 --- a/fs/read_write.c +++ b/fs/read_write.c @@ -1371,10 +1371,6 @@ ssize_t vfs_copy_file_range(struct file *file_in, loff_t pos_in, file_in->f_path.mnt != file_out->f_path.mnt) return -EXDEV; - /* forbid ranges in the same file */ - if (inode_in == inode_out) - return -EINVAL; - if (len == 0) return 0;