> I think there are lots of #defines and enums replicated in both
> nes_netlink.h and c4iw_netlink.h.  For example, the IWPM_* defines and
> enums.  It seems like we could put all this in a common header file to
> be included by all iwarp providers?  Say include/rdma/iw_portmap.h or
> something.  Basically the interface to the user mode daemon is the same
> for all providers, no?  Those bits should be in a common header.  (and
> maybe even a common core module if there is enough common functions).
> 
> Thoughts?

I agree.  This enum change looks off:

+       RDMA_NL_NES,
+       RDMA_NL_C4IW

Why is the communication with the daemon device specific?  Can the iwarp cm 
fully encapsulate these changes?

- Sean
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to