Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932189AbbD1AxZ (ORCPT ); Mon, 27 Apr 2015 20:53:25 -0400 Received: from p3plsmtpa06-01.prod.phx3.secureserver.net ([173.201.192.102]:56503 "EHLO p3plsmtpa06-01.prod.phx3.secureserver.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751840AbbD1AxY (ORCPT ); Mon, 27 Apr 2015 20:53:24 -0400 Message-ID: <553EDA01.9040708@talpey.com> Date: Mon, 27 Apr 2015 17:53:21 -0700 From: Tom Talpey User-Agent: Mozilla/5.0 (Windows NT 6.3; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Doug Ledford CC: "ira.weiny" , Michael Wang , Liran Liss , Roland Dreier , Sean Hefty , Hal Rosenstock , "linux-rdma@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Steve Wise , Jason Gunthorpe , Tom Tucker , Hoang-Nam Nguyen , "raisch@de.ibm.com" , Mike Marciniszyn , Eli Cohen , Faisal Latif , Jack Morgenstein , Or Gerlitz , Haggai Eran Subject: Re: [PATCH v6 01/26] IB/Verbs: Implement new callback query_transport() References: <1429878230-11749-1-git-send-email-yun.wang@profitbricks.com> <1429878230-11749-2-git-send-email-yun.wang@profitbricks.com> <553DE799.5050608@profitbricks.com> <20150427215229.GD5347@phlsvsds.ph.intel.com> <553ED159.2090006@talpey.com> <1430181360.44548.35.camel@redhat.com> In-Reply-To: <1430181360.44548.35.camel@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2454 Lines: 62 On 4/27/2015 5:36 PM, Doug Ledford wrote: > On Mon, 2015-04-27 at 17:16 -0700, Tom Talpey wrote: >> On 4/27/2015 2:52 PM, ira.weiny wrote: >>> On Mon, Apr 27, 2015 at 09:39:05AM +0200, Michael Wang wrote: >>>> On 04/24/2015 05:12 PM, Liran Liss wrote: >>>>> [snip] >>>> >>>> Like: >>>> >>>> enum rdma_protocol { >>>> RDMA_PROTOCOL_IB, >>>> RDMA_PROTOCOL_IBOE, >>>> RDMA_PROTOCOL_IWARP, >>>> RDMA_PROTOCOL_USNIC_UDP >>>> }; >>>> >>>> So we could use query_protocol() to ask device provide the protocol >>>> type, and there will be no mixing with the legacy transport type >>>> anymore :-) >>> >>> I'm ok with that. I like introducing a unique namespace which is clearly >>> different from the previous "transport" one. >> >> I agree the word "transport" takes things into the weeds. >> >> But on the topic of naming protocols, I've been wondering, is there >> some reason that "IBOE" is being used instead of "RoCE"? > > Because back in the day, when RoCE was accepted into the kernel, I'm > pretty sure it was prior to the IBTA's final stamp of approval and > before the name was set on RoCE, so IBoE was chosen upstream as the more > "correct" name because it properly denoted what it was deemed to truly > be: IB Verbs over Ethernet. Well history is all well and good, but it seems weird to not use the current, standard name in new code. It confuses me, anyway, because it seems like IBOE could easily mean something else. >> Also wondering, why add "UDP" to USNIC, is there a different USNIC? > > Yes, there are two transports, one a distinct ethertype and one that > encapsulates USNIC in UDP. But this new enum isn't about transport, it's about protocol. So is there one USNIC protocol, with a raw layering and a separate one with UDP? Or is it one USNIC protocol with two different framings? Seems there should be at least the USNIC protocol, without the _UDP decoration, and I don't see it in the enum. > >> Naming multiple layers together seems confusing and maybe in the end >> will create more code to deal with the differences. For example, what >> token will RoCEv2 take? RoCE_UDP, RoCE_v2 or ... ? > > Uncertain as of now. Ok, but it's imminent, right? What's the preference/guidance? -- 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/