From: Nick Piggin Paper bag time. Thanks to Randy for noticing that I didn't actually assign 'present' to anything. Unfortunately my original patch passed the few simple test cases I gave it, purely by coincidence. Signed-off-by: Nick Piggin Signed-off-by: Andrew Morton --- mm/mincore.c | 5 +++++ 1 file changed, 5 insertions(+) diff -puN mm/mincore.c~mincore-fill-in-results-properly mm/mincore.c --- a/mm/mincore.c~mincore-fill-in-results-properly +++ a/mm/mincore.c @@ -125,6 +125,8 @@ static long do_mincore(unsigned long add #endif } } + + vec[i] = present; } pte_unmap_unlock(ptep-1, ptl); @@ -135,6 +137,9 @@ none_mapped: pgoff = linear_page_index(vma, addr); for (i = 0; i < nr; i++, pgoff++) vec[i] = mincore_page(vma->vm_file->f_mapping, pgoff); + } else { + for (i = 0; i < nr; i++) + vec[i] = 0; } return nr; _