| Message ID | 20240816101912.1049329-1-zhangchunyan@iscas.ac.cn (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | riscv: Remove unused GENERATING_ASM_OFFSETS | expand |
Hi Chunyan, On 16/08/2024 12:19, Chunyan Zhang wrote: > The macro is not used in the current version of kernel, it looks like > can be removed to avoid a build warning: > > ../arch/riscv/kernel/asm-offsets.c: At top level: > ../arch/riscv/kernel/asm-offsets.c:7: warning: macro "GENERATING_ASM_OFFSETS" is not used [-Wunused-macros] > 7 | #define GENERATING_ASM_OFFSETS So this warning never shows up in a "normal" kernel compilation, but it happens when using W=2 (with quite a lot of other warnings). This unused define indeed seems completely useless, I guess not all unused defines fall into this category and we should be careful when removing them, but for this one it makes sense so: Reviewed-by: Alexandre Ghiti <alexghiti@rivosinc.com> Tested-by: Alexandre Ghiti <alexghiti@rivosinc.com> Thanks, Alex > > Fixes: 9639a44394b9 ("RISC-V: Provide a cleaner raw_smp_processor_id()") > Signed-off-by: Chunyan Zhang <zhangchunyan@iscas.ac.cn> > --- > arch/riscv/kernel/asm-offsets.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/arch/riscv/kernel/asm-offsets.c b/arch/riscv/kernel/asm-offsets.c > index b09ca5f944f7..cb09f0c4f62c 100644 > --- a/arch/riscv/kernel/asm-offsets.c > +++ b/arch/riscv/kernel/asm-offsets.c > @@ -4,8 +4,6 @@ > * Copyright (C) 2017 SiFive > */ > > -#define GENERATING_ASM_OFFSETS > - > #include <linux/kbuild.h> > #include <linux/mm.h> > #include <linux/sched.h>
On Fri, 16 Aug 2024 at 20:30, Alexandre Ghiti <alex@ghiti.fr> wrote: > > Hi Chunyan, > > On 16/08/2024 12:19, Chunyan Zhang wrote: > > The macro is not used in the current version of kernel, it looks like > > can be removed to avoid a build warning: > > > > ../arch/riscv/kernel/asm-offsets.c: At top level: > > ../arch/riscv/kernel/asm-offsets.c:7: warning: macro "GENERATING_ASM_OFFSETS" is not used [-Wunused-macros] > > 7 | #define GENERATING_ASM_OFFSETS > > So this warning never shows up in a "normal" kernel compilation, but it > happens when using W=2 (with quite a lot of other warnings). > > This unused define indeed seems completely useless, I guess not all > unused defines fall into this category and we should be careful when > removing them, but for this one it makes sense so: > > Reviewed-by: Alexandre Ghiti <alexghiti@rivosinc.com> > Tested-by: Alexandre Ghiti <alexghiti@rivosinc.com> I think this should be ready to be merged. Palmer, would you please pick this up to your tree? Or I can resend the patch if that's more convenient for you. Thanks, Chunyan > > Thanks, > > Alex > > > > > > Fixes: 9639a44394b9 ("RISC-V: Provide a cleaner raw_smp_processor_id()") > > Signed-off-by: Chunyan Zhang <zhangchunyan@iscas.ac.cn> > > --- > > arch/riscv/kernel/asm-offsets.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/arch/riscv/kernel/asm-offsets.c b/arch/riscv/kernel/asm-offsets.c > > index b09ca5f944f7..cb09f0c4f62c 100644 > > --- a/arch/riscv/kernel/asm-offsets.c > > +++ b/arch/riscv/kernel/asm-offsets.c > > @@ -4,8 +4,6 @@ > > * Copyright (C) 2017 SiFive > > */ > > > > -#define GENERATING_ASM_OFFSETS > > - > > #include <linux/kbuild.h> > > #include <linux/mm.h> > > #include <linux/sched.h>
diff --git a/arch/riscv/kernel/asm-offsets.c b/arch/riscv/kernel/asm-offsets.c index b09ca5f944f7..cb09f0c4f62c 100644 --- a/arch/riscv/kernel/asm-offsets.c +++ b/arch/riscv/kernel/asm-offsets.c @@ -4,8 +4,6 @@ * Copyright (C) 2017 SiFive */ -#define GENERATING_ASM_OFFSETS - #include <linux/kbuild.h> #include <linux/mm.h> #include <linux/sched.h>
The macro is not used in the current version of kernel, it looks like can be removed to avoid a build warning: ../arch/riscv/kernel/asm-offsets.c: At top level: ../arch/riscv/kernel/asm-offsets.c:7: warning: macro "GENERATING_ASM_OFFSETS" is not used [-Wunused-macros] 7 | #define GENERATING_ASM_OFFSETS Fixes: 9639a44394b9 ("RISC-V: Provide a cleaner raw_smp_processor_id()") Signed-off-by: Chunyan Zhang <zhangchunyan@iscas.ac.cn> --- arch/riscv/kernel/asm-offsets.c | 2 -- 1 file changed, 2 deletions(-)