On Wed, Aug 17, 2022 at 01:35:12PM -0700, John Hubbard wrote:
> How's this look to you:
>
> collected = collect_longterm_unpinnable_pages(&movable_page_list,
> nr_pages, pages);
> if (collected == 0)
> return 0;
>
> ret = migrate_longterm_unpinnable_pages(&movable_page_list, nr_pages,
> pages);
>
> /* If we got here, we have some unpinnable pages... */
>
> if (ret == 0) {
> /*
> * ...and we successfully migrated those pages. Which means that
> * the caller should retry the operation now.
> */
> ret = -EAGAIN;
return -EAGAIN
> }
>
> return ret;
But why return 0 from the helper function in the first place?
Jason
