linux-kernel - Re: [PATCH v3 1/3] riscv: remove irqflags.h inclusion in asm/bitops.h

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: <bb259f36-6a5a-1bb8-75de-9bb62ade44a8@kernel.org>
Date: Wed, 7 Jan 2026 15:34:48 -0700 (MST)
From: Paul Walmsley <pjw@...nel.org>
To: Yunhui Cui <cuiyunhui@...edance.com>
cc: aou@...s.berkeley.edu, alex@...ti.fr, andybnac@...il.com, 
    apatel@...tanamicro.com, ast@...nel.org, ben.dooks@...ethink.co.uk, 
    bjorn@...nel.org, bpf@...r.kernel.org, charlie@...osinc.com, cl@...two.org, 
    conor.dooley@...rochip.com, cyrilbur@...storrent.com, daniel@...earbox.net, 
    debug@...osinc.com, dennis@...nel.org, eddyz87@...il.com, 
    haoluo@...gle.com, john.fastabend@...il.com, jolsa@...nel.org, 
    kpsingh@...nel.org, linux-kernel@...r.kernel.org, linux-mm@...ck.org, 
    linux-riscv@...ts.infradead.org, linux@...musvillemoes.dk, 
    martin.lau@...ux.dev, palmer@...belt.com, pjw@...nel.org, 
    puranjay@...nel.org, pulehui@...wei.com, ruanjinjie@...wei.com, 
    rkrcmar@...tanamicro.com, samuel.holland@...ive.com, sdf@...ichev.me, 
    song@...nel.org, tglx@...utronix.de, tj@...nel.org, thuth@...hat.com, 
    yonghong.song@...ux.dev, yury.norov@...il.com, zong.li@...ive.com
Subject: Re: [PATCH v3 1/3] riscv: remove irqflags.h inclusion in
 asm/bitops.h

On Tue, 16 Dec 2025, Yunhui Cui wrote:

> The arch/riscv/include/asm/bitops.h does not functionally require
> including /linux/irqflags.h. Additionally, adding
> arch/riscv/include/asm/percpu.h causes a circular inclusion:
> kernel/bounds.c
> ->include/linux/log2.h
> ->include/linux/bitops.h
> ->arch/riscv/include/asm/bitops.h
> ->include/linux/irqflags.h
> ->include/linux/find.h
> ->return val ? __ffs(val) : size;
> ->arch/riscv/include/asm/bitops.h
> 
> The compilation log is as follows:
> CC      kernel/bounds.s
> In file included from ./include/linux/bitmap.h:11,
>                from ./include/linux/cpumask.h:12,
>                from ./arch/riscv/include/asm/processor.h:55,
>                from ./arch/riscv/include/asm/thread_info.h:42,
>                from ./include/linux/thread_info.h:60,
>                from ./include/asm-generic/preempt.h:5,
>                from ./arch/riscv/include/generated/asm/preempt.h:1,
>                from ./include/linux/preempt.h:79,
>                from ./arch/riscv/include/asm/percpu.h:8,
>                from ./include/linux/irqflags.h:19,
>                from ./arch/riscv/include/asm/bitops.h:14,
>                from ./include/linux/bitops.h:68,
>                from ./include/linux/log2.h:12,
>                from kernel/bounds.c:13:
> ./include/linux/find.h: In function 'find_next_bit':
> ./include/linux/find.h:66:30: error: implicit declaration of function '__ffs' [-Wimplicit-function-declaration]
>    66 |                 return val ? __ffs(val) : size;
>       |                              ^~~~~
> 
> Signed-off-by: Yunhui Cui <cuiyunhui@...edance.com>

Thanks, queued this patch for v6.19-rc.


- Paul