Re: [PATCH] mm: remove unused alloc_pages_bulk_list()

Yang Shi
2023-05-31T21:29:23Z

On Wed, May 31, 2023 at 8:24 AM Lorenzo Stoakes <lstoakes@gmail.com> wrote:
>
> On Mon, 19 Dec 2022 at 10:07, Mel Gorman <mgorman@techsingularity.net> wrote:
> >
> > On Sat, Dec 17, 2022 at 08:34:58AM +0000, Lorenzo Stoakes wrote:
> > I reckon we should give it another few months until May. There has been
> > one user recently that tried to use list but it turned out arrays were
> > more appropriate.
> >
>
> It being May 31st, it feels appropriate to chase this up :)
>
> David's series at [0] did initially use this function, before
> switching to the _array() variant. Other than that it seems that it
> remains unused.

Thanks for following this up. My patchset (use bulk allocator for
dm-crypt) also switched from list API to a new callback API suggested
by Mel. But I haven't heard anything back from Mel about it yet. I
forgot to follow up due to too many distractions.

>
> Cheers, Lorenzo
>
> [0]:https://lore.kernel.org/all/20230522205744.2825689-1-dhowells@redhat.com/
>
> > --
> > Mel Gorman
> > SUSE Labs
>
>
>
> --
> Lorenzo Stoakes
> https://ljs.io
>