From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0072.outbound.protection.outlook.com [104.47.42.72]) by dpdk.org (Postfix) with ESMTP id 4732D1E25 for ; Mon, 8 Oct 2018 11:38:09 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=1lfJMeD/zR0huB+bM7cV8avJUTXmjENpzhjuqZERthg=; b=byI0EHNpBFjSsTxUQszBVLo2ns9dJfFi5UzYXbDymBQ6y7gUUew/aQf8YS3wC3bIjVOrpQxKW6Btshz1KctVAYOJ3VvXyBFa4uDiq7g8cNlDjMBFkARSNisX5tNRbuNIcDkF+PzMP3du1FGHAARpfAXlnl0YQW/ulyVK6pX82dE= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.JacobKollanukkaran@cavium.com; Received: from jerin (115.113.156.3) by BL0PR07MB4996.namprd07.prod.outlook.com (2603:10b6:208:49::25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1207.18; Mon, 8 Oct 2018 09:37:59 +0000 Date: Mon, 8 Oct 2018 15:07:42 +0530 From: Jerin Jacob To: Thomas Monjalon Cc: Ferruh Yigit , "Ananyev, Konstantin" , Andrew Rybchenko , "Lu, Wenzhuo" , "Wu, Jingjing" , "Iremonger, Bernard" , "Mcnamara, John" , "Kovacevic, Marko" , Olivier Matz , "dev@dpdk.org" , "shahafs@mellanox.com" , "didier.pallard@6wind.com" Message-ID: <20181008093741.GA11081@jerin> References: <20180913134707.23698-1-jerin.jacob@caviumnetworks.com> <601d2413-e148-73c4-e7a5-59f09bd02451@intel.com> <20181008082421.GA3554@jerin> <2218090.RkeNvosNi6@xps> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2218090.RkeNvosNi6@xps> User-Agent: Mutt/1.10.1 (2018-07-13) X-Originating-IP: [115.113.156.3] X-ClientProxiedBy: BM1PR0101CA0012.INDPRD01.PROD.OUTLOOK.COM (2603:1096:b00:18::22) To BL0PR07MB4996.namprd07.prod.outlook.com (2603:10b6:208:49::25) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: abe96826-ff4e-4200-816f-08d62d01bfc1 X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(4534185)(7168020)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(2017052603328)(7153060)(7193020); SRVR:BL0PR07MB4996; X-Microsoft-Exchange-Diagnostics: 1; BL0PR07MB4996; 3:N9Fn72R4xGbwVC9Md688Z4jtwmDlX/gu1woEMhAr3SpO3EdCDvdTQYFKxM9H34DBABylSm5N2aWAjHaBuZiqgRR/bEe8GA2TlqCl4CUDlbfwC1WvaWlUs9m6A4k5y01+VvhQTnAeFPh/e+sUUBBSg2PJSu/iMLGGz0qNqw5dGeMWmpYpMIhHhVKnxBEdyJ966EVALOlYwlLYCotinHTAcR+jLMSXx6v4bPnuVqM+afbycqWoSUdIE9KtrLvNNwIS; 25:3NycpbxyfW39Tad5KJwbo0VXgZdD8A2LlEH2aCw1VanZuPBeWdklRfZZKCf9X+zXOQ2O7ymvJmUJEnqevkhS5qUzdPt1fL6/m648An/UQyVygdIzCCQzwkEjzF77V69TEhPmj8PeiPLnsg7eeYvaH3mKGTpyQ7njO5wDigtx52Ty+xDFcFR5XlDtY+u3V2BGgbFRD4bkLI/kpyG4FLJt+D6qfvAXmU2faZutjMJeoy5wuwObbs6IJz9uaCS8srFX6qGcQuARlnr0LekCuqkiE0S/ZuSHKWXjc5RmIHVjVxz8R9FeA1ooKIyxiErJb9suT7/jJqCi76Utbix6vTJhIQ==; 31:rShw036MFzPJ50gf7SMzWXZ9MuFcBvcb7uXUfJApAs10hWz7vr4kRQU7MgvsKAd1VAGV5sMx2x9/ocWtoZRHNmiAqX+90XgvJJrJRIGDBx15hNjizBiWkfuZk1dzdr9fafOLDZHn9lTJvln4q2DW2j16z4GjDogx7Q323a9pBHwT3+i/DWGCgKwXw6Jncds5lV42xityX6wENadmZRalu2mAABYDiIsvk2Ae2yj+Cck= X-MS-TrafficTypeDiagnostic: BL0PR07MB4996: X-Microsoft-Exchange-Diagnostics: 1; BL0PR07MB4996; 20:5ygBXpMm5cFnII7ZXidYuNZytSlP6llcG8hmNHlEhJGqo9/MIC4zrbQldpg8Y/ylWzizGwP0ZH9nOMEOtV+KQS9J6/JSKwFmKd5aPU1d11SrMBR/mQT+74hTHdXE22Ls2nCeuD4GOXyuGHvwnLcz5dE3DbADwusFpQQkRZnmM9nts3GNi+HK3mEKvXhfu1RAxfOjOyGUjaQi75MlXY7KP5rtwEtMfS4DCjTb/PhZFgTb1XwQ/ZtTyEeFTYPRqNGsiWhq/DAtEcgsK24w8PIYKA0OkFkY7wL2Sp7/2zpQamyn2QoNFXKGvaKpxJ/cRLaRRabyEd4bg/DcHKgHurOMW6dTm9+lTTDvGsL7vrrCDSrQgDDH2TQJtiVfnBw0MafQXcP9S0hGGAXaFzWBY+4kOgKQpCCXZByl0w97XEKjSxxRe36OFKDvqLfBeMRBJd7+UygN24fr9FHS9e47pBNbdj0aLCK34fPLR8oG/sRogPxV94EtayEBG9Ge9HYb+6xXHZwmMzXWzC9xYIROsxiZkLvTzO6y+O8xaelJ0vCJERIy2Iol0B09rre1AUiie633gaxQXhzhJumfhLB541E6hDJf0r+wa1uj+Pp4m9WG9sA= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(228905959029699)(211171220733660); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231355)(944501410)(52105095)(3002001)(10201501046)(93006095)(149066)(150057)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(20161123564045)(20161123560045)(20161123562045)(201708071742011)(7699051); SRVR:BL0PR07MB4996; BCL:0; PCL:0; RULEID:; SRVR:BL0PR07MB4996; X-Microsoft-Exchange-Diagnostics: 1; BL0PR07MB4996; 4:ZXoUCwNZPebTSg9xgwtgkLz345gE4FEHRfPESf81DDVgS3IXkeYkFszOroRnTlnSIC2IYEotHbQGpts1GSyAbZrH1dGuqlbjLNWpgbbrXjTFXz24RdrAY1zc9zGB3LrQh3cVDKLrXFonIhJDWZyxu0PP1ec9JS6IyyVqruobVLsVvHEB9ZnPgT9gYhKmlP1zvOKQdzjyVyyWImdBx6D3TlsB4q5cYIilZMDWrjEiY+J7lX6nsMl7pdNZn0IEWKz/QEnilwty2vdkIE84ebXDxZb0bhvz2OdmG/D6VnUn0Qvsna5K3b8dVY1jPuS+tyQxrDaEg+4hI2Fw7AcLPBy/0v+Dh8A4XwgUEIyutsgZ154= X-Forefront-PRVS: 081904387B X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(39860400002)(376002)(136003)(396003)(346002)(366004)(13464003)(57704003)(199004)(189003)(33896004)(55016002)(52116002)(6496006)(9686003)(76176011)(1076002)(229853002)(6116002)(3846002)(7416002)(53936002)(66066001)(956004)(11346002)(47776003)(486006)(23726003)(476003)(44832011)(6246003)(478600001)(58126008)(54906003)(72206003)(16526019)(33716001)(42882007)(14444005)(53546011)(386003)(186003)(50466002)(33656002)(106356001)(105586002)(446003)(4326008)(68736007)(316002)(5660300001)(6666003)(6916009)(16586007)(2906002)(8936002)(81166006)(8676002)(81156014)(55236004)(25786009)(97736004)(93886005)(7736002)(305945005)(26005)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:BL0PR07MB4996; H:jerin; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BL0PR07MB4996; 23:f9hxwz/rhydgZv1/sYVnmSad9bgiJqIc1b4ctcysr?= =?us-ascii?Q?GHSKzseIA+TgDeU2d/yMymru1GCtuK09PQ9z9XsHiS3XVYZivkvyscXVGG+i?= =?us-ascii?Q?JjaFAoaw/nMEZzgwO1LS9kkzd+vRjSJQv3fm/Nl2e+S5AZnshCCle1GuBibd?= =?us-ascii?Q?ouCR6AxcwSHG+OnmkjdPNLSfwte0+na1KNB4Xh/Xzn5Ld60F/EatGcbbE0+j?= =?us-ascii?Q?IeTQ+geKEzIfe5DBe0II0zT6rZsNko7d8WyEIKt7QZxTqdK5rUmyOhmZcJV1?= =?us-ascii?Q?ijkWmW1bZuh2qHE24nmJQ5eN9KZSG5Trj1ekGWUS5camZkKILIqocHRx5PoV?= =?us-ascii?Q?qetEO0WNz/Zbnr9HTZOGJhch9jMUCjN7cB5rwz864RL5Ku2ta0B7H0HzF+0G?= =?us-ascii?Q?buvzmUa00WCDY4x2pyt24k9bKvyecV8HWcNM12aHqxW8eMoZuhhaQmJbWZ3A?= =?us-ascii?Q?0wYkrZg3kXD0WgfPUAbDIPJtcFKnv8MnaRsfyuTHWbjPxYx5tsrsA1DiTbi2?= =?us-ascii?Q?/g8Rmk6PYAwVIPx6MoHPXrWTvh37rs4rHzcC2tYYuiwDRg6cqgA5Ie2FHr/v?= =?us-ascii?Q?HnSy9I3Mexoub4X0XihiRobFcz6OwWZdswE6tCU38MRmWMq5PJKwtq1baXaH?= =?us-ascii?Q?s7kCMdhoBN2nh873kboknkuoBwJhC1eAYzV2y0e/+ntf/C5HNsMhnVRFm1fK?= =?us-ascii?Q?xkpkzc70QeSd4gE5IVCMkVM1eg7/UwrIXZG5MgH6y4Fng2VU3U04Swd3wdO6?= =?us-ascii?Q?tSuDf63yJ+lJnT8W053pIHbQi4iU39Fqg8pG8kS73wEAyjsRYCFN7nzCLWXh?= =?us-ascii?Q?l1cD/WV38GwnZquOS5dLiueJD5viJI7JDlqrLmxqrCdgmPZcbf9BNMWXQZ8A?= =?us-ascii?Q?zEGM4JQq4ryPzSwnZmuyktgnxt+rJRr7B1DfU3ENEXEO2ARK9cir0HWgZmkN?= =?us-ascii?Q?Q60HTl0wOnF7Vo638CelLNleU4A8dB+0+giJFymO6c/o56FdKanggjjfsBVe?= =?us-ascii?Q?Ow403UP6w+ZhrkfN1s3o0JbuyFN1bc5CYX93mWGm15tumUepW9yIqKQdfdsR?= =?us-ascii?Q?INFeOi9ESNWvWKA1jSHQDe1w0C/R7VkpzRNY2xmpJLx/9y8FC+EGmO2OaRg7?= =?us-ascii?Q?W0BG/Qd0uW1ZDk5eAiMiDh9RoMQhjCmceezSEFTqlHuoJ57b6bvxng0I2aON?= =?us-ascii?Q?bODmv8kVRcqFZVYGovoy17ZYdvtNeZ4aPFLiAP0PM09t/UNoF1/vfE6lmDFt?= =?us-ascii?Q?cKBJ53oAe5MaL24E8srJgnZ4rOFO/hrCE6+uaYzTI856niPpxogy6iLtrvdv?= =?us-ascii?Q?PHXQ1FMsL0XtjS4WsFUdpX/Ibk8uZERWVKRxr0Tt38H+Pkss0jUhrp+W7FkI?= =?us-ascii?Q?bvpmw3BcaTaq0DI5T/aOQBE/9Z/fhjYBWY4FJPtLqFBLXuaOgXmZopBFNMXJ?= =?us-ascii?Q?7zYLzMQ1gDLm9EDSJ+CDcOevSv99l/HFgG45yjt0iSlLSln25ly?= X-Microsoft-Antispam-Message-Info: GIqozHzGqOg6GJNMrG2XgpINgqzMoMVeQJxVPFPBzTGqbPfCcyJwFsMWlNd+0qV3dd/6CiDqSke3OzUzNaJYliFC+sfMb3GknmGefsmL6LBnhOYacrIh0B1kqSJnZ51guci0oazlARzZ+vfgxUhez4PVMbC9hTMH5WsHaxYqnXWhTBrzsJYV1G7HzDQ8oaFfUFYL9Vooz/KEnueVxVu8RY/etXIiyQWy80qaXnq4cpNBmgVtrXVQf40UmswAnX5lSNs50O7ycWkYUxDQcnYXmRDW5VUowq6kWp63ki+XMjDr3jKjncDET7Rps9tJo1vUsP0S3dBa0lB4kXs83P0UoQe2OKwvJH2ScsgHOF3kit0= X-Microsoft-Exchange-Diagnostics: 1; BL0PR07MB4996; 6:vsTx46HBZUiMs+3FeGUzIlEF1TBt8W7btV+McbGKN1PdRVqR4J3jXJA/m2EJI+TBmxBmeVRHZFJT4OKHogMH6uQEH67hn/IdvDt+B2loVuPqzQTvtBO3Lol2C2aUi0pOHKRz+s2bonJuxIi/3/ZZdnX5ONhhQSBMsll7ns9QTjI5rOkdvD1aKFG6Gq39x5oWZFKpshrIXMKQ9j1lMAJE+QFiUvi1CxEHMLmF896evQMZUqwBMZAbnqji/vjy2VWCyPz1pciBXI6gpHy2u89W7hAEo/+tyqUpJbfw1YBS48OMbEaTekAiLYGFzRFKZMDftq8JP5aXgkXYBOVDgEwpAZTHAAfiWRSCLd+Qqy0rOWod64UIbWFeCe9ecV22wiZGsscih92yzSP5M0ehBy7LrG4R4kDPCsfUGkdkY3lfWvhXq/S6I1CcmeNu/q4/ko3Xcj3PhsFG9cTRqVz8s+DZqw==; 5:Zw+K5gv81RjtIT0XjGsUptXLJGmJCMJqmIxEo1V0kadenvw9smr3yloGDRWEHVJfqNgS+qam397M+CfKmxt4KX+OlBttCDK9m+6rxPPcVK8TuL1vkVfdGJd8YhhunUfYAfFCf7CCAZZbA+I2AVEVeA2B1YwVRlWRBUUTjhS7Pjg=; 7:xxAxDfBQSF+Y8GZyI6DA38j1I7hwOhHM4n+VtHpEXQ5j72ZRnUtXgiMKLROsroZldR917eiri+Hrs+UfY7ZeZlE3Xa/a4GrFt0oW/dpPlzUt6sNGERrryekISXWb3SC+ZBktFesW4gKSmYjWDRomxEJFIfS6LNEOhoCGl1N7qTu+uoOFWYtisu01wJ3SA57wShdzurUMeOCQ8JqQdVwWXwIMcyRF1/oBiaI32QtdVmWwhOFsAWw4xz93d00IbHtD SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Oct 2018 09:37:59.0162 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: abe96826-ff4e-4200-816f-08d62d01bfc1 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR07MB4996 Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP checksum definition X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Oct 2018 09:38:10 -0000 -----Original Message----- > Date: Mon, 08 Oct 2018 11:04:51 +0200 > From: Thomas Monjalon > To: Jerin Jacob , Ferruh Yigit > , "Ananyev, Konstantin" > > Cc: Andrew Rybchenko , "Lu, Wenzhuo" > , "Wu, Jingjing" , > "Iremonger, Bernard" , "Mcnamara, John" > , "Kovacevic, Marko" , > Olivier Matz , "dev@dpdk.org" , > "shahafs@mellanox.com" , "didier.pallard@6wind.com" > > Subject: Re: [dpdk-dev] [PATCH v2 1/4] ethdev: add Rx offload outer UDP > checksum definition > > 08/10/2018 10:24, Jerin Jacob: > > From: Ferruh Yigit > > > On 10/6/2018 1:18 PM, Ananyev, Konstantin wrote: > > > > From: Jerin Jacob [mailto:jerin.jacob@caviumnetworks.com] > > > >> From: Thomas Monjalon > > > >>> However, we should re-visit the flag PKT_RX_EIP_CKSUM_BAD. > > > >> > > > >> Do we need to block this patch due to the exiting PKT_RX_EIP_CKSUM_BAD > > > >> definition? > > > >> > > > >> I already added the author of the PKT_RX_EIP_CKSUM_BAD flag and ethdev and mbuf > > > >> maintainers in this list. So what else I need make forward progress > > > >> on this patch? > > > >> > > > >> I think, the definition of PKT_RX_EIP_CKSUM_BAD based on HW capability. It > > > >> is safe to assume that ALL HW can support CKSUM BAD if the feature is > > > >> available and hence it is more portable. > > > > > > > > Yes, as I remember PKT_RX_EIP_CKSUM_BAD is based on DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM. > > > > > > Switching to two bit won't reduce the portability, HW supports only reporting > > > CKSUM_BAD can set BAD || UNKNOWN. > > > > UNKNOWN is not a bit. It is represented as 0. It spec has 2 bit, then > > driver need to report GOOD as well. > > > > Same applies for PKT_RX_EL4_CKSUM as well. > > > > > > > > And I think patch is not blocked by PKT_RX_EIP_CKSUM_BAD, it can be changed > > > separately, for this patch question is can we represent PKT_RX_EL4_CKSUM_* with > > > two bits, to have BAD/GOOD/UNKNOWN? > > Yes, exact. > > PKT_RX_EIP_CKSUM_BAD must be left aside. > We should just avoid taking it as a reference. > And we can reconsider its definition later. OK. IMO, Using 2 bit scheme for tunneled checksum has following performance issue from driver side. Driver need to mark the packet as GOOD. All the HW can support detection of BAD. That not necessary mean GOOD in case of tunnel packet, so driver has to detect the packet is tunneled and packet is not BAD then mark GOOD. > > >