On 03/24/2015 02:08 PM, Boaz Harrosh wrote: > On 03/24/2015 12:51 PM, Dave Chinner wrote: <> > > Regarding splice_write: > It looks to me like you left the vector at iter_file_splice_write(). > If I understand correctly I think you need to call default_file_splice_write() > that uses memcpy, just as it was at the previous patches when you left > the vector NULL. So I think you need the same switch for write > as you do below for read. > I take this back It looks like iter_file_splice_write() is perfectly usable it relies on the source (Not us for write) being a page then kmaps and calls one of f_op->write(file,..) or new_sync_write(file, ...) so it looks like it all should work. So the only bad thing is the comment it looks like splice should just work also for dax. Will be testing exact same thing for ext4 and report Thanks Boaz > I'm doing the exact same code for ext4/2 right now and testing. > (Thanks for the loop pointer). > > Otherwise I have stared at this very carefully and it looks good > Did not test yet. > > Thanks > Boaz > <> -- 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
