[v2,0/9] A few cleanup and fixup patches for vmscan

Message

Miaohe Lin April 9, 2022, 9:33 a.m. UTC

Hi everyone,
This series contains a few patches to remove obsolete comment, introduce
helper to remove duplicated code and so no. Also we take all base pages
of THP into account in rare race condition. More details can be found in
the respective changelogs. Thanks!

---
v2:
  patch 1/9: drop code change and add a comment about MADV_FREE
  patch 2/9: simplify the code further and change to goto keep_locked
  patch 3/9: use folio, remove unneeded inline and break craze long lines
  patch 5/9: activate swap-backed executable folios after first usage too
  patch 9/9: new cleanup patch splitted from 5/9
  Many thanks Huang Ying, Matthew Wilcox, Christoph Hellwig, Muchun Song
  for review!
---
Miaohe Lin (9):
  mm/vmscan: add a comment about MADV_FREE pages check in
    folio_check_dirty_writeback
  mm/vmscan: remove unneeded can_split_huge_page check
  mm/vmscan: introduce helper function reclaim_page_list()
  mm/vmscan: save a bit of stack space in shrink_lruvec
  mm/vmscan: activate swap-backed executable folios after first usage
  mm/vmscan: take all base pages of THP into account when race with
    speculative reference
  mm/vmscan: take min_slab_pages into account when try to call
    shrink_node
  mm/vmscan: remove obsolete comment in kswapd_run
  mm/vmscan: use helper folio_is_file_lru()

 mm/vmscan.c | 92 ++++++++++++++++++++++++++---------------------------
 1 file changed, 45 insertions(+), 47 deletions(-)

Comments

Miaohe Lin April 9, 2022, 9:38 a.m. UTC | #1

On 2022/4/9 17:33, Miaohe Lin wrote:
> Hi everyone,
> This series contains a few patches to remove obsolete comment, introduce
> helper to remove duplicated code and so no. Also we take all base pages
> of THP into account in rare race condition. More details can be found in
> the respective changelogs. Thanks!

Very Sorry about it! There is some invalid address in my cc list. So I failed
to send them together twice. Sorry for making noise. :(

> 
> ---
> v2:
>   patch 1/9: drop code change and add a comment about MADV_FREE
>   patch 2/9: simplify the code further and change to goto keep_locked
>   patch 3/9: use folio, remove unneeded inline and break craze long lines
>   patch 5/9: activate swap-backed executable folios after first usage too
>   patch 9/9: new cleanup patch splitted from 5/9
>   Many thanks Huang Ying, Matthew Wilcox, Christoph Hellwig, Muchun Song
>   for review!
> ---
> Miaohe Lin (9):
>   mm/vmscan: add a comment about MADV_FREE pages check in
>     folio_check_dirty_writeback
>   mm/vmscan: remove unneeded can_split_huge_page check
>   mm/vmscan: introduce helper function reclaim_page_list()
>   mm/vmscan: save a bit of stack space in shrink_lruvec
>   mm/vmscan: activate swap-backed executable folios after first usage
>   mm/vmscan: take all base pages of THP into account when race with
>     speculative reference
>   mm/vmscan: take min_slab_pages into account when try to call
>     shrink_node
>   mm/vmscan: remove obsolete comment in kswapd_run
>   mm/vmscan: use helper folio_is_file_lru()
> 
>  mm/vmscan.c | 92 ++++++++++++++++++++++++++---------------------------
>  1 file changed, 45 insertions(+), 47 deletions(-)
>