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

[Sean Hefty]

On Fri, 10 Sep 2004 13:35:18 -0700
Roland Dreier <roland at topspin.com> wrote:

> 	struct ib_client {
> 		void (*add)   (struct ib_device *);
> 		void (*remove)(struct ib_device *);
> 
> 		struct list_head list;
> 	};
> 
> 	int ib_register_client  (struct ib_client *client);
> 	int ib_unregister_client(struct ib_client *client);

I agree with the behavior that you've defined.  I would use the names ib_reg_client/ib_dereg_client to better match the existing APIs.  (Same comments with "register/unregister" names below.)
 
> 	int ib_register_event_handler  (struct ib_device *device,
> 					struct ib_event_handler *event_handler);
> 	int ib_unregister_event_handler(struct ib_event_handler *event_handler);
> 	void ib_dispatch_event(struct ib_event *event);

What use do you see for ib_dispatch_event?