From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH v3 for-next 02/33] IB/core: Add kref to IB devices Date: Thu, 30 Apr 2015 15:28:42 -0600 Message-ID: <20150430212842.GB7709@obsidianresearch.com> References: <553DF294.4070507@mellanox.com> <553F931F.6000302@mellanox.com> <20150428174312.GB5497@obsidianresearch.com> <5540F8F4.5010906@dev.mellanox.co.il> <20150429164847.GA12781@obsidianresearch.com> <5541E5ED.7000606@mellanox.com> <1828884A29C6694DAF28B7E6B8A82373A8FC929B@ORSMSX109.amr.corp.intel.com> <20150430175252.GB32666@obsidianresearch.com> <1828884A29C6694DAF28B7E6B8A82373A8FC9419@ORSMSX109.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1828884A29C6694DAF28B7E6B8A82373A8FC9419-P5GAC/sN6hkd3b2yrw5b5LfspsVTdybXVpNB7YpNyf8@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Hefty, Sean" Cc: Matan Barak , Or Gerlitz , Somnath Kotur , Roland Dreier , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-rdma@vger.kernel.org On Thu, Apr 30, 2015 at 07:21:08PM +0000, Hefty, Sean wrote: > > But, how could RDMA CM work? Inbound CM messages will be filtered if > > the IP is not in the HW GID table? > > I'm not understanding the issue. > > If a device has some requirement to program its assigned IP > addresses into some HW table, I don't see why upper layers should be > made to care. Okay, I was only thinking about the first couple patches in this series. Three drivers will have this HW gid table, so having a driver helper library in the common code makes sense to me. But then the series it just seems to go crazy - what is with all these net dev patches? Now we are doing something with bonding? And a lot of screwy looking rocev2 gid type stuff? And driver updates? And 33 patches worth? You are totally right, this GID index and GID type stuff is getting *everywhere*, and it is hard to follow the *why* of all the changes are really needed. Matan, if you want to progress this, then split it up. Make a patch to library the existing roce GID table driver code and update the drivers. Just that bit alone has already spawned a huge discussion. Work the other ancillary unrelated patches in small chuncks through their trees. Then come back to rocev2 with an actual core API proposal patch set that can be discussed. Honestly, I'm not willing to even look at a patch set this big and rambly. 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