Commit 6e92f04a authored Jul 24, 2018 by Jack Morgenstein Committed by Greg Kroah-Hartman Jul 28, 2018

net/mlx4_core: Save the qpn from the input modifier in RST2INIT wrapper



[ Upstream commit 958c696f5a7274d9447a458ad7aa70719b29a50a ]

Function mlx4_RST2INIT_QP_wrapper saved the qp number passed in the qp
context, rather than the one passed in the input modifier.

However, the qp number in the qp context is not defined as a
required parameter by the FW. Therefore, drivers may choose to not
specify the qp number in the qp context for the reset-to-init transition.

Thus, we must save the qp number passed in the command input modifier --
which is always present. (This saved qp number is used as the input
modifier for command 2RST_QP when a slave's qp's are destroyed).

Fixes: c82e9aa0 ("mlx4_core: resource tracking for HCA resources used by guests")
Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
Signed-off-by: Tariq Toukan <tariqt@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent df20f746

drivers/net/ethernet/mellanox/mlx4/resource_tracker.c

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -2957,7 +2957,7 @@ int mlx4_RST2INIT_QP_wrapper(struct mlx4_dev *dev, int slave,
	u32 srqn = qp_get_srqn(qpc) & 0xffffff;		u32 srqn = qp_get_srqn(qpc) & 0xffffff;
	int use_srq = (qp_get_srqn(qpc) >> 24) & 1;		int use_srq = (qp_get_srqn(qpc) >> 24) & 1;
	struct res_srq *srq;		struct res_srq *srq;
	int local_qpn = be32_to_cpu(qpc->local_qpn) & 0xffffff;		int local_qpn = vhcr->in_modifier & 0xffffff;

	err = adjust_qp_sched_queue(dev, slave, qpc, inbox);		err = adjust_qp_sched_queue(dev, slave, qpc, inbox);
	if (err)		if (err)