summaryrefslogtreecommitdiff
path: root/mm
diff options
context:
space:
mode:
authorAndrew Morton <akpm@linux-foundation.org>2026-07-05 00:46:13 -0700
committerAndrew Morton <akpm@linux-foundation.org>2026-07-09 20:15:26 -0700
commitf16c5cfc8c1c114737de2b6e9ea7084ddbb5c4f2 (patch)
tree4097e8b35bcefee007734d9711d6c8215bd7df99 /mm
parent0507e1f6208bfa1d663aabb3783279a747d3231f (diff)
downloadlinux-next-f16c5cfc8c1c114737de2b6e9ea7084ddbb5c4f2.tar.gz
linux-next-f16c5cfc8c1c114737de2b6e9ea7084ddbb5c4f2.zip
mm-rmap-use-huge_ptep_get-in-try_to_migrate_one-checkpatch-fixes
ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'Commit a98a2f0c8ce1 ("mm/rmap: split migration into its own function")' #18: Commit a98a2f0c8ce1 copied the bug from try_to_unmap_one into WARNING: braces {} are not necessary for any arm of this statement #74: FILE: mm/rmap.c:2504: + if (folio_test_hugetlb(folio)) { [...] + } else { [...] total: 1 errors, 1 warnings, 31 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. ./patches/mm-rmap-use-huge_ptep_get-in-try_to_migrate_one.patch has style problems, please review. NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: Dev Jain <dev.jain@arm.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r--mm/rmap.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/mm/rmap.c b/mm/rmap.c
index 4c035578e564..9939400e77c7 100644
--- a/mm/rmap.c
+++ b/mm/rmap.c
@@ -2501,11 +2501,10 @@ static bool try_to_migrate_one(struct folio *folio, struct vm_area_struct *vma,
VM_BUG_ON_FOLIO(!pvmw.pte, folio);
address = pvmw.address;
- if (folio_test_hugetlb(folio)) {
+ if (folio_test_hugetlb(folio))
pteval = huge_ptep_get(mm, address, pvmw.pte);
- } else {
+ else
pteval = ptep_get(pvmw.pte);
- }
if (likely(pte_present(pteval))) {
pfn = pte_pfn(pteval);
} else {