From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Weiner Subject: Re: [PATCH v4 3/4] mm: account nr_isolated_xxx in [isolate|putback]_lru_page Date: Thu, 11 Jul 2019 14:07:27 -0400 Message-ID: <20190711180727.GC20341@cmpxchg.org> References: <20190711012528.176050-1-minchan@kernel.org> <20190711012528.176050-4-minchan@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190711012528.176050-4-minchan@kernel.org> Sender: linux-kernel-owner@vger.kernel.org To: Minchan Kim Cc: Andrew Morton , linux-mm , LKML , linux-api@vger.kernel.org, Michal Hocko , Tim Murray , Joel Fernandes , Suren Baghdasaryan , Daniel Colascione , Shakeel Butt , Sonny Rao , oleksandr@redhat.com, hdanton@sina.com, lizeb@google.com, Dave Hansen , "Kirill A . Shutemov" List-Id: linux-api@vger.kernel.org On Thu, Jul 11, 2019 at 10:25:27AM +0900, Minchan Kim wrote: > The isolate counting is pecpu counter so it would be not huge gain > to work them by batch. Rather than complicating to make them batch, > let's make it more stright-foward via adding the counting logic > into [isolate|putback]_lru_page API. > > * v1 > * fix accounting bug - Hillf > > Link: http://lkml.kernel.org/r/20190531165927.GA20067@cmpxchg.org > Acked-by: Michal Hocko > Suggested-by: Johannes Weiner > Signed-off-by: Minchan Kim This is tricky to review, but fwiw it looks correct to me. Acked-by: Johannes Weiner