From owner-svn-src-head@FreeBSD.ORG Mon Dec 1 14:48:39 2014 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id D996EC11; Mon, 1 Dec 2014 14:48:39 +0000 (UTC) Received: from bigwig.baldwin.cx (bigwig.baldwin.cx [IPv6:2001:470:1f11:75::1]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id B0A7A884; Mon, 1 Dec 2014 14:48:39 +0000 (UTC) Received: from ralph.baldwin.cx (pool-173-70-85-31.nwrknj.fios.verizon.net [173.70.85.31]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id 93891B987; Mon, 1 Dec 2014 09:48:38 -0500 (EST) From: John Baldwin To: Hans Petter Selasky Subject: Re: svn commit: r275358 - in head/sys: dev/bxe dev/cxgb dev/cxgbe dev/e1000 dev/ixgbe dev/ixl dev/mxge dev/netmap dev/oce dev/qlxgbe dev/qlxge dev/sfxge dev/virtio/network dev/vmware/vmxnet3 dev/vxge n... Date: Mon, 01 Dec 2014 09:13:55 -0500 Message-ID: <2301463.0AvfjFzmc1@ralph.baldwin.cx> User-Agent: KMail/4.14.2 (FreeBSD/10.1-STABLE; KDE/4.14.2; amd64; ; ) In-Reply-To: <201412011145.sB1BjPEI092904@svn.freebsd.org> References: <201412011145.sB1BjPEI092904@svn.freebsd.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (bigwig.baldwin.cx); Mon, 01 Dec 2014 09:48:38 -0500 (EST) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Dec 2014 14:48:40 -0000 On Monday, December 01, 2014 11:45:25 AM Hans Petter Selasky wrote: > Author: hselasky > Date: Mon Dec 1 11:45:24 2014 > New Revision: 275358 > URL: https://svnweb.freebsd.org/changeset/base/275358 > > Log: > Start process of removing the use of the deprecated "M_FLOWID" flag > from the FreeBSD network code. The flag is still kept around in the > "sys/mbuf.h" header file, but does no longer have any users. Instead > the "m_pkthdr.rsstype" field in the mbuf structure is now used to > decide the meaning of the "m_pkthdr.flowid" field. To modify the > "m_pkthdr.rsstype" field please use the existing "M_HASHTYPE_XXX" > macros as defined in the "sys/mbuf.h" header file. > > This patch introduces new behaviour in the transmit direction. > Previously network drivers checked if "M_FLOWID" was set in "m_flags" > before using the "m_pkthdr.flowid" field. This check has now now been > replaced by checking if "M_HASHTYPE_GET(m)" is different from > "M_HASHTYPE_NONE". In the future more hashtypes will be added, for > example hashtypes for hardware dedicated flows. > > "M_HASHTYPE_OPAQUE" indicates that the "m_pkthdr.flowid" value is > valid and has no particular type. This change removes the need for an > "if" statement in TCP transmit code checking for the presence of a > valid flowid value. The "if" statement mentioned above is now a direct > variable assignment which is then later checked by the respective > network drivers like before. > > Additional notes: > - The SCTP code changes will be committed as a separate patch. > - Removal of the "M_FLOWID" flag will also be done separately. > - The FreeBSD version has been bumped. > > MFC after: 1 month > Sponsored by: Mellanox Technologies Was this reviewed by anyone? I know I saw at least one thread on reworking M_FLOWID on one list, but it's not clear to me that anyone reviewed these specific changes. If someone did, please be sure to include that type of detail in future commits. If not, I would strongly encourage to still seek review even if changes have been agreed upon in the abstract. Phabricator makes this a lot easier to do than previously. -- John Baldwin