On Thu, Jan 09, 2020 at 12:26:54PM -0800, Roman Gushchin wrote: > The first argument of memcg_kmem_charge_memcg() and > __memcg_kmem_charge_memcg() is the page pointer and it's not used. > Let's drop it. > > Memcg pointer is passed as the last argument. Move it to > the first place for consistency with other memcg functions, > e.g. __memcg_kmem_uncharge_memcg() or try_charge(). > > Signed-off-by: Roman Gushchin <guro@xxxxxx> Looks good to me. Acked-by: Johannes Weiner <hannes@xxxxxxxxxxx>