| Message ID | 4E39594C.4090808@mellanox.co.il (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
> For older FW versions, fixing the usage of per port Mac table. > For each port we must define the base QP number, which is passed > to the HW. > Setting the correct value in SET_PORT FW command to enable the steering. > > Reported-by: Roland Dreier <roland@purestorage.com> Thanks, testing this now. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Aug 3, 2011 at 11:58 AM, Roland Dreier <roland@purestorage.com> wrote: >> For older FW versions, fixing the usage of per port Mac table. >> For each port we must define the base QP number, which is passed >> to the HW. >> Setting the correct value in SET_PORT FW command to enable the steering. > Thanks, testing this now. Yep, works well on my backrev FW boards. Dave, you want to merge this or should I? - R. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Roland Dreier <roland@purestorage.com> Date: Wed, 3 Aug 2011 12:06:56 -0700 > On Wed, Aug 3, 2011 at 11:58 AM, Roland Dreier <roland@purestorage.com> wrote: >>> For older FW versions, fixing the usage of per port Mac table. >>> For each port we must define the base QP number, which is passed >>> to the HW. >>> Setting the correct value in SET_PORT FW command to enable the steering. > >> Thanks, testing this now. > > Yep, works well on my backrev FW boards. Dave, you want to merge this > or should I? I'll take it, thanks for testing. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Yevgeny Petrilin <yevgenyp@mellanox.co.il> Date: Wed, 3 Aug 2011 17:21:00 +0300 > > For older FW versions, fixing the usage of per port Mac table. > For each port we must define the base QP number, which is passed > to the HW. > Setting the correct value in SET_PORT FW command to enable the steering. > > Reported-by: Roland Dreier <roland@purestorage.com> > Signed-off-by: Yevgeny Petrilin <yevgenyp@mellanox.co.il> Applied. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/mlx4/en_port.c b/drivers/net/mlx4/en_port.c index 5e71091..5ada5b4 100644 --- a/drivers/net/mlx4/en_port.c +++ b/drivers/net/mlx4/en_port.c @@ -128,7 +128,7 @@ int mlx4_SET_PORT_qpn_calc(struct mlx4_dev *dev, u8 port, u32 base_qpn, memset(context, 0, sizeof *context); context->base_qpn = cpu_to_be32(base_qpn); - context->n_mac = 0x7; + context->n_mac = 0x2; context->promisc = cpu_to_be32(promisc << SET_PORT_PROMISC_SHIFT | base_qpn); context->mcast = cpu_to_be32(m_promisc << SET_PORT_MC_PROMISC_SHIFT | diff --git a/drivers/net/mlx4/main.c b/drivers/net/mlx4/main.c index c94b342..f0ee35d 100644 --- a/drivers/net/mlx4/main.c +++ b/drivers/net/mlx4/main.c @@ -1117,6 +1117,8 @@ static int mlx4_init_port_info(struct mlx4_dev *dev, int port) info->port = port; mlx4_init_mac_table(dev, &info->mac_table); mlx4_init_vlan_table(dev, &info->vlan_table); + info->base_qpn = dev->caps.reserved_qps_base[MLX4_QP_REGION_ETH_ADDR] + + (port - 1) * (1 << log_num_mac); sprintf(info->dev_name, "mlx4_port%d", port); info->port_attr.attr.name = info->dev_name;
For older FW versions, fixing the usage of per port Mac table. For each port we must define the base QP number, which is passed to the HW. Setting the correct value in SET_PORT FW command to enable the steering. Reported-by: Roland Dreier <roland@purestorage.com> Signed-off-by: Yevgeny Petrilin <yevgenyp@mellanox.co.il> --- drivers/net/mlx4/en_port.c | 2 +- drivers/net/mlx4/main.c | 2 ++ 2 files changed, 3 insertions(+), 1 deletions(-)