mbox series

[v3,0/8] memcontrol code cleanup and simplification

Message ID 20210417043538.9793-1-songmuchun@bytedance.com (mailing list archive)
Headers show
Series memcontrol code cleanup and simplification | expand

Message

Muchun Song April 17, 2021, 4:35 a.m. UTC
This patch series is part of [1] patch series. Because those patches are
code cleanup or simplification. I gather those patches into a separate
series to make it easier to review.

[1] https://lore.kernel.org/linux-mm/20210409122959.82264-1-songmuchun@bytedance.com/

Changlogs in v3:
  1. Collect Acked-by and Review-by tags.
  2. Add a comment to patch 5 (suggested by Johannes).

  Thanks to Johannes, Shakeel and Michal's review.

Changlogs in v2:
  1. Collect Acked-by and Review-by tags.
  2. Add a new patch to rename lruvec_holds_page_lru_lock to page_matches_lruvec.
  3. Add a comment to patch 2.

  Thanks to Roman, Johannes, Shakeel and Michal's review.

Muchun Song (8):
  mm: memcontrol: fix page charging in page replacement
  mm: memcontrol: bail out early when !mm in get_mem_cgroup_from_mm
  mm: memcontrol: remove the pgdata parameter of mem_cgroup_page_lruvec
  mm: memcontrol: simplify lruvec_holds_page_lru_lock
  mm: memcontrol: rename lruvec_holds_page_lru_lock to
    page_matches_lruvec
  mm: memcontrol: simplify the logic of objcg pinning memcg
  mm: memcontrol: move obj_cgroup_uncharge_pages() out of css_set_lock
  mm: vmscan: remove noinline_for_stack

 include/linux/memcontrol.h | 43 ++++++++++--------------------
 mm/compaction.c            |  2 +-
 mm/memcontrol.c            | 65 +++++++++++++++++++++-------------------------
 mm/swap.c                  |  2 +-
 mm/vmscan.c                |  8 +++---
 mm/workingset.c            |  2 +-
 6 files changed, 50 insertions(+), 72 deletions(-)

Comments

Muchun Song April 26, 2021, 7:49 a.m. UTC | #1
On Sat, Apr 17, 2021 at 12:37 PM Muchun Song <songmuchun@bytedance.com> wrote:
>
> This patch series is part of [1] patch series. Because those patches are
> code cleanup or simplification. I gather those patches into a separate
> series to make it easier to review.
>
> [1] https://lore.kernel.org/linux-mm/20210409122959.82264-1-songmuchun@bytedance.com/
>

Hi Andrew,

Since all the patches in this series have at least three
Acked-by/Reviewed-by tags, I think that it is ready to
merge this to mm-tree. Maybe you forgot this. So this
is a reminder. :-) Thanks.


> Changlogs in v3:
>   1. Collect Acked-by and Review-by tags.
>   2. Add a comment to patch 5 (suggested by Johannes).
>
>   Thanks to Johannes, Shakeel and Michal's review.
>
> Changlogs in v2:
>   1. Collect Acked-by and Review-by tags.
>   2. Add a new patch to rename lruvec_holds_page_lru_lock to page_matches_lruvec.
>   3. Add a comment to patch 2.
>
>   Thanks to Roman, Johannes, Shakeel and Michal's review.
>
> Muchun Song (8):
>   mm: memcontrol: fix page charging in page replacement
>   mm: memcontrol: bail out early when !mm in get_mem_cgroup_from_mm
>   mm: memcontrol: remove the pgdata parameter of mem_cgroup_page_lruvec
>   mm: memcontrol: simplify lruvec_holds_page_lru_lock
>   mm: memcontrol: rename lruvec_holds_page_lru_lock to
>     page_matches_lruvec
>   mm: memcontrol: simplify the logic of objcg pinning memcg
>   mm: memcontrol: move obj_cgroup_uncharge_pages() out of css_set_lock
>   mm: vmscan: remove noinline_for_stack
>
>  include/linux/memcontrol.h | 43 ++++++++++--------------------
>  mm/compaction.c            |  2 +-
>  mm/memcontrol.c            | 65 +++++++++++++++++++++-------------------------
>  mm/swap.c                  |  2 +-
>  mm/vmscan.c                |  8 +++---
>  mm/workingset.c            |  2 +-
>  6 files changed, 50 insertions(+), 72 deletions(-)
>
> --
> 2.11.0
>