From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM02-BL2-obe.outbound.protection.outlook.com (mail-bl2nam02on0048.outbound.protection.outlook.com [104.47.38.48]) by dpdk.org (Postfix) with ESMTP id 4CEFD2030 for ; Wed, 13 Dec 2017 08:55:42 +0100 (CET) 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; bh=Yet8O+gVfVp7i+AKH2sVTGIMS6Cyl1rRzec4lzClEQ0=; b=AOf6flJXJVrOiezsA9G3FD8fLHbofZeJfCz5N2T0WRh0OcflP0DSoiBVLX1bL1CYkiRdTzWG5sLrPOi4DQKhVFRdxzH97hbaEzjRb8mztyNDg4GNgtq1gqQaPS01pZ0CBkHrXPRdabsLZI1Vf2wcZA0nQeNHDmakoUnP259lPv0= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.JacobKollanukkaran@cavium.com; Received: from jerin (122.167.65.15) by CO2PR07MB2519.namprd07.prod.outlook.com (10.166.201.6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.282.5; Wed, 13 Dec 2017 07:55:37 +0000 Date: Wed, 13 Dec 2017 13:25:19 +0530 From: Jerin Jacob To: Shahaf Shuler Cc: "Ananyev, Konstantin" , "dev@dpdk.org" , "Nicolau, Radu" , "arybchenko@solarflare.com" Message-ID: <20171213075518.GA18228@jerin> References: <20171123121419.144132-1-shahafs@mellanox.com> <2601191342CEEE43887BDE71AB9772585FAC8AEB@irsmsx105.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) X-Originating-IP: [122.167.65.15] X-ClientProxiedBy: BM1PR01CA0089.INDPRD01.PROD.OUTLOOK.COM (10.174.208.157) To CO2PR07MB2519.namprd07.prod.outlook.com (10.166.201.6) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: b812f122-15c0-4d31-db98-08d541fee6f7 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(5600026)(4604075)(4534020)(4602075)(4627115)(201703031133081)(201702281549075)(2017052603307); SRVR:CO2PR07MB2519; X-Microsoft-Exchange-Diagnostics: 1; CO2PR07MB2519; 3:I00sEhSjddqV37CjGqlOaFsLIrn2Dl5CnmD8ZEOY0q8BdUsFZKxP7Uv3jbcpqnMhk3eqXCOblSnCvJwWbOfbklvsmNxKhzkGq6Ax9useHRvEJhb2GoBQ2rKqCzlmLbCE7RETdaLFeXN3jm7cgZJ43cGtc43ZquzWSe6jSRX9DhI55aQNaGsyyvyeib5FmvM9jIsJdZzXcjARPEzrlqGkgB7ZhgPi+EdlY/IkzdWgphQ1XipwF1uYhRG9FpKBCh40; 25:ZGfVjsmZEnCbWtYTlWD1WevDzMQy+dIe0Wx+ll1oYDBPTZBNB1UjLitR7PBTaMmJ0wzDqusz4iaKh+z5E0b0klnBScq2u/yod/I7jwgSzoz9bqUy3wyobJb984QSv/gERW6z0l54hG83HbK6K3R9M3oI65mt4sH1ExzOiavZuInG0JflLrMzOyCgwmgmM1p3qHJlusO2bySaSORjXSvNFnlto0402ba6OQTth+PQ8tEyl8vlyHM3Su7bRCAR/OXZVWhYpMn7DIB63fyOEFv9hX2VQwvj1c1vIO4lNdfEnHFzix4irlRU/I66k+D+qUvcbfGby4mc7riDu5AtI6fBmA==; 31:LMP1oayypF+Qj0UY2+5gUxL3/rFKP9iCf+Lt67F6N8vCaawX7Q4Nq0+UTCxUE6XjYGS0E9DEYw7YI8cONLe7sTPHG2IO6/tCj22lhSROmTtvaPjdFla6ydv+vG9dEY61pobokUjvCXufxeG5DPiI8Nlt3C5/LXFIUErSY3Fs8dS84L3afoWfRl8ZDR/ioyICxVB8n6xm3RQp86q7I8QaV/AgxGOTeoWqPmuPJiX56cw= X-MS-TrafficTypeDiagnostic: CO2PR07MB2519: X-Microsoft-Exchange-Diagnostics: 1; CO2PR07MB2519; 20:w2PEXZ9gN858rxQETJN38OTEY8t4suXHMZIQvDdCZHtJLnhBsvJ3A33UZumHjUn+dsepX2+4Sdi6YC7AXUZJAKA2bRlljsT7w/mJ+krBrtr1qX8olzu2bs9GfgcNgkQ990n9pOKFk+n/SPaofvAz1bqM/tPrnawoOpWHKMUL6OTzNkBgB7aNyw3PXfO2Gn+Lw30TeRXnlhVSd7+cLs5XgMQNZEsd3av05Ay1A+O601+TUZP8IXEPGuR961Y1sXAHjXZS2x8f7oYW11lYGYpyBM6w4SOayl4QQvFb7xRNaLV3ZqP15Zy2L9olIc+Io7q4y8yFacg/UJmCEOzeonwUoofA8yEiJVbUDiXFHSO5V05gqVaoKD/YH/GFo0ay9fHsM7I9H3KGJZHD4fvyA5smv1ZxNQgVlBKWbSetvRXznLbJLKeSBJ7azKDSkhXKWHUJGSljqWnAyBqzSH75bSJGLdrcqHLpYAf70iA/4duo4Jb7OssJ56AAHTyXFfOAHX7Ui9rCZTrB2cRQaN9JND3icDAIUxTefrRJU0gfB1aUL/cFeySPJ8oA8l7TSGDh/DbtI888y59+waXHuDqkueuQK5ozcVW6f09pv/Ep5twlQig=; 4:IQdUrhVbBTUr62SzRtsIqD4MsWz5likrXcg+VISJ5rA6TJvMgBznDrPSSAuhhPLE9FaJ1FlEaGg7W3GoaVzZCJTlREfsYt7cAYnN7ok/69kgieh14gTRP2S9jcoBKmlotGxuGmVL0UJP69j6VFHiuHi01MLfdhqwt2XmQ1LvJEOHJOMP7LAP7jnD/JcksnaNi/5hw/o9i1BqAHTRt8nxeFYNHwLfqKd4hhFwP3M7/gtBaA4PFA53nzfG5DQkYbVSKZhn/l2SrrJFNcR5sc1g+mwhnwfQS8YT6tjxVkDppsoK4whC+z1Kby8vxlXueBTw X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(228905959029699); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040450)(2401047)(8121501046)(5005006)(3002001)(10201501046)(3231023)(93006095)(6041248)(20161123562025)(20161123555025)(20161123564025)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(6072148)(201708071742011); SRVR:CO2PR07MB2519; BCL:0; PCL:0; RULEID:(100000803101)(100110400095); SRVR:CO2PR07MB2519; X-Forefront-PRVS: 052017CAF1 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(346002)(376002)(366004)(189003)(199004)(13464003)(81156014)(9686003)(478600001)(1076002)(81166006)(16526018)(6246003)(66066001)(23726003)(42882006)(6666003)(229853002)(55016002)(8936002)(83506002)(316002)(53936002)(4326008)(58126008)(72206003)(6916009)(54906003)(2906002)(16586007)(68736007)(7736002)(2950100002)(105586002)(5660300001)(8676002)(25786009)(50466002)(33716001)(305945005)(33656002)(6496006)(52116002)(106356001)(93886005)(47776003)(6116002)(76176011)(3846002)(59450400001)(97736004)(33896004)(53546011)(386003)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:CO2PR07MB2519; H:jerin; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; CO2PR07MB2519; 23:Y2+XYssLEu0W4Ss1vnkVv2az8+H8KV9xXGd4KQ/NN?= =?us-ascii?Q?54H0cxCm79tle2g3rdCrK6QV4HyhSeCqxs0gqxnpKJ7y/qywahNP5MPlYlu1?= =?us-ascii?Q?83cWvcRP6lFMq/ybreK6Rk9RGbsbP0lxv2QOOTFnmidjGk8SBG0ffTQjBJEd?= =?us-ascii?Q?6KcaHjfQeNrGv2BZhYoUM1V7kSrLjDs+wCrcSYpXn9sQx4UhZzMF2hbgaZh0?= =?us-ascii?Q?bY80VXGWBX7vYaP4eN5g2E4BjnXedBDEZamfC5Gq6b+OR+A9FkID9po0/nOf?= =?us-ascii?Q?GRo1isKnIrodIJCrB+GkE6B7jafUKRf3s2QWhvjt19azn0zORNIUFxxfuydf?= =?us-ascii?Q?5iThTHpn6d/5ekZQWbe+c/TQmLplBeS37FbSptZOr2P16k2dnN72BWZiVav8?= =?us-ascii?Q?vBE3giC45yh9dDfStfoizOgupd/eLNEzKKlcXdHReXODlU4fAMjxVono9gLd?= =?us-ascii?Q?e42UWkdmxtHlYmiciYB6+1v9ldp68DykevzegA1KXTuRjFLDbWDhu+CUrhpV?= =?us-ascii?Q?3Rwnst4bNlTw6Ol1enz0TkWwB9fU5K20Otc8x0+nP9c6LuzPaUlRbplt71Cy?= =?us-ascii?Q?2+lxHv1PKSyXTqeZqNPYFIGRU17LFFzP0H49HRIEmpp1bbmrL85Y9khOHu56?= =?us-ascii?Q?3hXFy+bMOx7iGK3Y3LKet6MeQtDXXxqlG3jXtjPILDUxe4dEBqicaHD2ExWU?= =?us-ascii?Q?WOJd906QMGBtruTzjh1/63F84CKPHGZ2cj6d0ljZtVMLHA7wGSOXYzE/W90j?= =?us-ascii?Q?IgrRua/Kf9ANkl6fpcmVwy17piXpSD3+Jf5dQu1QoVoohUEhZey9QHQ1C9ql?= =?us-ascii?Q?/5IFhoq9MUkNOPHq/JJATSAHVtQNbU+1+O1ZGEYErWrsYCh2CGq/zgOsR9FB?= =?us-ascii?Q?OIsFZK/hI0C3SJ9fHeVYuoFDM8fjttEOeQmMC1HvCrjVY0GBAXrmmv8fWbbX?= =?us-ascii?Q?E91ZhNGDrtD45Atn7G8k2teNB3CfCet5RH399SO5UDlB4YY6S0qHs+6husqV?= =?us-ascii?Q?+9P9uz4AEJ3KBHdwTX0HJuXAyzt1tmsvM6PhhtEaIfy23ZfmHWgHSRIeI3FW?= =?us-ascii?Q?nm+XWVxweElGyF9eTIPBO2/QynuNjVgQtLi7T6RcLQZhVTvwQc35gyBKnd+8?= =?us-ascii?Q?teoeJMcJTw4565Ujiib2nUX3NYNIQrWduv8lSwqb0f/cYNGoFgIr8tH7bJt1?= =?us-ascii?Q?eAMZ+pskNhqfitq61TBaMEJOOsMxJVlltF/I65ir9GjB0Kgod7QYiRyMAJQZ?= =?us-ascii?Q?UxbBxj029IjBSI83RbeJqWBoBV0PhD5Maa5+wre?= X-Microsoft-Exchange-Diagnostics: 1; CO2PR07MB2519; 6:Nk95ONFCzLhZkrP1NF6WzmyhGIR4XzMy5DwjPC9V2RnVdOzGZ1NwP7sPVezVdQu3ZWBWRLaK2616cPg7NzMheCkevtvqsmw9Uf8dCuztH9zsqcy4dMxRQyGYmHcH2F4xXFdeH7oHnnohEyEp7f8A4wGsQ3skvXA/eVzZdU2d7LUGmgqFnEG7vD06Y9/206rTvBG6A32qCeSNariqs2BpIKajR+vrkDkE+4IQspHj6dTvzMZYxmZGDH9lvLwqb7XP2G8+Rd34RmnhVMBoTO0SdxcitSPYPIuxIyokonj7kHpcUEUES1lowon3H5lOS3W5ztqqFjQqCzeBdF2Xoi9OhogpIElVPPgBD9oDEb09ZEQ=; 5:93PosDjH9kO4vCUlFyMufWtUYXTTVuvudpRUBrZygseQE5ZFnVqsRQj0lEWtb7nE4CqugQzXHRIVTAduBCmCFjMN3ph7JBxL/HZiQi+GR4yaq1xgBGV+hoD/I3kbNkRUkOFdokPIGwRDiW3YvSq0yP2NQEU3eEon2CFeYOHcrUY=; 24:iJ9ciMnhraaSBKvk4dlGvRsA6ObpKmENtTpMzCjRYy6sRoZxmMV8gQ8OmwBn52q2CHcDTMwLLlEtkj5jg9fU1SWTBiAbD0lr7O2EbY62zGI=; 7:SiFwR9913v5IeBgrI/+2ROrwbkVYTpeokqVoatYi6DsZ3WlBez8lo1MdV20W09H36+zcro2Mx8FwJdUbQDIVs3UMY+81HgqIg62fIKCOVVLYasGih/YIFyzC891wB6FJnNQh2vpw6p1RFU8tXyNcDtCzEMBoL7kKsn7sg79MDHxuBe0ZW4pDLrFci7BO5AcDsEQz0hemZ2up1PVzh1x4hvkuqbWf3diXlh49GoMAYCbVy2tk+9E+76x+1T6Ynt5x SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Dec 2017 07:55:37.3419 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: b812f122-15c0-4d31-db98-08d541fee6f7 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO2PR07MB2519 Subject: Re: [dpdk-dev] [PATCH v2 05/39] examples/l3fwd: move to ethdev offloads API 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: Wed, 13 Dec 2017 07:55:42 -0000 -----Original Message----- > Date: Wed, 13 Dec 2017 07:21:01 +0000 > From: Shahaf Shuler > To: "Ananyev, Konstantin" , "dev@dpdk.org" > , "Nicolau, Radu" , > "arybchenko@solarflare.com" > Subject: Re: [dpdk-dev] [PATCH v2 05/39] examples/l3fwd: move to ethdev > offloads API > > Tuesday, December 12, 2017 7:12 PM, Ananyev, Konstantin: > > > -----Original Message----- > > > From: Shahaf Shuler [mailto:shahafs@mellanox.com] > > > Sent: Tuesday, December 12, 2017 12:26 PM > > > To: dev@dpdk.org; Ananyev, Konstantin > > ; > > > Nicolau, Radu ; arybchenko@solarflare.com > > > Subject: [PATCH v2 05/39] examples/l3fwd: move to ethdev offloads API > > > > > > Ethdev offloads API has changed since: > > > > > > commit ce17eddefc20 ("ethdev: introduce Rx queue offloads API") commit > > > cba7f53b717d ("ethdev: introduce Tx queue offloads API") > > > > > > This commit support the new API. > > > > > > Signed-off-by: Shahaf Shuler > > > --- > > > examples/l3fwd/main.c | 40 ++++++++++++++++++++++++++++++-------- > > -- > > > 1 file changed, 30 insertions(+), 10 deletions(-) > > > > > > diff --git a/examples/l3fwd/main.c b/examples/l3fwd/main.c index > > > 6229568..3bdf4d5 100644 > > > --- a/examples/l3fwd/main.c > > > +++ b/examples/l3fwd/main.c > > > @@ -149,11 +149,9 @@ struct lcore_params { > > > .mq_mode = ETH_MQ_RX_RSS, > > > .max_rx_pkt_len = ETHER_MAX_LEN, > > > .split_hdr_size = 0, > > > - .header_split = 0, /**< Header Split disabled */ > > > - .hw_ip_checksum = 1, /**< IP checksum offload enabled */ > > > - .hw_vlan_filter = 0, /**< VLAN filtering disabled */ > > > - .jumbo_frame = 0, /**< Jumbo Frame Support disabled */ > > > - .hw_strip_crc = 1, /**< CRC stripped by hardware */ > > > + .ignore_offload_bitfield = 1, > > > + .offloads = (DEV_RX_OFFLOAD_CRC_STRIP | > > > + DEV_RX_OFFLOAD_CHECKSUM), > > > }, > > > .rx_adv_conf = { > > > .rss_conf = { > > > @@ -163,6 +161,7 @@ struct lcore_params { > > > }, > > > .txmode = { > > > .mq_mode = ETH_MQ_TX_NONE, > > > + .offloads = DEV_TX_OFFLOAD_MBUF_FAST_FREE, > > > > Hmm, does it mean a new warning for all PMDs (majority) which don't > > support DEV_TX_OFFLOAD_MBUF_FAST_FREE? > > Good point. > Unlike other offloads which are must for the application proper run, this one it only for optimizing the performance and should be set only if PMD supports. > Am continuing to aggregate reasons why the DEV_TX_OFFLOAD_MBUF_FAST_FREE should not be defined as an offload. Anyway we passed that... > > I will fix on v3. Removing is not an option as the PMDs rely on that flag to will have the impact. # I see DEV_TX_OFFLOAD_MBUF_FAST_FREE as hint driver to depict the application requirements # All the drivers by default can support DEV_TX_OFFLOAD_MBUF_FAST_FREE(They are using the hint or not is a different question) So, How about setting DEV_TX_OFFLOAD_MBUF_FAST_FREE in all PMD driver as dummy one? I think, currently, it can be moved to old API to new API transition function till the drivers change to new offload flag scheme. We are planning to change nicvf driver to new offload scheme for this release so with this change, we have the performance impact on l3fwd application. I think, the other option could be to change usage/meaning of DEV_TX_OFFLOAD_MBUF_FAST_FREE flag where when the application needs multi-pool and reference count scheme then "it sets" the offload flags. If so, we don't need to set by default on the these applications.