Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-28 Thread Naoya Horiguchi
# sorry for late reply ... On Fri, Nov 18, 2016 at 02:56:24AM +0300, Kirill A. Shutemov wrote: > On Tue, Nov 08, 2016 at 08:31:52AM +0900, Naoya Horiguchi wrote: > > If one of callers of page migration starts to handle thp, memory management > > code > > start to see pmd migration entry, so we

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-28 Thread Naoya Horiguchi
# sorry for late reply ... On Fri, Nov 18, 2016 at 02:56:24AM +0300, Kirill A. Shutemov wrote: > On Tue, Nov 08, 2016 at 08:31:52AM +0900, Naoya Horiguchi wrote: > > If one of callers of page migration starts to handle thp, memory management > > code > > start to see pmd migration entry, so we

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-28 Thread Michal Hocko
On Tue 08-11-16 08:31:52, Naoya Horiguchi wrote: > If one of callers of page migration starts to handle thp, memory management > code > start to see pmd migration entry, so we need to prepare for it before > enabling. > This patch changes various code point which checks the status of given pmds

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-28 Thread Michal Hocko
On Tue 08-11-16 08:31:52, Naoya Horiguchi wrote: > If one of callers of page migration starts to handle thp, memory management > code > start to see pmd migration entry, so we need to prepare for it before > enabling. > This patch changes various code point which checks the status of given pmds

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-17 Thread Kirill A. Shutemov
On Tue, Nov 08, 2016 at 08:31:52AM +0900, Naoya Horiguchi wrote: > If one of callers of page migration starts to handle thp, memory management > code > start to see pmd migration entry, so we need to prepare for it before > enabling. > This patch changes various code point which checks the

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-17 Thread Kirill A. Shutemov
On Tue, Nov 08, 2016 at 08:31:52AM +0900, Naoya Horiguchi wrote: > If one of callers of page migration starts to handle thp, memory management > code > start to see pmd migration entry, so we need to prepare for it before > enabling. > This patch changes various code point which checks the

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Naoya Horiguchi
On Thu, Nov 10, 2016 at 05:28:20PM +0800, Hillf Danton wrote: > On Thursday, November 10, 2016 5:22 PM Naoya Horiguchi wrote: > > On Thu, Nov 10, 2016 at 05:08:07PM +0800, Hillf Danton wrote: > > > On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > > > > > > > @@ -1013,6 +1027,9 @@

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Naoya Horiguchi
On Thu, Nov 10, 2016 at 05:28:20PM +0800, Hillf Danton wrote: > On Thursday, November 10, 2016 5:22 PM Naoya Horiguchi wrote: > > On Thu, Nov 10, 2016 at 05:08:07PM +0800, Hillf Danton wrote: > > > On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > > > > > > > @@ -1013,6 +1027,9 @@

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Hillf Danton
On Thursday, November 10, 2016 5:22 PM Naoya Horiguchi wrote: > On Thu, Nov 10, 2016 at 05:08:07PM +0800, Hillf Danton wrote: > > On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > > > > > @@ -1013,6 +1027,9 @@ int do_huge_pmd_wp_page(struct fault_env *fe, pmd_t > > > orig_pmd) > >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Hillf Danton
On Thursday, November 10, 2016 5:22 PM Naoya Horiguchi wrote: > On Thu, Nov 10, 2016 at 05:08:07PM +0800, Hillf Danton wrote: > > On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > > > > > @@ -1013,6 +1027,9 @@ int do_huge_pmd_wp_page(struct fault_env *fe, pmd_t > > > orig_pmd) > >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Hillf Danton
On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > @@ -1013,6 +1027,9 @@ int do_huge_pmd_wp_page(struct fault_env *fe, pmd_t > orig_pmd) > if (unlikely(!pmd_same(*fe->pmd, orig_pmd))) > goto out_unlock; > > + if (unlikely(!pmd_present(orig_pmd))) > +

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Hillf Danton
On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > @@ -1013,6 +1027,9 @@ int do_huge_pmd_wp_page(struct fault_env *fe, pmd_t > orig_pmd) > if (unlikely(!pmd_same(*fe->pmd, orig_pmd))) > goto out_unlock; > > + if (unlikely(!pmd_present(orig_pmd))) > +

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Naoya Horiguchi
Hi Hillf, On Thu, Nov 10, 2016 at 05:08:07PM +0800, Hillf Danton wrote: > On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > > > @@ -1013,6 +1027,9 @@ int do_huge_pmd_wp_page(struct fault_env *fe, pmd_t > > orig_pmd) > > if (unlikely(!pmd_same(*fe->pmd, orig_pmd))) > >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Naoya Horiguchi
Hi Hillf, On Thu, Nov 10, 2016 at 05:08:07PM +0800, Hillf Danton wrote: > On Tuesday, November 08, 2016 7:32 AM Naoya Horiguchi wrote: > > > > @@ -1013,6 +1027,9 @@ int do_huge_pmd_wp_page(struct fault_env *fe, pmd_t > > orig_pmd) > > if (unlikely(!pmd_same(*fe->pmd, orig_pmd))) > >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Naoya Horiguchi
On Thu, Nov 10, 2016 at 02:06:14PM +0530, Anshuman Khandual wrote: > On 11/08/2016 05:01 AM, Naoya Horiguchi wrote: > > If one of callers of page migration starts to handle thp, memory management > > code > > start to see pmd migration entry, so we need to prepare for it before > > enabling. > >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Naoya Horiguchi
On Thu, Nov 10, 2016 at 02:06:14PM +0530, Anshuman Khandual wrote: > On 11/08/2016 05:01 AM, Naoya Horiguchi wrote: > > If one of callers of page migration starts to handle thp, memory management > > code > > start to see pmd migration entry, so we need to prepare for it before > > enabling. > >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Anshuman Khandual
On 11/08/2016 05:01 AM, Naoya Horiguchi wrote: > If one of callers of page migration starts to handle thp, memory management > code > start to see pmd migration entry, so we need to prepare for it before > enabling. > This patch changes various code point which checks the status of given pmds in

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-10 Thread Anshuman Khandual
On 11/08/2016 05:01 AM, Naoya Horiguchi wrote: > If one of callers of page migration starts to handle thp, memory management > code > start to see pmd migration entry, so we need to prepare for it before > enabling. > This patch changes various code point which checks the status of given pmds in

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-07 Thread Naoya Horiguchi
On Tue, Nov 08, 2016 at 08:23:50AM +0800, kbuild test robot wrote: > Hi Naoya, > > [auto build test ERROR on mmotm/master] > [also build test ERROR on v4.9-rc4 next-20161028] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-07 Thread Naoya Horiguchi
On Tue, Nov 08, 2016 at 08:23:50AM +0800, kbuild test robot wrote: > Hi Naoya, > > [auto build test ERROR on mmotm/master] > [also build test ERROR on v4.9-rc4 next-20161028] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: >

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-07 Thread kbuild test robot
Hi Naoya, [auto build test ERROR on mmotm/master] [also build test ERROR on v4.9-rc4 next-20161028] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-07 Thread kbuild test robot
Hi Naoya, [auto build test ERROR on mmotm/master] [also build test ERROR on v4.9-rc4 next-20161028] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-07 Thread Naoya Horiguchi
If one of callers of page migration starts to handle thp, memory management code start to see pmd migration entry, so we need to prepare for it before enabling. This patch changes various code point which checks the status of given pmds in order to prevent race between thp migration and the

[PATCH v2 07/12] mm: thp: check pmd migration entry in common path

2016-11-07 Thread Naoya Horiguchi
If one of callers of page migration starts to handle thp, memory management code start to see pmd migration entry, so we need to prepare for it before enabling. This patch changes various code point which checks the status of given pmds in order to prevent race between thp migration and the