netdev - Re: [PATCH net-next 2/3] IB/mlx4: Always use the correct port for mirrored multicast attachments

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 05 Feb 2015 17:04:44 +0300
From:	Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
To:	Or Gerlitz <ogerlitz@...lanox.com>,
	"David S. Miller" <davem@...emloft.net>
CC:	netdev@...r.kernel.org, Roland Dreier <roland@...nel.org>,
	Amir Vadai <amirv@...lanox.com>, Tal Alon <talal@...lanox.com>,
	Moni Shoua <monis@...lanox.com>
Subject: Re: [PATCH net-next 2/3] IB/mlx4: Always use the correct port for
 mirrored multicast attachments

Hello.

On 2/5/2015 4:52 PM, Or Gerlitz wrote:

> From: Moni Shoua <monis@...lanox.com>

> When attaching a QP to a multicast address in bonded mode, there was an
> assumption that the port of the QP must be #1. This assumption isn't the
> case under the flow which enables maximal usage of the physical ports.

> Fix it by always checking the port of the original flow and create the
> mirrored flow on the other port.

> Fixes: c6215745b66a ("IB/mlx4: Load balance ports in port aggregation mode")
> Signed-off-by: Moni Shoua <monis@...lanox.com>
> Signed-off-by: Or Gerlitz <ogerlitz@...lanox.com>
> ---
>   drivers/infiniband/hw/mlx4/main.c |    8 ++++++--
>   1 files changed, 6 insertions(+), 2 deletions(-)

> diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c
> index 2ed5b99..271d3f1 100644
> --- a/drivers/infiniband/hw/mlx4/main.c
> +++ b/drivers/infiniband/hw/mlx4/main.c
> @@ -1186,6 +1186,9 @@ static struct ib_flow *mlx4_ib_create_flow(struct ib_qp *qp,
>   			goto err_create_flow;
>   		i++;
>   		if (is_bonded) {
> +			/* Application always sees one port so the mirror rule
> +			 * must be on port #2
> +			 */

    Unrelated change?

>   			flow_attr->port = 2;
>   			err = __mlx4_ib_create_flow(qp, flow_attr,
>   						    domain, type[j],
> @@ -1279,14 +1282,15 @@ static int mlx4_ib_mcg_attach(struct ib_qp *ibqp, union ib_gid *gid, u16 lid)
>
>   	err = mlx4_multicast_attach(mdev->dev, &mqp->mqp, gid->raw, mqp->port,
>   				    !!(mqp->flags &
> -				       MLX4_IB_QP_BLOCK_MULTICAST_LOOPBACK),
> +				    MLX4_IB_QP_BLOCK_MULTICAST_LOOPBACK),

    This change is both unrelated and unneeded.

>   				    prot, &reg_id.id);
>   	if (err)
>   		goto err_malloc;
>
>   	reg_id.mirror = 0;
>   	if (mlx4_is_bonded(dev)) {
> -		err = mlx4_multicast_attach(mdev->dev, &mqp->mqp, gid->raw, 2,
> +		err = mlx4_multicast_attach(mdev->dev, &mqp->mqp, gid->raw,
> +					    (mqp->port == 1) ? 2 : 1,

    I doubt that these parens are needed.

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html