[ofw] (no subject)

[Hal Rosenstock]

On Mon, Sep 8, 2008 at 12:25 PM, Reuven Amitai <reuven at mellanox.co.il> wrote:
> Hi,
>
> The patch disable av port checking (part of check-in 1538).
> The problem of checking av port happen with mthca card (ConnectX cards are
> OK).
> OpenSM (runs on one machine) sends MAD request to another at the beginning.
> During request processing an av is created with mad svc port num. This field
> is taken from qp_attr.primary_port
> after query on the qp, but mthca set primary_port attr to be zero (among
> other fields in qp_attr).

Seems like mthca is not supporting query_qp properly. Wouldn't this be
a problem elsewhere as well ? If so, this is a workaround so why is
this be solved in IBAL rather than at the mthca level ? Shouldn't
mthca support query qp properly ?

-- Hal

> That will cause the av creation to fail and in consequence opensm fails
> to recognize the remote machine.
>
> Thanks, Reuven.
>
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>.
> Index: core/al/al_av.c
> ===================================================================
> --- core/al/al_av.c (revision 1545)
> +++ core/al/al_av.c (working copy)
> @@ -153,12 +153,17 @@
>    return IB_INVALID_PD_HANDLE;
>   }
>
> + /*
> + This check is disabled due to mthca problem. When registering mad svc,
> port num is zero
> + because ibv_query_qp retrun '0' as primary port.
> +
>   status = __check_av_port(h_pd->obj.p_ci_ca, p_av_attr);
>   if( status != IB_SUCCESS )
>   {
>    AL_PRINT_EXIT( TRACE_LEVEL_ERROR, AL_DBG_ERROR, ("IB_INVALID_PORT\n") );
>    return status;
>   }
> + */
>
>   /* Get an AV tracking structure. */
>   h_av = alloc_av();
>
> _______________________________________________
> ofw mailing list
> ofw at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
>