linux-kernel - Re: [PATCH v3 1/1] mm/rmap: inline folio_test_large_maybe_mapped

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <b3a951fe-577b-46de-a7c0-8e3a33a77913@redhat.com>
Date: Thu, 24 Apr 2025 22:28:01 +0200
From: David Hildenbrand <david@...hat.com>
To: Lance Yang <ioworker0@...il.com>, akpm@...ux-foundation.org
Cc: mingzhe.yang@...com, linux-mm@...ck.org, linux-kernel@...r.kernel.org,
 Lance Yang <lance.yang@...ux.dev>
Subject: Re: [PATCH v3 1/1] mm/rmap: inline
 folio_test_large_maybe_mapped_shared() into callers

On 24.04.25 17:56, Lance Yang wrote:
> From: Lance Yang <lance.yang@...ux.dev>
> 
> To prevent the function from being used when CONFIG_MM_ID is disabled, we
> intend to inline it into its few callers, which also would help maintain
> the expected code placement.
> 
> Suggested-by: David Hildenbrand <david@...hat.com>
> Signed-off-by: Lance Yang <lance.yang@...ux.dev>
> ---

Yeah, this way it's harder to abuse ... having the test functions is a 
leftover from when I had set/clear functions during development and 
experimented with using a pageflag (which involves atomics even on tail 
pages unfortunately).

Acked-by: David Hildenbrand <david@...hat.com>

-- 
Cheers,

David / dhildenb