Commit 58d7a965 authored by Gautam R A's avatar Gautam R A Committed by Leon Romanovsky
Browse files

RDMA/bnxt_re: Fix incorrect display of inactivity_cp in debugfs output 



The inactivity_cp parameter in debugfs was not being read or
written correctly, resulting in "Invalid argument" errors.

Fixed this by ensuring proper mapping of inactivity_cp in
both the map_cc_config_offset_gen0_ext0 and
bnxt_re_fill_gen0_ext0() functions.

Fixes: 656dff55 ("RDMA/bnxt_re: Congestion control settings using debugfs hook")
Signed-off-by: default avatarGautam R A <gautam-r.a@broadcom.com>
Link: https://patch.msgid.link/20250520035910.1061918-2-kalesh-anakkur.purayil@broadcom.com


Reviewed-by: default avatarKalesh AP <kalesh-anakkur.purayil@broadcom.com>
Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent d00d16bc

drivers/infiniband/hw/bnxt_re/debugfs.c

+5 −0
Original line number Diff line number Diff line
@@ -170,6 +170,9 @@ static int map_cc_config_offset_gen0_ext0(u32 offset, struct bnxt_qplib_cc_param 
	case CMDQ_MODIFY_ROCE_CC_MODIFY_MASK_TCP_CP:

		*val =  ccparam->tcp_cp;

		break;

	case CMDQ_MODIFY_ROCE_CC_MODIFY_MASK_INACTIVITY_CP:

		*val = ccparam->inact_th;

		break;

	default:

		return -EINVAL;

	}
@@ -247,7 +250,9 @@ static void bnxt_re_fill_gen0_ext0(struct bnxt_qplib_cc_param *ccparam, u32 offs 
		ccparam->tcp_cp = val;

		break;

	case CMDQ_MODIFY_ROCE_CC_MODIFY_MASK_TX_QUEUE:

		break;

	case CMDQ_MODIFY_ROCE_CC_MODIFY_MASK_INACTIVITY_CP:

		ccparam->inact_th = val;

		break;

	case CMDQ_MODIFY_ROCE_CC_MODIFY_MASK_TIME_PER_PHASE:

		ccparam->time_pph = val;