From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [RFCv2 00/15] RFCv2: Consolidated userspace RDMA library repo Date: Fri, 16 Sep 2016 10:16:26 -0600 Message-ID: <20160916161626.GB2833@obsidianresearch.com> References: <1471889618-1605-1-git-send-email-jgunthorpe@obsidianresearch.com> <1828884A29C6694DAF28B7E6B8A82373AB07FB7A@ORSMSX109.amr.corp.intel.com> <1828884A29C6694DAF28B7E6B8A82373AB07FBE1@ORSMSX109.amr.corp.intel.com> <20160913180342.GA6933@obsidianresearch.com> <1828884A29C6694DAF28B7E6B8A82373AB08075C@ORSMSX109.amr.corp.intel.com> <20160913222539.GB29095@obsidianresearch.com> <1828884A29C6694DAF28B7E6B8A82373AB080798@ORSMSX109.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Doug Ledford Cc: "Hefty, Sean" , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Devesh Sharma , Hal Rosenstock , "Marciniszyn, Mike" , Moni Shoua , Steve Wise , "Nikolova, Tatyana E" , Vladimir Sokolovsky , Yishai Hadas List-Id: linux-rdma@vger.kernel.org On Fri, Sep 16, 2016 at 11:12:17AM -0400, Doug Ledford wrote: > I would. Mainly because it is tied to both the kernel uAPI (via > netlink)and to librdmacm With what little I know about ibacm I'm inclined to agree. It seems like a utility for librdmacm, and is consuming a kernel interface. Reconsider Sean? > (such that librdmacm changes how it builds when ibacm is present, so > if we don't include it, then ibacm becomes a requirement of the > larger package if you want librdmacm to support the ibacm cache > service). I double checked and I couldn't find anything like this. librdmacm/src/amc.c builds unconditionally and contains no ifdefs, configure does not key on anything to do with acm. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html