[PATCH 4/6] mm: reintroduce vma_flags_test() as a singular flag test
Pedro Falcato
pfalcato at suse.de
Thu Mar 26 01:57:22 AEDT 2026
On Thu, Mar 05, 2026 at 10:50:17AM +0000, Lorenzo Stoakes (Oracle) wrote:
> Since we've now renamed vma_flags_test() to vma_flags_test_any() to be very
> clear as to what we are in fact testing, we now have the opportunity to
> bring vma_flags_test() back, but for explicitly testing a single VMA flag.
>
> This is useful, as often flag tests are against a single flag, and
> vma_flags_test_any(flags, VMA_READ_BIT) reads oddly and potentially causes
> confusion.
>
> We use sparse to enforce that users won't accidentally pass vm_flags_t to
> this function without it being flagged so this should make it harder to get
> this wrong.
>
> Of course, passing vma_flags_t to the function is impossible, as it is a
> struct.
>
> Also update the VMA tests to reflect this change.
>
> Signed-off-by: Lorenzo Stoakes (Oracle) <ljs at kernel.org>
Reviewed-by: Pedro Falcato <pfalcato at suse.de>
This is a lot nicer, though I am wondering if there is any difference in
codegen as well...
--
Pedro
More information about the Linux-erofs
mailing list