> +static bool > +xfs_is_dax_state_changing( > + unsigned int xflags, > + struct xfs_inode *ip) And I have no fricking idea what 'is_dax_state_changing' is supposed to mean for the caller. This needs a better name and/or a comment explaining the function. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>