lkml.org 
[lkml]   [2017]   [Oct]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] Add /proc/PID/{smaps, numa_maps} support for DAX
Date


>-----Original Message-----
>From: Dan Williams [mailto:dan.j.williams@intel.com]
>Sent: Tuesday, October 24, 2017 4:47 PM
>To: Du, Fan <fan.du@intel.com>
>Cc: Andrew Morton <akpm@linux-foundation.org>; Christoph Hellwig
><hch@lst.de>; linux-kernel@vger.kernel.org; Dave Hansen
><dave.hansen@linux.intel.com>
>Subject: Re: [PATCH] Add /proc/PID/{smaps, numa_maps} support for DAX
>
>On Tue, Oct 24, 2017 at 12:52 AM, Fan Du <fan.du@intel.com> wrote:
>>
>> So user could check those interface for more detailed
>> information about how much DAX mappings are currently
>> created.
>>
>> Here we use vma_is_dax method to find specific page
>> struture with DAX {huge, normal}page mappings,
>> vm_normal_page routine works as before without any
>> impact on the existing logical where _vm_normal_page
>> are called.
>>
>> Signed-off-by: Fan Du <fan.du@intel.com>
>> ---
>> fs/proc/task_mmu.c | 25 +++++++++++++++++++------
>> 1 file changed, 19 insertions(+), 6 deletions(-)
>>
>> diff --git a/fs/proc/task_mmu.c b/fs/proc/task_mmu.c
>> index 5589b4b..ba2e58c 100644
>> --- a/fs/proc/task_mmu.c
>> +++ b/fs/proc/task_mmu.c
>> @@ -528,7 +528,11 @@ static void smaps_pte_entry(pte_t *pte, unsigned
>long addr,
>> struct page *page = NULL;
>>
>> if (pte_present(*pte)) {
>> - page = vm_normal_page(vma, addr, *pte);
>> + if (!vma_is_dax(vma))
>> + page = vm_normal_page(vma, addr, *pte);
>> + else
>> + page = pte_page(*pte);
>
>See usage of pte_devmap, pmd_devmap, and pud_devmap in mm/gup.c. You
>need to check for that bit to know that there is a 'struct page' entry
>associated with a DAX mapping.

Thanks Dan for your review, got it!


>I believe we just need to fix up and
>resend his approach from last year:
>
> https://marc.info/?l=linux-kernel&m=148045064103357&w=2

Sure, I will rebase Dave's patch and send v2 soon.


\
 
 \ /
  Last update: 2017-10-24 12:25    [W:0.106 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site