Re: [External] [PATCH 2/3] mm: Charge active memcg when no mm is set

2021-04-02 Thread Muchun Song
On Sat, Apr 3, 2021 at 3:17 AM Dan Schatzberg wrote: > > set_active_memcg() worked for kernel allocations but was silently > ignored for user pages. > > This patch establishes a precedence order for who gets charged: > > 1. If there is a memcg associated with the page already, that memcg is >c

Re: [External] [PATCH 2/3] mm: Charge active memcg when no mm is set

2021-03-29 Thread Shakeel Butt
On Mon, Mar 29, 2021 at 9:13 AM Muchun Song wrote: > > On Mon, Mar 29, 2021 at 10:49 PM Dan Schatzberg > wrote: [...] > > Since remote memcg must hold a reference, we do not > need to do something like get_active_memcg() does. > Just use css_get to obtain a ref, it is simpler. Just > Like below.

Re: [External] [PATCH 2/3] mm: Charge active memcg when no mm is set

2021-03-29 Thread Muchun Song
On Mon, Mar 29, 2021 at 10:49 PM Dan Schatzberg wrote: > > set_active_memcg() worked for kernel allocations but was silently > ignored for user pages. > > This patch establishes a precedence order for who gets charged: > > 1. If there is a memcg associated with the page already, that memcg is >