diff options
author | Hugh Dickins <hugh@veritas.com> | 2005-10-29 18:15:59 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-10-29 21:40:37 -0700 |
commit | 65500d234e74fc4e8f18e1a429bc24e51e75de4a (patch) | |
tree | 2bae8c3622b6537dbd142ba2744c7cc9430d3b69 /init | |
parent | 7c1fd6b964860cdcf44b6b98d7dcd8cc16a0a26d (diff) | |
download | lwn-65500d234e74fc4e8f18e1a429bc24e51e75de4a.tar.gz lwn-65500d234e74fc4e8f18e1a429bc24e51e75de4a.zip |
[PATCH] mm: page fault handlers tidyup
Impose a little more consistency on the page fault handlers do_wp_page,
do_swap_page, do_anonymous_page, do_no_page, do_file_page: why not pass their
arguments in the same order, called the same names?
break_cow is all very well, but what it did was inlined elsewhere: easier to
compare if it's brought back into do_wp_page.
do_file_page's fallback to do_no_page dates from a time when we were testing
pte_file by using it wherever possible: currently it's peculiar to nonlinear
vmas, so just check that. BUG_ON if not? Better not, it's probably page
table corruption, so just show the pte: hmm, there's a pte_ERROR macro, let's
use that for do_wp_page's invalid pfn too.
Hah! Someone in the ppc64 world noticed pte_ERROR was unused so removed it:
restored (and say "pud" not "pmd" in its pud_ERROR).
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'init')
0 files changed, 0 insertions, 0 deletions