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 2668CC47DD9 for ; Wed, 27 Mar 2024 15:58:53 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id AA95B6B009A; Wed, 27 Mar 2024 11:58:52 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A59C16B009F; Wed, 27 Mar 2024 11:58:52 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 948856B00A0; Wed, 27 Mar 2024 11:58:52 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 787496B009A for ; Wed, 27 Mar 2024 11:58:52 -0400 (EDT) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 250AEA018C for ; Wed, 27 Mar 2024 15:58:52 +0000 (UTC) X-FDA: 81943277304.26.0BC41CD Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf16.hostedemail.com (Postfix) with ESMTP id E450A18000C for ; Wed, 27 Mar 2024 15:58:49 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=Gs3Exdbx; dmarc=none; spf=none (imf16.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1711555130; 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=O8YcRfB/DoKgecCwPFQi2t+GXFIKHrKPT+FPBZjiUo0=; b=A5nuSuNQl1oamkDAgZ3JP6GgIPShQvXAk52V2Q3jPVxcE3Udnndz2tZOzu0Q3Krx8i6+C/ e6HDnbdCdbrkv1LeVWVjsecRP3Wpn+nfNP3z6Kz9HA5oWQxyLf0296+XT+NyE/fadWstLC B1kdF2xo629TLL+RkIIl04F/4lXouTE= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=Gs3Exdbx; dmarc=none; spf=none (imf16.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1711555130; a=rsa-sha256; cv=none; b=rwv3Z6Cv5j/OZLXnO9Gyj3Prab9mSP+x3ni4jNdy1jyRAn5JdV/8+BqUYw1Imfg2THvpZz UBur5KWrsDGP/fmNqNOL8KZm3dZrGXzWPx9UEjb/vISVx1V53eL2bp7NIX7dfLLHOJtpOU hfLnx5LIk+y1jN4lfIyJxlV0EttUFNs= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=O8YcRfB/DoKgecCwPFQi2t+GXFIKHrKPT+FPBZjiUo0=; b=Gs3ExdbxglDssmjyTTl7sU8tKt ycLVT1eYch+I5f/R87BVhPLMpnJYjnnd66AM5N8XNakNKopbMjVKvBQ6rzmQm6OujGbAoc/4GAZpX OQYmMjAUVkXYvqnAGDpns3TK5dRR9P1XgecIM+pfPUBPwIWVqzhDNLWOy61FUdrkr7xBw7BvHjpHb 9HxvEzrnZHVmcE5HBcNU0vGbUEOCIr5qczVhHjUnyrKpJaBXhJWn6Y7NOE37BoyRcHSA0blKmcsSM LSXCzyBxtyp73NF2dS0YXRyIgVrj7IO88vDjAGLeAU5BBfzP/Imi4xr0go0hfjKc4DTc0DPlLfEXW pS1+SKqQ==; Received: from willy by casper.infradead.org with local (Exim 4.97.1 #2 (Red Hat Linux)) id 1rpVfe-00000004C6l-3pGJ; Wed, 27 Mar 2024 15:58:34 +0000 Date: Wed, 27 Mar 2024 15:58:34 +0000 From: Matthew Wilcox To: Zi Yan Cc: Kefeng Wang , Andrew Morton , Miaohe Lin , Naoya Horiguchi , David Hildenbrand , Oscar Salvador , Hugh Dickins , Jonathan Corbet , linux-mm@kvack.org, linux-doc@vger.kernel.org, Baolin Wang Subject: Re: [PATCH 2/6] mm: memory_hotplug: use more folio in do_migrate_range() Message-ID: References: <20240327141034.3712697-1-wangkefeng.wang@huawei.com> <20240327141034.3712697-3-wangkefeng.wang@huawei.com> <01BB19DC-3807-4954-A04A-55518301B6BA@nvidia.com> <1AF9E259-6189-4558-8929-598679808C06@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1AF9E259-6189-4558-8929-598679808C06@nvidia.com> X-Rspamd-Queue-Id: E450A18000C X-Rspam-User: X-Rspamd-Server: rspam04 X-Stat-Signature: hn78ay8r3dqisbw1ugy3rpgzcbm1is8d X-HE-Tag: 1711555129-812055 X-HE-Meta: U2FsdGVkX1/kPUsctqhHu6KdzAPIvYTlckHXvmtVEb2k3+3XYQz6E86/c7rOluLQG7RVKSa40XaLvvGDsLQVXQBNX9mvzOCmKm3aYhEgl96RyRj6N9d+3QJMbWbSkwiQsaya6fJCcXsNDrQ9XD30izVfVcqUINaNbdBsKueAF5YtO2kcUG6iaOduDTo8hPpv7N6NxbJdZ5GLN5JSxH5HVAp5qTsrTDtkZZYayMi2DwhQf5I9eTwrzk1e0zUfxnR4QUcM8mxyWyz2sg9w2zhyYDBIjMe/COoFs4IE0NgNkNS3a+WJqULOAThj8mrWMKdjUm6NBSBeRePBawEwpUSC1k13vBigMwH1ZPUcGlid2vOyqJCUhfwCOx2UknekXTam5R4naVXQ3YuGWq4A1ak99BlQ7ZnOnKTE7si/wupNJ7dbN9QRJ9muRDvD2pX5NmlbVzYlAadZPF1cVY0b+AA32CyVnqKGbe1u+EEvCynSI6IHkq6FKYHXYLjdhWBvYg68LB09lbZ3ScAqPZXWEdLnKGM/ydmFDzDdIj0Yeay6LL+U4ZaKIxGjUoKNh2FDlC5MEPo5VZ+honEONGzfHUA14ux9XhY2RdnaMswkO7+mUzAVD8uMqdeyZ0RcrXceWlSDc6PKJxog1wHyrMiOmG88nh76MT8LhoNPChcevC5FPtrDA/0ig7r+64vQHknTdc8En+hFLph7uMN//rwQQk3mJetYHsSDMk991HU8uD3rvNvRFergpD8SP1YEqiAxypiy4y0Y68izHL88TcC1cmCEzpiQUHsIqiKke+ZB7nsCrA2mENXZySbp/KN0VqNk169WQbxT+hDsAcpC9hEcphQG5sxlV4qp2W5jcqw4YDooF6ID+8ffBUt8im825gS9te9flRiGnURMdFtE/F0litNXpzh0DhvVxI9Gfu/OpL4Wj1B6Pdjj7cGP4fQtrBUe4cqoBEIzAyobv2dKX8hIChK ixGfvRz6 5Ffqoymaap2IouoK2JbSafVMIhRQrIltuqFFPt9aMdOsC3S3nY8xm5ZBbpQHejSgQurAdu5nyvNs3ipDDtjA2mmbN+AXNqJCZ78oHPi5XcxO283Crw9ex1VOUoWwtJkcBq7zU/Q3SmbNK6uoG4vV9pAsPpGrRjZaJWzkzNSyrVHb8QXcumiGuz74pzQ== 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, Mar 27, 2024 at 11:10:48AM -0400, Zi Yan wrote: > On 27 Mar 2024, at 10:54, Matthew Wilcox wrote: > > How much of this is safe without a refcount on the folio? > > folio_to_pfn() should be fine, isolate_hugetlb() checks the folio > under hugetlb_lock, but folio_nr_pages() might return a bogus number > that makes pfn go beyond end_pfn and ends for loop early. The code > below increases the refcount, so it might be better to move this > part of the code after refcount is increased. I really want to instill a little bit of fear in Kefeng. This is all really subtle code because it's running without a refcount. I've mostly stayed away from it because I know that I don't know what I'm doing. Kefeng has no idea that he doesn't know what he's doing. And so we get these patches, and they're sometimes subtly wrong, and it takes a lot of arguing and revision and thinking and Kefeng is doing very little of the thinking part! Kefeng, please stick to working on code that you understand. Or take the time to learn code you don't understand before sending patches to it. This kind of Leeroy Jenkins approach to development is not good.