diff options
| author | Hugh Dickins <hughd@google.com> | 2011-05-20 15:47:33 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-05-21 15:13:37 -0700 |
| commit | 10a22931499f4f1f579fb37b1ab0562f3ccfbe0d (patch) | |
| tree | 6fe3645c2e9118330b549e483afccbb809cbaaad | |
| parent | 23181925de608dea571fb5aba4a34900df54b973 (diff) | |
| download | lwn-10a22931499f4f1f579fb37b1ab0562f3ccfbe0d.tar.gz lwn-10a22931499f4f1f579fb37b1ab0562f3ccfbe0d.zip | |
tmpfs: fix highmem swapoff crash regression
commit e6c9366b2adb52cba64b359b3050200743c7568c upstream.
Commit 778dd893ae78 ("tmpfs: fix race between umount and swapoff")
forgot the new rules for strict atomic kmap nesting, causing
WARNING: at arch/x86/mm/highmem_32.c:81
from __kunmap_atomic(), then
BUG: unable to handle kernel paging request at fffb9000
from shmem_swp_set() when shmem_unuse_inode() is handling swapoff with
highmem in use. My disgrace again.
See
https://bugzilla.kernel.org/show_bug.cgi?id=35352
Reported-by: Witold Baryluk <baryluk@smp.if.uj.edu.pl>
Signed-off-by: Hugh Dickins <hughd@google.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | mm/shmem.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mm/shmem.c b/mm/shmem.c index 3ca835061291..21bdaf8fd21c 100644 --- a/mm/shmem.c +++ b/mm/shmem.c @@ -917,11 +917,12 @@ static int shmem_unuse_inode(struct shmem_inode_info *info, swp_entry_t entry, s if (size > ENTRIES_PER_PAGE) size = ENTRIES_PER_PAGE; offset = shmem_find_swp(entry, ptr, ptr+size); + shmem_swp_unmap(ptr); if (offset >= 0) { shmem_dir_unmap(dir); + ptr = shmem_swp_map(subdir); goto found; } - shmem_swp_unmap(ptr); } } lost1: |