diff options
author | Chen Gang <xili_gchen_5257@hotmail.com> | 2015-11-05 21:48:32 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-11-05 22:34:48 -0500 |
commit | e6ee219fdd69c87ceaeb421bcd753a63937f8f31 (patch) | |
tree | 7cecd90a11417a867288dd21b012ffab1e488a09 /mm | |
parent | 3ca65c19ddbb45f504edf92fe7126ecc94d56e36 (diff) |
mm/mmap.c: remove redundant statement "error = -ENOMEM"
It is still a little better to remove it, although it should be skipped
by "-O2".
Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>=0A=
Acked-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/mmap.c | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -1562,7 +1562,6 @@ unsigned long mmap_region(struct file *file, unsigned long addr, | |||
1562 | } | 1562 | } |
1563 | 1563 | ||
1564 | /* Clear old maps */ | 1564 | /* Clear old maps */ |
1565 | error = -ENOMEM; | ||
1566 | while (find_vma_links(mm, addr, addr + len, &prev, &rb_link, | 1565 | while (find_vma_links(mm, addr, addr + len, &prev, &rb_link, |
1567 | &rb_parent)) { | 1566 | &rb_parent)) { |
1568 | if (do_munmap(mm, addr, len)) | 1567 | if (do_munmap(mm, addr, len)) |