From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net] net: diag: Fix swapped src/dst in udp_dump_one. Date: Fri, 21 Sep 2018 07:49:07 -0700 (PDT) Message-ID: <20180921.074907.896069317193269306.davem@davemloft.net> References: <20180914062553.42648-1-lorenzo@google.com> <20180914.083038.452831271321088318.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, zenczykowski@gmail.com, dsahern@gmail.com, jeffv@google.com To: lorenzo@google.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:60138 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728313AbeIUUiU (ORCPT ); Fri, 21 Sep 2018 16:38:20 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Lorenzo Colitti Date: Fri, 21 Sep 2018 18:46:25 +0800 > Would you take a patch to add a one-line comment saying that this is > the way it is for backwards compatibility? If that comment were there > anyone else who finds this will not spend time debugging it and > immediately know what's going on. The fact that the ports are inverted > is not easy to spot on casual inspection. Sure.