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 54216C3601A for ; Mon, 7 Apr 2025 06:34:34 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4DEFC6B000E; Mon, 7 Apr 2025 02:34:33 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 48F2C6B0010; Mon, 7 Apr 2025 02:34:33 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 332226B0011; Mon, 7 Apr 2025 02:34:33 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 150436B000E for ; Mon, 7 Apr 2025 02:34:33 -0400 (EDT) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 6D08B1A0D59 for ; Mon, 7 Apr 2025 06:34:21 +0000 (UTC) X-FDA: 83306283522.28.C1EF005 Received: from sea.source.kernel.org (sea.source.kernel.org [172.234.252.31]) by imf03.hostedemail.com (Postfix) with ESMTP id BC28720007 for ; Mon, 7 Apr 2025 06:34:19 +0000 (UTC) Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=aI6L7cJ1; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf03.hostedemail.com: domain of rppt@kernel.org designates 172.234.252.31 as permitted sender) smtp.mailfrom=rppt@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1744007659; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=IpJ10cSqV4iX8HuG3VEKOAOvsKabgKSl6G/OtjOv91I=; b=ougxrgSHyiFb/2XuFOgpiruZg165jIDFf5vZvttdOFSHURJemgzLJulzYKnXZGMDcUND6t 0153e6aCubez2cf6O+a+cVzwDWwZ5ukbyUQHd0/xagC/g5H4TAfEYQcFxDdU5lh5JE8HS+ QrOLjXtBq+WLq83bP001uqU6qIUkpKQ= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1744007659; a=rsa-sha256; cv=none; b=GrTHf4ySxWngQTnP53N5Yi8bfcLyBCF5VzViTW0FydanBVtqnVIMviV006yLUlvW3BCIPG RpnvsciqVCK4+A3/fXcaIgt/ooTtR+KPESABpBXYkyPadM8mFnCXDm5t0xJkZOHIpmHvEy OTJG5dzFO5UciwWMju5bA84T/I3Z0EU= ARC-Authentication-Results: i=1; imf03.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=aI6L7cJ1; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf03.hostedemail.com: domain of rppt@kernel.org designates 172.234.252.31 as permitted sender) smtp.mailfrom=rppt@kernel.org Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id CB28F49DF2; Mon, 7 Apr 2025 06:34:17 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DA2BBC4CEDD; Mon, 7 Apr 2025 06:34:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1744007658; bh=mvvCxrEQvIfMjWL83+LMqX14F0AiZ8PBlfYYzUUNTTA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=aI6L7cJ14yFNNE8l1ZbzLG5VrShnHpIbzVWMDtLNG0ixNJkh62DShEjjfZsn3kXAY 1x8grI4qUvggQS+t4BOaQytI47qSWs2oIbqYmZUk8ANs5Q1Su4ZgAFRFAXtbvqDiPT Wu2I9jDPgs5P0emB3+Znl5cNxMfDKzYhj2Tg6juU3Z3IdfDTm5FTfvRqRMIJtEGjwm ji3o37CN9x0XgeZIq+pUDuMg/SgR8HEkQa7nWYgtGbdN4pAdhECk1V1KYVr+8up9Id xk051uAaMcDhK15B2P96lspEQULWm7ILXflJeXj1JYer8zC0pEa1tr+hopzCxiTPwp HH1bc8tDrLoEA== From: Mike Rapoport To: akpm@linux-foundation.org, yajun.deng@linux.dev, Wei Yang Cc: Mike Rapoport , linux-mm@kvack.org Subject: Re: [Patch v2 0/3] memblock: some fix for memmap_init_reserved_pages() Date: Mon, 7 Apr 2025 09:34:06 +0300 Message-ID: <174400754464.63656.4237088728418234476.b4-ty@kernel.org> X-Mailer: git-send-email 2.47.2 In-Reply-To: <20250318071948.23854-1-richard.weiyang@gmail.com> References: <20250318071948.23854-1-richard.weiyang@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: BC28720007 X-Stat-Signature: 74gjhha84c1po3fjaquuneqty4zi8dtn X-Rspam-User: X-HE-Tag: 1744007659-15110 X-HE-Meta: U2FsdGVkX19d/TmpcA/nkp6C/gv4EGli2dFwyOc49nb8aGRreYlbohWAu91t1npG0zJBNh7YLVDZZZ6yON6EE3WsnTY5ckS+DAzAdnI05U2mQzeXWsGEmvPq3MaYrszqqcXL4G2q2gM7dpRCE5St+/qGFVnoisAwP9ks5Ciyjq4tTTxTx+E0geiE2n86xzaAHIfxvn2Edmwq8TZszL9RTkHwSSJLoEjGdf0ZeCXHbyby8XItW4zpP35qP3Al2SJp+ujF6a7YNwVcT5y/GRoUagABPj1S4XPpMNq/CVIPLDWJBfdiOd6ji99sVgfl2ybAimyTU7rm0tVYroOvbabawlmaznR3kaM/ex5ZvqpZ5dExDYxNyEwmHAfG2vhdG+yfO0KwlKvyK//iOu60L6lVSAF8Y8CE7O5kM5wjAzJAOggX4pJ2cx+yyUocyZSGZpRl5l5/1C5kTDtV+vZR1TiEoQ0FRXyh/07oC9M/wpbgGnNkltu+kPVhW8wN6zcMc68X/ClBI+jcjP/SYqSkuT/OsX4a4ENAajjffPB13NJZN4+Dqfbv8y7QdUFsrc3+HKg9BLCSbBrtc1Wu58fQv7nUXEanjHNYYSEIo62JrYxfbEojshtifBT7JA7oAvMDLg8C2cORFlF2P0twOnjLra2pS62xSwkHI1hHhSlWMeh/tCIiPADHaX61BZnW6P+SNmwgTZ0I2c+cna/mXwp++ZODnchmx0lRevYGT4I+OJ/DGx/G40iFag65CgNjZ/6jXBP4k47n6iuolSrnoy93TR96eP3X9COgoka9b7rfYz9z9W/wbn5A5jMdEMtiVI9ud58Cj78FkE9HRcGNNhzA1MS39W2NykduUECtz6+2RwrAxEF76stk4O8rHIAnvO4iLR7RIACXV/UmzS4+Rw7NKxucdBdeDZKCfDiEgSAZvWN9k3OAnay55GBImYG95C0Sv0BfjJZmAD0mhqdiVkyR2DQ 3abdzqKs T+YCBTX4zZ3/WadZZCaaFbGSB1//VfXxQkLp1cggLOx6SLH4X2Ql8qA89kGCxLfmjVPDOFv2DlATRfcXP87JU57Hab/qsBaJXFUUckzLLsX4sr3DS9FFRN9iBM9VxsINCLQfglGHleVZz0CfWp8rslHf3vxuL6VZyCRzGn6nB2Q56RX+tqYAeB3EZmUjIJKSpSIVi1jqSy9y4PRihfFVGP25QrFllX/cHEmGyjxYBNTuiLJWPo1j5AVA+UrdWGwWVEh28LOhPf8UlbjLrx2FGqe5qYByCJmihVFTuQbgIbnEP63CsbBjPAPAlP+uq7HqLqSPjud5tf9K7Dt2Cj1Xa8UwSrnxqY+pTojamaCMXn+He/KQ= 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 Tue, 18 Mar 2025 07:19:45 +0000, Wei Yang wrote: > During code review, I found we may leave some reserved region with invalid nid > when memblock_set_node() double the array. The patch set propose a fix and > test case. > > Also I found memblock_set_node() accept size instead of end as second > parameter. Currently it is misused. > > [...] Applied to for-next branch of memblock.git tree, thanks! [1/3] mm/memblock: pass size instead of end to memblock_set_node() commit: 06eaa824fd239edd1eab2754f29b2d03da313003 [2/3] mm/memblock: repeat setting reserved region nid if array is doubled commit: eac8ea8736ccc09513152d970eb2a42ed78e87e8 [3/3] memblock tests: add test for memblock_set_node commit: 3b394dff15e14550a26b133fc7b556b5b526f6a5 tree: https://git.kernel.org/pub/scm/linux/kernel/git/rppt/memblock branch: for-next -- Sincerely yours, Mike.