Ira Weiny wrote: > On Tue, 19 Oct 2010 11:50:46 -0700 > "Hefty, Sean" <sean.he...@intel.com> 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
Just to be painfully clear ... A user-mode application would then only need to include ib_types.h + CM flavor of choice .h files ? > >> >> - Sean >> -- >> To unsubscribe from this list: send the line "unsubscribe >> linux-rdma" in >> the body of a message to majord...@vger.kernel.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 > wei...@llnl.gov -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html