Messages in this thread |  | | Date | Tue, 14 Apr 2015 12:21:37 -0600 | From | Jason Gunthorpe <> | Subject | Re: [PATCH v3 07/28] IB/Verbs: Reform IB-ulp ipoib |
| |
On Tue, Apr 14, 2015 at 06:02:47PM +0000, Hefty, Sean wrote: > > Yes, that is the only reasonable thing that could happen. > > > > init failure should only be possible under exceptional cases (OOM). > > > > The only system response is to call ib_umad_add_one again - so of > > course the first call had to completely clean up everything it did. > > A reasonable follow up change would be to replace the add device > callbacks with add port callbacks.
Yes, combined with a port argument to ib_set_client_data / ib_get_client_data it would be a nice simplifying clean up.
It would be nice to have sane error handling too :( In an ideal world the add call back should return an error and the thing that triggered it should unwind back to module load failure.
Jason
|  |