diff options
author | Andrea Arcangeli <aarcange@redhat.com> | 2016-10-07 17:01:25 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-10-07 18:46:29 -0700 |
commit | fb8c41e9ad1f356b06b46a63ada10b7dce2a5d94 (patch) | |
tree | dae7e1ad1d4e34ef6fb70c33f63f97ebe10722d1 /mm/sparse-vmemmap.c | |
parent | 6d2329f8872f23e46a19d240930571510ce525eb (diff) | |
download | linux-fb8c41e9ad1f356b06b46a63ada10b7dce2a5d94.tar.gz linux-fb8c41e9ad1f356b06b46a63ada10b7dce2a5d94.tar.bz2 linux-fb8c41e9ad1f356b06b46a63ada10b7dce2a5d94.zip |
mm: vma_adjust: remove superfluous confusing update in remove_next == 1 case
mm->highest_vm_end doesn't need any update.
After finally removing the oddness from vma_merge case 8 that was
causing:
1) constant risk of trouble whenever anybody would check vma fields
from rmap_walks, like it happened when page migration was
introduced and it read the vma->vm_page_prot from a rmap_walk
2) the callers of vma_merge to re-initialize any value different from
the current vma, instead of vma_merge() more reliably returning a
vma that already matches all fields passed as parameter
.. it is also worth to take the opportunity of cleaning up superfluous
code in vma_adjust(), that if not removed adds up to the hard
readability of the function.
Link: http://lkml.kernel.org/r/1474492522-2261-5-git-send-email-aarcange@redhat.com
Signed-off-by: Andrea Arcangeli <aarcange@redhat.com>
Cc: Rik van Riel <riel@redhat.com>
Cc: Hugh Dickins <hughd@google.com>
Cc: Mel Gorman <mgorman@techsingularity.net>
Cc: Jan Vorlicek <janvorli@microsoft.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/sparse-vmemmap.c')
0 files changed, 0 insertions, 0 deletions