[openib-general] Proposed device enumeration & async event APIs

Sean Hefty mshefty at ichips.intel.com
Mon Sep 13 10:14:35 PDT 2004


On Fri, 10 Sep 2004 19:34:26 -0700
Roland Dreier <roland at topspin.com> wrote:

> It makes sense to me to use ib_reg_phys_mr, ib_dereg_mr, etc as names
> of verbs since the IB spec uses the word "deregister" and the rest of
> the verb functions are pretty abbreviated.  Registering a memory
> region seems somehow a little different from registering a callback or
> a client too.

I think I'll leave those as is for now then.  A simple rename, if needed later, is easier to deal with than changes to parameters.
 
> In any case I would like to stick with the ib_register_client function
> names.  In fact in ib_mad.h maybe we should change ib_mad_reg and
> ib_mad_dereg to ib_register_mad_agent and ib_unregister_mad_agent
> (ib_mad_reg seems backwards from the rest of the API, where the verb
> comes before the noun -- eg ib_create_qp).

I agree.  The MAD names should change.  I'll submit a patch for this shortly.



More information about the general mailing list