nv-tegra-mirror/nvethernetrm
1
0
Fork 0
mirror of git://nv-tegra.nvidia.com/kernel/nvethernetrm.git synced 2025-12-24 10:34:24 +03:00
Code Issues Packages Projects Releases Wiki Activity

osi: core: Add a BYPASS FRP entry for XDCS

Browse Source 
Issue:
MAC Filters DCH/XDCS is not routing
to the proper DMA channel
after FRP enable.

Fix:
As per HW's suggestion add a BYPASS entry after user entries.

Bug 3899802

Change-Id: I330490ff94ed719ba3fc27f38b6dfaf1bc37369b
Signed-off-by: Mohan Thadikamalla <mohant@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/kernel/nvethernetrm/+/2826024
Reviewed-by: Bhadram Varka <vbhadram@nvidia.com>
Reviewed-by: Rakesh Goyal <rgoyal@nvidia.com>
Reviewed-by: Krishna Thota <kthota@nvidia.com>
GVS: Gerrit_Virtual_Submit <buildbot_gerritrpt@nvidia.com>
This commit is contained in:
Mohan Thadikamalla
2022-12-12 12:54:11 +05:30
committed by Bhadram Varka
parent f0af726970
commit 29242b7c12
1 changed files with 47 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

66
osi/core/frp.c
View File

@@ -370,8 +370,10 @@ done:
static nve32_t frp_hw_write(struct osi_core_priv_data *const osi_core, static nve32_t frp_hw_write(struct osi_core_priv_data *const osi_core,
struct core_ops *const ops_p) struct core_ops *const ops_p)
{ {
nve32_t ret, tmp; nve32_t ret = 0;
nve32_t tmp = 0;
struct osi_core_frp_entry *entry; struct osi_core_frp_entry *entry;
struct osi_core_frp_data bypass_entry = {};
nveu32_t frp_cnt = osi_core->frp_cnt, i = OSI_NONE; nveu32_t frp_cnt = osi_core->frp_cnt, i = OSI_NONE;
/* Disable the FRP in HW */ /* Disable the FRP in HW */
@@ -384,29 +386,55 @@ static nve32_t frp_hw_write(struct osi_core_priv_data *const osi_core,
goto hw_write_enable_frp; goto hw_write_enable_frp;
} }
/* Write FRP entries into HW */ /* Check space for XCS BYPASS rule */
for (i = 0; i < frp_cnt; i++) { if ((frp_cnt + 1U) > OSI_FRP_MAX_ENTRY) {
entry = &osi_core->frp_table[i]; ret = -1;
ret = ops_p->update_frp_entry(osi_core, i, &entry->data); OSI_CORE_ERR(osi_core->osd, OSI_LOG_ARG_HW_FAIL,
"No space for rules\n", OSI_NONE);
goto error;
}
/* Check HW table size for non-zero */
if (frp_cnt != 0U) {
/* Write FRP entries into HW */
for (i = 0; i < frp_cnt; i++) {
entry = &osi_core->frp_table[i];
ret = ops_p->update_frp_entry(osi_core, i,
&entry->data);
if (ret < 0) {
OSI_CORE_ERR(osi_core->osd, OSI_LOG_ARG_HW_FAIL,
"Fail to update FRP entry\n",
OSI_NONE);
goto hw_write_enable_frp;
}
}
/* Write BYPASS rule for XDCS */
bypass_entry.match_en = 0x0U;
bypass_entry.accept_frame = 1;
bypass_entry.reject_frame = 1;
ret = ops_p->update_frp_entry(osi_core, frp_cnt, &bypass_entry);
if (ret < 0) { if (ret < 0) {
OSI_CORE_ERR(osi_core->osd, OSI_LOG_ARG_HW_FAIL, OSI_CORE_ERR(osi_core->osd, OSI_LOG_ARG_HW_FAIL,
"Fail to update FRP entry\n", "Fail to update BYPASS entry\n",
OSI_NONE); OSI_NONE);
goto hw_write_enable_frp; goto hw_write_enable_frp;
} }
}
/* Update the NVE */ /* Update the NVE */
ret = ops_p->update_frp_nve(osi_core, (frp_cnt - 1U)); ret = ops_p->update_frp_nve(osi_core, frp_cnt);
if (ret < 0) { if (ret < 0) {
OSI_CORE_ERR(osi_core->osd, OSI_LOG_ARG_HW_FAIL, OSI_CORE_ERR(osi_core->osd, OSI_LOG_ARG_HW_FAIL,
"Fail to update FRP NVE\n", "Fail to update FRP NVE\n",
OSI_NONE); OSI_NONE);
} }
/* Enable the FRP in HW */ /* Enable the FRP in HW */
hw_write_enable_frp: hw_write_enable_frp:
tmp = ops_p->config_frp(osi_core, OSI_ENABLE); tmp = ops_p->config_frp(osi_core, OSI_ENABLE);
}
error:
return (ret < 0) ? ret : tmp; return (ret < 0) ? ret : tmp;
} }
@@ -616,6 +644,9 @@ static nve32_t frp_delete(struct osi_core_priv_data *const osi_core,
pos++; pos++;
} }
/* Update the frp_cnt entry */
osi_core->frp_cnt = (frp_cnt - count);
/* Write FRP Table into HW */ /* Write FRP Table into HW */
ret = frp_hw_write(osi_core, ops_p); ret = frp_hw_write(osi_core, ops_p);
if (ret < 0) { if (ret < 0) {
@@ -624,9 +655,6 @@ static nve32_t frp_delete(struct osi_core_priv_data *const osi_core,
OSI_NONE); OSI_NONE);
} }
/* Update the frp_cnt entry */
osi_core->frp_cnt = (frp_cnt - count);
done: done:
return ret; return ret;
} }