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/linux/kernel/git/next/linux-next.git/commit/?id=da4360877094368f6dfe75bbe804b0f0a5d575b0
Signed-off-by: Ebru Akagunduz <[email protected]>
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);
hugepage_vma_revalidate(mm, address);
...
pte = pte_offset_map(pmd, address);
What guarantees that 'pmd' is still valid?
Vlastimil
