diff options
author | Michael Neuling <mikey@neuling.org> | 2007-08-24 02:58:37 -0400 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2007-09-19 00:40:54 -0400 |
commit | 00efee7d5d0d7888aafbf0d2de76943ee8aca47a (patch) | |
tree | 672112cc5404b389f0547b4f5428ec06ca1ca122 /arch/powerpc/mm/slb.c | |
parent | 61a564fd2e7ab13ab11a6ce8305433baacf344ef (diff) |
[POWERPC] Remove barriers from the SLB shadow buffer update
After talking to an IBM POWER hypervisor (PHYP) design and development
guy, there seems to be no need for memory barriers when updating the SLB
shadow buffer provided we only update it from the current CPU, which we
do.
Also, these guys see no need in the future for these barriers.
Signed-off-by: Michael Neuling <mikey@neuling.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc/mm/slb.c')
-rw-r--r-- | arch/powerpc/mm/slb.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/arch/powerpc/mm/slb.c b/arch/powerpc/mm/slb.c index ff1811ac6c81..4bee1cfa9dea 100644 --- a/arch/powerpc/mm/slb.c +++ b/arch/powerpc/mm/slb.c | |||
@@ -59,14 +59,12 @@ static inline void slb_shadow_update(unsigned long ea, | |||
59 | { | 59 | { |
60 | /* | 60 | /* |
61 | * Clear the ESID first so the entry is not valid while we are | 61 | * Clear the ESID first so the entry is not valid while we are |
62 | * updating it. | 62 | * updating it. No write barriers are needed here, provided |
63 | * we only update the current CPU's SLB shadow buffer. | ||
63 | */ | 64 | */ |
64 | get_slb_shadow()->save_area[entry].esid = 0; | 65 | get_slb_shadow()->save_area[entry].esid = 0; |
65 | smp_wmb(); | ||
66 | get_slb_shadow()->save_area[entry].vsid = mk_vsid_data(ea, flags); | 66 | get_slb_shadow()->save_area[entry].vsid = mk_vsid_data(ea, flags); |
67 | smp_wmb(); | ||
68 | get_slb_shadow()->save_area[entry].esid = mk_esid_data(ea, entry); | 67 | get_slb_shadow()->save_area[entry].esid = mk_esid_data(ea, entry); |
69 | smp_wmb(); | ||
70 | } | 68 | } |
71 | 69 | ||
72 | static inline void slb_shadow_clear(unsigned long entry) | 70 | static inline void slb_shadow_clear(unsigned long entry) |