| Message ID | 20221006192052.1742948-1-keescook@chromium.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 4af609b216e8d9e8d4b03d49ca5b965e87c344b3 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | net: ethernet: mediatek: Remove -Warray-bounds exception | expand |
Hello: This patch was applied to netdev/net.git (master) by David S. Miller <davem@davemloft.net>: On Thu, 6 Oct 2022 12:20:52 -0700 you wrote: > GCC-12 emits false positive -Warray-bounds warnings with > CONFIG_UBSAN_SHIFT (-fsanitize=shift). This is fixed in GCC 13[1], > and there is top-level Makefile logic to remove -Warray-bounds for > known-bad GCC versions staring with commit f0be87c42cbd ("gcc-12: disable > '-Warray-bounds' universally for now"). > > Remove the local work-around. > > [...] Here is the summary with links: - net: ethernet: mediatek: Remove -Warray-bounds exception https://git.kernel.org/netdev/net/c/4af609b216e8 You are awesome, thank you!
diff --git a/drivers/net/ethernet/mediatek/Makefile b/drivers/net/ethernet/mediatek/Makefile index fe66ba8793cf..45ba0970504a 100644 --- a/drivers/net/ethernet/mediatek/Makefile +++ b/drivers/net/ethernet/mediatek/Makefile @@ -11,8 +11,3 @@ mtk_eth-$(CONFIG_NET_MEDIATEK_SOC_WED) += mtk_wed_debugfs.o endif obj-$(CONFIG_NET_MEDIATEK_SOC_WED) += mtk_wed_ops.o obj-$(CONFIG_NET_MEDIATEK_STAR_EMAC) += mtk_star_emac.o - -# FIXME: temporarily silence -Warray-bounds on non W=1+ builds -ifndef KBUILD_EXTRA_WARN -CFLAGS_mtk_ppe.o += -Wno-array-bounds -endif
GCC-12 emits false positive -Warray-bounds warnings with CONFIG_UBSAN_SHIFT (-fsanitize=shift). This is fixed in GCC 13[1], and there is top-level Makefile logic to remove -Warray-bounds for known-bad GCC versions staring with commit f0be87c42cbd ("gcc-12: disable '-Warray-bounds' universally for now"). Remove the local work-around. [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105679 Signed-off-by: Kees Cook <keescook@chromium.org> --- drivers/net/ethernet/mediatek/Makefile | 5 ----- 1 file changed, 5 deletions(-)