> -----Original Message----- > From: Jason Gunthorpe [mailto:jgunthorpe@xxxxxxxxxxxxxxxxxxxx] > Sent: Tuesday, May 17, 2016 7:57 PM > To: Mark Bloch <markb@xxxxxxxxxxxx> > Cc: Doug Ledford <dledford@xxxxxxxxxx>; leon@xxxxxxxxxx; linux- > rdma@xxxxxxxxxxxxxxx > Subject: Re: [PATCH rdma-next V2 5/5] IB/core: Integrate IB address > resolution module into core > > On Tue, May 17, 2016 at 05:00:03AM +0000, Mark Bloch wrote: > > > Are you talking about the changes in ibnl_add_client? > > Those changes were because of a different module (ib_sa.ko) > > Bleck, why is this such a mess? Can't you get everything into one > module so we don't need this dynamic stuff?? Well, seeing as ib_sa.ko doesn't have to be loaded we will still need A dynamic part somewhere, to handle the registration/deregistration. We will probably need a new module to manage that. I think this is doable, but do we need it? I agree there might be a more elegant solution, but I don't see any other modules that will use it. Do we have a use case other than ib_sa/ib_core? Mark -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html