From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BC057C433EF for ; Mon, 29 Nov 2021 08:41:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1348627AbhK2Iol (ORCPT ); Mon, 29 Nov 2021 03:44:41 -0500 Received: from smtp-out1.suse.de ([195.135.220.28]:39288 "EHLO smtp-out1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1350040AbhK2Imh (ORCPT ); Mon, 29 Nov 2021 03:42:37 -0500 Received: from relay2.suse.de (relay2.suse.de [149.44.160.134]) by smtp-out1.suse.de (Postfix) with ESMTP id B6ECA212CB; Mon, 29 Nov 2021 08:39:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1638175157; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=y1/HRvQWe5bupilVHXWVZDusBm8nr9LZt0uqBedKdKA=; b=J4qijL9j7NKJC0s4vnqOlrDs2BSFPsMfOKHgq75fzXUbUMmG6kDFsHC446/t8l9HnhxQGi 1X48f8fJWJ9sQfbIUtxwtWBvay0Vz0aMdUE83pfyyc4cYhcZsz7fW82PhBT10czrrivQxi qE3pMx7MyDOyufnfhAhM+rP1LoUdXcM= Received: from suse.cz (unknown [10.100.201.86]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by relay2.suse.de (Postfix) with ESMTPS id 8242FA3B83; Mon, 29 Nov 2021 08:39:17 +0000 (UTC) Date: Mon, 29 Nov 2021 09:39:16 +0100 From: Michal Hocko To: Hao Lee Cc: Matthew Wilcox , Linux MM , Johannes Weiner , vdavydov.dev@gmail.com, Shakeel Butt , cgroups@vger.kernel.org, LKML Subject: Re: [PATCH] mm: reduce spinlock contention in release_pages() Message-ID: References: <20211125080238.GA7356@haolee.io> <20211125123133.GA7758@haolee.io> <20211126162623.GA10277@haolee.io> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20211126162623.GA10277@haolee.io> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri 26-11-21 16:26:23, Hao Lee wrote: [...] > I will try Matthew's idea to use semaphore or mutex to limit the number of BE > jobs that are in the exiting path. This sounds like a feasible approach for > our scenario... I am not really sure this is something that would be acceptable. Your problem is resource partitioning. Papering that over by a lock is not the right way to go. Besides that you will likely hit a hard question on how many tasks to allow to run concurrently. Whatever the value some workload will very likely going to suffer. We cannot assume admin to chose the right value because there is no clear answer for that. Not to mention other potential problems - e.g. even more priority inversions etc. -- Michal Hocko SUSE Labs