On 2/4/23 09:40, Masahiro Yamada wrote: > On Sun, Feb 5, 2023 at 2:30 AM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote: >> >> Hi, >> >> I'm seeing thousands of these warnings: >> (gcc 12.2.0) >> >> >> ./include/generated/autoksyms.h:7:9: warning: ISO C99 requires whitespace after the macro name >> 7 | #define __KSYM_TOC. 1 >> | ^~~~~~~~~~ >> >> In file included from ../include/asm-generic/export.h:57, >> from ./arch/powerpc/include/generated/asm/export.h:1, >> from ../arch/powerpc/kernel/misc.S:17: >> ./include/generated/autoksyms.h:7:9: warning: missing whitespace after the macro name >> 7 | #define __KSYM_TOC. 1 >> | ^~~~~~~~~~ >> >> >> Can anything be done about them? >> >> Thanks. >> -- >> ~Randy > > > > Hmm, I just thought this issue was fixed by > commit 29500f15b54b63ad0ea60b58e85144262bd24df2 > > > > Does this happen only for GCC 12? > > Does the following patch fix the issue? > (I did not test it.) > > I have been running build tests all day (have probably done around 500 builds) and I'm not seeing the build problem now. I don't know why not or what changed. I have not applied the patch below. I'm just trying to reproduce the problem and cannot do so. (Bad me: I had 3 build failures earlier but I didn't save the failing .config files! I didn't think that it would be so difficult to reproduce.) Thanks for your help. > > > > diff --git a/scripts/gen_autoksyms.sh b/scripts/gen_autoksyms.sh > index 12bcfae940ee..70d4ab3621f6 100755 > --- a/scripts/gen_autoksyms.sh > +++ b/scripts/gen_autoksyms.sh > @@ -54,7 +54,7 @@ EOT > } | sed -e 's/ /\n/g' | sed -n -e '/^$/!p' | > # Remove the dot prefix for ppc64; symbol names with a dot (.) hold entry > # point addresses. > -sed -e 's/^\.//' | > +sed -e 's/^\.//g' | > sort -u | > # Ignore __this_module. It's not an exported symbol, and will be resolved > # when the final .ko's are linked. -- ~Randy
