diff options
author | Hugh Dickins <hugh@veritas.com> | 2006-02-14 16:52:59 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-02-14 19:09:33 -0500 |
commit | 16bf134840da3920ded1290973c56ec214636f12 (patch) | |
tree | 1aacddb873bd383030a65e13e78c2ced38edfa23 | |
parent | d98c7a09843621f1b145ca5ae8ed03ff04085edb (diff) |
[PATCH] compound page: no access_process_vm check
The PageCompound check before access_process_vm's set_page_dirty_lock is no
longer necessary, so remove it. But leave the PageCompound checks in
bio_set_pages_dirty, dio_bio_complete and nfs_free_user_pages: at least some
of those were introduced as a little optimization on hugetlb pages.
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | kernel/ptrace.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/ptrace.c b/kernel/ptrace.c index 5f33cdb6fff5..d2cf144d0af5 100644 --- a/kernel/ptrace.c +++ b/kernel/ptrace.c | |||
@@ -242,8 +242,7 @@ int access_process_vm(struct task_struct *tsk, unsigned long addr, void *buf, in | |||
242 | if (write) { | 242 | if (write) { |
243 | copy_to_user_page(vma, page, addr, | 243 | copy_to_user_page(vma, page, addr, |
244 | maddr + offset, buf, bytes); | 244 | maddr + offset, buf, bytes); |
245 | if (!PageCompound(page)) | 245 | set_page_dirty_lock(page); |
246 | set_page_dirty_lock(page); | ||
247 | } else { | 246 | } else { |
248 | copy_from_user_page(vma, page, addr, | 247 | copy_from_user_page(vma, page, addr, |
249 | buf, maddr + offset, bytes); | 248 | buf, maddr + offset, bytes); |