From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chuck Lever Subject: Re: device attr cleanup (was: Handle mlx4 max_sge_rd correctly) Date: Mon, 14 Dec 2015 19:19:28 -0500 Message-ID: References: <566753E3.9060301@redhat.com> <20151208225940.GB27609@obsidianresearch.com> <20151208230244.GA10701@infradead.org> <20151209005203.GD16976@phlsvsds.ph.intel.com> <20151209184235.GB4522@infradead.org> <20151210014556.GA32059@phlsvsds.ph.intel.com> Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2104\)) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <20151210014556.GA32059-W4f6Xiosr+yv7QzWx2u06xL4W9x8LtSr@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Christoph Hellwig Cc: "ira.weiny" , Jason Gunthorpe , Or Gerlitz , Doug Ledford , Sagi Grimberg , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Steve Wise , Or Gerlitz List-Id: linux-rdma@vger.kernel.org > On Dec 9, 2015, at 8:45 PM, ira.weiny wrote: > > On Wed, Dec 09, 2015 at 10:42:35AM -0800, Christoph Hellwig wrote: >> On Tue, Dec 08, 2015 at 07:52:03PM -0500, ira.weiny wrote: >>> Searching patchworks... >>> >>> I'm a bit worried about the size of the patch and I would like to see it split >>> up for review. But I agree Christophs method is better long term. >> >> I'd be happy to split it up if I could see a way to split it. So if >> anyone has an idea you're welcome! > > Well this is a ~3300 line patch which is pretty hard to review in total. > >> >>> Christoph do you have this on github somewhere? Perhaps it is split but I'm >>> not finding in on patchworks? >> >> No need for github, we have much better (and older) git hosting sites :) >> >> http://git.infradead.org/users/hch/rdma.git/shortlog/refs/heads/ib_device_attr Tested-by: Chuck Lever With NFS/RDMA client and server. > Another nice side effect of this patch is to get rid of all the struct > ib_device_attr allocations which are littered all over the ULPs. > > For the core, srp, ipoib, qib, hfi1 bits. Generally the rest looks fine I just > did not have time to really go through it line by line. > > Reviewed-by: Ira Weiny > > Doug this is going to conflict with the rdmavt work. So if you take this could > you respond on the list. -- Chuck Lever chucklever-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org -- 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