* Re: [PATCH] mm: Remove stray references to struct pagevec
[not found] <20260225-pagevec_cleanup-v1-1-38e2246363d2@columbia.edu>
@ 2026-02-25 20:47 ` Matthew Wilcox
2026-02-25 20:52 ` David Hildenbrand (Arm)
2026-02-25 21:30 ` Chris Li
2 siblings, 0 replies; 3+ messages in thread
From: Matthew Wilcox @ 2026-02-25 20:47 UTC (permalink / raw)
To: Tal Zussman
Cc: David Howells, Marc Dionne, Jaegeuk Kim, Chao Yu, Andrew Morton,
David Hildenbrand, Lorenzo Stoakes, Liam R. Howlett,
Vlastimil Babka, Mike Rapoport, Suren Baghdasaryan, Michal Hocko,
Chris Li, Kairui Song, Kemeng Shi, Nhat Pham, Baoquan He,
Barry Song, linux-afs, linux-kernel, linux-f2fs-devel, linux-mm
On Wed, Feb 25, 2026 at 12:57:16PM -0500, Tal Zussman wrote:
> struct pagevec was removed in commit 1e0877d58b1e ("mm: remove struct
> pagevec"). Remove remaining forward declarations and change
> __folio_batch_release()'s declaration to match its definition.
>
> Signed-off-by: Tal Zussman <tz2294@columbia.edu>
Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
> ---
> Happy to rename PAGEVEC_SIZE and pagevec.h if desired too, but that'd be
> ~65 more lines of churn... Although it looks like there are plenty of .c
> files that include it for no reason, so could be a good opportunity to
> remove those.
Yes, there's lots of cleanup work here that I didn't get around to!
Thanks for taking on this much, and any more work you want to do in this
area will be lovely.
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] mm: Remove stray references to struct pagevec
[not found] <20260225-pagevec_cleanup-v1-1-38e2246363d2@columbia.edu>
2026-02-25 20:47 ` [PATCH] mm: Remove stray references to struct pagevec Matthew Wilcox
@ 2026-02-25 20:52 ` David Hildenbrand (Arm)
2026-02-25 21:30 ` Chris Li
2 siblings, 0 replies; 3+ messages in thread
From: David Hildenbrand (Arm) @ 2026-02-25 20:52 UTC (permalink / raw)
To: Tal Zussman, David Howells, Marc Dionne, Jaegeuk Kim, Chao Yu,
Andrew Morton, Lorenzo Stoakes, Liam R. Howlett, Vlastimil Babka,
Mike Rapoport, Suren Baghdasaryan, Michal Hocko, Chris Li,
Kairui Song, Kemeng Shi, Nhat Pham, Baoquan He, Barry Song,
Matthew Wilcox
Cc: linux-afs, linux-kernel, linux-f2fs-devel, linux-mm
On 2/25/26 18:57, Tal Zussman wrote:
> struct pagevec was removed in commit 1e0877d58b1e ("mm: remove struct
> pagevec"). Remove remaining forward declarations and change
> __folio_batch_release()'s declaration to match its definition.
>
> Signed-off-by: Tal Zussman <tz2294@columbia.edu>
Acked-by: David Hildenbrand (Arm) <david@kernel.org>
--
Cheers,
David
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] mm: Remove stray references to struct pagevec
[not found] <20260225-pagevec_cleanup-v1-1-38e2246363d2@columbia.edu>
2026-02-25 20:47 ` [PATCH] mm: Remove stray references to struct pagevec Matthew Wilcox
2026-02-25 20:52 ` David Hildenbrand (Arm)
@ 2026-02-25 21:30 ` Chris Li
2 siblings, 0 replies; 3+ messages in thread
From: Chris Li @ 2026-02-25 21:30 UTC (permalink / raw)
To: Tal Zussman
Cc: David Howells, Marc Dionne, Jaegeuk Kim, Chao Yu, Andrew Morton,
David Hildenbrand, Lorenzo Stoakes, Liam R. Howlett,
Vlastimil Babka, Mike Rapoport, Suren Baghdasaryan, Michal Hocko,
Kairui Song, Kemeng Shi, Nhat Pham, Baoquan He, Barry Song,
Matthew Wilcox, linux-afs, linux-kernel, linux-f2fs-devel,
linux-mm
Thanks for doing this.
On Wed, Feb 25, 2026 at 9:57 AM Tal Zussman <tz2294@columbia.edu> wrote:
>
> struct pagevec was removed in commit 1e0877d58b1e ("mm: remove struct
> pagevec"). Remove remaining forward declarations and change
> __folio_batch_release()'s declaration to match its definition.
>
> Signed-off-by: Tal Zussman <tz2294@columbia.edu>
Acked-by: Chris Li <chrisl@kernel.org>
Chris
^ permalink raw reply [flat|nested] 3+ messages in thread