Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; 
  s=zapps768; d=zoho.com; 
  h=date:from:reply-to:to:cc:message-id:in-reply-to:references:subject:mime-version:content-type:user-agent; 
  b=NH/0WVHs8pl5r7c4SwmzFlzo30SkQTMHFrgv/+asfm3G+PavvDWfMPKBSjYMX+L/jPxGsysbaH8n
    BBfhHBImml6Znd6gVmDN9+O+U+Pw3GwCOra4zqzDEyJJbBMVWGV6  
Date:   Thu, 05 Jul 2018 00:55:27 +0800
From:   Huaisheng Ye <yehs2007@zoho.com>
Reply-To: yehs2007@zoho.com
To:     "\"Dan Williams\"" <dan.j.williams@intel.com>
Cc:     "\"linux-nvdimm\"" <linux-nvdimm@lists.01.org>,
        "\"Ross Zwisler\"" <ross.zwisler@linux.intel.com>,
        "\"Matthew Wilcox\"" <willy@infradead.org>,
        "\"Vishal L Verma\"" <vishal.l.verma@intel.com>,
        "\"Dave Jiang\"" <dave.jiang@intel.com>,
        "\"Martin Schwidefsky\"" <schwidefsky@de.ibm.com>,
        "\"Heiko Carstens\"" <heiko.carstens@de.ibm.com>,
        "\"Al Viro\"" <viro@zeniv.linux.org.uk>,
        "\"Martin K. Petersen\"" <martin.petersen@oracle.com>,
        "\"Jens Axboe\"" <axboe@kernel.dk>,
        "\"Greg KH\"" <gregkh@linuxfoundation.org>,
        "\"Bart Van Assche\"" <bart.vanassche@wdc.com>,
        "\"Jan Kara\"" <jack@suse.com>,
        "\"Linux Kernel Mailing List\"" <linux-kernel@vger.kernel.org>,
        "\"linux-s390\"" <linux-s390@vger.kernel.org>,
        "\"linux-fsdevel\"" <linux-fsdevel@vger.kernel.org>,
        "\"colyli\"" <colyli@suse.de>,
        "\"NingTing Cheng\"" <chengnt@lenovo.com>,
        "\"Huaisheng Ye\"" <yehs1@lenovo.com>
Message-ID: <16466385fd1.ddcc3b7a51249.6994139134661160061@zoho.com>
In-Reply-To: <CAPcyv4hgk89NKGhmZ1Z3XsGFzJAn1y+46WJn_61r8rj5bmqv6A@mail.gmail.com>
References: <20180704163823.9980-1-yehs2007@zoho.com> <CAPcyv4hgk89NKGhmZ1Z3XsGFzJAn1y+46WJn_61r8rj5bmqv6A@mail.gmail.com>
Subject: Re: [PATCH v2 0/4] Assigning NULL to gfn of dax_direct_access if
 useless
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
User-Agent: Zoho Mail
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


 ---- On Thu, 05 Jul 2018 00:48:40 +0800 Dan Williams <dan.j.williams@intel.com> wrote ---- 
 > On Wed, Jul 4, 2018 at 9:38 AM, Huaisheng Ye <yehs2007@zoho.com> wrote: 
 > > From: Huaisheng Ye <yehs1@lenovo.com> 
 > > 
 > > Changes since v1 [1]: 
 > > * Collect Jan's reviewed-by. 
 > > * According to Dan's suggestion, update the unit test infrastructure 
 > >   tools/testing/nvdimm/pmem-dax.c for checking the validity of gfn. 
 > > 
 > > [1]: https://lkml.org/lkml/2018/7/4/81 
 > > 
 > > --- 
 > > 
 > > Some functions within fs/dax don't need to get gfn from direct_access. 
 >  
 > Any reason you are calling it 'gfn'? I'm assuming that is a typo and 
 > it should be 'pfn'. 

Oh, sorry.
I made a mistake, it is a typo. I will correct it right now.

---
Cheers,
Huaisheng Ye