Michal Hocko <mho...@suse.com> 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 <linmia...@huawei.com> > >Acked-by: Michal Hocko <mho...@suse.com> > >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.