From: Ross Zwisler Subject: Re: [PATCH 09/13] dax: Allow dax_iomap_fault() to return pfn Date: Mon, 21 Aug 2017 12:45:01 -0600 Message-ID: <20170821184501.GA26220@linux.intel.com> References: <20170817160815.30466-1-jack@suse.cz> <20170817160815.30466-10-jack@suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-fsdevel@vger.kernel.org, linux-nvdimm@lists.01.org, Andy Lutomirski , linux-ext4@vger.kernel.org, linux-xfs@vger.kernel.org, Christoph Hellwig , Ross Zwisler , Dan Williams , Boaz Harrosh To: Jan Kara Return-path: Received: from mga09.intel.com ([134.134.136.24]:40003 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753353AbdHUSpD (ORCPT ); Mon, 21 Aug 2017 14:45:03 -0400 Content-Disposition: inline In-Reply-To: <20170817160815.30466-10-jack@suse.cz> Sender: linux-ext4-owner@vger.kernel.org List-ID: On Thu, Aug 17, 2017 at 06:08:11PM +0200, Jan Kara wrote: > For synchronous page fault dax_iomap_fault() will need to return PFN > which will then need to inserted into page tables after fsync() > completes. Add necessary parameter to dax_iomap_fault(). > > Signed-off-by: Jan Kara Yep, this seems like a nice, straightforward way of doing things. I like this better than the vmf->orig_pte solution from the previous RFC. Reviewed-by: Ross Zwisler