The quilt patch titled
Subject: filemap: remove redundant folio_test_large check in filemap_free_folio
has been removed from the -mm tree. Its filename was
filemap-remove-redundant-folio_test_large-check-in-filemap_free_folio.patch
This patch was dropped because it was merged into the mm-stable branch
of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm
------------------------------------------------------
From: Guanjun <guanjun@xxxxxxxxxxxxxxxxx>
Subject: filemap: remove redundant folio_test_large check in filemap_free_folio
Date: Thu, 13 Feb 2025 13:56:12 +0800
The folio_test_large() check in filemap_free_folio() is unnecessary
because folio_nr_pages(), which is called internally already performs this
check. Removing the redundant condition simplifies the code and avoids
double validation.
This change improves code readability and reduces unnecessary operations
in the folio freeing path.
Link: https://lkml.kernel.org/r/20250213055612.490993-1-guanjun@xxxxxxxxxxxxxxxxx
Signed-off-by: Guanjun <guanjun@xxxxxxxxxxxxxxxxx>
Acked-by: David Hildenbrand <david@xxxxxxxxxx>
Cc: Matthew Wilcox <willy@xxxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
mm/filemap.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
--- a/mm/filemap.c~filemap-remove-redundant-folio_test_large-check-in-filemap_free_folio
+++ a/mm/filemap.c
@@ -227,15 +227,12 @@ void __filemap_remove_folio(struct folio
void filemap_free_folio(struct address_space *mapping, struct folio *folio)
{
void (*free_folio)(struct folio *);
- int refs = 1;
free_folio = mapping->a_ops->free_folio;
if (free_folio)
free_folio(folio);
- if (folio_test_large(folio))
- refs = folio_nr_pages(folio);
- folio_put_refs(folio, refs);
+ folio_put_refs(folio, folio_nr_pages(folio));
}
/**
_
Patches currently in -mm which might be from guanjun@xxxxxxxxxxxxxxxxx are
