On Wed, Jun 22, 2016 at 05:27:08PM +0200, Christoph Hellwig wrote: > The last patch is what started the series: XFS currently uses the > direct I/O locking strategy for DAX because DAX was overloaded onto > the direct I/O path. For XFS this means that we only take a shared > inode lock instead of the normal exclusive one for writes IFF they > are properly aligned. While this is fine for O_DIRECT which requires > explicit opt-in from the application it's not fine for DAX where we'll > suddenly lose expected and required synchronization of the file system > happens to use DAX undeneath. Except we did that *intentionally* - by definition there is no cache to bypass with DAX and so all IO is "direct". That, combined with the fact that all Linux filesystems except XFS break the POSIX exclusive writer rule you are quoting to begin with, it seemed pointless to enforce it for DAX.... So, before taking any patches to change that behaviour in XFS, a wider discussion about the policy needs to be had. I don't think we should care about POSIX here - if you have an application that needs this serialisation, turn off DAX. That's why I made it a per-inode inheritable flag and why the mount option will go away over time. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- 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
