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,
