Re: [PATCH v8 1/1] f2fs: dax: implement direct access
From: Jaegeuk Kim
Date: Tue Jul 25 2017 - 22:16:28 EST
Hi Dan,
On 07/25, Dan Williams wrote:
> [ adding linux-nvdimm ]
>
> On Thu, Jul 20, 2017 at 5:10 AM, sunqiuyang <sunqiuyang@xxxxxxxxxx> wrote:
> > From: Qiuyang Sun <sunqiuyang@xxxxxxxxxx>
> >
> > This patch implements Direct Access (DAX) in F2FS, including:
> > - a mount option to choose whether to enable DAX or not
>
> We're in the process of walking back and potentially deprecating the
> use of the dax mount option for xfs and ext4 since dax can have
> negative performance implications if page cache memory happens to be
> faster than pmem. It should be limited to applications that
> specifically want the semantic, not globally enabled for the entire
> mount. xfs has went ahead and added the XFS_DIFLAG2_DAX indoe flag for
> per-inode enabling of dax.
Thank you so much for pointing this out. So, is there a plan to define a
generic inode flag to enable dax via inode_set_flag? Or, does each filesystem
need to handle it individually likewise xfs?
>
> I'm wondering if any new filesystem that adds dax support at this
> point should do so with inode flags and not a mount option?
Anyway, in such the case, I have to postpone merging this patch for a while.
Thanks,