On Wed, Jul 07, 2021 at 01:08:51PM -0400, Johannes Weiner wrote: > On Wed, Jun 30, 2021 at 05:00:29AM +0100, Matthew Wilcox (Oracle) wrote: > > -static void __unlock_page_memcg(struct mem_cgroup *memcg) > > +static void __memcg_unlock(struct mem_cgroup *memcg) > > This is too generic a name. There are several locks in the memcg, and > this one only locks the page->memcg bindings in the group. Fair. __memcg_move_unlock looks like the right name to me?