Re: ib mad definitions

[Ira Weiny <weiny2-i2BcT+NCU+M@public.gmane.org>]

On Tue, 19 Oct 2010 11:50:46 -0700
"Hefty, Sean" <sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote:

> > ib_types depends on complib at the moment (fixable)
> > ibutils depends on OpenSM (it will anyway -- non-issue)
> > somethings in ib_types are ugly, byteswapping (non-issue; deal with it
> > later)
> > OpenSM may _not_ include umad and therefore miss defines. (fixable?)
> > 
> > As for this last item, would it be a big deal to require umad for the
> > header
> > only?  Does umad not compile somewhere that other vendor layers are used?
> > I
> > think it is much better for OpenSM to require umad than for other MAD
> > processing software to require OpenSM.  Also, would splitting ib_types help
> > this at all?
> 
> I'll propose the following:
> 
> 1. Add to libibumad/include/infiniband:
> 
>    umad_types.h - basic mad, rmpp headers
>    umad_sa.h    - SA attributes
>    umad_cm.h    - CM messages
> 
> 2. Include umad_types.h and umad_sa.h from ib_types.h
> 3. Include umad_cm.h from ib_cm_types.h
> 
> We start with a minimal set of definitions to umad and add/move other definitions later as needed, creating new header files where appropriate (umad_smi.h, umad_pm.h, etc.)
> 
> If we can get some basic agreement on this, I'll start on the patches immediately.  In an ideal world, the new header files would work on any platform.

I agree,
Ira

> 
> - Sean
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://BLOCKEDvger.kernel.org/majordomo-info.html
> 


-- 
Ira Weiny
Math Programmer/Computer Scientist
Lawrence Livermore National Lab
925-423-8008
weiny2-i2BcT+NCU+M@public.gmane.org
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html