Re: [PATCH] mm: memcontrol: eliminate redundant check in __mem_cgroup_insert_exceeded()

From: linmiaohe
Date: Fri Oct 09 2020 - 22:14:28 EST


Michal Hocko <mhocko@xxxxxxxx> wrote:
> On Fri 09-10-20 09:51:04, Miaohe Lin wrote:
>> The mz->usage_in_excess >= mz_node->usage_in_excess check is exactly
>> the else case of mz->usage_in_excess < mz_node->usage_in_excess. So we
>> could replace else if (mz->usage_in_excess >=
>> mz_node->usage_in_excess) with else equally.
>>
>> Signed-off-by: Miaohe Lin <linmiaohe@xxxxxxxxxx>
>
>Acked-by: Michal Hocko <mhocko@xxxxxxxx>
>
>I believe this is a result of a very unreadable code. Resp. the comment makes it hard to follow. It would be slightly better to simply drop the comment which doesn't really explain much IMHO.
>

Agree. Will do it in v2. Many thanks.