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 1B182CD1296 for ; Wed, 10 Apr 2024 08:53:02 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 78EB16B0089; Wed, 10 Apr 2024 04:53:01 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 73EFE6B008A; Wed, 10 Apr 2024 04:53:01 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 606AB6B008C; Wed, 10 Apr 2024 04:53:01 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 42AFE6B0089 for ; Wed, 10 Apr 2024 04:53:01 -0400 (EDT) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 5965D1406CE for ; Wed, 10 Apr 2024 08:53:00 +0000 (UTC) X-FDA: 81993007320.07.8044EAD Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.223.131]) by imf19.hostedemail.com (Postfix) with ESMTP id 2513C1A0004 for ; Wed, 10 Apr 2024 08:52:57 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=jPGHdLkF; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=MO178eB9; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=jPGHdLkF; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=MO178eB9; spf=pass (imf19.hostedemail.com: domain of osalvador@suse.de designates 195.135.223.131 as permitted sender) smtp.mailfrom=osalvador@suse.de; dmarc=pass (policy=none) header.from=suse.de ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1712739178; 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=a4/3Q4DJ9o/sc+7kC2HhXGX78ya0FrJXHzbJCF/H0r8=; b=keFBGzPumKvJcXDI0M3fm7HYqanFj4HE0XDFMwWkbjQOV3hKfa/CePLE+0u9LE84BjS4j0 /PGbx6OyUQP+oYROlIWT8QgWtDtN2+L9xV9xrgHR10jKQ4nNIGtuRfq8Sbw64H93PsZkzW LO7rEfmwdE4k3FUx4KikmMOGKQ5JX7g= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1712739178; a=rsa-sha256; cv=none; b=6Uud1p69HgZ5Qn+kZGY1Oh0P6rZ9++0Z5BK8QdDJ4YeKZzM8GbJfIKUd/6P/5BiPk5RmGF 6HUyTL3gXzl3SFU70x1scYLitJfBm6mIOiGfeTnKjSBSU/NNcDZvEQSrcuizvCPnYxHijv wuSvQ6DUn/txLxPXf3PI/i3ZVeh8JLA= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=jPGHdLkF; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=MO178eB9; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=jPGHdLkF; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=MO178eB9; spf=pass (imf19.hostedemail.com: domain of osalvador@suse.de designates 195.135.223.131 as permitted sender) smtp.mailfrom=osalvador@suse.de; dmarc=pass (policy=none) header.from=suse.de Received: from imap2.dmz-prg2.suse.org (imap2.dmz-prg2.suse.org [10.150.64.98]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 3281A5C8C1; Wed, 10 Apr 2024 08:52:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1712739176; 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=a4/3Q4DJ9o/sc+7kC2HhXGX78ya0FrJXHzbJCF/H0r8=; b=jPGHdLkFAW9b9GD4dsBdDAaVAWrr3d7nadyMQlBfXlDj6j6x7p6Jx4JSl47UkEV77fYEmy G3hqNvrzH3Yc6UIqJqRee1+zjqhz21bzolVKxhBvBhofJw2IlAcc8nagCFOZT3blHLRSJC nq64WvUf4uVyForMkjt6AlLr7vegrVQ= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1712739176; 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=a4/3Q4DJ9o/sc+7kC2HhXGX78ya0FrJXHzbJCF/H0r8=; b=MO178eB9uPnWzO2/WxY/xl4HlULtieXGKwMjyrsNF5sggJA1ozfvLh6yxatquurgq8zwON 5dx5rShwTg0411DQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1712739176; 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=a4/3Q4DJ9o/sc+7kC2HhXGX78ya0FrJXHzbJCF/H0r8=; b=jPGHdLkFAW9b9GD4dsBdDAaVAWrr3d7nadyMQlBfXlDj6j6x7p6Jx4JSl47UkEV77fYEmy G3hqNvrzH3Yc6UIqJqRee1+zjqhz21bzolVKxhBvBhofJw2IlAcc8nagCFOZT3blHLRSJC nq64WvUf4uVyForMkjt6AlLr7vegrVQ= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1712739176; 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=a4/3Q4DJ9o/sc+7kC2HhXGX78ya0FrJXHzbJCF/H0r8=; b=MO178eB9uPnWzO2/WxY/xl4HlULtieXGKwMjyrsNF5sggJA1ozfvLh6yxatquurgq8zwON 5dx5rShwTg0411DQ== Received: from imap2.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap2.dmz-prg2.suse.org (Postfix) with ESMTPS id CB17E1390D; Wed, 10 Apr 2024 08:52:55 +0000 (UTC) Received: from dovecot-director2.suse.de ([10.150.64.162]) by imap2.dmz-prg2.suse.org with ESMTPSA id RwnqLmdTFmZwVwAAn2gu4w (envelope-from ); Wed, 10 Apr 2024 08:52:55 +0000 Date: Wed, 10 Apr 2024 10:52:54 +0200 From: Oscar Salvador To: Miaohe Lin Cc: akpm@linux-foundation.org, naoya.horiguchi@nec.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] mm/memory-failure: fix deadlock when hugetlb_optimize_vmemmap is enabled Message-ID: References: <20240407085456.2798193-1-linmiaohe@huawei.com> <13aa38af-46a1-3894-32bd-c3eb6ef67359@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <13aa38af-46a1-3894-32bd-c3eb6ef67359@huawei.com> X-Rspamd-Queue-Id: 2513C1A0004 X-Rspam-User: X-Rspamd-Server: rspam11 X-Stat-Signature: zhhuu8fhhgroycdc7d178qzxd1fooed3 X-HE-Tag: 1712739177-869776 X-HE-Meta: U2FsdGVkX1/qgT1G6la3mNsWhp9+5UsiGgcfYrbKHGqZm1ql0PN7nqgSc8yxPmLFMq40MXA3hBsHlDDhBXmxXgkJsBMmhgNPJe1M0/XAa+ZW3msrTENERJwHEOJQuf85Owg0tsRWKZ5TeweOv+6rEtjp0UD6mhGVsolzSsubbm2VTfKjX8IZpmhpRcwy4hBPYSu3lFkMzmkcvoOYlrOcNt4byIk42uBLVo4u6K9/OvsQKRjrTnjsHTVjcnCrbdfYvWKIyWjtKCTpGBE2uBDRC/5IobmDo8v2SuXzU4MpCoXRTa/r+oYYJs+G88TY9s/SiS6xf81sNmOoGPi3rQnzoJ8ICztua2YEJ4QBNeSz2+WED9SXPiTH/0xhMD7fMM7Y+2dj38XzuFLdZdbXvZ9yqNuhnMGqwA7yA/dgqIJOz1RPHf7A6GDEH8jLj+5ZRb37gZrnGIU/+VRcngu6jHlGIlOVnBQJCdGhekY2LwqbXcliPKpm/z7lMvmtVrUUO3r7G9v+dsrM/v41vxF7pG8p/2Fm0wgvaTs8hfXiGuHsYHjsuxwVf2Ru84IPfQWfYbncaSPS0VdkVdyq+Pn+Ug8k8ai+R3WNl2tGaaF8ERrdrLAX31jSiVW30/ZXFaSUJKC19XKgemASmn3ixKfjZVL6R6S1Ce4H1hq9yz4SA8wxMJOqzhNJQGIhGXeIlTJKxtrbFtO4uzNvQca2evqfXJbsw8YUcx5F8+t1JWZzTFgqr+r0NmsYREZiy7H5S1nB39JW6wUZtxei0i/m5deSSyFqgC9hwKk3pfUOore1W+8lby7MvmceIS+/ODEyUk0TkggXMi8tFRajtxcvwlUHi+ILVVq1EdDRkbayCayQzlob96g0jUxRDYouuH4iiSs/N1/hYtbmc/BnAWfU3nywV7J3BRZ/eYOxPYkNFaw93ncgojL2TWyxFeDpKITD4WPAfXm7/aKsIhg5Tlb2sZkn1Yr rDa6bLNX OYxEj8z30ZqOqj5m6FHPTj3/mqE+8epJQt1oR46aApsTqHzUDAoxNDceloiIdsIITttbY3ePMlHfuw6prMh34A03ilnxu6Rf9uEWBgwirYecuNMNdbYUsnR5yRoiw7FPaZtei/PrhyWnGkjM9tsPuPzq3mYWrH2DSKInXFFVOuUhdB6N42712X6/OG9Zt8UYGboyjCbOhgVliswN+Pzmfvchbd6LXLp8Y9bEz 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, Apr 10, 2024 at 03:52:14PM +0800, Miaohe Lin wrote: > AFAICS, iff check_pages_enabled static key is enabled and in hard offline mode, > check_new_pages() will prevent those pages from ending up in a PCP queue again > when refilling PCP list. Because PageHWPoison pages will be taken as 'bad' pages > and skipped when refill PCP list. Yes, but check_pages_enabled static key is only enabled when either CONFIG_DEBUG_PAGEALLOC or CONFIG_DEBUG_VM are set, which means that under most of the systems that protection will not take place. Which takes me to a problem we had in the past where we were handing over hwpoisoned pages from PCP lists happily. Now, with for soft-offline mode, we worked hard to stop doing that because soft-offline is a non-disruptive operation and no one should get killed. hard-offline is another story, but still I think that extending the comment to include the following would be a good idea: "Disabling pcp before dissolving the page was a deterministic approach because we made sure that those pages cannot end up in any PCP list. Draining PCP lists expels those pages to the buddy system, but nothing guarantees that those pages do not get back to a PCP queue if we need to refill those." Just to remind ourselves of the dangers of a non-deterministic approach. Thanks -- Oscar Salvador SUSE Labs