[openib-general] Re: [PATCH] fix umad object lifetime stuff

[Sean Hefty]

Roland Dreier wrote:
> I just committed the following patch for user_mad.c, which fixes
> various issues with possibly freeing various data structures before
> the last reference is gone.  For example, cdev_del() might return
> before the last reference to the cdev is gone, so freeing a structure
> containing the cdev is wrong at that point.  (Side note: it's
> essentially impossible to use cdev_init() safely unless the cdev in
> question is statically allocated as part of the module).

I can't say that I grasp the relationship between the cdev_* and class_* calls 
yet, but should umad and ucm create their own classes?  I'm trying to add the 
ucma, and I'm wondering if we should add another infiniband_blah class, versus 
adding an rdma_cm entry somewhere else.

- Sean