From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH v1 08/12] IB/cma: Add net_dev and private data checks to RDMA CM Date: Thu, 16 Jul 2015 12:22:20 -0600 Message-ID: <20150716182220.GE3680@obsidianresearch.com> References: <1434976961-27424-1-git-send-email-haggaie@mellanox.com> <1434976961-27424-9-git-send-email-haggaie@mellanox.com> <20150713181414.GJ23832@obsidianresearch.com> <55A63CAC.60603@mellanox.com> <20150715184934.GD23588@obsidianresearch.com> <20150715210342.GA32516@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org To: Liran Liss Cc: Haggai Eran , Doug Ledford , "linux-rdma@vger.kernel.org" , "netdev@vger.kernel.org" , Guy Shapiro , Shachar Raindel , Yotam Kenneth List-Id: linux-rdma@vger.kernel.org On Thu, Jul 16, 2015 at 12:01:55PM +0000, Liran Liss wrote: > - Name space lookup is done based on BTH.pkey, private_data.IP, and > optionally GRH.DGID (if present, for extra validation) Just changing the pkey to BTH.pkey would be fine by me. Using GRH.DGID if available instead of the primary path hack is also smart, I pointed that out at the beginning.. Jason