common/cnxk: fix link cfg for sdp
Checks
Commit Message
Link configure registers are invalid and should not be accessed
for SDP ports. But while on txq release which does SQ flush calls
backpressure disable API which configures these link registers.
Fixes: 58debb813a8d ("common/cnxk: enable TM to listen on Rx pause frames")
Signed-off-by: Harman Kalra <hkalra@marvell.com>
---
drivers/common/cnxk/roc_nix_tm.c | 3 +++
1 file changed, 3 insertions(+)
Comments
On Wed, Feb 28, 2024 at 12:25 AM Harman Kalra <hkalra@marvell.com> wrote:
>
> Link configure registers are invalid and should not be accessed
> for SDP ports. But while on txq release which does SQ flush calls
> backpressure disable API which configures these link registers.
>
> Fixes: 58debb813a8d ("common/cnxk: enable TM to listen on Rx pause frames")
>
> Signed-off-by: Harman Kalra <hkalra@marvell.com>
Updated the git commit as follows and applied to
dpdk-next-net-mrvl/for-main. Thanks
common/cnxk: fix link config for SDP
Link configure registers are invalid and should not be accessed
for SDP ports. But while on Txq release which does SQ flush calls
back pressure disable API which configures these link registers.
Fixes: 58debb813a8d ("common/cnxk: enable TM to listen on Rx pause frames")
Cc: stable@dpdk.org
Signed-off-by: Harman Kalra <hkalra@marvell.com>
@@ -328,6 +328,9 @@ nix_tm_bp_config_set(struct roc_nix *roc_nix, uint16_t sq, uint16_t tc,
uint8_t k = 0;
int rc = 0, i;
+ if (roc_nix_is_sdp(roc_nix))
+ return 0;
+
sq_s = nix->sqs[sq];
if (!sq_s)
return -ENOENT;