x86: UV: remove uv_flush_tlb_others() WARN_ON
authorCliff Wickman <cpw@sgi.com>
Fri, 6 Mar 2009 23:30:56 +0000 (17:30 -0600)
committerIngo Molnar <mingo@elte.hu>
Sun, 8 Mar 2009 10:17:15 +0000 (11:17 +0100)
commit3a450de1365d20afde406f0d9b2931a5e4a4fd6a
tree3631a02fc90e6fbf098182b99d1d7eb7870f74ba
parentccea34b5d0fbab081496d1860f31acee99fa8a6d
x86: UV: remove uv_flush_tlb_others() WARN_ON

In uv_flush_tlb_others() (arch/x86/kernel/tlb_uv.c),
the "WARN_ON(!in_atomic())" fails if CONFIG_PREEMPT is not enabled.

And CONFIG_PREEMPT is not enabled by default in the distribution that
most UV owners will use.

We could #ifdef CONFIG_PREEMPT the warning, but that is not good form.
And there seems to be no suitable fix to in_atomic() when CONFIG_PREMPT
is not on.

As Ingo commented:

  > and we have no proper primitive to test for atomicity. (mainly
  > because we dont know about atomicity on a non-preempt kernel)

So we drop the WARN_ON.

Signed-off-by: Cliff Wickman <cpw@sgi.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/tlb_uv.c