From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: device attr cleanup (was: Handle mlx4 max_sge_rd correctly) Date: Wed, 9 Dec 2015 10:44:48 -0800 Message-ID: <20151209184448.GC4522@infradead.org> References: <566753E3.9060301@redhat.com> <20151208225940.GB27609@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Or Gerlitz Cc: Jason Gunthorpe , Doug Ledford , Sagi Grimberg , Christoph Hellwig , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Steve Wise , Or Gerlitz List-Id: linux-rdma@vger.kernel.org On Wed, Dec 09, 2015 at 01:13:29AM +0200, Or Gerlitz wrote: > > Christoph patch is here indeed, it does two things > > 1. remove all the ULP device attr alloc, device query, attr free hassle > 2. adds tons of new fields to struct ib_device > > I think it just goes too much and needlessly adds tons of these new > fields directly to struct ib_device where we can have them all well > scoped into ib_device_attr member or pointer from struct ib_device What's the benefit of that? And looking at the existing members of struct ib_device what determines if it goes straight into the device or the attribute? There is a reason why we don't do this weird attr split in other Linux subsystems, and making IB follow this pattern makes everyone feel right at home instead of wondering about the weird attribute. -- 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