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 021F4C83F1A for ; Fri, 11 Jul 2025 05:35:02 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 9865C6B0093; Fri, 11 Jul 2025 01:35:02 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 95C8A6B0095; Fri, 11 Jul 2025 01:35:02 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 899306B0098; Fri, 11 Jul 2025 01:35:02 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 7989E6B0093 for ; Fri, 11 Jul 2025 01:35:02 -0400 (EDT) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id F19DE161FD8 for ; Fri, 11 Jul 2025 05:35:01 +0000 (UTC) X-FDA: 83650870002.04.EC852C8 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.19]) by imf04.hostedemail.com (Postfix) with ESMTP id 0DD4140006 for ; Fri, 11 Jul 2025 05:34:58 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=lMVO5yoL; spf=none (imf04.hostedemail.com: domain of andriy.shevchenko@linux.intel.com has no SPF policy when checking 198.175.65.19) smtp.mailfrom=andriy.shevchenko@linux.intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1752212099; 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=x1sKRFqJqMEAo918KZaEs9YDl9YncshqMg5v07qiP2g=; b=XepyFaw2pmbcfwxGtNUlRjSSiylyYEDPxXbCu293M0Mrhk3usZc8ol7tg+JPCr3lR26+YX fOhF+A1sRdOjGTDOzVkBM/yh/R3WyTpsvLOb6Z3Lw39276vJwGYC0YoC1pOf9pEFLVz+Db sxeo0mvhEfe3DZnj9EmgBrqRAPEe78w= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=lMVO5yoL; spf=none (imf04.hostedemail.com: domain of andriy.shevchenko@linux.intel.com has no SPF policy when checking 198.175.65.19) smtp.mailfrom=andriy.shevchenko@linux.intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1752212099; a=rsa-sha256; cv=none; b=cRQdbofSHJk/GARshOpFOpFAHVkHDwUKV5bUz8ejd282tiNzKUGpHlwgsDlZApveRwOHp5 WcCmZYbqZmhfKSWhVsBxc7nuSymPpWrcm49eG/74Fvot/kKDMfTzyLyDT7YxHyQUFOk0Cw avUy8MyHwOKF8D0IYewFI0c5hoh1Jnw= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1752212099; x=1783748099; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=Rkkzr1JNQrtqP/eOaaEnhMbpJc2uhM/lz5zcM3e8+QY=; b=lMVO5yoLf469lF1GLRsqSWEy+C0RFy1iqzBKHHOIpxWtxpQTvzqQ5DeS tb4sy+DRr/sBKxwv8OvTThVtNPTvAMBaqvoIzsc/guEk1BcxDeLFgpPLs aO9vGhZibWBZJo6XXTrh38cC4d7stYclm5wcEM+IBW/oyrVonA2rl0+by qnYMa3nUf/kmOC4Vb+AtKZ35zL3SW9AKe5ERzLGCTsmXIQPHfOahL6BPd eQkBnVTgXmMWtua8C4FwniZSAxMmkOLuLwSQY24s45WfrZVbmRy06T4C3 QHIF5RfjRBTxT0VpQHNDjNn0tcJtNFqoN7kMklurHSxFEBNBYoXwJ34GA w==; X-CSE-ConnectionGUID: KJsb4cAfR3SGUdYc0iASfQ== X-CSE-MsgGUID: 7jYz8i2yTki+dQYcfRR/Dw== X-IronPort-AV: E=McAfee;i="6800,10657,11490"; a="54359796" X-IronPort-AV: E=Sophos;i="6.16,302,1744095600"; d="scan'208";a="54359796" Received: from orviesa010.jf.intel.com ([10.64.159.150]) by orvoesa111.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Jul 2025 22:34:58 -0700 X-CSE-ConnectionGUID: xYIRpqcUTqaPjOqFVZetgw== X-CSE-MsgGUID: sksYXizAT/uZGtntblvMhw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.16,302,1744095600"; d="scan'208";a="155693108" Received: from smile.fi.intel.com ([10.237.72.52]) by orviesa010.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Jul 2025 22:34:55 -0700 Received: from andy by smile.fi.intel.com with local (Exim 4.98.2) (envelope-from ) id 1ua6PL-0000000EQT6-2CVV; Fri, 11 Jul 2025 08:34:51 +0300 Date: Fri, 11 Jul 2025 08:34:51 +0300 From: Andy Shevchenko To: Alistair Popple Cc: Leon Romanovsky , linux-mm@kvack.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev, =?iso-8859-1?B?Suly9G1l?= Glisse , Andrew Morton , Nathan Chancellor , Nick Desaulniers , Bill Wendling , Justin Stitt Subject: Re: [PATCH v2 1/1] mm/hmm: Move pmd_to_hmm_pfn_flags() to the respective #ifdeffery Message-ID: References: <20250710082403.664093-1-andriy.shevchenko@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo X-Rspam-User: X-Stat-Signature: 7sheipnm5f88abhprfjumbr7xpr399wi X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 0DD4140006 X-HE-Tag: 1752212098-506850 X-HE-Meta: U2FsdGVkX19VWVsBXYpMjHqox1yWlNRqdz9luFz66YcqskR9La6+X0/cKFl6JkGN1B89V+BFX+k4cfRBQClkPB1rq35Z4pm2qJzeKSYjYZhkhTGq10fLOxRzyYSKg+zgHBgaFhmRNJI3Fmqptapn8gfxnhgfMfFhlfgiPxLAns3KIcyvcC+77oSAUOK1k8wBbt9NvkGIaUwhFwmRgYqKkuUU3np64b/ZjXKxzFDKHxU2N2pqjbkczf41y49iFR+Y7qPU2yE+/FxKsGOVlfBIwCFPsMeksDMLuVQgLAmuaC/WQ/5tDMgCF0jFg3Ab6Zm5WE4aIw89Gbb2fQJ/n/ByPqsVCXT0cTnkyq0RPH3mM4OuAqGFRooQi2zUZ5/VV9BxrREAgTLGE3PYR189LFhJr7Unmi2PGFnh+nRYIJHNQQ1vOl7Yqrw8XPOY3ODXbEbElVrLyX+kF6AdwcdxKQXQ/TCGjXzVs1wlB7iFi+kRovG39nIqB5UinqTQagqGH/+xhMnSYcMnWW7uXvHc8rX04q5u7rmiilO0+ObLK/8T8CYUblPjWv2QuM0PkRARMeXBW131jvmZyq6mhJ41tAv+j9hNLWxICwXkoIuKFerw94AVzvewGWU5hTpEJDRPrx0ZY/JeVTfccvZ9/lM0LccI69xh52sLOsxCgwkwL/X70YeobuLiPhjVjlw0X16SNlBmUesp+5kKU3sg6Q1fPobAEj+gMzWzv3RWYZ5U1wBsjJ+cs7QJBBtJoT54H0PDGBJOnxfP06T3ZgGqlgkiDNBiJhGBzaGMMXSsr0D59lAzGJJnSlwla51XvH5949j+AyTQ+DH6KcRO7I9KRYx55AY8Ai/xgLboht4Ba93g8f7B862pnO4dZy5zBSIKxbyx8W0B2c11inkp2yygdfhv00nYFJPQUs7Uyy+RN66LZPK2AHqkeIzAT1UjakkgsDjAT09/KzAatAr5cztIOyi4mMO N/0QdruO r6oVda/xHffkANylzHoWRwWZxfiEc1jHOqp4e/4GqXE0oIb+YzSPRhtrcYUoMmNs1coPfX95mhMhD1rW8cL5nYhH7vXZHAGNn20PkxvGn7IQCu0FEu2RXnHoL5UbvAtEKAT8Ww+7oZEWwNUuo23HY7lJZpLyaZvbNbuRk34FI3EnTvv+lZW4ctKSL/6WwFPM0N/AcYGsu1JryqheDlyYKtEiUIOSk/36xG9PdsacdmrtmcvhP5MdQryed+1NfA1xP2FjUMtKb4/2xrd67N5jHeNpvzjVbm3/nPCRcA1IeV0ut/GU/9pRGwALIQ1Y3i9le4XTFyWA5NFXNXWo= 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 Fri, Jul 11, 2025 at 09:35:49AM +1000, Alistair Popple wrote: > Shouldn't the Fixes tag be for the commit that introduced the #ifdeffery in the > first place? Ie: 992de9a8b751 ("mm/hmm: allow to mirror vma of a file on a DAX > backed filesystem") Looks indeed better, I was lost in the maze of the history of mm/hmm.c changes. Fixes: 992de9a8b751 ("mm/hmm: allow to mirror vma of a file on a DAX backed filesystem") The problem is that backporting to that, if needed, requires a bit of adaptation. Nevertheless, I added it above, and we actually can have two Fixes tags to make backporting easier for those (we don't have LTS kernels older that the second commit anyway), Andrew, can you add the above tag as well? > As far as I can tell that is what would have introduced the warning. Other than > that it looks good though so feel free to add: > > Reviewed-by: Alistair Popple Thank you for the review! -- With Best Regards, Andy Shevchenko