Open MPI logo

Hardware Locality Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |   all Hardware Locality Development mailing list

Subject: Re: [hwloc-devel] ibverbs -> not just infiniband!
From: Brice Goglin (Brice.Goglin_at_[hidden])
Date: 2009-10-21 08:10:21

Jeff Squyres wrote:
> FWIW: the name has been OpenFabrics / OFED for a few years now. Not
> that I wholly disagree -- we're still stuck with the "openib" module
> name in Open MPI because we named it several years ago when it was
> called OpenIB -- but I think the "OpenFabrics" name is pretty stable.
> OpenIB was an informal name that existed before there was an official
> organization behind it. OpenFabrics is the legal entity that was
> created to support all things related to this technology, so I don't
> think that name will be changing any time soon. Indeed, there's a
> *lot* of money put into the marketing and branding with the name
> "OpenFabrics".
> There hasn't [yet?] been discussion of renaming <infiniband/verbs.h>
> (or some of the other IB-centric struct/symbol names), but the whole
> package is very definitely marketed as "OpenFabrics verbs", not
> "InfiniBand verbs" (although the IB vendors certainly don't correct
> this misconception ;-) ).
> So I still would like to rename this file before release.

Ok :) I think I'd vote for some like ofed-verbs.h then, it'd match the
existing glibc-sched.h and linux-libnuma.h

> By the way, this file actually only works for Linux so far. Unless we
> Sun is porting the OpenFabrics verbs to Solaris (to replace their DAPL
> stack). There is also talk of porting the verbs API to MS Windows,
> although I'm not tracking that effort at all. If all this comes to
> fruition, it'll be 3 different platforms that expose the same verbs API.

I thought verbs already existed on more than Linux actually. What I
meant is that *our* ibverbs.h code is Linux specific (it uses a sysfs
specific nice feature of OFED/Linux). If we want to keep this file
portable, we'll need to port hwloc_ibverbs_get_device_cpuset() to
non-Linux OS one day, which means we need a #ifdef LINUX in this public
header. However, IIRC our #define LINUX_SYS is internal only so far.