Commit 1a9f064f authored by Rusty Russell's avatar Rusty Russell Committed by Linus Torvalds

m32r: make flush_cpumask non-volatile.

We cast away the volatile, but really, why make it volatile at all?
We already do a mb() inside the cpumask_empty() loop.
Signed-off-by: default avatarRusty Russell <rusty@rustcorp.com.au>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 95c607d9
...@@ -45,7 +45,7 @@ static volatile unsigned long flushcache_cpumask = 0; ...@@ -45,7 +45,7 @@ static volatile unsigned long flushcache_cpumask = 0;
/* /*
* For flush_tlb_others() * For flush_tlb_others()
*/ */
static volatile cpumask_t flush_cpumask; static cpumask_t flush_cpumask;
static struct mm_struct *flush_mm; static struct mm_struct *flush_mm;
static struct vm_area_struct *flush_vma; static struct vm_area_struct *flush_vma;
static volatile unsigned long flush_va; static volatile unsigned long flush_va;
...@@ -415,7 +415,7 @@ static void flush_tlb_others(cpumask_t cpumask, struct mm_struct *mm, ...@@ -415,7 +415,7 @@ static void flush_tlb_others(cpumask_t cpumask, struct mm_struct *mm,
*/ */
send_IPI_mask(&cpumask, INVALIDATE_TLB_IPI, 0); send_IPI_mask(&cpumask, INVALIDATE_TLB_IPI, 0);
while (!cpumask_empty((cpumask_t*)&flush_cpumask)) { while (!cpumask_empty(&flush_cpumask)) {
/* nothing. lockup detection does not belong here */ /* nothing. lockup detection does not belong here */
mb(); mb();
} }
...@@ -468,7 +468,7 @@ void smp_invalidate_interrupt(void) ...@@ -468,7 +468,7 @@ void smp_invalidate_interrupt(void)
__flush_tlb_page(va); __flush_tlb_page(va);
} }
} }
cpumask_clear_cpu(cpu_id, (cpumask_t*)&flush_cpumask); cpumask_clear_cpu(cpu_id, &flush_cpumask);
} }
/*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*/ /*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*=*/
......
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