coolsnowwolf / lede

Lean's LEDE source
Other
29.96k stars 19.55k forks source link

为何编译内核会出现如下警告提示?!会有什么影响吗? #11211

Open zxlhhyccc opened 1 year ago

zxlhhyccc commented 1 year ago

image

net/netfilter/nf_conntrack_ecache.c:136:12: warning: '__nf_conntrack_eventmask_report' defined but not used [-Wunused-function]
  136 | static int __nf_conntrack_eventmask_report(struct nf_conntrack_ecache *e,
      |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  CC [M]  net/netfilter/nf_conntrack_proto_gre.o
  CC      drivers/scsi/scsi_sysfs.o
  CC      drivers/base/container.o
  CC      drivers/base/property.o
  CC [M]  net/netfilter/nf_conntrack_dscpremark_ext.o
  AR      drivers/scsi/built-in.a
  AR      drivers/nvme/host/built-in.a
  AR      drivers/nvme/target/built-in.a
  AR      drivers/nvme/built-in.a
  CC      drivers/mtd/parsers/ofpart_core.o
  CC      drivers/mtd/parsers/qcomsmempart.o
  CC      drivers/base/cacheinfo.o
  CC [M]  net/netfilter/nf_conntrack_netlink.o
  CC      net/ipv4/udplite.o
  AR      drivers/mtd/parsers/built-in.a
  CC      drivers/mtd/mtdsplit/mtdsplit.o
  CC      drivers/base/swnode.o
  CC      drivers/mtd/mtdsplit/mtdsplit_squashfs.o
net/netfilter/nf_conntrack_netlink.c:3130:1: warning: 'ctnetlink_expect_event' defined but not used [-Wunused-function]
 3130 | ctnetlink_expect_event(unsigned int events, const struct nf_exp_event *item)
      | ^~~~~~~~~~~~~~~~~~~~~~
  CC      net/ipv4/udp_offload.o
WYC-2020 commented 1 year ago

没使用到这个函数而已,有一个宏把这个函数包含其中即可

zxlhhyccc commented 1 year ago

没使用到这个函数而已,有一个宏把这个函数包含其中即可

看了一下,没看到哪个宏包含了这个函数!

WYC-2020 commented 1 year ago

没使用到这个函数而已,有一个宏把这个函数包含其中即可

看了一下,没看到哪个宏包含了这个函数!

明天给你看,因为这个我修过

zxlhhyccc commented 1 year ago

没使用到这个函数而已,有一个宏把这个函数包含其中即可

看了一下,没看到哪个宏包含了这个函数!

明天给你看,因为这个我修过

请问大佬能发我你修改的么?谢谢了!

WYC-2020 commented 1 year ago

没使用到这个函数而已,有一个宏把这个函数包含其中即可

看了一下,没看到哪个宏包含了这个函数!

明天给你看,因为这个我修过

请问大佬能发我你修改的么?谢谢了!

CONFIG_NF_CONNTRACK_CHAIN_EVENTS 这个宏,我这个可能 你不一定能直接用,因为可能位置不一样,我有自己的一些patch 可能位置与你有偏移 你拿去试下吧,不行就按 这个自己修 5.15 952-add-net-conntrack-events-support-multiple-registrant.patch 6.1 952-add-net-conntrack-events-support-multiple-registrant.patch

zxlhhyccc commented 1 year ago

谢谢大佬,请问这个补丁在编译内核的时候会有我截图的那些没有用到的函数的警告吗?

WYC-2020 commented 1 year ago

谢谢大佬,请问这个补丁在编译内核的时候会有我截图的那些没有用到的函数的警告吗?

不会

zxlhhyccc commented 1 year ago

谢谢大佬,请问这个补丁在编译内核的时候会有我截图的那些没有用到的函数的警告吗?

不会

请问大佬,有5.10内核的952补丁么?烦请大佬发一个。谢谢!

WYC-2020 commented 1 year ago

谢谢大佬,请问这个补丁在编译内核的时候会有我截图的那些没有用到的函数的警告吗?

不会

请问大佬,有5.10内核的952补丁么?烦请大佬发一个。谢谢!

5.10 不会有这个报警吧,这个l有啊

zxlhhyccc commented 1 year ago

谢谢大佬,请问这个补丁在编译内核的时候会有我截图的那些没有用到的函数的警告吗?

不会

请问大佬,有5.10内核的952补丁么?烦请大佬发一个。谢谢!

5.10 不会有这个报警吧,这个l有啊

不记得有没有这个报警,我是想准备着。。。

ZqinKing commented 1 year ago

不用理会,或者在gcc编译的时候增加-Wnounused-function即可。

zxlhhyccc commented 1 year ago

不用理会,或者在gcc编译的时候增加-Wnounused-function即可。

这个是编译内核,应该在哪里增加?不过我认为@WYC-2020修改的补丁效果更好。

e6e6 commented 1 year ago

上游打开了 CONFIG_WERROR=y 所以,其实还是建议更新下补丁

WYC-2020 commented 1 year ago

CONFIG_WERROR

打开这个 那有你玩的了,内核编译只要有警告统一当错误处理,有能力的可以打开 慢慢修吧哈哈

KaneGreen commented 1 year ago

粗略看了一下,Linux内核从5.15开始默认启用CONFIG_WERROR=y(似乎后来又放弃了);然后5.18再次对x86启用该项。 OpenWrt官方从23.05(目前还是rc版本)开始启用CONFIG_WERROR=y个人感觉以后启用CONFIG_WERROR=y会是大势所趋。

目前我看到的只有两处Warning: net/netfilter/nf_conntrack_ecache.c文件第136行左右的__nf_conntrack_eventmask_report函数和 net/netfilter/nf_conntrack_netlink.c文件第3123行左右的ctnetlink_expect_event函数。 警告类型均是defined but not usedunused-function

应该是和这个patch导致某些这些函数在新的#ifdef条件编译中没有被使用有关:https://github.com/coolsnowwolf/lede/blob/master/target/linux/generic/hack-5.15/952-add-net-conntrack-events-support-multiple-registrant.patch

如果能修复,CONFIG_WERROR=y就能开启了。

e6e6 commented 6 months ago

上游在gcc中默认开启了 CONFIG_WERROR

https://github.com/openwrt/openwrt/blob/65f34e99284f33be96518ac48de7d0beb972633e/toolchain/gcc/Config.version#L21-L24