Messages in this thread | | | Date | Wed, 2 May 2018 08:30:40 -0400 | From | Johannes Weiner <> | Subject | Re: [PATCH v2] mm: introduce memory.min |
| |
On Mon, Apr 23, 2018 at 01:36:10PM +0100, Roman Gushchin wrote: > @@ -59,6 +59,12 @@ enum memcg_memory_event { > MEMCG_NR_MEMORY_EVENTS, > }; > > +enum mem_cgroup_protection { > + MEMCG_PROT_NONE, > + MEMCG_PROT_LOW, > + MEMCG_PROT_HIGH,
Ha, HIGH doesn't make much sense, but I went back and it's indeed what I suggested. Must have been a brainfart. This should be
MEMCG_PROT_NONE, MEMCG_PROT_LOW, MEMCG_PROT_MIN
right? To indicate which type of protection is applying.
The rest of the patch looks good:
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
| |