From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Theodore Y. Ts'o" Subject: Re: [PATCH v2 1/3] ext4, dax: add ext4_bmap to ext4_dax_aops Date: Sat, 15 Sep 2018 21:32:31 -0400 Message-ID: <20180916013231.GD8652@thunk.org> References: <20180914145114.11122-1-toshi.kani@hpe.com> <20180914145114.11122-2-toshi.kani@hpe.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: jack@suse.cz, dan.j.williams@intel.com, adilger.kernel@dilger.ca, linux-ext4@vger.kernel.org, linux-nvdimm@lists.01.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org To: Toshi Kani Return-path: Content-Disposition: inline In-Reply-To: <20180914145114.11122-2-toshi.kani@hpe.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org On Fri, Sep 14, 2018 at 08:51:12AM -0600, Toshi Kani wrote: > Ext4 mount path calls .bmap to the journal inode. This currently > works for the DAX mount case because ext4_iget() always set > 'ext4_da_aops' to any regular files. > > In preparation to fix ext4_iget() to set 'ext4_dax_aops' for ext4 > DAX files, add ext4_bmap() to 'ext4_dax_aops'. .bmap works for > DAX inodes. [1] > > [1]: https://lkml.org/lkml/2018/9/12/803 > Fixes: 5f0663bb4a64 ("ext4, dax: introduce ext4_dax_aops") > Signed-off-by: Toshi Kani > Suggested-by: Jan Kara > Cc: Jan Kara > Cc: Dan Williams > Cc: "Theodore Ts'o" > Cc: Andreas Dilger > Cc: Thanks, applied. - Ted