From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boaz Harrosh Subject: Re: [PATCH 5/8] xfs: add DAX file operations support Date: Tue, 24 Mar 2015 14:24:22 +0200 Message-ID: <55115776.4000006@plexistor.com> References: <1427194266-2885-1-git-send-email-david@fromorbit.com> <1427194266-2885-6-git-send-email-david@fromorbit.com> <551153C4.6000808@plexistor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, willy@linux.intel.com, jack@suse.cz To: Boaz Harrosh , Dave Chinner , xfs@oss.sgi.com Return-path: Received: from mail-wi0-f176.google.com ([209.85.212.176]:35163 "EHLO mail-wi0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751643AbbCXMY0 (ORCPT ); Tue, 24 Mar 2015 08:24:26 -0400 Received: by wibdy8 with SMTP id dy8so73396082wib.0 for ; Tue, 24 Mar 2015 05:24:25 -0700 (PDT) In-Reply-To: <551153C4.6000808@plexistor.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: 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 > <>