Commit 4877b14f authored by Andrew Morton's avatar Andrew Morton Committed by Linus Torvalds

[PATCH] rmap 34 vm_flags page_table_lock

From: Hugh Dickins <hugh@veritas.com>

First of a batch of seven rmap patches, based on 2.6.6-mm3.  Probably the
final batch: remaining issues outstanding can have isolated patches.  The
first half of the batch is good for anonmm or anon_vma, the second half of the
batch replaces my anonmm rmap by Andrea's anon_vma rmap.

Judge for yourselves which you prefer.  I do think I was wrong to call
anon_vma more complex than anonmm (its lists are easier to understand than my
refcounting), and I'm happy with its vma merging after the last patch.  It
just comes down to whether we can spare the extra 24 bytes (maximum, on
32-bit) per vma for its advantages in swapout and mremap.

rmap 34 vm_flags page_table_lock

Why do we guard vm_flags mods with page_table_lock when it's already
down_write guarded by mmap_sem?  There's probably a historical reason, but no
sign of any need for it now.  Andrea added a comment and removed the instance
from mprotect.c, Hugh plagiarized his comment and removed the instances from
madvise.c and mlock.c.  Huge leap in scalability...  not expected; but this
should stop people asking why those spinlocks.
parent 114c71ee
...@@ -31,7 +31,9 @@ static long madvise_behavior(struct vm_area_struct * vma, unsigned long start, ...@@ -31,7 +31,9 @@ static long madvise_behavior(struct vm_area_struct * vma, unsigned long start,
return -EAGAIN; return -EAGAIN;
} }
spin_lock(&mm->page_table_lock); /*
* vm_flags is protected by the mmap_sem held in write mode.
*/
VM_ClearReadHint(vma); VM_ClearReadHint(vma);
switch (behavior) { switch (behavior) {
...@@ -44,7 +46,6 @@ static long madvise_behavior(struct vm_area_struct * vma, unsigned long start, ...@@ -44,7 +46,6 @@ static long madvise_behavior(struct vm_area_struct * vma, unsigned long start,
default: default:
break; break;
} }
spin_unlock(&mm->page_table_lock);
return 0; return 0;
} }
......
...@@ -32,10 +32,13 @@ static int mlock_fixup(struct vm_area_struct * vma, ...@@ -32,10 +32,13 @@ static int mlock_fixup(struct vm_area_struct * vma,
goto out; goto out;
} }
} }
spin_lock(&mm->page_table_lock); /*
* vm_flags is protected by the mmap_sem held in write mode.
* It's okay if try_to_unmap_one unmaps a page just after we
* set VM_LOCKED, make_pages_present below will bring it back.
*/
vma->vm_flags = newflags; vma->vm_flags = newflags;
spin_unlock(&mm->page_table_lock);
/* /*
* Keep track of amount of locked VM. * Keep track of amount of locked VM.
......
...@@ -213,10 +213,12 @@ mprotect_fixup(struct vm_area_struct *vma, struct vm_area_struct **pprev, ...@@ -213,10 +213,12 @@ mprotect_fixup(struct vm_area_struct *vma, struct vm_area_struct **pprev,
goto fail; goto fail;
} }
spin_lock(&mm->page_table_lock); /*
* vm_flags and vm_page_prot are protected by the mmap_sem
* held in write mode.
*/
vma->vm_flags = newflags; vma->vm_flags = newflags;
vma->vm_page_prot = newprot; vma->vm_page_prot = newprot;
spin_unlock(&mm->page_table_lock);
success: success:
change_protection(vma, start, end, newprot); change_protection(vma, start, end, newprot);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment