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 B3452C83F1B for ; Wed, 16 Jul 2025 07:48:08 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 16FED6B0093; Wed, 16 Jul 2025 03:48:08 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0F9A26B0095; Wed, 16 Jul 2025 03:48:08 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F033B6B0096; Wed, 16 Jul 2025 03:48:07 -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 DCA246B0093 for ; Wed, 16 Jul 2025 03:48:07 -0400 (EDT) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 48B5612BF47 for ; Wed, 16 Jul 2025 07:48:07 +0000 (UTC) X-FDA: 83669349414.11.813DBCD Received: from mout-p-101.mailbox.org (mout-p-101.mailbox.org [80.241.56.151]) by imf08.hostedemail.com (Postfix) with ESMTP id E4023160009 for ; Wed, 16 Jul 2025 07:48:04 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=pankajraghav.com header.s=MBO0001 header.b=1izeZ3gS; spf=pass (imf08.hostedemail.com: domain of kernel@pankajraghav.com designates 80.241.56.151 as permitted sender) smtp.mailfrom=kernel@pankajraghav.com; dmarc=pass (policy=quarantine) header.from=pankajraghav.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1752652085; 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=N1EqZLQw7tah4DL/vfuoIKpuF0J4DZz2EQrgNRTQ+98=; b=ALT5qoTKW3+add7hgOe8wEkSxDJam2OuHg65GIWQzhXd9FaRqm53UNjmJRZiaVUKsFm05p yj7+6XjgG9Dv3hfZmxPHia+sS7N4NctjCdjByjeHTgTuCtWbkK19IJzB+k0YRFJ5lLo1lc I9D0JEBJMFkEUbdle5OkcmiSTH0cUmU= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1752652085; a=rsa-sha256; cv=none; b=bvaZYbwOFZt/eQffo2sAnBXKctlDDmK+kydNY4tkdvUfRbj4N19o1/k2V50jzchlq/5/le 56wHJY2xF6xU9En4NUsyJv0An60N40py1bfP42pqM9cn+0TxA25W8fmvdjzg5I+1x3aXtO 1bd3mHMAJQStlKbOhTdwwLRJIA5OiTg= ARC-Authentication-Results: i=1; imf08.hostedemail.com; dkim=pass header.d=pankajraghav.com header.s=MBO0001 header.b=1izeZ3gS; spf=pass (imf08.hostedemail.com: domain of kernel@pankajraghav.com designates 80.241.56.151 as permitted sender) smtp.mailfrom=kernel@pankajraghav.com; dmarc=pass (policy=quarantine) header.from=pankajraghav.com Received: from smtp2.mailbox.org (smtp2.mailbox.org [10.196.197.2]) (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 mout-p-101.mailbox.org (Postfix) with ESMTPS id 4bhp7h18QTz9st4; Wed, 16 Jul 2025 09:48:00 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pankajraghav.com; s=MBO0001; t=1752652080; h=from:from:reply-to:subject:subject: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=N1EqZLQw7tah4DL/vfuoIKpuF0J4DZz2EQrgNRTQ+98=; b=1izeZ3gS36y6dVm2SXoZQMgV4uu6K4WUpBSOW12sVGpNYwQGQUku64h5dqtCwnmuWO8J5U gze6egm9P63dUUxgFRq1lyFsY4tku2LSw/nT9YrVBi2V29TQt/Wy+So/cM/xCepnOkiWpO rFtLohVOjXS0PdibyNaaS91YXkE2cz4qWaCG30DbcQfNCDqTVmFLWuS8jaajRNnl1BpeRE L5B79psuz5kOZs2RQaRygFGh/BzQPcZknyZrxKRAdA5fmoI6nhPI55QRYayZbedL2Ypg1b YKExuvbtKbqKend7+zQM0gy+XtM9HSaqrytymH5pd59Y7ilCDpdL5Bd0dXef8Q== Date: Wed, 16 Jul 2025 09:47:47 +0200 From: "Pankaj Raghav (Samsung)" To: Lorenzo Stoakes Cc: Suren Baghdasaryan , Ryan Roberts , Baolin Wang , Borislav Petkov , Ingo Molnar , "H . Peter Anvin" , Vlastimil Babka , Zi Yan , Mike Rapoport , Dave Hansen , Michal Hocko , David Hildenbrand , Andrew Morton , Thomas Gleixner , Nico Pache , Dev Jain , "Liam R . Howlett" , Jens Axboe , linux-kernel@vger.kernel.org, willy@infradead.org, linux-mm@kvack.org, x86@kernel.org, linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, "Darrick J . Wong" , mcgrof@kernel.org, gost.dev@samsung.com, hch@lst.de, Pankaj Raghav Subject: Re: [PATCH v2 1/5] mm: move huge_zero_page declaration from huge_mm.h to mm.h Message-ID: References: <20250707142319.319642-1-kernel@pankajraghav.com> <20250707142319.319642-2-kernel@pankajraghav.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspamd-Queue-Id: E4023160009 X-Stat-Signature: howtkppdbzcyiax9w6k4ftfa9rj8snu6 X-Rspam-User: X-Rspamd-Server: rspam07 X-HE-Tag: 1752652084-892901 X-HE-Meta: U2FsdGVkX19h+LlrVBgzcqslJEF1euyLYNwkBStllzAu5P0vPlcxG2xmXIv5JJVibEGuwhKjWQDw/r/g6GK+SU0AuLEMSdeyNTy/IPTa3akPx6bCJqFhfJz80iJbmq6W9iecNpiDCjl6Smz6arZtSfmC6JXTjoZRllfrLrBGd3/dVVNcWd89e1Qd81pJlRaF/ovopM2r+qbwebfdB0lihJa3jFv0ccrsaABrKlxSH5awfs1SuzN/uozFiQMtl9YUnebLnHjZVuqfPfg6RZrhbgFVWHY2js1K4XHdNSqu9J8VgicARNyC57b/dzTzY7pckB31jh/VIbu1uT5ceGO3gkOaG6OuI/Of/knG4CASOKR4n7CiVo3E93TWBFygolXH2h15OgyVFpy+matqSqSATA383khHCmEZDpfTNOkt+z+mICjBfBn8+IRwHKK4delFjlqbBhAcJdeaIwDhE4RSiNXy2a4xqmkY048v5zZx1vfuGHU7kJN/9w0Ae1WpiODae/FWRg96lEDGVy3SLK8wiqwLTy6M//kj1DD+QD0SaxV9u24Q4vMLTv/4GVvA5SERUtmCYddT1iXrHFpZhFqvuv4I2OpduxldKhEX+1rcyyc93L2itk+S5+hvUZnfnfSCjMxX6JzSG4K4h1QXRe6GSrMmRis2Myz+ZlGWEcZCp/56jRLxY7GuvSfOm/wZLUgJ4g0zZ2Ce7IZ5fupc3NM02rHlRR+EzzooC2XF7VDuxMquN3eVcuLSQLn9bi+uLRtZhOAS8cftiJX20GO93mdvTp8mv2cA33LSv5TLSJw7z8mWxmLgI5ImeiJp+fbep5tF4/43bwuUrgEOfNzUZrMrREA6kFR64YZKhgOIPh4sZzrQqm75bUgaoE/76Xldg7ZSL6IEvx5RAxgXwsqKvOJv4p43YfpfXhPmD1+RHQ+Wi0vekK/OTI6J7E45qLiFIJ9S1G+R45aYbj1i9Ne6fDT 2jhu84kk GUqMRB4PwMZmjr8Fmm29z6Cfq9jg+LlvTX304uMbtygpcDes05TLCLEZuBZPIMK5K5ZxJ7bfYyo4kboF8cYcdL5STkx9dK7j9FyrkZyP3biK6cKLbsgwSpXTTSOuhZxmMWtUUNNfd2RBnr0d6xe1SXto/wDI+dSisvInagtmDGhRER5Nh7CSFStGD42GEnw2R57MkyKqdHQykBO6k67yIbY9KE45qeEMUffaSq3sSxy19G4MqnHJrpn6F8muwWuMhfdiMfCba7UBRJR0Uw7/maRjSI3licPHqP1IrWXjem9OWIKm8lsfdM0atFQ== 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, Jul 15, 2025 at 03:08:40PM +0100, Lorenzo Stoakes wrote: > On Mon, Jul 07, 2025 at 04:23:15PM +0200, Pankaj Raghav (Samsung) wrote: > > From: Pankaj Raghav > > > > Move the declaration associated with huge_zero_page from huge_mm.h to > > mm.h. This patch is in preparation for adding static PMD zero page as we > > will be reusing some of the huge_zero_page infrastructure. > > Hmm this is really iffy. > > The whole purpose of huge_mm.h is to handle huge page stuff, and now you're > moving it to a general header... not a fan of this - now we have _some_ > huge stuff in mm.h and some stuff here. > > Yes this might be something we screwed up already, but that's not a recent > to perpetuate mistakes. > > Surely you don't _need_ to do this and this is a question of fixing up > header includes right? > > Or is them some horrible cyclical header issue here? > > Also your commit message doesn't give any reason as to why you _need_ to do > this also. For something like this where you're doing something that at > face value seems to contradict the purpose of these headers, you need to > explain why. > In one of the earlier versions, David asked me to experiment by moving some of these declarations to mm.h and see how it looks. Mainly because, as you guessed it later, we can use it without THP being enabled. But I see that you strongly feel against moving this to mm.h (and I see why). I can move it back to huge_mm.h. Thanks -- Pankaj