Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-27 Thread kbuild test robot
Hi Fan, Thank you for the patch! Yet we hit a small issue. [auto build test WARNING on linus/master] [also build test WARNING on v4.14-rc6 next-20171018] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/c

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-27 Thread Dan Williams
al Hocko >>Cc: Du, Fan ; a...@linux-foundation.org; h...@lst.de; >>Williams, Dan J ; linux-kernel@vger.kernel.org; >>linux-...@vger.kernel.org >>Subject: Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX >> >>On 10/26/2017 07:31 AM, Michal Hocko wrote:

RE: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-26 Thread Du, Fan
Dan J ; linux-kernel@vger.kernel.org; >linux-...@vger.kernel.org >Subject: Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX > >On 10/26/2017 07:31 AM, Michal Hocko wrote: >> On Thu 26-10-17 07:24:14, Dave Hansen wrote: >>> Actually, I don't remember whether

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-26 Thread Dan Williams
On Thu, Oct 26, 2017 at 8:07 AM, Michal Hocko wrote: > On Thu 26-10-17 07:51:20, Dave Hansen wrote: >> On 10/26/2017 07:31 AM, Michal Hocko wrote: >> > On Thu 26-10-17 07:24:14, Dave Hansen wrote: >> >> Actually, I don't remember whether it was tooling or just confused >> >> humans. I *think* Dan

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-26 Thread Michal Hocko
On Thu 26-10-17 07:51:20, Dave Hansen wrote: > On 10/26/2017 07:31 AM, Michal Hocko wrote: > > On Thu 26-10-17 07:24:14, Dave Hansen wrote: > >> Actually, I don't remember whether it was tooling or just confused > >> humans. I *think* Dan was trying to write test cases for huge page DAX > >> suppo

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-26 Thread Dave Hansen
On 10/26/2017 07:31 AM, Michal Hocko wrote: > On Thu 26-10-17 07:24:14, Dave Hansen wrote: >> Actually, I don't remember whether it was tooling or just confused >> humans. I *think* Dan was trying to write test cases for huge page DAX >> support and couldn't figure out whether or not it was using

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-26 Thread Michal Hocko
On Thu 26-10-17 07:24:14, Dave Hansen wrote: > On 10/26/2017 07:16 AM, Michal Hocko wrote: > >> The original motivation was for DAX. They have parallel large page > >> infrastructure separate from hugetlbfs and THP. Their constraints about > >> when they can use large pages differ from the normal

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-26 Thread Dave Hansen
On 10/26/2017 07:16 AM, Michal Hocko wrote: >> The original motivation was for DAX. They have parallel large page >> infrastructure separate from hugetlbfs and THP. Their constraints about >> when they can use large pages differ from the normal mm cases, so it is >> hard to tell when large pages

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-26 Thread Michal Hocko
On Wed 25-10-17 10:14:34, Dave Hansen wrote: > On 10/25/2017 02:30 AM, Michal Hocko wrote: > >> > >> 7f6c1780-7f6c17e0 rw-s 00:06 20559 /dev/dax12.0 > >> Size: 6144 kB > >> . > >> . > >> . > >> Ptes@2MB: 6144 kB > > This says how but it doesn't tell

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-25 Thread Dave Hansen
On 10/25/2017 02:30 AM, Michal Hocko wrote: >> >> 7f6c1780-7f6c17e0 rw-s 00:06 20559 /dev/dax12.0 >> Size: 6144 kB >> . >> . >> . >> Ptes@2MB:6144 kB > This says how but it doesn't tell why and who is going to use the > information and what for.

Re: [PATCH 2/2] Add /proc/PID/{smaps, numa_maps} support for DAX

2017-10-25 Thread Michal Hocko
On Wed 25-10-17 08:27:35, Fan Du 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