forked from Openwrt/openwrt
fb2c6e9d4d
Removed because they are upstream: generic/backport-5.15/704-15-v5.19-net-mtk_eth_soc-move-MAC_MCR-setting-to-mac_finish.patch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-5.15.y&id=c5c0760adc260d55265c086b9efb350ea6dda38b generic/pending-5.15/735-net-mediatek-mtk_eth_soc-release-MAC_MCR_FORCE_LINK-.patch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-5.15.y&id=448cc8b5f743985f6d1d98aa4efb386fef4c3bf2 generic/pending-5.15/736-net-ethernet-mtk_eth_soc-fix-PPE-hanging-issue.patch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-5.15.y&id=9fcadd125044007351905d40c405fadc2d3bb6d6 Add new configuration symbols for tegra target. Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
45 lines
1.5 KiB
Diff
45 lines
1.5 KiB
Diff
From e10a35abb3da12b812cfb6fc6137926a0c81e39a Mon Sep 17 00:00:00 2001
|
|
From: Daniel Golle <daniel@makrotopia.org>
|
|
Date: Sun, 10 Sep 2023 22:40:30 +0100
|
|
Subject: [PATCH] net: ethernet: mtk_eth_soc: fix uninitialized variable
|
|
|
|
Variable dma_addr in function mtk_poll_rx can be uninitialized on
|
|
some of the error paths. In practise this doesn't matter, even random
|
|
data present in uninitialized stack memory can safely be used in the
|
|
way it happens in the error path.
|
|
|
|
However, in order to make Smatch happy make sure the variable is
|
|
always initialized.
|
|
|
|
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
|
|
Signed-off-by: David S. Miller <davem@davemloft.net>
|
|
---
|
|
drivers/net/ethernet/mediatek/mtk_eth_soc.c | 5 +++--
|
|
1 file changed, 3 insertions(+), 2 deletions(-)
|
|
|
|
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
|
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
|
|
@@ -1940,11 +1940,11 @@ static int mtk_poll_rx(struct napi_struc
|
|
u8 *data, *new_data;
|
|
struct mtk_rx_dma_v2 *rxd, trxd;
|
|
int done = 0, bytes = 0;
|
|
+ dma_addr_t dma_addr = DMA_MAPPING_ERROR;
|
|
|
|
while (done < budget) {
|
|
unsigned int pktlen, *rxdcsum;
|
|
struct net_device *netdev;
|
|
- dma_addr_t dma_addr;
|
|
u32 hash, reason;
|
|
int mac = 0;
|
|
|
|
@@ -2121,7 +2121,8 @@ release_desc:
|
|
else
|
|
rxd->rxd2 = RX_DMA_PREP_PLEN0(ring->buf_size);
|
|
|
|
- if (MTK_HAS_CAPS(eth->soc->caps, MTK_36BIT_DMA))
|
|
+ if (MTK_HAS_CAPS(eth->soc->caps, MTK_36BIT_DMA) &&
|
|
+ likely(dma_addr != DMA_MAPPING_ERROR))
|
|
rxd->rxd2 |= RX_DMA_PREP_ADDR64(dma_addr);
|
|
|
|
ring->calc_idx = idx;
|