From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hal Rosenstock Subject: Re: ib-diags: compatability issue with ibstat Date: Fri, 23 Sep 2011 12:38:25 -0400 Message-ID: References: <1828884A29C6694DAF28B7E6B8A8237316E66F5B@ORSMSX101.amr.corp.intel.com> <1828884A29C6694DAF28B7E6B8A8237316E67718@ORSMSX101.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1747400220==" Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: ofw-bounces@lists.openfabrics.org Errors-To: ofw-bounces@lists.openfabrics.org To: "Hefty, Sean" Cc: "linux-rdma (linux-rdma@vger.kernel.org)" , "ofw@lists.openfabrics.org" , "Ira Weiny (weiny2@llnl.gov)" List-Id: linux-rdma@vger.kernel.org --===============1747400220== Content-Type: multipart/alternative; boundary=0016367fb52f2cf13e04ad9e7016 --0016367fb52f2cf13e04ad9e7016 Content-Type: text/plain; charset=UTF-8 On Fri, Sep 23, 2011 at 12:33 PM, Hal Rosenstock wrote: > > > On Fri, Sep 23, 2011 at 12:21 PM, Hefty, Sean wrote: > >> > > The only way to determine whether fdr10 is active or not is via the >> > > vendor proprietary MAD. That info may be reflected in some other API >> > > (and/or file) so that MAD does not need to be reissued. In a separate >> > > thread on linux-rdma, there was discussion on a couple of different >> > > ways to do that from verbs and in this thread that there's no sysfs >> > > equivalent in Windows. You've already stated that the Windows support >> > > is using libibverbs for libibumad support so it seems appropriate to >> > > me to do the same here (in Windows at least). >> > >> > The proper place for is_fdr10 is in libibumad (and then we wouldn't be >> > discussing libibverbs w/ibstat) but that was not done to avoid a >> > change to the umad port structure. >> >> ibstat determines is_fdr10 by reading a file. > > Yes, same file that active_speed comes from (which umad already parses). > That same data could just as easily be exported as a port attribute, which >> would make it belong to ibverbs, rather than umad. >> > Sure; that's another way. > >> If the only point of exporting is_fdr10 from the kernel is for an ib >> management diag to display the value, then it's not a useful value for verbs >> applications. >> > > fdr10 is in the same category as active_speed. What's the use for that > other than display ? Anyhow, display seems useful to me to know the local > port speed. > > -- Hal > --0016367fb52f2cf13e04ad9e7016 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

On Fri, Sep 23, 2011 at 12:33 PM, Hal Ro= senstock <= hal.rosenstock@gmail.com> wrote:


On Fri= , Sep 23, 2011 at 12:21 PM, Hefty, Sean <sean.hefty@intel.com> wrote:
> > The only way to determine whether fdr10 is active or not is = via the
> > vendor proprietary MAD. That info may be reflected in some other = API
> > (and/or file) so that MAD does not need to be reissued. In a sepa= rate
> > thread on linux-rdma, there was discussion on a couple of differe= nt
> > ways to do that from verbs and in this thread that there's no= sysfs
> > equivalent in Windows. You've already stated that the Windows= support
> > is using libibverbs for libibumad support so it seems appropriate= to
> > me to do the same here (in Windows at least).
>
> The proper place for is_fdr10 is in libibumad (and then we wouldn'= t be
> discussing libibverbs w/ibstat) but that was not done to avoid a
> change to the umad port structure.

ibstat determines is_fdr10 by reading a file. =C2=A0
=C2=A0
Yes, same file that active= _speed comes from (which umad already parses).
=C2=A0
That same data could just as easily be exported as a port attribute, which = would make it belong to ibverbs, rather than umad.
=C2=A0
Sure; that's another way.<= /div>
=C2=A0

If the only point of exporting is_fdr10 from the kernel is for an ib manage= ment diag to display the value, then it's not a useful value for verbs = applications.

fdr10 is in the same category as ac= tive_speed. What's the use for=C2=A0that other than display ? Anyhow, d= isplay seems useful to me to know the local port speed.
=C2=A0
-- Hal

--0016367fb52f2cf13e04ad9e7016-- --===============1747400220== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ ofw mailing list ofw@lists.openfabrics.org http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw --===============1747400220==--