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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4C214C4332F for ; Wed, 30 Nov 2022 12:50:44 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id C35526B0074; Wed, 30 Nov 2022 07:50:43 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id BBE506B0075; Wed, 30 Nov 2022 07:50:43 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A5E8E6B0078; Wed, 30 Nov 2022 07:50:43 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 93FB46B0074 for ; Wed, 30 Nov 2022 07:50:43 -0500 (EST) Received: from smtpin22.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 4FC1D12036D for ; Wed, 30 Nov 2022 12:50:43 +0000 (UTC) X-FDA: 80190092766.22.A3CB1C3 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by imf14.hostedemail.com (Postfix) with ESMTP id C4FA210000D for ; Wed, 30 Nov 2022 12:50:41 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 515121F8D4; Wed, 30 Nov 2022 12:50:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1669812640; 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=8ZGYbHjMMJD2KtxpoYl71o8+mthDszQIMORViCtN+H4=; b=TEYdT2ejsqxumwVeGAtrs/wb5t8Z7WykQ86E9LmitKLafwNZ3z4nIFifyvnAoyM2lSntZk 4ETEFoMHzkEz3lOxQ/oZ49G7lqHhgEx9C8HKAAVubxMqpZ9B4BG6KeTAM+reO31e5xywk0 aevyUeC9LoRpjosj/FGvwSFKdixPZHk= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 346E21331F; Wed, 30 Nov 2022 12:50:40 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id qHV9CqBRh2O3SgAAMHmgww (envelope-from ); Wed, 30 Nov 2022 12:50:40 +0000 Date: Wed, 30 Nov 2022 13:50:39 +0100 From: Michal Hocko To: Johannes Weiner Cc: Hugh Dickins , Andrew Morton , Linus Torvalds , Shakeel Butt , Stephen Rothwell , linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] mm: remove lock_page_memcg() from rmap Message-ID: References: <20221123181838.1373440-1-hannes@cmpxchg.org> <16dd09c-bb6c-6058-2b3-7559b5aefe9@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1669812642; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=8ZGYbHjMMJD2KtxpoYl71o8+mthDszQIMORViCtN+H4=; b=Zw2ZtU/JSeDL326XSbaTBCweiArbBpVMpbZCQXqyJ8RuI2h3Isw1SLM1yewj5sUQXL+ndd PcQnZAyrYks0hYNzrqxTzKQQpWcUz5RvN/LSckmRMK5ihZd7kmTdC1K6jNxe+XGuauSR9K /fJdamO+djGQ8qH9RwuXTSUvFlfMeU8= ARC-Authentication-Results: i=1; imf14.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=TEYdT2ej; spf=pass (imf14.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.29 as permitted sender) smtp.mailfrom=mhocko@suse.com; dmarc=pass (policy=quarantine) header.from=suse.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1669812642; a=rsa-sha256; cv=none; b=O4vs5REOOGKyvQQbkaX8Xzxj9BtfJS2bjMqaAl8OvHYa9zgK18q7Zf5xI/lYTZzmZuHotF hCZ0XPqHS4ZH7xXgXLVc5Sh5dvqyGrvhVVuQngFw8rkJvybX2/6zpQq7Wo0JWKilczTlBl 7IS5pldFFCKq+TB8llYH2lKUwqO39wo= X-Stat-Signature: x6g9qp15qwgzocgige5tkhbynkzbxhb9 X-Rspam-User: X-Rspamd-Queue-Id: C4FA210000D X-Rspamd-Server: rspam11 Authentication-Results: imf14.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=TEYdT2ej; spf=pass (imf14.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.29 as permitted sender) smtp.mailfrom=mhocko@suse.com; dmarc=pass (policy=quarantine) header.from=suse.com X-HE-Tag: 1669812641-955941 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Tue 29-11-22 14:08:34, Johannes Weiner wrote: [...] > Short of further restricting the pages that can be moved, I don't see > how we can get rid of the cgroup locks in rmap after all. :( :( > We can try limiting move candidates to present ptes. But maybe it's > indeed time to deprecate the legacy charge moving altogether, and get > rid of the entire complication. > > Hugh, Shakeel, Michal, what do you think? I do agree with Hugh that going part way will likely not solve the general maintenance burden. I am OK with dropping the functionality but we should at least add a big fat warning if anybody wants to enable memory.move_charge_at_immigrate. If there are any workload which absolutely depend on the feature (which would make them impossible to migrate to v2 btw) then we want to hear about that and address in some way. An incremental deprecation by limiting to present ptes sounds like this semantic change might get overlooked for an extended time. -- Michal Hocko SUSE Labs