From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753788AbbHOPtR (ORCPT ); Sat, 15 Aug 2015 11:49:17 -0400 Received: from mail-ig0-f177.google.com ([209.85.213.177]:38291 "EHLO mail-ig0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752297AbbHOPtO (ORCPT ); Sat, 15 Aug 2015 11:49:14 -0400 MIME-Version: 1.0 In-Reply-To: <20150815091154.GA21290@lst.de> References: <1439484671-15718-1-git-send-email-ross.zwisler@linux.intel.com> <1439484671-15718-8-git-send-email-ross.zwisler@linux.intel.com> <1439571318.16263.3.camel@linux.intel.com> <20150815091154.GA21290@lst.de> Date: Sat, 15 Aug 2015 08:49:14 -0700 Message-ID: Subject: Re: [PATCH v2 7/7] pmem, dax: have direct_access use __pmem annotation From: Dan Williams To: Christoph Hellwig Cc: Ross Zwisler , "linux-kernel@vger.kernel.org" , "linux-nvdimm@lists.01.org" , Jonathan Corbet , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Jens Axboe , Martin Schwidefsky , Heiko Carstens , linux390@de.ibm.com, Alexander Viro , Matthew Wilcox , Jeff Layton , Andrew Morton , Omar Sandoval , Boaz Harrosh , Miklos Szeredi , Jan Kara , Wolfram Sang , =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , Gerald Schaefer , Ameen Ali , "Martin K. Petersen" , Sagi Grimberg , Mike Snitzer , Tejun Heo , Shaohua Li , Ming Lei , linux-doc@vger.kernel.org, linuxppc-dev , linux-s390@vger.kernel.org, linux-fsdevel , Dave Chinner Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Aug 15, 2015 at 2:11 AM, Christoph Hellwig wrote: > On Fri, Aug 14, 2015 at 09:58:16AM -0700, Dan Williams wrote: >> > I'll merge with your code for v3. >> >> Sounds, let me go rebase the __pfn_t patches on -mm so we'all lined up >> and collision free. > > I'm doubt that we'll have PFN mapping ready for 4.3. I'd rather see > Ross series goes first, and move the patch to remove the size argument > from ->direct access [1] over to this series as well. > > [1] https://git.kernel.org/cgit/linux/kernel/git/djbw/nvdimm.git/commit/?h=pfn&id=8e15e69fb9e61ac563c5a7ffd9dd9a7b545cced3 Yes, let's do it. The need for DAX persistence guarantees is a higher priority than solving the DAX vs PMEM unbind bug. Especially if we can defer __pfn_t and get some basic struct page mapping into 4.3 as well.