From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hal Rosenstock Subject: Re: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through netlink Date: Wed, 10 Jun 2015 15:51:54 -0400 Message-ID: <5578955A.4070001@dev.mellanox.co.il> References: <1433861837-26177-1-git-send-email-kaike.wan@intel.com> <1433861837-26177-5-git-send-email-kaike.wan@intel.com> <55787838.3020606@dev.mellanox.co.il> <20150610191026.GA28334@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150610191026.GA28334-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jason Gunthorpe Cc: kaike.wan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, John Fleck , Ira Weiny List-Id: linux-rdma@vger.kernel.org On 6/10/2015 3:10 PM, Jason Gunthorpe wrote: > On Wed, Jun 10, 2015 at 01:47:36PM -0400, Hal Rosenstock wrote: >> On 6/9/2015 10:57 AM, kaike.wan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org wrote: >>> From: Kaike Wan >>> >>> This patch routes a SA pathrecord query to netlink first >> >> Should only unicast PRs be done in this manner or should API support >> enabling for unicast and/or multicast ? >> >> AFAIK kernel doesn't query multicast PRs now (queries MCMRs) but this >> seems like it would help make it future proof and not have to take >> timeout on local query unless app supports it. > > It is a good question. We can clearly extend toward that, using a MGID > as the DGID and adding additional nested netlink fields. > > However, does it make sense? If it doesn't make sense, then should MGIDs as DGIDs never be requested via the PR netlink API ? > I don't think user space should be > joining on behalf of the kernel... Agreed. > Do we ever query without a join? Query is possible prior to join. The use case I'm aware of is doing query PR for MGID rather than query MCMR. That case could be viewed just like unicast PRs. -- Hal > Jasno > -- 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