diff options
| author | Hugh Dickins <hugh@veritas.com> | 2006-10-31 18:40:39 +0000 |
|---|---|---|
| committer | Paul Mackerras <paulus@samba.org> | 2006-11-01 14:52:48 +1100 |
| commit | 96268889ee369b36203b7a06e8aabb197270216e (patch) | |
| tree | 90e6553e23ebd4c23b3e39a90d9cb70bc691e7aa /arch/powerpc | |
| parent | 5fe8e8b88e68e517637e3f8287f1fee89e2d9252 (diff) | |
| download | op-kernel-dev-96268889ee369b36203b7a06e8aabb197270216e.zip op-kernel-dev-96268889ee369b36203b7a06e8aabb197270216e.tar.gz | |
[POWERPC] Make high hugepage areas preempt safe
Checking source for other get_paca()->field preemption dangers found that
open_high_hpage_areas does a structure copy into its paca while preemption
is enabled: unsafe however gcc accomplishes it. Just remove that copy:
it's done safely afterwards by on_each_cpu, as in open_low_hpage_areas.
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Acked-by: David Gibson <dwg@au1.ibm.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc')
| -rw-r--r-- | arch/powerpc/mm/hugetlbpage.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c index 5615acc..fd68b74 100644 --- a/arch/powerpc/mm/hugetlbpage.c +++ b/arch/powerpc/mm/hugetlbpage.c @@ -480,9 +480,6 @@ static int open_high_hpage_areas(struct mm_struct *mm, u16 newareas) mm->context.high_htlb_areas |= newareas; - /* update the paca copy of the context struct */ - get_paca()->context = mm->context; - /* the context change must make it to memory before the flush, * so that further SLB misses do the right thing. */ mb(); |
