On Fri, Apr 20, 2018 at 05:36:32PM +0100, Roman Gushchin wrote: > We do store memory.min, memory.low and memory.max actual values > in struct page_counter fields, while memory.high value is located > in the struct mem_cgroup directly, which is not very consistent. > > This patch moves the high field from struct mem_cgroup to > struct page_counter to simplify the code and make handling > of all limits/boundaries clearer. I would prefer not doing this. Yes, it looks a bit neater if all these things are next to each other in the struct, but on the other hand it separates the high variable from high_work, and it adds an unnecessary setter function as well. Plus, nothing in the page_counter code actually uses the value, it really isn't part of that abstraction layer.
