Received-SPF: pass (google.com: domain of linux-kernel+bounces-209015-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
From: Stephen Brennan <stephen.s.brennan@oracle.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-mm@kvack.org, Vlastimil Babka <vbabka@suse.cz>,
        "Vishal Moola
 (Oracle)" <vishal.moola@gmail.com>,
        Omar Sandoval <osandov@osandov.com>,
        David Hildenbrand <david@redhat.com>,
        "Matthew Wilcox (Oracle)"
 <willy@infradead.org>,
        Hao Ge <gehao@kylinos.cn>, linux-debuggers@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4] mm: convert page type macros to enum
In-Reply-To: <20240607212738.bf55318aebd7172fadaa11c5@linux-foundation.org>
References: <20240607202954.1198180-1-stephen.s.brennan@oracle.com>
 <20240607212738.bf55318aebd7172fadaa11c5@linux-foundation.org>
Date: Mon, 10 Jun 2024 15:41:32 -0700
Message-ID: <87ikygo1yb.fsf@oracle.com>
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: 
	=?us-ascii?Q?JgkY5l4Y1cZ4f6l4xGlqZAAYmQmN8SrHq6LlkFtipIW2axYV4zuRgburPoFM?=
 =?us-ascii?Q?lR+RgByH9RvRZylCXuS8PNek+SOw/U47z72CCpz5mIsRiqBq1YG+pNI3DmXp?=
 =?us-ascii?Q?AdcLeg1MLq59s7Xj2j3qWIADobQnKhUGvS4Wv9uSDrdiJ8W3Dh5lD23PMo8Z?=
 =?us-ascii?Q?xY0frCTYJpGb42CmavLDRG0C9JKmHbbTJCLVd/pInFlNdq2Sch1WxIy+bEs9?=
 =?us-ascii?Q?JGbiV2Q4UodCI7vAK5LfDsbUUy92YH0cGObGW+ZdhbDwPs2NLO1q9Bkp15Mg?=
 =?us-ascii?Q?axPHBdznufbYqF1mfWDyYA4vYVEKnF1UsDGQTPWMDXj+RtAA1l1eqSc+Fy9i?=
 =?us-ascii?Q?kA56DUtwu7MIubQLkLyeewJMcMMVXnsaqgwMWzDKYWNvh+mSfXStC/ztJx9m?=
 =?us-ascii?Q?bmrC5hLJ1cj7n4NS9vRl9YHxQESSAAwax581zLb6gAgGEP0gYwZ2uMgDz2BB?=
 =?us-ascii?Q?WwJspV5Fya2uvNGm+45ll6+UD+jnnFMaegekh5Z46SSlnSV3AC3szgZH+4Vg?=
 =?us-ascii?Q?v7n6zDSM/usCuWNSPQPOPwRgPzBCbgBbmBu/gFy8JxgMZVg+N/XRDxpM9pom?=
 =?us-ascii?Q?A1P9Hy9YtYznnl60piUiGubAWw77CVGCO6gKYBU1C6G+4o7spWh2ajkGviko?=
 =?us-ascii?Q?X5Wgeqg0sGw6vjx4LIyoLwN0O7iyiZY536jM0Oe+i8aNj4v20wbDH2pYiy5w?=
 =?us-ascii?Q?+Fk8Uu3arr/EoL4NCgGOqMp8t7MBijGok5HKhkkJrTTnZmmRB1x2x+pSgMpq?=
 =?us-ascii?Q?DVf5vXltM9IzYf+ZF4QW+fI39qPGvyiq2kRPn4Y3xxz1Mp/MOBgj5JJXnqhF?=
 =?us-ascii?Q?6QUAqhOJxUkObeJNog8uM7mABanRnTNU2ggCwnnP/9sNNb86wUVDMwVNSi7u?=
 =?us-ascii?Q?VMGaW4qrG/LbJxaEV3cmANfJa7RwiFYe5U+MlL89x5ZJZoje3wNBBu8lJCbN?=
 =?us-ascii?Q?h0YzoutEdbt/hrMkMgLA2h63C3y9Vu0DIucWsvxwfKXttOQobqqKhEbnBPqt?=
 =?us-ascii?Q?ZmlJhqSezdiZHgkkxmCWoL5nezXmGDKZAIn2OvK1sN6SCLJipsbqFLVGZD7I?=
 =?us-ascii?Q?2Rt3CIhMuufHzPenlIztGpEQz+/DO1/NaBWNs2RbQfMqGBw/ojd7qspIpjoX?=
 =?us-ascii?Q?D8eN6AvMNa/6mxkXTHnxJIT5T/p0Gs+RwX7zSteb2zpD5bn8AD9ZhEpJZFi2?=
 =?us-ascii?Q?5PAHdpNKs4fbC0sg0Pcj5x26hBlkO7TuMZVkLlM0uIRgxgKFsCr7cY4wzTnP?=
 =?us-ascii?Q?lOZkvU2HhyEc6LM/agxEpZJHNzsea/deyerKKJw6YpwirempnIAFZiHnuBd+?=
 =?us-ascii?Q?LXu8nw+lSkW+NLM/9nAswUzSFTzha1QQUtGDN8+P8uB0mfXapqkgPcwhclky?=
 =?us-ascii?Q?FLqQ7Iyu4Waxzn/STZKewkryIamG5EYB04itoeqh1O/Y83UbimYo7A7rQ5Dw?=
 =?us-ascii?Q?3pdG3HtFl85yjtNAPGTqJiP4IQ+8r9A/1sKFPn5qKmDp4A+tE4jwkOTHNpYu?=
 =?us-ascii?Q?v8PdMUpnWKJeMMm0PRtF+T6Ab7YtFB3eBHcBtaXylWVGml02HGM4jNnvuKkZ?=
 =?us-ascii?Q?3IcQWAomTMYkKPJ+iebPKc1Kry4AuzjIIOnXNlH3SrnloEimDwmG8PRG2fu/?=
 =?us-ascii?Q?rw=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	KzDvMUUUZ5amdNKYi1/QWviHKgVoyAAbXx/kgWd/3X9Qkf3IzSJ00ISnxIwr1Upl4xisw9HZuBNh/IyymBZioaQ/rLeAVN+jl4FibAi/HPQh/9t/NM+hNUpnqAN7F1wmMvRaDcja+N0ho+H33FbA+ZorNsNCVZk0H4ull6ezJStUWxhPx2yQLAIE41o7YhTzLnHWVUuMWnTO0A92Ae6bfGMyBUZK80hPOl3TbuMWyN0SkNWIbnd3pHG9YCmDisepJL5KO+TcAXmdwyqbMZ0Kxn3B1PRZBe5O6LOR/c8J5suUkR+pjFKhWq4EUhVTYYenXCvwcHVem5q7uwVsAgE5oFYGl+FpbkGb3+JpKa6pt+iroPTc0+WAn5fCM30S7l1c9EPPZxFqkmhgopTOWexMI1Pen+qOexLifNbFrM0o4cqnDqYjF3ybzqum90Bna6X7NQBQn9Hu/FpJBVISHgW0Py+WYuPvdyjUWR+7xdLUAVxcs6fP1itqjANd3dIiR8XAcwhNUWidZ7JU69o7ui+TsrrCtr21LrGmm20H71RrrRuhhkhEdG9uApmzF3q/X3SD6xZMYHXwBxqM3Qn+WcuKgbEgejBQgEqQWv9SzZma2GI=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fc600c35-824f-4c66-0166-08dc899e7b3f
X-MS-Exchange-CrossTenant-AuthSource: PH8PR10MB6597.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jun 2024 22:41:34.6895
 (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: KQJ81QGSJcXgP+QjgiqmyAGhxnsb4nRm8rTAz04EdPnnuqovilIdy16DZJ5V3cyaVadmge3/DRnOU4PPB4ZOqjpittNn2+sfQRQo5lOLy6c=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR10MB6275
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.680,FMLib:17.12.28.16
 definitions=2024-06-10_06,2024-06-10_01,2024-05-17_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 malwarescore=0
 mlxlogscore=999 bulkscore=0 suspectscore=0 spamscore=0 mlxscore=0
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2405010000 definitions=main-2406100167
X-Proofpoint-GUID: 9H37eLw5_pT2BayIYAJQkzEosy-rXPBG
X-Proofpoint-ORIG-GUID: 9H37eLw5_pT2BayIYAJQkzEosy-rXPBG

Andrew Morton <akpm@linux-foundation.org> writes:

> On Fri,  7 Jun 2024 13:29:53 -0700 Stephen Brennan <stephen.s.brennan@oracle.com> wrote:
>
>> Changing PG_slab from a page flag to a page type in commit 46df8e73a4a3
>> ("mm: free up PG_slab") in has the unintended consequence of removing
>> the PG_slab constant from kernel debuginfo. The commit does add the
>> value to the vmcoreinfo note, which allows debuggers to find the value
>> without hardcoding it. However it's most flexible to continue
>> representing the constant with an enum. To that end, convert the page
>> type fields into an enum. Debuggers will now be able to detect that
>> PG_slab's type has changed from enum pageflags to enum pagetype.
>> 
>> Fixes: 46df8e73a4a3 ("mm: free up PG_slab")
>
> Should we backport this into 6.9.x?

Hi Andrew,

Looks like commit 46df8e73a4a3 ("mm: free up PG_slab") is introduced in
the v6.10-rc's, and not backported to 6.9. So PG_slab is still part of
enum pageflags in 6.9. From the perspective of the issue which motivated
this patch, there's no reason to backport.

Backporting could make the other enum pagetype constants available in
6.9, but I'm not sure there are any users who would care for that. I'd
say there's no need.

Thanks,
Stephen