Messages in this thread Patch in this message | | | From | Nadav Amit <> | Subject | [RFC PATCH 2/8] mm/madvise: remove unnecessary check on madvise_dontneed_free() | Date | Sun, 26 Sep 2021 09:12:53 -0700 |
| |
From: Nadav Amit <namit@vmware.com>
madvise_dontneed_free() is called only from madvise_vma() and the behavior is always either MADV_FREE or MADV_DONTNEED. There is no need to check again in madvise_dontneed_free() if the behavior is any different.
Cc: Andrea Arcangeli <aarcange@redhat.com> Cc: Andrew Morton <akpm@linux-foundation.org> Cc: Minchan Kim <minchan@kernel.org> Cc: Colin Cross <ccross@google.com> Cc: Suren Baghdasarya <surenb@google.com> Cc: Mike Rapoport <rppt@linux.vnet.ibm.com> Signed-off-by: Nadav Amit <namit@vmware.com> --- mm/madvise.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/mm/madvise.c b/mm/madvise.c index a2b05352ebfe..fe843513a4e8 100644 --- a/mm/madvise.c +++ b/mm/madvise.c @@ -820,10 +820,8 @@ static long madvise_dontneed_free(struct vm_area_struct *vma, if (behavior == MADV_DONTNEED) return madvise_dontneed_single_vma(vma, start, end); - else if (behavior == MADV_FREE) + else /* behavior == MADV_FREE */ return madvise_free_single_vma(vma, start, end); - else - return -EINVAL; } static long madvise_populate(struct vm_area_struct *vma, -- 2.25.1
| |