Re: [PATCH] wifi: ath: add struct_group for struct ath_cycle_counters

[Christian Lamparter <chunkeey@gmail.com>]

On 6/4/23 03:48, Shiji Yang wrote:
> Thank you all,
> This patch may not look so beautiful, but its main purpose is to raise
> some awareness about this strange compilation warning.
> 
> This problem only occurs in mips gcc 12.3 (maybe only on arc mips/ath79).

It also occures with PPC464/APM82181. Could it be that that it occurs on
"big-endian" archs?


> In:
> 
>> --- a/drivers/net/wireless/ath/ath9k/main.c
>> +++ b/drivers/net/wireless/ath/ath9k/main.c
>> @@ -135,8 +135,8 @@ void ath9k_ps_wakeup(struct ath_softc *sc)
>> 	if (power_mode != ATH9K_PM_AWAKE) {
>> 		spin_lock(&common->cc_lock);
>> 		ath_hw_cycle_counters_update(common);
>> -		memset(&common->cc_survey, 0, sizeof(common->cc_survey));
>> -		memset(&common->cc_ani, 0, sizeof(common->cc_ani));
>> +		memset(&common->cc_survey.cnts, 0, sizeof(common->cc_survey.cnts));
>> +		memset(&common->cc_ani.cnts, 0, sizeof(common->cc_ani.cnts));
>> 		spin_unlock(&common->cc_lock);
>> 	}
> 
> The compiler will only warn the second memset() `memset(&common->cc_ani, 0, sizeof(common->cc_ani));`
> detected write beyond size of field. However, `cc_survey` and `cc_survey` are the same structure.
> I have no idea about this warning. I will be very grateful if someone can provide some tips or a
> better solution.
> 
> The full log:
> make[4]: Entering directory '/home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24'
> make[5]: 'Kconfig.versions' is up to date.
> make[7]: 'Kconfig.versions' is up to date.
> make[8]: 'conf' is up to date.
> boolean symbol CRYPTO_LIB_ARC4 tested for 'm'? test forced to 'n'
> #
> # configuration written to .config
> #
> Building backport-include/backport/autoconf.h ... done.
>    CC [M]  /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/drivers/net/wireless/ath/ath9k/main.o
> In file included from ./include/linux/string.h:253,
>                   from /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/backport-include/linux/string.h:3,
>                   from ./include/linux/bitmap.h:11,
>                   from ./include/linux/cpumask.h:12,
>                   from ./arch/mips/include/asm/processor.h:15,
>                   from ./arch/mips/include/asm/thread_info.h:16,
>                   from ./include/linux/thread_info.h:60,
>                   from ./include/asm-generic/current.h:5,
>                   from ./arch/mips/include/generated/asm/current.h:1,
>                   from ./include/linux/sched.h:12,
>                   from /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/backport-include/linux/sched.h:4,
>                   from ./include/linux/delay.h:23,
>                   from /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/backport-include/linux/delay.h:3,
>                   from /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/drivers/net/wireless/ath/ath9k/main.c:18:
> In function 'fortify_memset_chk',
>      inlined from 'ath9k_ps_wakeup' at /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/drivers/net/wireless/ath/ath9k/main.c:140:3:
> ./include/linux/fortify-string.h:314:25: error: call to '__write_overflow_field' declared with attribute warning: detected write beyond size of field (1st parameter); maybe use struct_group()? [-Werror=attribute-warning]
>    314 |                         __write_overflow_field(p_size_field, size);
>        |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> cc1: all warnings being treated as errors
> make[11]: *** [scripts/Makefile.build:250: /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/drivers/net/wireless/ath/ath9k/main.o] Error 1
> make[10]: *** [scripts/Makefile.build:500: /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/drivers/net/wireless/ath/ath9k] Error 2
> make[9]: *** [scripts/Makefile.build:500: /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/drivers/net/wireless/ath] Error 2
> make[8]: *** [scripts/Makefile.build:500: /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/drivers/net/wireless] Error 2
> make[7]: *** [Makefile:2012: /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24] Error 2
> make[6]: *** [Makefile.build:13: modules] Error 2
> make[5]: *** [Makefile.real:93: modules] Error 2
> make[4]: *** [Makefile:121: modules] Error 2
> make[4]: Leaving directory '/home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24'
> make[3]: *** [Makefile:401: /home/db/openwrt/build_dir/target-mips_24kc_musl/linux-ath79_nand/backports-6.1.24/.built] Error 2
> make[3]: Leaving directory '/home/db/openwrt/package/kernel/mac80211'
> time: package/kernel/mac80211/regular/compile#4.94#0.95#21.75
>      ERROR: package/kernel/mac80211 failed to build (build variant: regular).
> make[2]: *** [package/Makefile:120: package/kernel/mac80211/compile] Error 1
> make[2]: Leaving directory '/home/db/openwrt'
> make[1]: *** [package/Makefile:114: /home/db/openwrt/staging_dir/target-mips_24kc_musl/stamp/.package_compile] Error 2
> make[1]: Leaving directory '/home/db/openwrt'
> make: *** [/home/db/openwrt/include/toplevel.mk:231: world] Error 2
> 
> Best Regards,
> Shiji Yang