Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sat, 7 Jan 2023 12:55:37 -0800
From:   Mike Kravetz <mike.kravetz@oracle.com>
To:     Matthew Wilcox <willy@infradead.org>
Cc:     Sidhartha Kumar <sidhartha.kumar@oracle.com>,
        linux-kernel@vger.kernel.org, linux-mm@kvack.org,
        akpm@linux-foundation.org, songmuchun@bytedance.com,
        tsahu@linux.ibm.com, jhubbard@nvidia.com
Subject: Re: [PATCH mm-unstable 8/8] mm/hugetlb: convert
 demote_free_huge_page to folios
Message-ID: <Y7ncSXYzCmjzmS87@monkey>
References: <20230103191340.116536-1-sidhartha.kumar@oracle.com>
 <20230103191340.116536-9-sidhartha.kumar@oracle.com>
 <Y7jGyH9Atv3sPjcZ@monkey>
 <Y7jLVJ9kvBFCUhNd@casper.infradead.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Y7jLVJ9kvBFCUhNd@casper.infradead.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?wXgGNbKE6mQCQYnQOEILFUt7642zo616Y9flV9nKO4DwraM+jeBSZsIPXXLx?=
 =?us-ascii?Q?kDeX/aY2mlok4l5ZRJ9am7vRqiwNLR9M7j8o+eMkJj1U7m0onwur9ztkt8aU?=
 =?us-ascii?Q?U6g3gDVlUsQYiWpmCMmPzvW/m+EPZ2CQ2c/P6bEk+ZvkTJ+N3meqkuzpxv9h?=
 =?us-ascii?Q?xC1uRnbObMIHv7aSW4lxUxQ1fuhZd8Y3AHuyiaYlEADjNJONQcLq4hF4L8kh?=
 =?us-ascii?Q?lRMGNFSzFAcy2DQE77mIcbP0TViyomTd1sp6WTKptmh6leksQeyuPuhNlRyg?=
 =?us-ascii?Q?r18R4h+4egV7Xf0i6e12P0AuCVJZ1AXAGxmHA8DzfP57GIWKz73XiGDZi6qa?=
 =?us-ascii?Q?pHRQ8g1tBCfrfJZwN4MQiZHdHMsJbPmq9caeSSJrZN16x3eR33Kc7ncdTiCW?=
 =?us-ascii?Q?Eh0eHxsuswz09porBQGAi/L3oN5A106O99CMnc4rE2N/cWCaY0LIcrU42x1v?=
 =?us-ascii?Q?6N0CQseFVXX4oi60RDHkDOf2rC5w/PVoH0vA1D1fNVTNaCC0BKuAWptnIAbc?=
 =?us-ascii?Q?lun4qdR3rNW1g7/tkS35NauqrlAcwCRfdKLBw40vFzdPpnOUhwy2g8SYJlHU?=
 =?us-ascii?Q?pEj2qMQ0sG68tZjZw7idDtnC+5vAB+LxZyybKJXciQAdLRQ0NMDKjAPuzd0m?=
 =?us-ascii?Q?PFeVl+EMuqokWL8HjhIVLDw8uGD7ZoOp8QLhqbnvaslpkvCUNJWvCpH1+Ec+?=
 =?us-ascii?Q?UWEPT2BNoh2ushcIhnWWVHmoXw11qV/UfF0rqUoO9Rpo1hR32L0zJVabhnXz?=
 =?us-ascii?Q?QGLUFZdWHVfnBHZtTO5ROIhWQwAojaTiSiUb8krwDdTn2K7fybADUcJUzchj?=
 =?us-ascii?Q?baoqiOmQPeXd0b3gbK9OQCuuF1MszkPpcuaGq2iNnZ65akzI7FlB57CJGnr5?=
 =?us-ascii?Q?iQayS8bhoG/dDWp3LXgqDsvCJcfVPB4GZEoRLzfv9kWhNyibmp7Sit3aPyi3?=
 =?us-ascii?Q?R+0rxTsrvZeSdcdJiGfIUmXzKfep1iZzpZ0aIVuN2hPb9tSeC3jGQBVufQeT?=
 =?us-ascii?Q?bkpavYcseVpgLRVL3xvPNKfbwkgcwFB884zuxxQjrbkz3rH1ksoMYaDA+q3r?=
 =?us-ascii?Q?GB9nOKErYpdBGt5eA43/HsiQoqeFKSeirG4/oiX30bcrClCeOES4bhacDZgr?=
 =?us-ascii?Q?5oczL7ZO+QStW+pn4wmaImIRVzKrCdtASuHeSucLw2do/kdFW4LESqb/4HZy?=
 =?us-ascii?Q?r70ZZdQ0b+rpLXiTc2fh6IAxUllpC3lLkL12ul3fe8lyJ+kGN+ymqrgo4G3M?=
 =?us-ascii?Q?TlBuXlNgDCQcPborjmHUbx7AlDtoQp+TKdSHgkbzN1CTsNrE0V5wouFZWvYB?=
 =?us-ascii?Q?jnJOUCo6Z76Zn/xf668/w+emdcwDALsZuwy3P3gkc69TVacRqe9UQgAoe5Sn?=
 =?us-ascii?Q?ouIrghd/FRdmid5N1sfMYNMkAX9CBTjJ28bX7PseI1NtGanxuds4U3bJaOPD?=
 =?us-ascii?Q?i1C0sqkkh6HR/2Z0CGsjifQdNoN1UfFNyG1fus+4eZK7xhojz9FgDHO+5pi5?=
 =?us-ascii?Q?YZSwCcENnnZYiqksnCGERI7IGztQBUi5+CmnU52BbdGfUvukqi0oMPOjqyxI?=
 =?us-ascii?Q?3fTkCgg8a4YHEky0DFCR0xMgTBj0xE9mYn+C8jj1Zst5B8PENcYwAsH59ohu?=
 =?us-ascii?Q?Og=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?UTFud54HcnqOJP+Sro368Dxx7jjDjt/ntR4YtUwJWUcx11Rpf2NuIKJwX/fa?=
 =?us-ascii?Q?3cFQcxjni70D2vYL/vBBZMW5DRpFqInjRyJxP/mArqia2c0QFI8yltgVL0rA?=
 =?us-ascii?Q?W4SdwbK5xPqTRzYaHSwOYN1+jWS3DvP0xNWh/6MQUMI2qLi1CtttPgHPutcA?=
 =?us-ascii?Q?8sGo3St1/XZ8i8au4USR8IuM+k5n8TpRhzYIhU5NR7to1HyerAPP1cAqSs5W?=
 =?us-ascii?Q?IETiXksYC5t6+DbttyvPAPhvN+SEu3Rl/ZltV9/OcKxKZQJquiTQOSbeSL5y?=
 =?us-ascii?Q?KBhf4YCX/EgcJRdCgSB3efxgg1CQTSvlgTdMqCsAI4Yhu6duaABktYTMrdM4?=
 =?us-ascii?Q?z2IQKEl20e7Rg2roxNpxaIWSTpN3b1HwP9/tq7s4AB8gcyuMKme2wU0P0ido?=
 =?us-ascii?Q?6YkbzRal/8d4y88rT4UbV2144bJXYRYvOybZosBrn9mv02jYDSB1/qbcpsA2?=
 =?us-ascii?Q?CHr4oM2xoQgFiPzLNEz93q6v6TEER7Wn+rOJWtfnvoZEawgXiHI7HDiGn8aa?=
 =?us-ascii?Q?V1wcQxZTmlryqG+5W0W3Ve7gfMR8PSXaiMiCZPeCtLua3sN1C2xixF02NzZq?=
 =?us-ascii?Q?3LtG2cewO3KLovmNc9HKablyKRW3voZz+XYGO1thb9IIu06dxDy0mnvClaks?=
 =?us-ascii?Q?JdS0mODk31WGSxfNSkYgfgWRESQm4wH8l1a/CDhHSNkK+P0BEBL1Q+EbJ/3U?=
 =?us-ascii?Q?/swQ7KcAtynzPrGcgYdZiew4CbSP0ty2mvUGkpmcIIlEhiApidyem7weVe6N?=
 =?us-ascii?Q?0nItsMwToDZ7GJYX6XU74mzRJsFoM+AFmJi6Q5xH90zPrD6RX/nn8yszVz3X?=
 =?us-ascii?Q?mXBB7UuSueZqfaqlCe0x5bq2SmLXZ1G5FT3Pij9HTPOU7u5O4Ys9QsFY29ym?=
 =?us-ascii?Q?s1f3q+RYIVqHiC/sfmhfGkjHNpOS7rxztzz9wDL89AokqVg16Z424oTwZuX4?=
 =?us-ascii?Q?JLnriryxHBid+VVr9n8W6GSFONTnGHHWaunwuweXi5k=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: de450557-f5a3-402a-01e2-08daf0f188fb
X-MS-Exchange-CrossTenant-AuthSource: BY5PR10MB4196.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Jan 2023 20:55:40.6969
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 5xUQIMz9+RqwpdjdTybIQFkH/8mdrUF5/5y4fFbMcd9wREbsc7QZQay5P1HUCjNSB//bnKf/8SuFfAI1mmOR6g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR10MB4972
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.923,Hydra:6.0.545,FMLib:17.11.122.1
 definitions=2023-01-07_11,2023-01-06_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 mlxlogscore=999
 suspectscore=0 adultscore=0 phishscore=0 mlxscore=0 malwarescore=0
 spamscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2212070000 definitions=main-2301070163
X-Proofpoint-ORIG-GUID: 9m9N5idqEi1EyABeWPndsCpgtdThyA3K
X-Proofpoint-GUID: 9m9N5idqEi1EyABeWPndsCpgtdThyA3K
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 01/07/23 01:31, Matthew Wilcox wrote:
> On Fri, Jan 06, 2023 at 05:11:36PM -0800, Mike Kravetz wrote:
> > On 01/03/23 13:13, Sidhartha Kumar wrote:
> > > @@ -3477,15 +3477,15 @@ static int demote_free_huge_page(struct hstate *h, struct page *page)
> > >  	mutex_lock(&target_hstate->resize_lock);
> > >  	for (i = 0; i < pages_per_huge_page(h);
> > >  				i += pages_per_huge_page(target_hstate)) {
> > > -		subpage = nth_page(page, i);
> > > -		folio = page_folio(subpage);
> > > +		subpage = folio_page(folio, i);
> > > +		subfolio = page_folio(subpage);
> > 
> > No problems with the code, but I am not in love with the name subfolio.
> > I know it is patterned after 'subpage'.  For better or worse, the term
> > subpage is used throughout the kernel.  This would be the first usage of
> > the term 'subfolio'.
> > 
> > Matthew do you have any comments on the naming?  It is local to hugetlb,
> > but I would hate to see use of the term subfolio based on its introduction
> > here.
> 
> I'm really not a fan of it either.  I intended to dive into this patch
> and understand the function it's modifying, in the hopes of suggesting
> a better name and/or method.

At a high level, this routine is splitting a very large folio (1G for
example) into multiple large folios of a smaller size (512 2M folios for
example).  The loop is iterating through the very large folio at
increments of the smaller large folio.  subfolio (previously subpage) is
used to point to the smaller large folio within the loop.

-- 
Mike Kravetz