mm: push down lock_page_memcg()

[PATCH v2 0/3] mm: push down lock_page_memcg()

Johannes Weiner posted 3 patches 2 years, 9 months ago

Download series mbox

Documentation/admin-guide/cgroup-v1/memory.rst | 11 ++++-
mm/memcontrol.c                                | 56 ++++++++++++++++++------
mm/rmap.c                                      | 26 ++++-------
3 files changed, 60 insertions(+), 33 deletions(-)

Expand all Fold all

[PATCH v2 0/3] mm: push down lock_page_memcg()

Posted by Johannes Weiner 2 years, 9 months ago

New series based on the discussion in the previous thread around
getting lock_page_memcg() out of rmap.

I beat on this with concurrent high-frequency moving of tasks that
partially share a swapped out shmem file. I didn't spot anything
problematic. That said, it is quite subtle, and Hugh, I'd feel better
if you could also subject it to your torture suite ;)

Thanks!

Against yesterday's mm-unstable.

 Documentation/admin-guide/cgroup-v1/memory.rst | 11 ++++-
 mm/memcontrol.c                                | 56 ++++++++++++++++++------
 mm/rmap.c                                      | 26 ++++-------
 3 files changed, 60 insertions(+), 33 deletions(-)

Re: [PATCH v2 0/3] mm: push down lock_page_memcg()

Posted by Michal Hocko 2 years, 9 months ago

On Tue 06-12-22 18:13:38, Johannes Weiner wrote:
> New series based on the discussion in the previous thread around
> getting lock_page_memcg() out of rmap.
> 
> I beat on this with concurrent high-frequency moving of tasks that
> partially share a swapped out shmem file. I didn't spot anything
> problematic. That said, it is quite subtle, and Hugh, I'd feel better
> if you could also subject it to your torture suite ;)

For the whole series
Acked-by: Michal Hocko <mhocko@suse.com>
Thanks!
> 
> Thanks!
> 
> Against yesterday's mm-unstable.
> 
>  Documentation/admin-guide/cgroup-v1/memory.rst | 11 ++++-
>  mm/memcontrol.c                                | 56 ++++++++++++++++++------
>  mm/rmap.c                                      | 26 ++++-------
>  3 files changed, 60 insertions(+), 33 deletions(-)
> 

-- 
Michal Hocko
SUSE Labs