diff options
| author | Hugh Dickins <[email protected]> | 2013-10-16 20:47:09 +0000 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2013-10-17 04:35:53 +0000 |
| commit | 57a8f0cdb87da776bf0e4ce7554a9133854fa779 (patch) | |
| tree | fe920bc16fe4ef97e94d31fce5f25bbecaff6e57 /lib/spinlock_debug.c | |
| parent | mm: fix BUG in __split_huge_page_pmd (diff) | |
| download | kernel-57a8f0cdb87da776bf0e4ce7554a9133854fa779.tar.gz kernel-57a8f0cdb87da776bf0e4ce7554a9133854fa779.zip | |
mm: revert mremap pud_free anti-fix
Revert commit 1ecfd533f4c5 ("mm/mremap.c: call pud_free() after fail
calling pmd_alloc()").
The original code was correct: pud_alloc(), pmd_alloc(), pte_alloc_map()
ensure that the pud, pmd, pt is already allocated, and seldom do they
need to allocate; on failure, upper levels are freed if appropriate by
the subsequent do_munmap(). Whereas commit 1ecfd533f4c5 did an
unconditional pud_free() of a most-likely still-in-use pud: saved only
by the near-impossiblity of pmd_alloc() failing.
Signed-off-by: Hugh Dickins <[email protected]>
Cc: Chen Gang <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'lib/spinlock_debug.c')
0 files changed, 0 insertions, 0 deletions