diff options
| author | Hugh Dickins <hughd@google.com> | 2012-02-08 17:13:40 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-02-13 11:06:11 -0800 |
| commit | 91b08ca08c8f5fbad289950086d6fb7ab0642d63 (patch) | |
| tree | a9229fcb99dd4f846ca1577c213d822345f52a28 /mm/swap.c | |
| parent | b9134812300fd0e3c242c50af5c563065d2f1b1e (diff) | |
| download | kernel_samsung_smdk4412-91b08ca08c8f5fbad289950086d6fb7ab0642d63.zip kernel_samsung_smdk4412-91b08ca08c8f5fbad289950086d6fb7ab0642d63.tar.gz kernel_samsung_smdk4412-91b08ca08c8f5fbad289950086d6fb7ab0642d63.tar.bz2 | |
mm: fix UP THP spin_is_locked BUGs
commit b9980cdcf2524c5fe15d8cbae9c97b3ed6385563 upstream.
Fix CONFIG_TRANSPARENT_HUGEPAGE=y CONFIG_SMP=n CONFIG_DEBUG_VM=y
CONFIG_DEBUG_SPINLOCK=n kernel: spin_is_locked() is then always false,
and so triggers some BUGs in Transparent HugePage codepaths.
asm-generic/bug.h mentions this problem, and provides a WARN_ON_SMP(x);
but being too lazy to add VM_BUG_ON_SMP, BUG_ON_SMP, WARN_ON_SMP_ONCE,
VM_WARN_ON_SMP_ONCE, just test NR_CPUS != 1 in the existing VM_BUG_ONs.
Signed-off-by: Hugh Dickins <hughd@google.com>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'mm/swap.c')
| -rw-r--r-- | mm/swap.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -667,7 +667,7 @@ void lru_add_page_tail(struct zone* zone, VM_BUG_ON(!PageHead(page)); VM_BUG_ON(PageCompound(page_tail)); VM_BUG_ON(PageLRU(page_tail)); - VM_BUG_ON(!spin_is_locked(&zone->lru_lock)); + VM_BUG_ON(NR_CPUS != 1 && !spin_is_locked(&zone->lru_lock)); SetPageLRU(page_tail); |