From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf0-f70.google.com (mail-lf0-f70.google.com [209.85.215.70]) by kanga.kvack.org (Postfix) with ESMTP id C1C4D6B0270 for ; Wed, 4 Jul 2018 10:44:26 -0400 (EDT) Received: by mail-lf0-f70.google.com with SMTP id g82-v6so1331540lfg.4 for ; Wed, 04 Jul 2018 07:44:26 -0700 (PDT) Received: from mail-sor-f65.google.com (mail-sor-f65.google.com. [209.85.220.65]) by mx.google.com with SMTPS id h7-v6sor1027670lfl.100.2018.07.04.07.44.24 for (Google Transport Security); Wed, 04 Jul 2018 07:44:25 -0700 (PDT) Date: Wed, 4 Jul 2018 17:44:22 +0300 From: Vladimir Davydov Subject: Re: Message-ID: <20180704144422.geugzjycujftwwd7@esperanza> References: <20180624200907.ufjxk6l2biz6xcm2@esperanza> <20180703145235.28050-1-bigeasy@linutronix.de> <20180703141429.c752e3342426b9f8d48ef255@linux-foundation.org> <20180703214429.tntoxzb66zikhukc@linutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180703214429.tntoxzb66zikhukc@linutronix.de> Sender: owner-linux-mm@kvack.org List-ID: To: Sebastian Andrzej Siewior Cc: Andrew Morton , linux-mm@kvack.org, tglx@linutronix.de, Kirill Tkhai On Tue, Jul 03, 2018 at 11:44:29PM +0200, Sebastian Andrzej Siewior wrote: > On 2018-07-03 14:14:29 [-0700], Andrew Morton wrote: > > > > > Reply-To: "[PATCH 0/4] mm/list_lru": add.list_lru_shrink_walk_irq@mail.linuxfoundation.org.and.use.it () > > > > Well that's messed up. > > indeed it is. This should get into Subject: > > > On Tue, 3 Jul 2018 16:52:31 +0200 Sebastian Andrzej Siewior wrote: > > > > > My intepretation of situtation is that Vladimir Davydon is fine patch #1 > > > and #2 of the series [0] but dislikes the irq argument and struct > > > member. It has been suggested to use list_lru_shrink_walk_irq() instead > > > the approach I went on in "mm: list_lru: Add lock_irq member to > > > __list_lru_init()". > > > > > > This series is based on the former two patches and introduces > > > list_lru_shrink_walk_irq() (and makes the third patch of series > > > obsolete). > > > In patch 1-3 I tried a tiny cleanup so the different locking > > > (spin_lock() vs spin_lock_irq()) is simply lifted to the caller of the > > > function. > > > > > > [0] The patch > > > mm: workingset: remove local_irq_disable() from count_shadow_nodes() > > > and > > > mm: workingset: make shadow_lru_isolate() use locking suffix > > > > > > > This isn't a very informative [0/n] changelog. Some overall summary of > > the patchset's objective, behaviour, use cases, testing results, etc. > > The patches should be threaded as a reply to 3/3 of the series so I > assumed it was enough. And while Vladimir complained about 2/3 and 3/3 > the discussion went on in 2/3 where he suggested to go on with the _irq > function. And testing, well with and without RT the function was invoked > as part of swapping (allocating memory until OOM) without complains. > > > I'm seeing significant conflicts with Kirill's "Improve shrink_slab() > > scalability (old complexity was O(n^2), new is O(n))" series, which I > > merged eight milliseconds ago. Kirill's patchset is large but fairly > > straightforward so I expect it's good for 4.18. So I suggest we leave > > things a week or more then please take a look at redoing this patchset > > on top of that work? > > If Vladimir is okay with to redo and nobody else complains then I could > rebase these four patches on top of your tree next week. IMHO this approach is more straightforward than the one with the per list_lru flag. For all patches, Reviewed-by: Vladimir Davydov