The patch titled Subject: mm-madvise-split-out-mmap-locking-operations-for-madvise-fix-checkpatch-fixes has been added to the -mm mm-unstable branch. Its filename is mm-madvise-split-out-mmap-locking-operations-for-madvise-fix-checkpatch-fixes.patch This patch will shortly appear at https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/mm-madvise-split-out-mmap-locking-operations-for-madvise-fix-checkpatch-fixes.patch This patch will later appear in the mm-unstable branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/process/submit-checklist.rst when testing your code *** The -mm tree is included into linux-next via the mm-everything branch at git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm and is updated there every 2-3 working days ------------------------------------------------------ From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> Subject: mm-madvise-split-out-mmap-locking-operations-for-madvise-fix-checkpatch-fixes Date: Tue Feb 11 03:55:07 PM PST 2025 WARNING: suspect code indent for conditional statements (8, 12) #57: FILE: mm/madvise.c:1598: + if (is_memory_failure(behavior)) + return 0; WARNING: Statements should start on a tabstop #58: FILE: mm/madvise.c:1599: + return 0; WARNING: suspect code indent for conditional statements (8, 12) #72: FILE: mm/madvise.c:1612: + if (is_memory_failure(behavior)) + return; WARNING: Statements should start on a tabstop #73: FILE: mm/madvise.c:1613: + return; Please run checkpatch prior to sending patches Cc: Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- mm/madvise.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/mm/madvise.c~mm-madvise-split-out-mmap-locking-operations-for-madvise-fix-checkpatch-fixes +++ a/mm/madvise.c @@ -1596,7 +1596,7 @@ static bool is_memory_failure(int behavi static int madvise_lock(struct mm_struct *mm, int behavior) { if (is_memory_failure(behavior)) - return 0; + return 0; if (madvise_need_mmap_write(behavior)) { if (mmap_write_lock_killable(mm)) @@ -1610,7 +1610,7 @@ static int madvise_lock(struct mm_struct static void madvise_unlock(struct mm_struct *mm, int behavior) { if (is_memory_failure(behavior)) - return; + return; if (madvise_need_mmap_write(behavior)) mmap_write_unlock(mm); _ Patches currently in -mm which might be from akpm@xxxxxxxxxxxxxxxxxxxx are mm-vmscan-extract-calculated-pressure-balance-as-a-function-fix.patch mm-madvise-split-out-mmap-locking-operations-for-madvise-fix-checkpatch-fixes.patch mm-madvise-split-out-mmap-locking-operations-for-madvise-fix-fix.patch mm-madvise-remove-redundant-mmap_lock-operations-from-process_madvise-fix.patch lib-plistc-add-shortcut-for-plist_requeue-fix.patch