diff options
| author | Hugh Dickins | 2013-10-16 13:47:09 -0700 |
|---|---|---|
| committer | Linus Torvalds | 2013-10-16 21:35:53 -0700 |
| commit | 57a8f0cdb87da776bf0e4ce7554a9133854fa779 (patch) | |
| tree | fe920bc16fe4ef97e94d31fce5f25bbecaff6e57 /fs | |
| parent | 750e8165f5e87b6a142be953640eabb13a9d350a (diff) | |
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 <hughd@google.com>
Cc: Chen Gang <gang.chen@asianux.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs')
0 files changed, 0 insertions, 0 deletions