Re: [PATCH 3/4] mm, swap: cleanup swap entry allocation parameter

[David Hildenbrand]

> > > diff --git a/mm/swapfile.c b/mm/swapfile.c
> > > index 732e07c70ce9..534b21aeef5a 100644
> > > --- a/mm/swapfile.c
> > > +++ b/mm/swapfile.c
> > > @@ -1425,7 +1425,7 @@ static bool swap_sync_discard(void)
> > >    * Context: Caller needs to hold the folio lock.
> > >    * Return: Whether the folio was added to the swap cache.
> > >    */
> > > -int folio_alloc_swap(struct folio *folio, gfp_t gfp)
> > > +int folio_alloc_swap(struct folio *folio)
> > >   {
> > >          unsigned int order = folio_order(folio);
> > >          unsigned int size = 1 << order;
> >
> > One trivial issue for myself, I better update the kerneldoc too...
> > sorry about this:
>
> LGTM, with the kerneldoc update:
>
> Acked-by: Nhat Pham <nphamcs@xxxxxxxxx>

Reviewed-by: David Hildenbrand <david@xxxxxxxxxx>

--
Cheers

David / dhildenb