Re: [PATCH v3 3/3] mm: skip KASAN tagging for page-allocated page tables

[Dev Jain <dev.jain@arm.com>]

On 24/04/26 11:11 pm, Catalin Marinas wrote:
> On Fri, Apr 24, 2026 at 06:31:57PM +0530, Dev Jain wrote:
>> From: Muhammad Usama Anjum <usama.anjum@arm.com>
>>
>> Page tables are always accessed via the linear mapping with a match-all
>> tag, so HW-tag KASAN never checks them. For page-allocated tables (PTEs
>> and PGDs etc), avoid the tag setup and poisoning overhead by using
>> __GFP_SKIP_KASAN. SLUB-backed page tables are unchanged for now. (They
>> aren't widely used and require more SLUB related skip logic. Leave it
>> later.)
>>
>> Reviewed-by: Ryan Roberts <ryan.roberts@arm.com>
>> Signed-off-by: Muhammad Usama Anjum <usama.anjum@arm.com>
>> Acked-by: David Hildenbrand (Arm) <david@kernel.org>
>> Reviewed-by: Catalin Marinas <catalin.marinas@arm.com>
> 
> Your signed-off-by is missing. You must add it if you are reposting
> someone else's patches.

Okay I read submitting-patches properly and didn't know we should
add SOB even if we are sending it as it is ...

"Any further SoBs (Signed-off-by:'s) following the author's SoB are from
people handling and transporting the patch, but were not involved in its
development. SoB chains should reflect the **real** route a patch took
as it was propagated to the maintainers and ultimately to Linus, with
the first SoB entry signalling primary authorship of a single author."

So I'll re-order the tags as

Reviewed-by: Ryan Roberts <ryan.roberts@arm.com>
Reviewed-by: Catalin Marinas <catalin.marinas@arm.com>
Acked-by: David Hildenbrand (Arm) <david@kernel.org>
Signed-off-by: Muhammad Usama Anjum <usama.anjum@arm.com>
Signed-off-by: Dev Jain <dev.jain@arm.com>

> 
> Also I only got cc'ed on patch 3. You should normally cc all people on
> all patches. Maybe you could skip this if the patches are some
> independent cleanups but even so, I'd still cc all the others at least
> on the cover letter.

Oops didn't realize my script was not taking a union of the result of
get_maintainer on each patch. I'll fix that.
>