On Tue 27-09-16 19:13:50, Christoph Hellwig wrote: > On Tue, Sep 27, 2016 at 07:17:07PM +0200, Jan Kara wrote: > > OK, the changelog is stale but I actually took care to integrate this with > > your iomap patches and for the new invalidation code in iomap_dax_actor() > > to work we need this additional information... > > It's not just the changelogs (which will need updates on more than this > patch), but also the content. We're not using get_blocks for DAX > anymore, so this patch should not be needed anymore. Ah, you are right. After reading the code again this patch is not needed anymore because xfs_file_iomap_begin() returns IOMAP_F_NEW whenever we allocated block regardless whether it was zeroed-out or not. But if I get it right, all the IS_DAX checks in __xfs_get_blocks() can be dropped now, cannot they? Honza -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR -- 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
