Re: [PATCH] mm, thp: fix locking inconsistency in collapse_huge_page

2016-06-08 Thread Sergey Senozhatsky
On (06/06/16 15:05), Vlastimil Babka wrote: [..] > I think this does fix the inconsistency, thanks. > > But looking at collapse_huge_page() as of latest -next, I wonder if there's > another problem: > > pmd = mm_find_pmd(mm, address); > ... > up_read(&mm->mmap_sem); > down_write(&mm->mmap_sem); >

Re: [PATCH] mm, thp: fix locking inconsistency in collapse_huge_page

2016-06-06 Thread Vlastimil Babka
On 06/03/2016 02:28 PM, Ebru Akagunduz wrote: After creating revalidate vma function, locking inconsistency occured due to directing the code path to wrong label. This patch directs to correct label and fix the inconsistency. Related commit that caused inconsistency: http://git.kernel.org/cgit/l

[PATCH] mm, thp: fix locking inconsistency in collapse_huge_page

2016-06-03 Thread Ebru Akagunduz
After creating revalidate vma function, locking inconsistency occured due to directing the code path to wrong label. This patch directs to correct label and fix the inconsistency. Related commit that caused inconsistency: http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=d