linux-kernel - Re: [PATCH] mips: fix compilation error

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: <aK2f3C9bT-cvPR87@casper.infradead.org>
Date: Tue, 26 Aug 2025 12:51:56 +0100
From: Matthew Wilcox <willy@...radead.org>
To: Andrew Morton <akpm@...ux-foundation.org>,
	Nicola Vetrini <nicola.vetrini@...il.com>
Cc: linux-kernel@...r.kernel.org, linux-mips@...r.kernel.org,
	regressions@...ts.linux.dev, tsbogend@...ha.franken.de,
	anders.roxell@...aro.org, naresh.kamboju@...aro.org
Subject: Re: [PATCH] mips: fix compilation error

On Mon, Aug 25, 2025 at 11:42:45PM +0200, Nicola Vetrini wrote:
> The following build error occurs on a mips build configuration
> (32r2el_defconfig and similar ones)
> ./arch/mips/include/asm/cacheflush.h:42:34: error: passing argument 2 of ‘set_bit’
> from incompatible pointer type [-Werror=incompatible-pointer-types]
>    42 |         set_bit(PG_dcache_dirty, &(folio)->flags)
>       |                                  ^~~~~~~~~~~~~~~
>       |                                  |
>       |                                  memdesc_flags_t *
> 
> This is due to changes introduced by
> commit 30f45bf18d55 ("mm: introduce memdesc_flags_t"), which did not update
> these usage sites.
> 
> Link: https://lore.kernel.org/lkml/CA+G9fYvkpmqGr6wjBNHY=dRp71PLCoi2341JxOudi60yqaeUdg@mail.gmail.com/
> Fixes: 30f45bf18d55 ("mm: introduce memdesc_flags_t")
> Signed-off-by: Nicola Vetrini <nicola.vetrini@...il.com>
> ---
> First time sending a Linux patch, so I may have gotten some part of
> it wrong.

Well, Naresh was the one who got it wrong BY NOT CCing THE AUTHOR
OF THE PATCH THAT IT WAS BISECTED TO.  Honestly, Naresh, do better.
syzbot manages this, and it's a shell script.

Anyway, this can just be fixed up as the patch is still in mm-new.
Andrew, please apply.

> Bisection was done at the provided link.
> ---
>  arch/mips/include/asm/cacheflush.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/mips/include/asm/cacheflush.h b/arch/mips/include/asm/cacheflush.h
> index 1f14132b3fc9..5d283ef89d90 100644
> --- a/arch/mips/include/asm/cacheflush.h
> +++ b/arch/mips/include/asm/cacheflush.h
> @@ -37,11 +37,11 @@
>  #define PG_dcache_dirty			PG_arch_1
>  
>  #define folio_test_dcache_dirty(folio)		\
> -	test_bit(PG_dcache_dirty, &(folio)->flags)
> +	test_bit(PG_dcache_dirty, &(folio)->flags.f)
>  #define folio_set_dcache_dirty(folio)	\
> -	set_bit(PG_dcache_dirty, &(folio)->flags)
> +	set_bit(PG_dcache_dirty, &(folio)->flags.f)
>  #define folio_clear_dcache_dirty(folio)	\
> -	clear_bit(PG_dcache_dirty, &(folio)->flags)
> +	clear_bit(PG_dcache_dirty, &(folio)->flags.f)
>  
>  extern void (*flush_cache_all)(void);
>  extern void (*__flush_cache_all)(void);
> 
> base-commit: 6c68f4c0a147c025ae0b25fab688c7c47964a02f
> -- 
> 2.43.0
>