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 C6F9CC61D9B for ; Wed, 22 Nov 2023 13:56:29 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 30B496B060D; Wed, 22 Nov 2023 08:56:29 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2BBA76B060F; Wed, 22 Nov 2023 08:56:29 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 184CC6B0610; Wed, 22 Nov 2023 08:56:29 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 00EE26B060D for ; Wed, 22 Nov 2023 08:56:28 -0500 (EST) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 9E6EF1A0BF5 for ; Wed, 22 Nov 2023 13:56:28 +0000 (UTC) X-FDA: 81485740056.12.A04E59E Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by imf20.hostedemail.com (Postfix) with ESMTP id 735AB1C0021 for ; Wed, 22 Nov 2023 13:56:26 +0000 (UTC) Authentication-Results: imf20.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=a8PFBE4b; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf20.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1700661386; 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=4PL01vdipAxXI2et46ovWIE1C2j61g1DsIeVKK4hMJ8=; b=IKy2wChe2j/sqAwC0lhKSI9cKoQDNOXDzwOQ9YWMV7WmiQwYqoniMgEEEInnoUAjejrCrz 3Pe6KzekFy3td5l0S/dpTomaJjddOPPD+vAndMtNcQfA/ZIDWUTu2pJR0iSTLDiuJIhWrE Z5uf9sNyBPfZc2SNVdmUuIhQ2Qgn5oM= ARC-Authentication-Results: i=1; imf20.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=a8PFBE4b; dmarc=pass (policy=quarantine) header.from=suse.com; spf=pass (imf20.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1700661386; a=rsa-sha256; cv=none; b=BAqiVoxQG4sGk4/KZTvro8g2wUw1fIHKSDbpO5t+sUP/9D0agYZhHP2P1tah9riT5/pshd ECGXRoHm1utUN32dWvazktTelB1ONJAH931XWaI1wtnkRCZbi2Vz17pINRKPXU9l/WwKxl y1mJRC+4myXr/DzBAfd90EmU5F4QK9g= 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-out1.suse.de (Postfix) with ESMTPS id 7A1AE21979; Wed, 22 Nov 2023 13:56:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1700661384; 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=4PL01vdipAxXI2et46ovWIE1C2j61g1DsIeVKK4hMJ8=; b=a8PFBE4bG4ry/xym88djaUEBZuMT6DkQChwAEsfDOWoIXWMauOOabCr1VcT2jZ/XMqyOrE 7fj98k6G520QLOvWtK0Dqur9JjfkfdgEXvKy4shV9GKRKT6ig2M2/f0SbhW4x0HUSmQryG wFVgLZoKlK6bFq7HzJJdpMEMVqVQLPk= 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 524FF13467; Wed, 22 Nov 2023 13:56:24 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id qv2DEIgIXmWndwAAMHmgww (envelope-from ); Wed, 22 Nov 2023 13:56:24 +0000 Date: Wed, 22 Nov 2023 14:56:23 +0100 From: Michal Hocko To: Marcelo Tosatti Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, Vlastimil Babka , Andrew Morton , David Hildenbrand , Peter Xu Subject: Re: [patch 0/2] mm: too_many_isolated can stall due to out of sync VM counters Message-ID: References: <20231113233420.446465795@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspamd-Queue-Id: 735AB1C0021 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: 3mueyrdxs8deemixucjmk4cmcx5o8e3s X-HE-Tag: 1700661386-676133 X-HE-Meta: U2FsdGVkX1//yKkNCZnNxuc9FHjcjaBoqkBtXTAxZVxHRJ4pH09QCeYF/E7CMmxh9fD/eKTbTrEx+3mjzNb0enABZUBhtiHQB1DWSpQHwlrv/Y8BtyeT8fq5j/GC1E9AtlNKwI21nerzEJGUvgpERPHg3uSaOYTWwfB8TJrgIRz2u4mgMAfEwfOwDpoES5jUtX1yo8dgmUxOCHRGiVhk+yZdLX4pX3p0IoQpong4aAQOYalRTinVJJ2MjRWsXDWWePYXiDWTwof5thMYzmQOJNhENfjbHl7xHncV8lOHWxTIDe8vnYjsFleTtd5IcXqRzY9Yewrui/wuBvMaBjt6gaHKeRAlSy5K0GZJnmVYC5VQ7Edi7RZedYaqSH9jWy54Z6aRcUPEzR7YKdlWUj/EEaH2caB/b/fico6aKNznNlbnbBH2jP81uTEkWad0RynBWQL4+951WBXWEPb1BvVt8ZoEP3OvnchevamkQ4feoAORHdY8OGfVvafQqL509sJuynRemD9lMMvVc6DCD8yKp5hKy2R4Q63E5ZKj9HErtuoNWFMcJPxaVnJy5BynRzvvkq8jtQyvbU6w2+D8d3P2LTtPgenKCULFu31m6iVpxJ/jyXW5VLM+jpsraJRbMufIqwn9vnqjFGtSvzAWDoSyCq5C9uHIdJ5w8f6dEcPwqW6qewlG+3rjSoSaiUKz4U6djeoAuBz1Hkgu78kqqlw1vSqFk9tsHahbfJDKjhqE0RO+kVWUop8onaIxFUNPpqyvjLsl6JSvOoRlRMcyIIC3f5phQkzkkAU6TwizEQVrl8NgpctrdGD41Oq+UqUJ0DJIfaTqYYpUtu8UJt1aS/WH+PBVMe0Q8fPgdkNjUX7rO10M62mYgMB9kPRJUY/ScT2ddwJk/+ehZC6qyAWAI4mt95sMxBw0MHHF521pcVwpsb8LsNiWqkqPswohb93yiwMHHmCcEw85O22Y0//FIDF TTXK+Cz+ OWDj3LpZ/nMbxo4oDjppBo40cTMbkszvER97HFxBoLpV2JrSmMRRc2xojYydVbKP1cY8HBxLiS9XY2M/GMxc3EYRsNAonZcauE21nU+zMWGAGnXgDn3/w7UIm/rzEm1WF6P+H4q3+R+QnfZvNAfuLfRG69IMEsdKYHbhMHQ1xW/lN0SZzVH3SIxdbZ83NqyDd42uVfV2MESqyKRRnulbNg+3HRBNYzV9kHApBSNf0b7k3ihE= 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: List-Subscribe: List-Unsubscribe: On Wed 22-11-23 08:26:02, Marcelo Tosatti wrote: [...] > Michal, > > Let me know if you have any objections to the patch, thanks. I do not think you have exaplained how the patch helps nor you have shown it has fixed the described problem. You seem to be very focused on the specific snapshot which I do agree shows that the data is out of sync and that there is throttling happening when strictly speaking it should noti. But (let me repeat) those discrepancies are so small that it is very likely that concurrent reclaimers will be stalled (just take one to isolate those pages) anyway. Maybe this leads to an earlier OOM killer invocation as untrottled reclaimers will be able to conclude there is no progress rather than being throttled on the direct reclaim. That being said I am not saying the patch is incorrect. Nevertheless, I do not think we want to merge this patch without a better understanding what is going on in your specific case and what kind of runtime difference does the patch make in that case. From your previous email it seems like the actual case is mostly memory stress test that manages to fill out the memory to push almost all the file LRU while anon LRU is not reclaimable for some reason. That shouldn't be terribly hard to reproduce. -- Michal Hocko SUSE Labs