[ofw] (no subject)

[Leonid Keller]

I'll add query_qp support to MTHCA and we'll return av port checking. 

> -----Original Message-----
> From: Hal Rosenstock [mailto:hal.rosenstock at gmail.com] 
> Sent: Tuesday, September 09, 2008 3:56 PM
> To: Reuven Amitai
> Cc: Leonid Keller; Tzachi Dar; ofw at lists.openfabrics.org
> Subject: Re: [ofw] (no subject)
> 
> 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
> >
>