From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 038A737B8 for ; Tue, 24 Nov 2015 16:49:30 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga101.jf.intel.com with ESMTP; 24 Nov 2015 07:49:29 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,338,1444719600"; d="scan'208";a="846156228" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.208.62]) by fmsmga001.fm.intel.com with SMTP; 24 Nov 2015 07:49:28 -0800 Received: by (sSMTP sendmail emulation); Tue, 24 Nov 2015 15:49:27 +0025 Date: Tue, 24 Nov 2015 15:49:27 +0000 From: Bruce Richardson To: "Mrzyglod, DanielX T" Message-ID: <20151124154926.GB17696@bricha3-MOBL3> References: <1448377959-4440-1-git-send-email-danielx.t.mrzyglod@intel.com> <7ADD74816B4C8A45B56203CBA65FE5A61D98E602@IRSMSX107.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <7ADD74816B4C8A45B56203CBA65FE5A61D98E602@IRSMSX107.ger.corp.intel.com> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH] net: fix build with gcc 4.4.7 and strict aliasing X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 24 Nov 2015 15:49:31 -0000 On Tue, Nov 24, 2015 at 03:47:28PM +0000, Mrzyglod, DanielX T wrote: > This error fix this situation for IPv6 checksum offload error on RHEL65 > Any optimalisation above -O0 provide error in IPv6 checksum > > > Step 1 : start testpmd > ./x86_64-native-linuxapp-gcc/app/testpmd -c 0x6 -n 4 -- -i --portmask=0x3 --disable-hw-vlan --enable-rx-cksum --crc-strip --txqflags=0 > > Step 2 : settings and start > set verbose 1 > set fwd csum > start > > Step 3 : calculate correct checksum values of IPv6/TCP and IPv6/UDP by scapy > Packets info: IPv6/UDP:Ether(dst="02:00:00:00:00:00", src="90:e2:ba:4a:33:5c")/IPv6(src="::2")/UDP()/("X"*46) > IPv6/TCP: Ether(src="52:00:00:00:00:00", dst="90:e2:ba:4a:33:5d")/IPv6(src="::1")/TCP()/("X"*46) > > Step 4 : Send two packets with wrong checksum value,and calculate the right checksum value by port,packets received on another port > Send packets info: IPv6/UDP：Ether(dst="90:e2:ba:4a:33:5d", src="52:00:00:00:00:00")/IPv6(src="::1")/UDP(chksum=0xf)/("X"*46) > IPv6/TCP：Ether(dst="90:e2:ba:4a:33:5d", src="52:00:00:00:00:00")/IPv6(src="::1")/TCP(chksum=0xf)/("X"*46) > > RESULTS: > 'IPv6/TCP': ['0xd41']}, 'IPv6/UDP': ['0x7d07'], > > EXPECTED RESULTS: > 'IPv6/TCP': ['0x9f5e']}, 'IPv6/UDP': ['0xf26'] Hi Daniel, this sort of detail - perhaps in abbreviated form - should be in the commit message for the patch. Can you perhaps do a V2 to include it? /Bruce