From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hugh Dickins Subject: Re: [PATCH 00/13] mm: clean up some lru related pieces Date: Fri, 18 Sep 2020 13:46:59 -0700 (PDT) Message-ID: References: <20200918030051.650890-1-yuzhao@google.com> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=date:from:to:cc:subject:in-reply-to:message-id:references :user-agent:mime-version; bh=2ZugjWO+M2gB8NAEaOpinUsTCy83ITVXPQJJCuUh0tk=; b=WjGWRtLtK/iF2JBGFvAFhBIIWnfYlGitLxgWOb55+90M8NWyeyAYHjMqullm40yEpn lmqlNP/RdMdof/dlgrXD7LIEh4QFBziiZqBFLkYpx6nqO0LhGPGEOc0jUI2Om0ofv1UI S/Hy5kwkEn3elzHAQrdsgG/Q9wGl2K6SM0TmW2KTpFSeBnGKYLHA64mphmfHRP2oHqcu Nw/MpK/R0Xy+kPNtblr1tupxRaInMGtJ8M9fnfZcWJFSLNg21CPbG28w+j/zksLAp7Vl 8clu8q+f9R+5RyuSHTYM6Cv81znd/ZP0aNzgFVSvrVYxc8q3XTCLKcwPTT/gq68bMO0y Um1g== In-Reply-To: <20200918030051.650890-1-yuzhao-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org> List-ID: Content-Type: TEXT/PLAIN; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Yu Zhao Cc: Andrew Morton , Michal Hocko , Alex Shi , Steven Rostedt , Ingo Molnar , Johannes Weiner , Vladimir Davydov , Roman Gushchin , Shakeel Butt , Chris Down , Yafang Shao , Vlastimil Babka , Huang Ying , Pankaj Gupta , Matthew Wilcox , Konstantin Khlebnikov , Minchan Kim , Jaewon Kim , cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org On Thu, 17 Sep 2020, Yu Zhao wrote: > Hi Andrew, > > I see you have taken this: > mm: use add_page_to_lru_list()/page_lru()/page_off_lru() > Do you mind dropping it? > > Michal asked to do a bit of additional work. So I thought I probably > should create a series to do more cleanups I've been meaning to. > > This series contains the change in the patch above and goes a few > more steps farther. It's intended to improve readability and should > not have any performance impacts. There are minor behavior changes in > terms of debugging and error reporting, which I have all highlighted > in the individual patches. All patches were properly tested on 5.8 > running Chrome OS, with various debug options turned on. > > Michal, > > Do you mind taking a looking at the entire series? > > Thank you. > > Yu Zhao (13): > mm: use add_page_to_lru_list() > mm: use page_off_lru() > mm: move __ClearPageLRU() into page_off_lru() > mm: shuffle lru list addition and deletion functions > mm: don't pass enum lru_list to lru list addition functions > mm: don't pass enum lru_list to trace_mm_lru_insertion() > mm: don't pass enum lru_list to del_page_from_lru_list() > mm: rename page_off_lru() to __clear_page_lru_flags() > mm: inline page_lru_base_type() > mm: VM_BUG_ON lru page flags > mm: inline __update_lru_size() > mm: make lruvec_lru_size() static > mm: enlarge the int parameter of update_lru_size() > > include/linux/memcontrol.h | 14 ++-- > include/linux/mm_inline.h | 115 ++++++++++++++------------------- > include/linux/mmzone.h | 2 - > include/linux/vmstat.h | 2 +- > include/trace/events/pagemap.h | 11 ++-- > mm/compaction.c | 2 +- > mm/memcontrol.c | 10 +-- > mm/mlock.c | 2 +- > mm/swap.c | 53 ++++++--------- > mm/vmscan.c | 28 +++----- > 10 files changed, 95 insertions(+), 144 deletions(-) > > -- > 2.28.0.681.g6f77f65b4e-goog Sorry, Yu, I may be out-of-line in sending this: but as you know, Alex Shi has a long per-memcg lru_lock series playing in much the same area (particularly conflicting in mm/swap.c and mm/vmscan.c): a patchset that makes useful changes, that I'm very keen to help into mmotm a.s.a.p (but not before I've completed diligence). We've put a lot of effort into its testing, I'm currently reviewing it patch by patch (my general silence indicating that I'm busy on that, but slow as ever): so I'm a bit discouraged to have its stability potentially undermined by conflicting cleanups at this stage. If there's general agreement that your cleanups are safe and welcome (Michal's initial reaction sheds some doubt on that), great: I hope that Andrew can fast-track them into mmotm, then Alex rebase on top of them, and I then re-test and re-review. But if that quick agreement is not forthcoming, may I ask you please to hold back, and resend based on top of Alex's next posting? Thanks, Hugh