# git rev-parse -q --verify 21f9c8a13bb2a0c24d9c6b86bc0896542a28c197^{commit} 21f9c8a13bb2a0c24d9c6b86bc0896542a28c197 already have revision, skipping fetch # git checkout -q -f -B kisskb 21f9c8a13bb2a0c24d9c6b86bc0896542a28c197 # git clean -qxdf # < git log -1 # commit 21f9c8a13bb2a0c24d9c6b86bc0896542a28c197 # Author: Linus Torvalds # Date: Thu Aug 11 08:40:01 2022 -0700 # # Revert "Makefile.extrawarn: re-enable -Wformat for clang" # # This reverts commit 258fafcd0683d9ccfa524129d489948ab3ddc24c. # # The clang -Wformat warning is terminally broken, and the clang people # can't seem to get their act together. # # This test program causes a warning with clang: # # #include # # int main(int argc, char **argv) # { # printf("%hhu\n", 'a'); # } # # resulting in # # t.c:5:19: warning: format specifies type 'unsigned char' but the argument has type 'int' [-Wformat] # printf("%hhu\n", 'a'); # ~~~~ ^~~ # %d # # and apparently clang people consider that a feature, because they don't # want to face the reality of how either C character constants, C # arithmetic, and C varargs functions work. # # The rest of the world just shakes their head at that kind of # incompetence, and turns off -Wformat for clang again. # # And no, the "you should use a pointless cast to shut this up" is not a # valid answer. That warning should not exist in the first place, or at # least be optinal with some "-Wformat-me-harder" kind of option. # # [ Admittedly, there's also very little reason to *ever* use '%hh[ud]' in # C, but what little reason there is is entirely about 'I want to see # only the low 8 bits of the argument'. So I would suggest nobody ever # use that format in the first place, but if they do, the clang # behavious is simply always wrong. Because '%hhu' takes an 'int'. It's # that simple. ] # # Reported-by: Sudip Mukherjee (Codethink) # Cc: Nick Desaulniers # Signed-off-by: Linus Torvalds # < /opt/cross/kisskb/korg/gcc-11.1.0-nolibc/sh4-linux/bin/sh4-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-11.1.0-nolibc/sh4-linux/bin/sh4-linux-ld --version # < git log --format=%s --max-count=1 21f9c8a13bb2a0c24d9c6b86bc0896542a28c197 # < make -s -j 120 ARCH=sh O=/kisskb/build/linus_sh-defconfig_sh4-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/sh4-linux/bin/sh4-linux- defconfig # < make -s -j 120 ARCH=sh O=/kisskb/build/linus_sh-defconfig_sh4-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/sh4-linux/bin/sh4-linux- help # make -s -j 120 ARCH=sh O=/kisskb/build/linus_sh-defconfig_sh4-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/sh4-linux/bin/sh4-linux- olddefconfig # make -s -j 120 ARCH=sh O=/kisskb/build/linus_sh-defconfig_sh4-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/sh4-linux/bin/sh4-linux- Generating include/generated/machtypes.h :1517:2: warning: #warning syscall clone3 not implemented [-Wcpp] /kisskb/src/arch/sh/kernel/machvec.c: In function 'sh_mv_setup': /kisskb/src/arch/sh/kernel/machvec.c:105:33: warning: array subscript 'struct sh_machine_vector[0]' is partly outside array bounds of 'long int[1]' [-Warray-bounds] 105 | sh_mv = *(struct sh_machine_vector *)&__machvec_start; | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from /kisskb/src/arch/sh/kernel/machvec.c:13: /kisskb/src/arch/sh/include/asm/sections.h:7:13: note: while referencing '__machvec_start' 7 | extern long __machvec_start, __machvec_end; | ^~~~~~~~~~~~~~~ In file included from /kisskb/src/arch/sh/include/asm/hw_irq.h:6, from /kisskb/src/include/linux/irq.h:596, from /kisskb/src/include/asm-generic/hardirq.h:17, from /kisskb/src/arch/sh/include/asm/hardirq.h:9, from /kisskb/src/include/linux/hardirq.h:11, from /kisskb/src/include/linux/interrupt.h:11, from /kisskb/src/include/linux/serial_core.h:13, from /kisskb/src/include/linux/serial_sci.h:6, from /kisskb/src/arch/sh/kernel/cpu/sh4a/setup-shx3.c:10: /kisskb/src/include/linux/sh_intc.h:100:63: warning: division 'sizeof (void *) / sizeof (void)' does not compute the number of array elements [-Wsizeof-pointer-div] 100 | #define _INTC_ARRAY(a) a, __same_type(a, NULL) ? 0 : sizeof(a)/sizeof(*a) | ^ /kisskb/src/include/linux/sh_intc.h:107:9: note: in expansion of macro '_INTC_ARRAY' 107 | _INTC_ARRAY(sense_regs), _INTC_ARRAY(ack_regs), \ | ^~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:124:15: note: in expansion of macro 'INTC_HW_DESC' 124 | .hw = INTC_HW_DESC(vectors, groups, mask_regs, \ | ^~~~~~~~~~~~ /kisskb/src/arch/sh/kernel/cpu/sh4a/setup-shx3.c:309:8: note: in expansion of macro 'DECLARE_INTC_DESC' 309 | static DECLARE_INTC_DESC(intc_desc, "shx3", vectors, groups, | ^~~~~~~~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:100:63: warning: division 'sizeof (void *) / sizeof (void)' does not compute the number of array elements [-Wsizeof-pointer-div] 100 | #define _INTC_ARRAY(a) a, __same_type(a, NULL) ? 0 : sizeof(a)/sizeof(*a) | ^ /kisskb/src/include/linux/sh_intc.h:107:34: note: in expansion of macro '_INTC_ARRAY' 107 | _INTC_ARRAY(sense_regs), _INTC_ARRAY(ack_regs), \ | ^~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:124:15: note: in expansion of macro 'INTC_HW_DESC' 124 | .hw = INTC_HW_DESC(vectors, groups, mask_regs, \ | ^~~~~~~~~~~~ /kisskb/src/arch/sh/kernel/cpu/sh4a/setup-shx3.c:309:8: note: in expansion of macro 'DECLARE_INTC_DESC' 309 | static DECLARE_INTC_DESC(intc_desc, "shx3", vectors, groups, | ^~~~~~~~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:100:63: warning: division 'sizeof (void *) / sizeof (void)' does not compute the number of array elements [-Wsizeof-pointer-div] 100 | #define _INTC_ARRAY(a) a, __same_type(a, NULL) ? 0 : sizeof(a)/sizeof(*a) | ^ /kisskb/src/include/linux/sh_intc.h:107:34: note: in expansion of macro '_INTC_ARRAY' 107 | _INTC_ARRAY(sense_regs), _INTC_ARRAY(ack_regs), \ | ^~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:124:15: note: in expansion of macro 'INTC_HW_DESC' 124 | .hw = INTC_HW_DESC(vectors, groups, mask_regs, \ | ^~~~~~~~~~~~ /kisskb/src/arch/sh/kernel/cpu/sh4a/setup-shx3.c:322:8: note: in expansion of macro 'DECLARE_INTC_DESC' 322 | static DECLARE_INTC_DESC(intc_desc_irq, "shx3-irq", vectors_irq, groups, | ^~~~~~~~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:100:63: warning: division 'sizeof (void *) / sizeof (void)' does not compute the number of array elements [-Wsizeof-pointer-div] 100 | #define _INTC_ARRAY(a) a, __same_type(a, NULL) ? 0 : sizeof(a)/sizeof(*a) | ^ /kisskb/src/include/linux/sh_intc.h:107:9: note: in expansion of macro '_INTC_ARRAY' 107 | _INTC_ARRAY(sense_regs), _INTC_ARRAY(ack_regs), \ | ^~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:124:15: note: in expansion of macro 'INTC_HW_DESC' 124 | .hw = INTC_HW_DESC(vectors, groups, mask_regs, \ | ^~~~~~~~~~~~ /kisskb/src/arch/sh/kernel/cpu/sh4a/setup-shx3.c:337:8: note: in expansion of macro 'DECLARE_INTC_DESC' 337 | static DECLARE_INTC_DESC(intc_desc_irl, "shx3-irl", vectors_irl, groups, | ^~~~~~~~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:100:63: warning: division 'sizeof (void *) / sizeof (void)' does not compute the number of array elements [-Wsizeof-pointer-div] 100 | #define _INTC_ARRAY(a) a, __same_type(a, NULL) ? 0 : sizeof(a)/sizeof(*a) | ^ /kisskb/src/include/linux/sh_intc.h:107:34: note: in expansion of macro '_INTC_ARRAY' 107 | _INTC_ARRAY(sense_regs), _INTC_ARRAY(ack_regs), \ | ^~~~~~~~~~~ /kisskb/src/include/linux/sh_intc.h:124:15: note: in expansion of macro 'INTC_HW_DESC' 124 | .hw = INTC_HW_DESC(vectors, groups, mask_regs, \ | ^~~~~~~~~~~~ /kisskb/src/arch/sh/kernel/cpu/sh4a/setup-shx3.c:337:8: note: in expansion of macro 'DECLARE_INTC_DESC' 337 | static DECLARE_INTC_DESC(intc_desc_irl, "shx3-irl", vectors_irl, groups, | ^~~~~~~~~~~~~~~~~ /kisskb/src/fs/ext4/readpage.c: In function 'ext4_mpage_readpages': /kisskb/src/fs/ext4/readpage.c:407:1: warning: the frame size of 1132 bytes is larger than 1024 bytes [-Wframe-larger-than=] 407 | } | ^ /kisskb/src/fs/mpage.c: In function '__mpage_writepage': /kisskb/src/fs/mpage.c:634:1: warning: the frame size of 1144 bytes is larger than 1024 bytes [-Wframe-larger-than=] 634 | } | ^ /kisskb/src/fs/mpage.c: In function 'do_mpage_readpage': /kisskb/src/fs/mpage.c:308:1: warning: the frame size of 1092 bytes is larger than 1024 bytes [-Wframe-larger-than=] 308 | } | ^ Kernel: arch/sh/boot/zImage is ready Completed OK # rm -rf /kisskb/build/linus_sh-defconfig_sh4-gcc11 # Build took: 0:00:27.847251