From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: Re: [PATCH v2 net-next 1/3] rds: Changing IP address internal representation to struct in6_addr Date: Fri, 6 Jul 2018 10:26:32 -0700 Message-ID: <7c6de609-13cb-8648-82fe-7a0332ecc941@oracle.com> References: <693463ed87c5956eed05a6266c4edf354ab3d51a.1530086216.git.ka-cheong.poon@oracle.com> <6a7b0951-d950-1555-a5b2-622d6f7b9f8f@oracle.com> <20180706101505.GA32098@oracle.com> <64196e9e-bb90-9452-5fbf-2004e2bd68e3@oracle.com> <20180706152558.GJ30983@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Sowmini Varadhan , davem@davemloft.net, netdev@vger.kernel.org, rds-devel@oss.oracle.com To: Ka-Cheong Poon Return-path: Received: from userp2130.oracle.com ([156.151.31.86]:60240 "EHLO userp2130.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932834AbeGFR0j (ORCPT ); Fri, 6 Jul 2018 13:26:39 -0400 In-Reply-To: <20180706152558.GJ30983@oracle.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: Hi Ka-Cheong, On 7/6/2018 8:25 AM, Sowmini Varadhan wrote: > On (07/06/18 23:08), Ka-Cheong Poon wrote: >> >> As mentioned in a previous mail, it is unclear why the >> port number is transport specific. Most Internet services >> use the same port number running over TCP/UDP as shown >> in the IANA database. And the IANA RDS registration is >> the same. What is the rationale of having a transport >> specific number in the RDS implementation? > > because not every transport may need a port number. > Lets keep separate port for RDMA and TCP transport. This has been already useful for wireshark dissector and can also help for eBPF like external tooling. The fragment format and re-assembly is different across transports. I do see your point and also agree that port number isn't transport specific and in case we need to add another transport, what port to use. But may be till then lets keep the existing behavior. As such this port switch is not related to IPv6 support as such so lets deal with it separately. Regards, Santosh