Skip to content

Commit e307b5a

Browse files
Hariprasad Kelamdavem330
authored andcommitted
octeontx2-af: Fix pause frame configuration
The current implementation's default Pause Forward setting is causing unnecessary network traffic. This patch disables Pause Forward to address this issue. Fixes: 1121f6b ("octeontx2-af: Priority flow control configuration support") Signed-off-by: Hariprasad Kelam <[email protected]> Signed-off-by: Sunil Kovvuri Goutham <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent c3e0414 commit e307b5a

File tree

1 file changed

+7
-4
lines changed
  • drivers/net/ethernet/marvell/octeontx2/af

1 file changed

+7
-4
lines changed

drivers/net/ethernet/marvell/octeontx2/af/rpm.c

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -373,6 +373,11 @@ void rpm_lmac_pause_frm_config(void *rpmd, int lmac_id, bool enable)
373373
cfg |= RPMX_MTI_MAC100X_COMMAND_CONFIG_TX_P_DISABLE;
374374
rpm_write(rpm, lmac_id, RPMX_MTI_MAC100X_COMMAND_CONFIG, cfg);
375375

376+
/* Disable forward pause to driver */
377+
cfg = rpm_read(rpm, lmac_id, RPMX_MTI_MAC100X_COMMAND_CONFIG);
378+
cfg &= ~RPMX_MTI_MAC100X_COMMAND_CONFIG_PAUSE_FWD;
379+
rpm_write(rpm, lmac_id, RPMX_MTI_MAC100X_COMMAND_CONFIG, cfg);
380+
376381
/* Enable channel mask for all LMACS */
377382
if (is_dev_rpm2(rpm))
378383
rpm_write(rpm, lmac_id, RPM2_CMR_CHAN_MSK_OR, 0xffff);
@@ -616,12 +621,10 @@ int rpm_lmac_pfc_config(void *rpmd, int lmac_id, u8 tx_pause, u8 rx_pause, u16 p
616621

617622
if (rx_pause) {
618623
cfg &= ~(RPMX_MTI_MAC100X_COMMAND_CONFIG_RX_P_DISABLE |
619-
RPMX_MTI_MAC100X_COMMAND_CONFIG_PAUSE_IGNORE |
620-
RPMX_MTI_MAC100X_COMMAND_CONFIG_PAUSE_FWD);
624+
RPMX_MTI_MAC100X_COMMAND_CONFIG_PAUSE_IGNORE);
621625
} else {
622626
cfg |= (RPMX_MTI_MAC100X_COMMAND_CONFIG_RX_P_DISABLE |
623-
RPMX_MTI_MAC100X_COMMAND_CONFIG_PAUSE_IGNORE |
624-
RPMX_MTI_MAC100X_COMMAND_CONFIG_PAUSE_FWD);
627+
RPMX_MTI_MAC100X_COMMAND_CONFIG_PAUSE_IGNORE);
625628
}
626629

627630
if (tx_pause) {

0 commit comments

Comments
 (0)