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@xxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html