Re: [PATCH V1] mm/list_lru: make the case where mlru is NULL as unlikely

[Muchun Song <muchun.song@linux.dev>]

> On Feb 27, 2025, at 16:22, Jingxiang Zeng <jingxiangzeng.cas@gmail.com> wrote:
> 
> From: Zeng Jingxiang <linuszeng@tencent.com>
> 
> In the following memcg_list_lru_alloc() function, mlru here is almost
> always NULL, so in most cases this should save a function call, mark
> mlru as unlikely to optimize the code, and reusing the mlru for the
> next attempt when the tree insertion fails.
>        do {
>                xas_lock_irqsave(&xas, flags);
>                if (!xas_load(&xas) && !css_is_dying(&pos->css)) {
>                        xas_store(&xas, mlru);
>                        if (!xas_error(&xas))
>                                mlru = NULL;
>                }
>                xas_unlock_irqrestore(&xas, flags);
>        } while (xas_nomem(&xas, GFP_KERNEL));
>>      if (mlru)
>                kfree(mlru);
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202412290924.UTP7GH2Z-lkp@intel.com/
> Signed-off-by: Zeng Jingxiang <linuszeng@tencent.com>
> Suggested-by: Johannes Weiner <hannes@cmpxchg.org>

Reviewed-by: Muchun Song <muchun.song@linux.dev>

Thanks.