Re: [next -v1 3/5] memcg: simplify the mem_cgroup_update_lru_size function

From: Yu Zhao
Date: Fri Dec 06 2024 - 00:34:42 EST


On Thu, Dec 5, 2024 at 6:45 PM Chen Ridong <chenridong@xxxxxxxxxxxxxxx> wrote:
>
> From: Chen Ridong <chenridong@xxxxxxxxxx>
>
> In the `mem_cgroup_update_lru_size` function, the `lru_size` should be
> updated by adding `nr_pages` regardless of whether `nr_pages` is greater
> than 0 or less than 0. To simplify this function, add a check for
> `nr_pages` == 0. When `nr_pages` is not equal to 0, perform the same
> actions.
>
> Signed-off-by: Chen Ridong <chenridong@xxxxxxxxxx>

NAK.

The commit that added that clearly explains why it was done that way.