Re: [PATCH v2 2/4] mm: rename my_zero_pfn() to zero_pfn()
From: Christophe Leroy (CS GROUP)
Date: Mon Feb 09 2026 - 10:12:50 EST
Le 09/02/2026 à 15:40, Mike Rapoport a écrit :
From: "Mike Rapoport (Microsoft)" <rppt@xxxxxxxxxx>
my_zero_pfn() is a silly name.
Rename zero_pfn variable to zero_page_pfn and my_zero_pfn() function to
zero_pfn().
While on it, move extern declarations of zero_page_pfn outside the
functions that use it and add a comment about what ZERO_PAGE is.
I guess the idea of the extern declaration being inside its only external users was to ensure nobody would start using it directly, hence bypassing the overloadable helpers.
Should we at least call the variable __zero_page_pfn instead in order to warn people that using it directly requires care ?
Christophe