Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966247AbbDVQ5i (ORCPT ); Wed, 22 Apr 2015 12:57:38 -0400 Received: from quartz.orcorp.ca ([184.70.90.242]:43686 "EHLO quartz.orcorp.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965464AbbDVQ5e (ORCPT ); Wed, 22 Apr 2015 12:57:34 -0400 Date: Wed, 22 Apr 2015 10:57:24 -0600 From: Jason Gunthorpe To: Doug Ledford Cc: Liran Liss , Michael Wang , Roland Dreier , Sean Hefty , Hal Rosenstock , "linux-rdma@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "hal@dev.mellanox.co.il" , Tom Tucker , Steve Wise , Hoang-Nam Nguyen , "raisch@de.ibm.com" , Mike Marciniszyn , Eli Cohen , Faisal Latif , Jack Morgenstein , Or Gerlitz , Haggai Eran , Ira Weiny , Tom Talpey Subject: Re: [PATCH v5 00/27] IB/Verbs: IB Management Helpers Message-ID: <20150422165724.GC19500@obsidianresearch.com> References: <5534B8C9.506@profitbricks.com> <1429714792.45956.110.camel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1429714792.45956.110.camel@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Broken-Reverse-DNS: no host name found for IP address 10.0.0.183 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1311 Lines: 33 On Wed, Apr 22, 2015 at 10:59:52AM -0400, Doug Ledford wrote: > > 2)The name rdma_tech_* is lame. > > rdma_transport_*(), adhering to the above (*) remark, is much better. > > For example, both IB and ROCE *do* use the same transport. > > I especially want to second this. I haven't really been happy with the > rdma_tech_* names at all. I'm not excited about the names either.. cap_ is bad because it pollutes the global namespace. rdma_tech_ .. as used, this is selecting the standard the port implements. The word 'standard' is a better choice than 'transport', and 'technology' is often synonymous with 'standard'. Meh. I've said it already, but this patch set has probably gotten too big. If we could just do the cap conversion without messing with other stuff, or adding rdma_tech, that would really be the best. Nobody seems to like the rdma_tech parts of this series. I'd also drop '[PATCH v5 09/27] IB/Verbs: Reform IB-core verbs/uverbs_cmd/sysfs' - that is UAPI stuff, it could be done as a followup someday, not worth the risk right now. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/