2020-02-26 22:49:48

by Jeff Moyer

[permalink] [raw]
Subject: Re: [PATCH V4 00/13] Enable per-file/per-directory DAX operations V4

Hi, Ira,

[email protected] writes:

> From: Ira Weiny <[email protected]>
>
> https://github.com/weiny2/linux-kernel/pull/new/dax-file-state-change-v4
>
> Changes from V3:
> https://lore.kernel.org/lkml/[email protected]/
>
> * Remove global locking... :-D
> * put back per inode locking and remove pre-mature optimizations
> * Fix issues with Directories having IS_DAX() set
> * Fix kernel crash issues reported by Jeff
> * Add some clean up patches
> * Consolidate diflags to iflags functions
> * Update/add documentation
> * Reorder/rename patches quite a bit

I left out patches 1 and 2, but applied the rest and tested. This
passes xfs tests in the following configurations:
1) MKFS_OPTIONS="-m reflink=0" MOUNT_OPTIONS="-o dax"
2) MKFS_OPTIONS="-m reflink=0"
but with the added configuration step of setting the dax attribute on
the mounted test directory.

I also tested to ensure that reflink fails when a file has the dax
attribute set. I've got more testing to do, but figured I'd at least
let you know I've been looking at it.

Thanks!
Jeff