On Mon, Mar 14, 2022 at 10:54:03AM +0100, Michal Hocko wrote: >On Sat 12-03-22 07:16:22, Wei Yang wrote: >> There is no tree operation if mz is not on-tree. > >This doesn't explain problem you are trying to solve nor does it make >much sense to me TBH. > This just tries to make the code looks consistent. >> Let's remove the extra check. > >What would happen if the mz was already in the excess tree and the >excess has grown? The purpose mem_cgroup_update_tree() is to update the soft limit tree. And the approach is to remove and add it back to the tree with new excess. I don't get your point for this question. -- Wei Yang Help you, Help me