aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristian Marangi <ansuelsmth@gmail.com>2023-05-09 23:32:29 +0200
committerChristian Marangi <ansuelsmth@gmail.com>2023-05-12 03:15:44 +0200
commitd088282ddd88dfaba222b60d9db9f13a87948865 (patch)
tree82123e48de711ecd71f8dbb63130907d13125f05
parent220739aa0d2fd6e50a4ac74e8ef71ff9f3bfb116 (diff)
downloadupstream-d088282ddd88dfaba222b60d9db9f13a87948865.tar.gz
upstream-d088282ddd88dfaba222b60d9db9f13a87948865.tar.bz2
upstream-d088282ddd88dfaba222b60d9db9f13a87948865.zip
kernel: lantiq: vrx518_tc: add patch fix compilation warning for ptm_tc
Alloc MIB struct dynamically instead to fix compilation warning for stack limit. Fix compilation warning: /__w/openwrt/openwrt/openwrt/build_dir/target-arm-openwrt-linux-muslgnueabi_musl/linux-ipq40xx_chromium/vrx518_tc_drv-ugw_8.5.2.10/dcdp/ptm_tc.c: In function 'ptm_tc_get_stats': /__w/openwrt/openwrt/openwrt/build_dir/target-arm-openwrt-linux-muslgnueabi_musl/linux-ipq40xx_chromium/vrx518_tc_drv-ugw_8.5.2.10/dcdp/ptm_tc.c:463:1: error: the frame size of 1032 bytes is larger than 1024 bytes [-Werror=frame-larger-than=] 463 | } | ^ cc1: all warnings being treated as errors Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
-rw-r--r--package/kernel/lantiq/vrx518_tc/patches/205-dcdp-ptm_tc-dynamically-alloc-mib.patch206
1 files changed, 206 insertions, 0 deletions
diff --git a/package/kernel/lantiq/vrx518_tc/patches/205-dcdp-ptm_tc-dynamically-alloc-mib.patch b/package/kernel/lantiq/vrx518_tc/patches/205-dcdp-ptm_tc-dynamically-alloc-mib.patch
new file mode 100644
index 0000000000..ca4c041b21
--- /dev/null
+++ b/package/kernel/lantiq/vrx518_tc/patches/205-dcdp-ptm_tc-dynamically-alloc-mib.patch
@@ -0,0 +1,206 @@
+--- a/dcdp/ptm_tc.c
++++ b/dcdp/ptm_tc.c
+@@ -298,15 +298,19 @@ static int ptm_tc_get_stats(struct ptm_e
+ )
+ {
+ struct rtnl_link_stats64 *stat;
+- struct wan_rx_mib_table rx_mib;
++ struct wan_rx_mib_table *rx_mib;
+ unsigned int cur_cnt, last_cnt;
+ struct intel_tc_ptm_sl_stats *ptm_sl_stats;
+ struct ptm_priv *ptm_tc;
++ int ret = 0;
+ if (!priv || !stats)
+ return -EFAULT;
+ ptm_tc = priv->ptm_tc;
+ if (!ptm_tc)
+ return -EFAULT;
++ rx_mib = kzalloc(sizeof(*rx_mib), GFP_KERNEL);
++ if (!rx_mib)
++ return -ENOMEM;
+ if (bonding)
+ stats->tc_info = TC_PTM_BND_MODE;
+ else
+@@ -340,11 +344,11 @@ static int ptm_tc_get_stats(struct ptm_e
+ ? cur_cnt - last_cnt
+ : cur_cnt + ((unsigned int)(-1) - last_cnt);
+
+- tc_mem_read(priv, &rx_mib,
++ tc_mem_read(priv, rx_mib,
+ fpi_addr(__RX_GIF_MIB_BASE),
+- sizeof(rx_mib)
++ sizeof(*rx_mib)
+ );
+- ptm_sl_stats->wrx_gif_drop_pdu = rx_mib.wrx_dropdes_pdu;
++ ptm_sl_stats->wrx_gif_drop_pdu = rx_mib->wrx_dropdes_pdu;
+
+ cur_cnt = tc_r32(GIF0_RX_CRC_ERR_CNT);
+ last_cnt = priv->ptm_mib.rx_crc_err_pdu[0];
+@@ -358,7 +362,7 @@ static int ptm_tc_get_stats(struct ptm_e
+ ? cur_cnt - last_cnt
+ : cur_cnt + ((unsigned int)(-1) - last_cnt);
+
+- ptm_sl_stats->wrx_gif_total_bytes = rx_mib.wrx_total_bytes;
++ ptm_sl_stats->wrx_gif_total_bytes = rx_mib->wrx_total_bytes;
+ cur_cnt = sb_r32(__US_TC_LOCAL_Q_CFG_CTXT_BASE +
+ offsetof(desq_cfg_ctxt_t, deq_pkt_cnt) / 4);
+ last_cnt = priv->ptm_mib.tx_total_pdu[0];
+@@ -376,90 +380,108 @@ static int ptm_tc_get_stats(struct ptm_e
+ /* For bonding information */
+ if (bonding) {
+ int i;
+- struct intel_tc_ptm_bonding_stats ptm_ds;
+- struct intel_tc_ptm_bonding_stats ptm_us;
++ struct intel_tc_ptm_bonding_stats *ptm_ds;
++ struct intel_tc_ptm_bonding_stats *ptm_us;
+ struct intel_tc_ptm_bonding_stats *ptm_bonding_stats;
++
++ ptm_ds = kzalloc(sizeof(*ptm_ds), GFP_KERNEL);
++ if (!ptm_ds) {
++ ret = -ENOMEM;
++ goto free_rx_mib;
++ }
++
++ ptm_us = kzalloc(sizeof(*ptm_us), GFP_KERNEL);
++ if (!ptm_us) {
++ ret = -ENOMEM;
++ goto free_ptm_ds;
++ }
++
+ priv = tc_ep_priv(0);
+- ptm_tc_bond_get_stats(priv, &ptm_ds);
++ ptm_tc_bond_get_stats(priv, ptm_ds);
+ priv = tc_ep_priv(1);
+- ptm_tc_bond_get_stats(priv, &ptm_us);
++ ptm_tc_bond_get_stats(priv, ptm_us);
+ ptm_bonding_stats =
+ &(stats->stats.ptm_tc_stats.pmt_bonding_stats);
+ for (i = 0; i < 8; i++)
+ ptm_bonding_stats->us_gif_mib[i] =
+- ptm_ds.us_gif_mib[i] + ptm_us.us_gif_mib[i];
++ ptm_ds->us_gif_mib[i] + ptm_us->us_gif_mib[i];
+ for (i = 0; i < 8; i++) {
+ ptm_bonding_stats->ds_gif_mib[i].rx_frag_byte_cnt =
+- ptm_ds.ds_gif_mib[i].rx_frag_byte_cnt
+- + ptm_us.ds_gif_mib[i].rx_frag_byte_cnt;
++ ptm_ds->ds_gif_mib[i].rx_frag_byte_cnt
++ + ptm_us->ds_gif_mib[i].rx_frag_byte_cnt;
+ ptm_bonding_stats->ds_gif_mib[i].rx_byte_cnt =
+- ptm_ds.ds_gif_mib[i].rx_byte_cnt
+- + ptm_us.ds_gif_mib[i].rx_byte_cnt;
++ ptm_ds->ds_gif_mib[i].rx_byte_cnt
++ + ptm_us->ds_gif_mib[i].rx_byte_cnt;
+ ptm_bonding_stats->ds_gif_mib[i].rx_of_frag_byte_cnt =
+- ptm_ds.ds_gif_mib[i].rx_of_frag_byte_cnt
+- + ptm_us.ds_gif_mib[i].rx_of_frag_byte_cnt;
++ ptm_ds->ds_gif_mib[i].rx_of_frag_byte_cnt
++ + ptm_us->ds_gif_mib[i].rx_of_frag_byte_cnt;
+ ptm_bonding_stats->ds_gif_mib[i].rx_of_byte_cnt =
+- ptm_ds.ds_gif_mib[i].rx_of_byte_cnt
+- + ptm_us.ds_gif_mib[i].rx_of_byte_cnt;
++ ptm_ds->ds_gif_mib[i].rx_of_byte_cnt
++ + ptm_us->ds_gif_mib[i].rx_of_byte_cnt;
+ ptm_bonding_stats->ds_gif_mib[i].rx_oor_frag_byte_cnt =
+- ptm_ds.ds_gif_mib[i].rx_oor_frag_byte_cnt
+- + ptm_us.ds_gif_mib[i].rx_oor_frag_byte_cnt;
++ ptm_ds->ds_gif_mib[i].rx_oor_frag_byte_cnt
++ + ptm_us->ds_gif_mib[i].rx_oor_frag_byte_cnt;
+ ptm_bonding_stats->ds_gif_mib[i].rx_miss_frag_byte_cnt =
+- ptm_ds.ds_gif_mib[i].rx_miss_frag_byte_cnt
+- + ptm_us.ds_gif_mib[i].rx_miss_frag_byte_cnt;
++ ptm_ds->ds_gif_mib[i].rx_miss_frag_byte_cnt
++ + ptm_us->ds_gif_mib[i].rx_miss_frag_byte_cnt;
+ ptm_bonding_stats->ds_gif_mib[i].rx_to_frag_byte_cnt =
+- ptm_ds.ds_gif_mib[i].rx_to_frag_byte_cnt
+- + ptm_us.ds_gif_mib[i].rx_to_frag_byte_cnt;
++ ptm_ds->ds_gif_mib[i].rx_to_frag_byte_cnt
++ + ptm_us->ds_gif_mib[i].rx_to_frag_byte_cnt;
+ }
+ for (i = 0; i < 4; i++) {
+ ptm_bonding_stats->ds_bg_mib[i].conform_pkt_cnt =
+- ptm_ds.ds_bg_mib[i].conform_pkt_cnt
+- + ptm_us.ds_bg_mib[i].conform_pkt_cnt;
++ ptm_ds->ds_bg_mib[i].conform_pkt_cnt
++ + ptm_us->ds_bg_mib[i].conform_pkt_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].conform_frag_cnt =
+- ptm_ds.ds_bg_mib[i].conform_frag_cnt
+- + ptm_us.ds_bg_mib[i].conform_frag_cnt;
++ ptm_ds->ds_bg_mib[i].conform_frag_cnt
++ + ptm_us->ds_bg_mib[i].conform_frag_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].conform_byte_cnt =
+- ptm_ds.ds_bg_mib[i].conform_byte_cnt
+- + ptm_us.ds_bg_mib[i].conform_byte_cnt;
++ ptm_ds->ds_bg_mib[i].conform_byte_cnt
++ + ptm_us->ds_bg_mib[i].conform_byte_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].no_sop_pkt_cnt =
+- ptm_ds.ds_bg_mib[i].no_sop_pkt_cnt
+- + ptm_us.ds_bg_mib[i].no_sop_pkt_cnt;
++ ptm_ds->ds_bg_mib[i].no_sop_pkt_cnt
++ + ptm_us->ds_bg_mib[i].no_sop_pkt_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].no_sop_frag_cnt =
+- ptm_ds.ds_bg_mib[i].no_sop_frag_cnt
+- + ptm_us.ds_bg_mib[i].no_sop_frag_cnt;
++ ptm_ds->ds_bg_mib[i].no_sop_frag_cnt
++ + ptm_us->ds_bg_mib[i].no_sop_frag_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].no_sop_byte_cnt =
+- ptm_ds.ds_bg_mib[i].no_sop_byte_cnt
+- + ptm_us.ds_bg_mib[i].no_sop_byte_cnt;
++ ptm_ds->ds_bg_mib[i].no_sop_byte_cnt
++ + ptm_us->ds_bg_mib[i].no_sop_byte_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].no_eop_pkt_cnt =
+- ptm_ds.ds_bg_mib[i].no_eop_pkt_cnt
+- + ptm_us.ds_bg_mib[i].no_eop_pkt_cnt;
++ ptm_ds->ds_bg_mib[i].no_eop_pkt_cnt
++ + ptm_us->ds_bg_mib[i].no_eop_pkt_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].no_eop_frag_cnt =
+- ptm_ds.ds_bg_mib[i].no_eop_frag_cnt
+- + ptm_us.ds_bg_mib[i].no_eop_frag_cnt;
++ ptm_ds->ds_bg_mib[i].no_eop_frag_cnt
++ + ptm_us->ds_bg_mib[i].no_eop_frag_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].no_eop_byte_cnt =
+- ptm_ds.ds_bg_mib[i].no_eop_byte_cnt
+- + ptm_us.ds_bg_mib[i].no_eop_byte_cnt;
++ ptm_ds->ds_bg_mib[i].no_eop_byte_cnt
++ + ptm_us->ds_bg_mib[i].no_eop_byte_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].oversize_pkt_cnt =
+- ptm_ds.ds_bg_mib[i].oversize_pkt_cnt
+- + ptm_us.ds_bg_mib[i].oversize_pkt_cnt;
++ ptm_ds->ds_bg_mib[i].oversize_pkt_cnt
++ + ptm_us->ds_bg_mib[i].oversize_pkt_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].oversize_frag_cnt =
+- ptm_ds.ds_bg_mib[i].oversize_frag_cnt
+- + ptm_us.ds_bg_mib[i].oversize_frag_cnt;
++ ptm_ds->ds_bg_mib[i].oversize_frag_cnt
++ + ptm_us->ds_bg_mib[i].oversize_frag_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].oversize_byte_cnt =
+- ptm_ds.ds_bg_mib[i].oversize_byte_cnt
+- + ptm_us.ds_bg_mib[i].oversize_byte_cnt;
++ ptm_ds->ds_bg_mib[i].oversize_byte_cnt
++ + ptm_us->ds_bg_mib[i].oversize_byte_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].noncosec_pkt_cnt =
+- ptm_ds.ds_bg_mib[i].noncosec_pkt_cnt
+- + ptm_us.ds_bg_mib[i].noncosec_pkt_cnt;
++ ptm_ds->ds_bg_mib[i].noncosec_pkt_cnt
++ + ptm_us->ds_bg_mib[i].noncosec_pkt_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].noncosec_frag_cnt =
+- ptm_ds.ds_bg_mib[i].noncosec_frag_cnt
+- + ptm_us.ds_bg_mib[i].noncosec_frag_cnt;
++ ptm_ds->ds_bg_mib[i].noncosec_frag_cnt
++ + ptm_us->ds_bg_mib[i].noncosec_frag_cnt;
+ ptm_bonding_stats->ds_bg_mib[i].noncosec_byte_cnt =
+- ptm_ds.ds_bg_mib[i].noncosec_byte_cnt
+- + ptm_us.ds_bg_mib[i].noncosec_byte_cnt;
++ ptm_ds->ds_bg_mib[i].noncosec_byte_cnt
++ + ptm_us->ds_bg_mib[i].noncosec_byte_cnt;
+ }
++ kfree(ptm_us);
++free_ptm_ds:
++ kfree(ptm_ds);
+ }
+- return 0;
++free_rx_mib:
++ kfree(rx_mib);
++ return ret;
+ }
+ static int ptm_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
+ {