From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Hefty, Sean" Subject: RE: [PATCH 3/3] IB/sa: route SA pathrecord query through netlink Date: Tue, 19 May 2015 20:27:22 +0000 Message-ID: <1828884A29C6694DAF28B7E6B8A82373A8FDD2D8@ORSMSX109.amr.corp.intel.com> References: <1431975616-23529-1-git-send-email-kaike.wan@intel.com> <1431975616-23529-4-git-send-email-kaike.wan@intel.com> <20150519190742.GO18675@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <20150519190742.GO18675-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> Content-Language: en-US Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jason Gunthorpe , "Wan, Kaike" , "Hal Rosenstock (hal-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org)" Cc: "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "Fleck, John" , "Weiny, Ira" List-Id: linux-rdma@vger.kernel.org > I have mixed feelings about using the SA message format for this. > > In-kernel we don't use that format, and many of the newer APIs were > designed around passing the GMP,Forward,Return path tuple, not just a > naked GMP. I think the choice of the netlink protocol is a largely matter of taste. In the internal review of these patches, the netlink hook was previously in the mad layer, which would have allowed redirecting any mad to user space. (The code only attempted to redirect SA queries.) Trying to hook from within ib_mad added a fair amount of complexity, that moving up into sa_query avoided. The proposed interface would allow moving the changes back down, versus limiting this only to the SA query calls. (And I'm not meaning to imply that limiting this only to SA queries is a bad choice.) > I'd rather see the netlink data ask for what the kernel needs (eg UD > path to X, RC path to X) and return the new format we've already > established for AF_IB Does anyone else have input on the netlink protocol? Hal/Ira? MADs seem more flexible, but higher-level structures more efficient and natural. -- 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