Commit 4a76ef03 authored Feb 14, 2007 by Nick Piggin Committed by Linus Torvalds Feb 15, 2007

[PATCH] mincore: fill in results properly



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 <npiggin@suse.de>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent 30fcffed

mm/mincore.c

+5 −0

Original line number	Diff line number	Diff line
		@@ -125,6 +125,8 @@ static long do_mincore(unsigned long addr, unsigned char *vec, unsigned long pag
		#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;

Admin message