From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 28E35A0548; Thu, 11 Nov 2021 19:31:18 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D966A410F7; Thu, 11 Nov 2021 19:31:17 +0100 (CET) Received: from JPN01-OS2-obe.outbound.protection.outlook.com (mail-eopbgr1410104.outbound.protection.outlook.com [40.107.141.104]) by mails.dpdk.org (Postfix) with ESMTP id A525B40E28; Thu, 11 Nov 2021 19:31:15 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YhLrnrPL+chQC3QGRYalh7pcbsBdFGKVCaWbEYyy3u8kMl9zuNO5TDH5dqbpQgtsMDcfo0mXv9KyEUgbGlICibZ+3YdLAEVyWNUOK7HtxwSWfJQLpsmbqapRXAiBo+EfESwRf4bbr0ztvAQqJBTWhEF3SWiNahordg2pmMy07nxtHTMgRzFVEVcfsCVHmFtqklzGjR7ymQL81vGdgaNm0D7C7sPJ7s0bzucuelFZ1GDG02oViMfnLD91OtJ4LCS+G2qmHLr48t7DjmA1eaLw2+cHa1WJDzotNO2o7MG9JRSExqpMxh/I2OxjZVQZerxXfWW0h2sUEl3/tUAmiokJoA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=EStj1DdQt0O7RUotVfIUUhh7nTquQHlg68pw4eczs3k=; b=ZRf6+ypS6fMLmvj8kR5L+s3QJgLhkp3mDCIZcKqnXyab0QLGWgT6akIg/zzI0/vQMCA4wg2T0x9LLTwmeDneShsmTnXr1s7xs+/cTjyg5Qn7EU/CHZ/B4LMXERdCqx9vm6jK+ojGBctFuScO/AzsXeohE4yBcN2Lh/uzLJZieQPtbTm5walJu+9pJovtXuxI2caQFv4nzszFeGDFGbICd7Uqiy7LMgtOKdzfzRkpTwzFvQRi2fFv7HWaF+IAGiLLT4AXKb+pmrgcj0RYgUDVADzX4q8r2kIg8cTGIgHQGwgMdpEcip2MppF07cMzfyoN5HMSq0lOMCTqzmcjQ8GNPQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=connect.ust.hk; dmarc=pass action=none header.from=connect.ust.hk; dkim=pass header.d=connect.ust.hk; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=connect.ust.hk; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=EStj1DdQt0O7RUotVfIUUhh7nTquQHlg68pw4eczs3k=; b=1VKEA/vYmPMGWhvTDnPW6I8fZbYO8X8G6W0KRFfoSsu8S/gZ0Q3nmecGi+uR9s9lDX9pHfnLvADKzWWb4JnEBcoUv1TxF0tGc3oU1P+3IUney07Dpqzr0egvUOxyCsRnIpbOgop8fbML9GTsHDGcXrpfRe7HLCgfHl/37Jj8IFA= Received: from TYCP286MB1188.JPNP286.PROD.OUTLOOK.COM (2603:1096:400:b7::8) by TYAP286MB0105.JPNP286.PROD.OUTLOOK.COM (2603:1096:404:8039::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4669.11; Thu, 11 Nov 2021 18:31:12 +0000 Received: from TYCP286MB1188.JPNP286.PROD.OUTLOOK.COM ([fe80::c0af:a534:cead:3a04]) by TYCP286MB1188.JPNP286.PROD.OUTLOOK.COM ([fe80::c0af:a534:cead:3a04%7]) with mapi id 15.20.4690.025; Thu, 11 Nov 2021 18:31:12 +0000 From: YE Chengfeng To: Slava Ovsiienko , Ferruh Yigit , "dev@dpdk.org" CC: Raslan Darawsheh , Matan Azrad , "stable@dpdk.org" Subject: Re: [PATCH] net/mlx5: remove redundant "set used" Thread-Topic: [PATCH] net/mlx5: remove redundant "set used" Thread-Index: AQHX1tjhBIiFFw1jBUWmmrCXbIuNuKv+BxeAgAA0xYCAAAVvAIAAPBcpgAAMbwCAAB0J2Q== Date: Thu, 11 Nov 2021 18:31:11 +0000 Message-ID: References: <20211111084751.26721-1-viacheslavo@nvidia.com> In-Reply-To: Accept-Language: zh-CN, en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: nvidia.com; dkim=none (message not signed) header.d=none;nvidia.com; dmarc=none action=none header.from=connect.ust.hk; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: dd80ffc3-fa31-413e-a98e-08d9a541700a x-ms-traffictypediagnostic: TYAP286MB0105: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:7691; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: PQEdYhIUUT/tch3KFoRycmIQmSoCcmAy8u5LHFuY+PoTtkeDtZFVVXWU0zrk/QIvpEt8Yc70wSesIJASLOdoJvuroE+nhysdE0FAddf7v/X/bR9jTld0tncjL64AdqHv4K5KkbTUEbg15snOfPTyKxj8I9JhpeAh4DtJ+5dadcb0BSvND29nBq8/jqLwrXYR1ReSMx1bZKHaKmxXh8pIxQaKKH4dThqsjcBpOOx89mNdzWNaW/6yilCeFapMqfTBW1VamA1uax2/Ics7uVU9AjT3m5z8yx84pBw8uhMQjaOpR5L929Nfx0gamy7IMV0WFq+yx7qG9D+w8TQRSZ+CGLwHxw9Qbqf/UiFOxtnxkYMqD/S+9Ts9CylAlvfVPT6y0r+D65mtRHPFjvPEysaL8vc1oHrn9SNn0oWJZZR0iktdMBNrPtY4lp7Rw8xAOdHpidvL2FXuqw1m/1h9g310x+oGoXibogtjkVPUzb3SQ4G4gej1LtMVlJlmtuk4H0rAZuScw7Ihv2LQUhL0GQ4F6Hb+4RZzcS4LGulHYIXvEJvVB2G6XSqKZWSEcG6nsTiMhq+n/qokoAtMKBgwz+1y/UKS8tZNgNyPX4ACHDZn0/EhebvYTNmo8Zvx61/D++OOPkiIO4NjXKlfEPwg5UKtrakr7VMcrHnsAIk2CQBcid3aK0gwnl61Q+9dMDdwDP0yuADWQ+o6UHmYTAy0pXMxQIf2RrP4KIbIruDIsIqgZXaT5HaPRsew5I74jKexOwWp/SXvArn01LNfHakqFbp+C+znSQizoC0jaKqAsCIJgk4= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:TYCP286MB1188.JPNP286.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(4636009)(366004)(53546011)(66476007)(7696005)(4326008)(66556008)(66446008)(122000001)(45080400002)(55016002)(6506007)(64756008)(166002)(54906003)(2906002)(66946007)(76116006)(71200400001)(91956017)(508600001)(5660300002)(86362001)(83380400001)(966005)(19627235002)(33656002)(186003)(26005)(8676002)(8936002)(52536014)(9686003)(316002)(38070700005)(110136005)(786003)(38100700002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?gb2312?B?VFZvQXU5cEtZeDVrV28zNlhmYUswVU9kY2dkR2RiM2t2SFRGZFVtRC9kQTQw?= =?gb2312?B?Vm5iUWtocDU4N0xBVUE1UjExYVE3TUg5UUJzWVVveE1WOERmNFhBY0orQWda?= =?gb2312?B?UHczRGM4UGFWemF1Z0w0aHdNd1Rkd29YNVZWZkFoSEFzdnBQNERGazM3Tmlr?= =?gb2312?B?Z0NHNlE4MEFJZUVHc0c5WnkzaHpsQjYyOW96cStLNTloK0xyN0tuRHRoeGd4?= =?gb2312?B?eFZCTm01T0N5T0k3c0RzZjZ5NkVCK3dnRElhb3RiWUw0SFVSSzEzTGg2TFY4?= =?gb2312?B?S3ZEdHc5MmFYNDNsdk5uN2NQZ2srNFJ2cFE1Z3hSMG5QcUppc0JyNDloMFBh?= =?gb2312?B?ekRTeHgzTXRHNFBhWDJUUnBkVU9YdWNJbzY2T3NqWmdaODJjUEtNcURDRi8v?= =?gb2312?B?aEdJdDFvK215M1NIdzdOV2FKQ1dhcC9aaDIxL3lSVW5sUittN3Bqc3NldzUr?= =?gb2312?B?TEpJc0krSDRuZ2thR3h3R3JRS3Rnb1ZJdUY4NG54UDQrM0ZEZDNZeFJjeCtU?= =?gb2312?B?WnNBTzExYXF3WWVFVC9VSEFoNFhsZlhLNG0rQm1PZW1wUW9QdXNOTE00Y2lE?= =?gb2312?B?cXVFRDZlUzFiNkIzRjJPbng1YSs1cFhyaHcvN3d2WVp3R3c3S0gzYmY2Znlx?= =?gb2312?B?M0hmQmp4c2RiazA3bTd5YVJ5Yi9HL25kWmhYTUsyckUvT3BiMkxHZFdJb0RC?= =?gb2312?B?S3JrcjhPTzlWaTlXRXhEZHd4dGVkNTZRUzZmVnpoSk9KbHlZVU44MEtMcjFC?= =?gb2312?B?aWFWUGtlY0Zmdituam5IYmVnOWV2SXRGUmhNNVhWS3gxNUVEYlRINWZ0ZUVI?= =?gb2312?B?OW5Wdkd5YXhLZ3owcXdISE9CMjF3WDdlTWxuMEMvUHZqV1llMG90K1hPZTho?= =?gb2312?B?dGZGNjRhcUtBY3VkT1l0MmxFY1dqcC9jMTVLVlJuUFZZV2ZBWEdvZmw2dlRi?= =?gb2312?B?bWpHaE00bnZ5WFd6b2NEZk5GK3pnemJqd2pxZndRTS9QeFRWRHBJc2g1L0Ew?= =?gb2312?B?QkNzMk5RQlkwK0Yyd2Q4NVJTL3pkVXpiNEJlb3ZkM1VtRmc4c0JmVUNaR0d4?= =?gb2312?B?WEpSR1N5SE9vQVBLb3lnYkhkd3RyOGRyMTlOcWIwUlRaRWp2L0IrdkdyQUQx?= =?gb2312?B?NkJmYm1QUkx4QWJlK2tWVHNGeEMxSnNjU2Z0MDRRbEZnakE0LzJsR0FxUmxj?= =?gb2312?B?bVdSQkJneEd5c0tqcFRsa1hrNVdSOGVmY1hZaVd4eGR3MkFFeHJxRy90WmdD?= =?gb2312?B?RSt1L3ptM2pLelk2S0ovdmQxWTdicDMybmxQQlIvRlV0cmh1cENxdHhLZ2gy?= =?gb2312?B?eUQwU3NBcGVqSGpWZVdCcWJhZWt6U1M1TkJaN3ZMYUhyT01DNFZEdHYrWC9J?= =?gb2312?B?ZUhvbFNzUGlUY255ckhJL1ZOUzRkN0tFUFBEY3RQRG83TDlzMlZGM1B6ZU8x?= =?gb2312?B?OXY0REx4aUtSNXJpNnNQUnJKZkZ1UExGQkxtL0hZbGMxZWo2OFFndXBWL2FI?= =?gb2312?B?d1BxcVVrMm1SWnhDYjJGL05ocUl4VDUyWTdpamxXWVluRncwVkZRNnVFL3Aw?= =?gb2312?B?TitxZEVXajU4NjExSWVIUXRDZzVFNysyMDBIOGZHYnE5ZXVXbVlqNHZnOHBw?= =?gb2312?B?d0U3OUFDQmlFTjdIOHprTGlyb01DME1LT1dqemswUTNLMVB0M2RVaDNxbUtI?= =?gb2312?B?NGExb3BwY0MzN3d1aTVVN252UDQ1bEU4R3pJZ0M1YVFhaXVCMk90aXplRlEr?= =?gb2312?B?UUltcXFVdW9pNFZrM3ZDaGFqQ2hkd1NNTDRFR29lMEN3K2hDY3RIVWQ3M3F1?= =?gb2312?B?TmdvYVhtQldSUzNjZ2lIdUJubytvL082NGVQNDlBTFllZ1ZMTW8xOFJyZEE2?= =?gb2312?B?cWswU3U3Z29JVnYyOXUrL0tJdHlvK2hTeW52OUdHcWtPOHBDNC8wZS9yT2Rt?= =?gb2312?B?dTRxYVpWUXRtM2hjcFB0NUh0Mll4emZQRFRPZDVwaUNKTEVidXNWcWVxNGRl?= =?gb2312?B?TGVNdUp4MUlkVUxDam5CZGh1T0Vhb0xlR0s3VXlZWTdMTnkvSi83VCtxYVY1?= =?gb2312?B?d1JCTENDYUc3NytQc3VyVGxkQzVSZmRBRUxqaHBOcWJJc3hIQkoxbnNhZTR4?= =?gb2312?B?NjFOa0pQcno3SktpQUltejJSMGM5T0NHUHlpU0dBWnRaQWYxbCs4aW5mRFYz?= =?gb2312?Q?02W+hdKrC8kHwAd1Yvfs9S6EE2fL9onK10lUFCpa3baL?= Content-Type: multipart/alternative; boundary="_000_TYCP286MB1188C9B68B14429317F388398A949TYCP286MB1188JPNP_" MIME-Version: 1.0 X-OriginatorOrg: connect.ust.hk X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: TYCP286MB1188.JPNP286.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: dd80ffc3-fa31-413e-a98e-08d9a541700a X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Nov 2021 18:31:11.6200 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 6c1d4152-39d0-44ca-88d9-b8d6ddca0708 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: 6JC261ZSeEYuDixbPuktZcVN5qV/bcu8rfhP/lKnJ3gtjybKNmUS0UPHSJnIwA7MBTLkR1bHF4Z0fu7NK3HOiQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYAP286MB0105 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org --_000_TYCP286MB1188C9B68B14429317F388398A949TYCP286MB1188JPNP_ Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: base64 R290IGl0Lg0KDQq78cihIE91dGxvb2sgZm9yIGlPUzxodHRwczovL2FrYS5tcy9vMHVrZWY+DQpf X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXw0Kt6K8/sjLOiBTbGF2YSBPdnNpaWVua28g PHZpYWNoZXNsYXZvQG52aWRpYS5jb20+DQq3osvNyrG85DogRnJpZGF5LCBOb3ZlbWJlciAxMiwg MjAyMSAxMjo0NzowNCBBTQ0KytW8/sjLOiBZRSBDaGVuZ2ZlbmcgPGN5ZWFhQGNvbm5lY3QudXN0 LmhrPjsgRmVycnVoIFlpZ2l0IDxmZXJydWgueWlnaXRAaW50ZWwuY29tPjsgZGV2QGRwZGsub3Jn IDxkZXZAZHBkay5vcmc+DQqzrcvNOiBSYXNsYW4gRGFyYXdzaGVoIDxyYXNsYW5kQG52aWRpYS5j b20+OyBNYXRhbiBBenJhZCA8bWF0YW5AbnZpZGlhLmNvbT47IHN0YWJsZUBkcGRrLm9yZyA8c3Rh YmxlQGRwZGsub3JnPg0K1vfM4jogUkU6IFtQQVRDSF0gbmV0L21seDU6IHJlbW92ZSByZWR1bmRh bnQgInNldCB1c2VkIg0KDQoNCkZyb206IFlFIENoZW5nZmVuZyA8Y3llYWFAY29ubmVjdC51c3Qu aGs+DQpTZW50OiBUaHVyc2RheSwgTm92ZW1iZXIgMTEsIDIwMjEgMTg6MDgNClRvOiBTbGF2YSBP dnNpaWVua28gPHZpYWNoZXNsYXZvQG52aWRpYS5jb20+OyBGZXJydWggWWlnaXQgPGZlcnJ1aC55 aWdpdEBpbnRlbC5jb20+OyBkZXZAZHBkay5vcmcNCkNjOiBSYXNsYW4gRGFyYXdzaGVoIDxyYXNs YW5kQG52aWRpYS5jb20+OyBNYXRhbiBBenJhZCA8bWF0YW5AbnZpZGlhLmNvbT47IHN0YWJsZUBk cGRrLm9yZw0KU3ViamVjdDogUmU6IFtQQVRDSF0gbmV0L21seDU6IHJlbW92ZSByZWR1bmRhbnQg InNldCB1c2VkIg0KDQoNCg0KTm8gcHJvYmxlbSwgSSB3aWxsIHNlbmQgdGhlIG5ldyBwYXRjaC4N Cg0KW1NPXSBUaGFuayB5b3UuDQoNCg0KDQpCdXQgd2hhdCBhYm91dCB0aGUgY29tbWl0IG1lc3Nh Z2UgYW5kIHRpdGxlLCBzaG91bGQgSSB1c2UgdGhlIHByZXZpb3VzIG9uZT8NCg0KbmV0L21seDU6 IGZpeCBtdXRleCB1bmxvY2sgaW4gdHhwcCBjbGVhbnVwDQoNCltTT10gUGxlYXNlLCAgc2VuZCBh cyB0aGUgbmV4dCB2ZXJzaW9uICh2Nj8pIG9mIHlvdXIgcGF0Y2guIEp1c3Qgc3F1YXNoIG15IHBy b3Bvc2Fscw0KYW5kIGtlZXAgeW91ciB0aXRsZSAoobBuZXQvbWx4NTogZml4IG11dGV4IHVubG9j ayBpbiB0eHBwIGNsZWFudXChsSkgYW5kIGNvbW1pdCBtZXNzYWdlLg0KDQoNCg0KV2l0aCBiZXN0 IHJlZ2FyZHMsDQpTbGF2YQ0KDQoNCg0KDQoNCg0KDQq78cihIE91dGxvb2sgZm9yIGlPUzxodHRw czovL2FwYzAxLnNhZmVsaW5rcy5wcm90ZWN0aW9uLm91dGxvb2suY29tLz91cmw9aHR0cHMlM0El MkYlMkZha2EubXMlMkZvMHVrZWYmZGF0YT0wNCU3QzAxJTdDY3llYWElNDBjb25uZWN0LnVzdC5o ayU3Q2IwNTRiZDhmOGYzYzQyZTFmMGJiMDhkOWE1MzJlNWQ1JTdDNmMxZDQxNTIzOWQwNDRjYTg4 ZDliOGQ2ZGRjYTA3MDglN0MxJTdDMCU3QzYzNzcyMjQ2MDMwNzE2MzU5OCU3Q1Vua25vd24lN0NU V0ZwYkdac2IzZDhleUpXSWpvaU1DNHdMakF3TURBaUxDSlFJam9pVjJsdU16SWlMQ0pCVGlJNklr MWhhV3dpTENKWFZDSTZNbjAlM0QlN0MxMDAwJnNkYXRhPTN2JTJCSUFFZEhTZnNCJTJCRW9BbUts ZEZxeWJlSENDM2loTFo2SFNuV3prVEQwJTNEJnJlc2VydmVkPTA+DQoNCl9fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fDQoNCreivP7IyzogU2xhdmEgT3ZzaWllbmtvIDx2aWFjaGVzbGF2 b0BudmlkaWEuY29tPG1haWx0bzp2aWFjaGVzbGF2b0BudmlkaWEuY29tPj4NCreiy83KsbzkOiBU aHVyc2RheSwgTm92ZW1iZXIgMTEsIDIwMjEgODoyNzozMCBQTQ0KytW8/sjLOiBGZXJydWggWWln aXQgPGZlcnJ1aC55aWdpdEBpbnRlbC5jb208bWFpbHRvOmZlcnJ1aC55aWdpdEBpbnRlbC5jb20+ PjsgZGV2QGRwZGsub3JnPG1haWx0bzpkZXZAZHBkay5vcmc+IDxkZXZAZHBkay5vcmc8bWFpbHRv OmRldkBkcGRrLm9yZz4+DQqzrcvNOiBSYXNsYW4gRGFyYXdzaGVoIDxyYXNsYW5kQG52aWRpYS5j b208bWFpbHRvOnJhc2xhbmRAbnZpZGlhLmNvbT4+OyBNYXRhbiBBenJhZCA8bWF0YW5AbnZpZGlh LmNvbTxtYWlsdG86bWF0YW5AbnZpZGlhLmNvbT4+OyBzdGFibGVAZHBkay5vcmc8bWFpbHRvOnN0 YWJsZUBkcGRrLm9yZz4gPHN0YWJsZUBkcGRrLm9yZzxtYWlsdG86c3RhYmxlQGRwZGsub3JnPj47 IFlFIENoZW5nZmVuZyA8Y3llYWFAY29ubmVjdC51c3QuaGs8bWFpbHRvOmN5ZWFhQGNvbm5lY3Qu dXN0LmhrPj4NCtb3zOI6IFJFOiBbUEFUQ0hdIG5ldC9tbHg1OiByZW1vdmUgcmVkdW5kYW50ICJz ZXQgdXNlZCINCg0KDQoNCkhpLCBGZXJydWgNCg0KPiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0t LQ0KPiBGcm9tOiBGZXJydWggWWlnaXQgPGZlcnJ1aC55aWdpdEBpbnRlbC5jb208bWFpbHRvOmZl cnJ1aC55aWdpdEBpbnRlbC5jb20+Pg0KPiBTZW50OiBUaHVyc2RheSwgTm92ZW1iZXIgMTEsIDIw MjEgMTQ6MDgNCj4gVG86IFNsYXZhIE92c2lpZW5rbyA8dmlhY2hlc2xhdm9AbnZpZGlhLmNvbTxt YWlsdG86dmlhY2hlc2xhdm9AbnZpZGlhLmNvbT4+OyBkZXZAZHBkay5vcmc8bWFpbHRvOmRldkBk cGRrLm9yZz4NCj4gQ2M6IFJhc2xhbiBEYXJhd3NoZWggPHJhc2xhbmRAbnZpZGlhLmNvbTxtYWls dG86cmFzbGFuZEBudmlkaWEuY29tPj47IE1hdGFuIEF6cmFkDQo+IDxtYXRhbkBudmlkaWEuY29t PG1haWx0bzptYXRhbkBudmlkaWEuY29tPj47IHN0YWJsZUBkcGRrLm9yZzxtYWlsdG86c3RhYmxl QGRwZGsub3JnPjsgQ2hlbmdmZW5nIFllDQo+IDxjeWVhYUBjb25uZWN0LnVzdC5oazxtYWlsdG86 Y3llYWFAY29ubmVjdC51c3QuaGs+Pg0KPiBTdWJqZWN0OiBSZTogW1BBVENIXSBuZXQvbWx4NTog cmVtb3ZlIHJlZHVuZGFudCAic2V0IHVzZWQiDQo+DQo+IE9uIDExLzExLzIwMjEgODo1OSBBTSwg U2xhdmEgT3ZzaWllbmtvIHdyb3RlOg0KPiA+IEhpLCBGZXJydWgNCj4gPg0KPiA+IEkndmUgYWxz byBpbnNwZWN0ZWQgdGhlIG1seDUgUE1EIGNvZGUgZm9yIFJURV9TRVRfVVNFRCgpIGZvciB0aGUN Cj4gPiBzaW1pbGFyIGlzc3VlcyByZWxhdGVkIHRvIHRoZSBNTFg1X0FTU0VSVCgpLg0KPiA+DQo+ ID4gVGhlIHBhdGNoDQo+ID4gaHR0cHM6Ly9hcGMwMS5zYWZlbGlua3MucHJvdGVjdGlvbi5vdXRs b29rLmNvbS8/dXJsPWh0dHAlM0ElMkYlMkZwYXRjaGVzLmRwZGsub3JnJTJGcHJvamVjdCUyRmRw ZGslMkZwYXRjaCUyRjIwMjExMTExMDg0NzUxLjI2NzIxLTEtJmFtcDtkYXRhPTA0JTdDMDElN0Nj eWVhYSU0MGNvbm5lY3QudXN0LmhrJTdDODc1NTNhNWNkNDM2NDdkYjcwODAwOGQ5YTUwZWEzMjEl N0M2YzFkNDE1MjM5ZDA0NGNhODhkOWI4ZDZkZGNhMDcwOCU3QzElN0MwJTdDNjM3NzIyMzA0NTY2 ODI4MTg1JTdDVW5rbm93biU3Q1RXRnBiR1pzYjNkOGV5SldJam9pTUM0d0xqQXdNREFpTENKUUlq b2lWMmx1TXpJaUxDSkJUaUk2SWsxaGFXd2lMQ0pYVkNJNk1uMCUzRCU3QzEwMDAmYW1wO3NkYXRh PXdvVE1naHNFMFM3Y2tvbzBBSTAxWWJkV3JKZUx0Yk1DdTRHZUU4SkRpVWMlM0QmYW1wO3Jlc2Vy dmVkPTA8aHR0cHM6Ly9hcGMwMS5zYWZlbGlua3MucHJvdGVjdGlvbi5vdXRsb29rLmNvbS8/dXJs PWh0dHAlM0ElMkYlMkZwYXRjaGVzLmRwZGsub3JnJTJGcHJvamVjdCUyRmRwZGslMkZwYXRjaCUy RjIwMjExMTExMDg0NzUxLjI2NzIxLTEtJmRhdGE9MDQlN0MwMSU3Q2N5ZWFhJTQwY29ubmVjdC51 c3QuaGslN0NiMDU0YmQ4ZjhmM2M0MmUxZjBiYjA4ZDlhNTMyZTVkNSU3QzZjMWQ0MTUyMzlkMDQ0 Y2E4OGQ5YjhkNmRkY2EwNzA4JTdDMSU3QzAlN0M2Mzc3MjI0NjAzMDcxNzM1NTUlN0NVbmtub3du JTdDVFdGcGJHWnNiM2Q4ZXlKV0lqb2lNQzR3TGpBd01EQWlMQ0pRSWpvaVYybHVNeklpTENKQlRp STZJazFoYVd3aUxDSlhWQ0k2TW4wJTNEJTdDMTAwMCZzZGF0YT1GSDFCRU9kYURuNHZnTEhKJTJC alY4RHJIOWtaSVZDVnBvVlVFblBCMkdIV3clM0QmcmVzZXJ2ZWQ9MD4NCj4gdmlhYw0KPiA+IGhl c2xhdm9AbnZpZGlhLmNvbS88bWFpbHRvOmhlc2xhdm9AbnZpZGlhLmNvbS8+DQo+ID4gc2hvdWxk IHJlZmluZSB0aGUgZmV3IGZvdW5kIG9uZXMuDQo+ID4NCj4gPiBJIGRvIG5vdCBtaW5kIGFib3V0 IHNxdWFzaGluZyB3aXRoICJuZXQvbWx4NTogZml4IG11dGV4IHVubG9jayBpbiB0eHBwDQo+IGNs ZWFudXAiDQo+ID4gQWZ0ZXIgZ2V0dGluZyB0aGlzIGNvZGUgaW4gVXBzdHJlYW0gd2lsbCBjYXJl IGFib3V0IHRoZSB2ZXJzaW9uIGZvciBMVFMuDQo+ID4NCj4NCj4gSXQgd2lsbCBjYXVzZSBhZGRp dGlvbmFsIGNvbXBsZXhpdHkgZm9yIHRoZSBMVFMsIHNpbmNlIGEgc21hbGwgcGFydCBvZiB0aGUg YmVsb3cNCj4gZml4IHdpbGwgYmUgb3JpZ2luYXRlZCBmcm9tIENoZW5nZmVuZydzIGNoYW5nZS4g VG8gaGVscCBMVFMsIHdoYXQgZG8geW91IHRoaW5rDQo+IC0gRmlyc3QgZ2V0IHlvdXIgZml4IG9u IHRvcCBvZiBjdXJyZW50IHRhc2sNCj4gLSBIYXZlIGEgbmV3IHZlcnNpb24gZnJvbSBDaGVuZ2Zl bmcgb24gdG9wIG9mIGxhdGVzdCBoZWFkLCB3aXRoICdjbGFpbV96ZXJvJw0KPiB1c2FnZT8NCldv dWxkIGJlIG5pY2UsIEkgaGF2ZSBubyBhbnkgb2JqZWN0aW9ucy4NCkNoZW5nZmVuZywgY291bGQg eW91IHBsZWFzZSwgc3F1YXNoIChvciB3cml0ZSBieSB5b3Vyc2VsZikgbXkgcHJvcG9zZWQgdXBk YXRlcw0KYW5kIHNlbmQgdGhlIG5leHQgdmVyc2lvbiBvZiB5b3VyIHBhdGNoIHdpdGggImNsYWlt X3plcm8oKSI/DQoNCj4gU28gb25seSB5b3VyIHVwZGF0ZSBuZWVkIHRvIGJlIG1lcmdlZCB0byBM VFMgcmVsZWFzZXMuDQpZZXMsIGFncmVlLCBpdCBpcyBldmVuIGJldHRlciB0aGFuIG15IHByb3Bv c2FsLg0KDQpXaXRoIGJlc3QgcmVnYXJkcywNClNsYXZhDQoNCj4gPg0KPiA+PiAtLS0tLU9yaWdp bmFsIE1lc3NhZ2UtLS0tLQ0KPiA+PiBGcm9tOiBWaWFjaGVzbGF2IE92c2lpZW5rbyA8dmlhY2hl c2xhdm9AbnZpZGlhLmNvbTxtYWlsdG86dmlhY2hlc2xhdm9AbnZpZGlhLmNvbT4+DQo+ID4+IFNl bnQ6IFRodXJzZGF5LCBOb3ZlbWJlciAxMSwgMjAyMSAxMDo0OA0KPiA+PiBUbzogZGV2QGRwZGsu b3JnPG1haWx0bzpkZXZAZHBkay5vcmc+DQo+ID4+IENjOiBmZXJydWgueWlnaXRAaW50ZWwuY29t PG1haWx0bzpmZXJydWgueWlnaXRAaW50ZWwuY29tPjsgUmFzbGFuIERhcmF3c2hlaCA8cmFzbGFu ZEBudmlkaWEuY29tPG1haWx0bzpyYXNsYW5kQG52aWRpYS5jb20+PjsNCj4gPj4gTWF0YW4gQXpy YWQgPG1hdGFuQG52aWRpYS5jb208bWFpbHRvOm1hdGFuQG52aWRpYS5jb20+Pjsgc3RhYmxlQGRw ZGsub3JnPG1haWx0bzpzdGFibGVAZHBkay5vcmc+DQo+ID4+IFN1YmplY3Q6IFtQQVRDSF0gbmV0 L21seDU6IHJlbW92ZSByZWR1bmRhbnQgInNldCB1c2VkIg0KPiA+Pg0KPiA+PiBUaGUgcGF0Y2gg anVzdCByZWZpbmVzIHRoZSBjb2RlIGFuZCByZXBsYWNlcyB0aGUgcGFpcnMgb2YNCj4gPj4gTUxY NV9BU1NFUlQoKSBhbmQNCj4gPj4gUlRFX1NFVF9VU0VEKCkgd2l0aCBlcXVpdmFsZW50IGNsYWlt X3plcm8oKS4NCj4gPj4NCj4gPj4gQ2M6IHN0YWJsZUBkcGRrLm9yZzxtYWlsdG86c3RhYmxlQGRw ZGsub3JnPg0KPiA+Pg0KPiA+PiBTaWduZWQtb2ZmLWJ5OiBWaWFjaGVzbGF2IE92c2lpZW5rbyA8 dmlhY2hlc2xhdm9AbnZpZGlhLmNvbTxtYWlsdG86dmlhY2hlc2xhdm9AbnZpZGlhLmNvbT4+DQo+ ID4+IC0tLQ0KPiA+PiAgIGRyaXZlcnMvbmV0L21seDUvbWx4NV90eHBwLmMgfCAzMCArKysrKysr KysrLS0tLS0tLS0tLS0tLS0tLS0tLS0NCj4gPj4gICAxIGZpbGUgY2hhbmdlZCwgMTAgaW5zZXJ0 aW9ucygrKSwgMjAgZGVsZXRpb25zKC0pDQo+ID4+DQo+ID4+IGRpZmYgLS1naXQgYS9kcml2ZXJz L25ldC9tbHg1L21seDVfdHhwcC5jDQo+ID4+IGIvZHJpdmVycy9uZXQvbWx4NS9tbHg1X3R4cHAu YyBpbmRleCA3MzYyNmYwZThmLi5hZjc3ZTkxZTRjIDEwMDY0NA0KPiA+PiAtLS0gYS9kcml2ZXJz L25ldC9tbHg1L21seDVfdHhwcC5jDQo+ID4+ICsrKyBiL2RyaXZlcnMvbmV0L21seDUvbWx4NV90 eHBwLmMNCj4gPj4gQEAgLTg5MCw3ICs4OTAsNiBAQCBtbHg1X3R4cHBfc3RhcnQoc3RydWN0IHJ0 ZV9ldGhfZGV2ICpkZXYpDQo+ID4+ICAgICBzdHJ1Y3QgbWx4NV9wcml2ICpwcml2ID0gZGV2LT5k YXRhLT5kZXZfcHJpdmF0ZTsNCj4gPj4gICAgIHN0cnVjdCBtbHg1X2Rldl9jdHhfc2hhcmVkICpz aCA9IHByaXYtPnNoOw0KPiA+PiAgICAgaW50IGVyciA9IDA7DQo+ID4+IC0gIGludCByZXQ7DQo+ ID4+DQo+ID4+ICAgICBpZiAoIXByaXYtPmNvbmZpZy50eF9wcCkgew0KPiA+PiAgICAgICAgICAg ICAvKiBQYWNrZXQgcGFjaW5nIGlzIG5vdCByZXF1ZXN0ZWQgZm9yIHRoZSBkZXZpY2UuICovIEBA IC0NCj4gPj4gOTAzLDE0ICs5MDIsMTQgQEAgbWx4NV90eHBwX3N0YXJ0KHN0cnVjdCBydGVfZXRo X2RldiAqZGV2KQ0KPiA+PiAgICAgICAgICAgICByZXR1cm4gMDsNCj4gPj4gICAgIH0NCj4gPj4g ICAgIGlmIChwcml2LT5jb25maWcudHhfcHAgPiAwKSB7DQo+ID4+IC0gICAgICAgICAgcmV0ID0g cnRlX21idWZfZHluZmxhZ19sb29rdXANCj4gPj4gLQ0KPiA+PiAgICAgKFJURV9NQlVGX0RZTkZM QUdfVFhfVElNRVNUQU1QX05BTUUsIE5VTEwpOw0KPiA+PiAtICAgICAgICAgIGlmIChyZXQgPCAw KQ0KPiA+PiArICAgICAgICAgIGVyciA9IHJ0ZV9tYnVmX2R5bmZsYWdfbG9va3VwDQo+ID4+ICsg ICAgICAgICAgICAgICAgICAoUlRFX01CVUZfRFlORkxBR19UWF9USU1FU1RBTVBfTkFNRSwNCj4g Pj4gTlVMTCk7DQo+ID4+ICsgICAgICAgICAgLyogTm8gZmxhZyByZWdpc3RlcmVkIG1lYW5zIG5v IHNlcnZpY2UgbmVlZGVkLiAqLw0KPiA+PiArICAgICAgICAgIGlmIChlcnIgPCAwKQ0KPiA+PiAg ICAgICAgICAgICAgICAgICAgIHJldHVybiAwOw0KPiA+PiArICAgICAgICAgIGVyciA9IDA7DQo+ ID4+ICAgICB9DQo+ID4+IC0gIHJldCA9IHB0aHJlYWRfbXV0ZXhfbG9jaygmc2gtPnR4cHAubXV0 ZXgpOw0KPiA+PiAtICBNTFg1X0FTU0VSVCghcmV0KTsNCj4gPj4gLSAgUlRFX1NFVF9VU0VEKHJl dCk7DQo+ID4+ICsgIGNsYWltX3plcm8ocHRocmVhZF9tdXRleF9sb2NrKCZzaC0+dHhwcC5tdXRl eCkpOw0KPiA+PiAgICAgaWYgKHNoLT50eHBwLnJlZmNudCkgew0KPiA+PiAgICAgICAgICAgICBw cml2LT50eHBwX2VuID0gMTsNCj4gPj4gICAgICAgICAgICAgKytzaC0+dHhwcC5yZWZjbnQ7DQo+ ID4+IEBAIC05MjQsOSArOTIzLDcgQEAgbWx4NV90eHBwX3N0YXJ0KHN0cnVjdCBydGVfZXRoX2Rl diAqZGV2KQ0KPiA+PiAgICAgICAgICAgICAgICAgICAgIHJ0ZV9lcnJubyA9IC1lcnI7DQo+ID4+ ICAgICAgICAgICAgIH0NCj4gPj4gICAgIH0NCj4gPj4gLSAgcmV0ID0gcHRocmVhZF9tdXRleF91 bmxvY2soJnNoLT50eHBwLm11dGV4KTsNCj4gPj4gLSAgTUxYNV9BU1NFUlQoIXJldCk7DQo+ID4+ IC0gIFJURV9TRVRfVVNFRChyZXQpOw0KPiA+PiArICBjbGFpbV96ZXJvKHB0aHJlYWRfbXV0ZXhf dW5sb2NrKCZzaC0+dHhwcC5tdXRleCkpOw0KPiA+PiAgICAgcmV0dXJuIGVycjsNCj4gPj4gICB9 DQo+ID4+DQo+ID4+IEBAIC05NDQsMjggKzk0MSwyMSBAQCBtbHg1X3R4cHBfc3RvcChzdHJ1Y3Qg cnRlX2V0aF9kZXYgKmRldikgIHsNCj4gPj4gICAgIHN0cnVjdCBtbHg1X3ByaXYgKnByaXYgPSBk ZXYtPmRhdGEtPmRldl9wcml2YXRlOw0KPiA+PiAgICAgc3RydWN0IG1seDVfZGV2X2N0eF9zaGFy ZWQgKnNoID0gcHJpdi0+c2g7DQo+ID4+IC0gIGludCByZXQ7DQo+ID4+DQo+ID4+ICAgICBpZiAo IXByaXYtPnR4cHBfZW4pIHsNCj4gPj4gICAgICAgICAgICAgLyogUGFja2V0IHBhY2luZyBpcyBh bHJlYWR5IGRpc2FibGVkIGZvciB0aGUgZGV2aWNlLiAqLw0KPiA+PiAgICAgICAgICAgICByZXR1 cm47DQo+ID4+ICAgICB9DQo+ID4+ICAgICBwcml2LT50eHBwX2VuID0gMDsNCj4gPj4gLSAgcmV0 ID0gcHRocmVhZF9tdXRleF9sb2NrKCZzaC0+dHhwcC5tdXRleCk7DQo+ID4+IC0gIE1MWDVfQVNT RVJUKCFyZXQpOw0KPiA+PiAtICBSVEVfU0VUX1VTRUQocmV0KTsNCj4gPj4gKyAgY2xhaW1femVy byhwdGhyZWFkX211dGV4X2xvY2soJnNoLT50eHBwLm11dGV4KSk7DQo+ID4+ICAgICBNTFg1X0FT U0VSVChzaC0+dHhwcC5yZWZjbnQpOw0KPiA+PiAgICAgaWYgKCFzaC0+dHhwcC5yZWZjbnQgfHwg LS1zaC0+dHhwcC5yZWZjbnQpIHsNCj4gPj4gLSAgICAgICAgICByZXQgPSBwdGhyZWFkX211dGV4 X3VubG9jaygmc2gtPnR4cHAubXV0ZXgpOw0KPiA+PiAtICAgICAgICAgIE1MWDVfQVNTRVJUKCFy ZXQpOw0KPiA+PiAtICAgICAgICAgIFJURV9TRVRfVVNFRChyZXQpOw0KPiA+PiArICAgICAgICAg IGNsYWltX3plcm8ocHRocmVhZF9tdXRleF91bmxvY2soJnNoLT50eHBwLm11dGV4KSk7DQo+ID4+ ICAgICAgICAgICAgIHJldHVybjsNCj4gPj4gICAgIH0NCj4gPj4gICAgIC8qIE5vIHJlZmVyZW5j ZXMgYW55IG1vcmUsIGRvIGFjdHVhbCBkZXN0cm95LiAqLw0KPiA+PiAgICAgbWx4NV90eHBwX2Rl c3Ryb3koc2gpOw0KPiA+PiAtICByZXQgPSBwdGhyZWFkX211dGV4X3VubG9jaygmc2gtPnR4cHAu bXV0ZXgpOw0KPiA+PiAtICBNTFg1X0FTU0VSVCghcmV0KTsNCj4gPj4gLSAgUlRFX1NFVF9VU0VE KHJldCk7DQo+ID4+ICsgIGNsYWltX3plcm8ocHRocmVhZF9tdXRleF91bmxvY2soJnNoLT50eHBw Lm11dGV4KSk7DQo+ID4+ICAgfQ0KPiA+Pg0KPiA+PiAgIC8qDQo+ID4+IC0tDQo+ID4+IDIuMTgu MQ0KPiA+DQo= --_000_TYCP286MB1188C9B68B14429317F388398A949TYCP286MB1188JPNP_ Content-Type: text/html; charset="gb2312" Content-Transfer-Encoding: quoted-printable
Got it.

=BB=F1=C8=A1 Outlook for iOS

=B7=A2=BC=FE=C8=CB: Slava O= vsiienko <viacheslavo@nvidia.com>
=B7=A2=CB=CD=CA=B1=BC=E4: Friday, November 12, 2021 12:47:04 AM
=CA=D5=BC=FE=C8=CB: YE Chengfeng <cyeaa@connect.ust.hk>; Ferru= h Yigit <ferruh.yigit@intel.com>; dev@dpdk.org <dev@dpdk.org> =B3=AD=CB=CD: Raslan Darawsheh <rasland@nvidia.com>; Matan Azr= ad <matan@nvidia.com>; stable@dpdk.org <stable@dpdk.org>
=D6=F7=CC=E2: RE: [PATCH] net/mlx5: remove redundant "set used&= quot;
 

From: YE Chengfeng <cyeaa@con= nect.ust.hk>
Sent: Thursday, November 11, 2021 18:08
To: Slava Ovsiienko <viacheslavo@nvidia.com>; Ferruh Yigit <= ;ferruh.yigit@intel.com>; dev@dpdk.org
Cc: Raslan Darawsheh <rasland@nvidia.com>; Matan Azrad <mat= an@nvidia.com>; stable@dpdk.org
Subject: Re: [PATCH] net/mlx5: remove redundant "set used"=

 

No problem, I will send the new patch. 

[SO] Thank you.

 

But what about the commit message and title, should I use the previous= one?

net/mlx5: fix mutex unlock in txpp cl=
eanup
[SO] Please,  send as the next version (v6?) of your patch. Jus=
t squash my proposals
and keep your title (=A1=B0net/mlx5: fix mutex unl= ock in txpp cleanup=A1=B1) and commit message.
 
With best regards,
Slava
 
=
 =
;

 

=BB=F1=C8=A1 Outlook for iOS


=B7=A2=BC=FE=C8=CB: Slava Ovsiienko <viacheslavo@= nvidia.com>
=B7= =A2=CB=CD=CA=B1=BC=E4: Thursday, November 11, 2021 8:27:30 PM
=CA= =D5=BC=FE=C8=CB: Fer= ruh Yigit <ferruh.yigit@intel.com= >; dev@dpdk.org <dev@dpdk.org> =B3= =AD=CB=CD: Raslan Darawsheh <rasland@nvidia.com= >; Matan Azrad <matan@nvidia.com<= /a>>; stable@dpdk.org <stable@dpdk.org>; YE Chengfeng <cyeaa@connect.ust.hk>
=D6= =F7=CC=E2: RE: [PATC= H] net/mlx5: remove redundant "set used"

 

Hi, Ferruh

> -----Original Message-----
> From: Ferruh Yigit <ferru= h.yigit@intel.com>
> Sent: Thursday, November 11, 2021 14:08
> To: Slava Ovsiienko <viac= heslavo@nvidia.com>; dev@dpdk.org
> Cc: Raslan Darawsheh <rasland= @nvidia.com>; Matan Azrad
> <matan@nvidia.com>; stable@dpdk.org; Chengfeng Ye
> <cyeaa@connect.ust.hk&g= t;
> Subject: Re: [PATCH] net/mlx5: remove redundant "set used" >
> On 11/11/2021 8:59 AM, Slava Ovsiienko wrote:
> > Hi, Ferruh
> >
> > I've also inspected the mlx5 PMD code for RTE_SET_USED() for the<= br> > > similar issues related to the MLX5_ASSERT().
> >
> > The patch
> > https://apc01.safelinks.protection.outlook.com/?url=3Dhttp%3A%2F%2Fpatches.= dpdk.org%2Fproject%2Fdpdk%2Fpatch%2F20211111084751.26721-1-&amp;data=3D= 04%7C01%7Ccyeaa%40connect.ust.hk%7C87553a5cd43647db708008d9a50ea321%7C6c1d4= 15239d044ca88d9b8d6ddca0708%7C1%7C0%7C637722304566828185%7CUnknown%7CTWFpbG= Zsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C= 1000&amp;sdata=3DwoTMghsE0S7ckoo0AI01YbdWrJeLtbMCu4GeE8JDiUc%3D&amp= ;reserved=3D0
> viac
> > heslavo@nvidia.com/ > > should refine the few found ones.
> >
> > I do not mind about squashing with "net/mlx5: fix mutex unlo= ck in txpp
> cleanup"
> > After getting this code in Upstream will care about the version f= or LTS.
> >
>
> It will cause additional complexity for the LTS, since a small part of= the below
> fix will be originated from Chengfeng's change. To help LTS, what do y= ou think
> - First get your fix on top of current task
> - Have a new version from Chengfeng on top of latest head, with 'claim= _zero'
> usage?
Would be nice, I have no any objections.
Chengfeng, could you please, squash (or write by yourself) my proposed upda= tes
and send the next version of your patch with "claim_zero()"?

> So only your update need to be merged to LTS releases.
Yes, agree, it is even better than my proposal.

With best regards,
Slava

> >
> >> -----Original Message-----
> >> From: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
> >> Sent: Thursday, November 11, 2021 10:48
> >> To: dev@dpdk.org
> >> Cc: ferruh.yigit@in= tel.com; Raslan Darawsheh <ras= land@nvidia.com>;
> >> Matan Azrad <matan@nvi= dia.com>; stable@dpdk.org
> >> Subject: [PATCH] net/mlx5: remove redundant "set used&qu= ot;
> >>
> >> The patch just refines the code and replaces the pairs of
> >> MLX5_ASSERT() and
> >> RTE_SET_USED() with equivalent claim_zero().
> >>
> >> Cc: stable@dpdk.org > >>
> >> Signed-off-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
> >> ---
> >>   drivers/net/mlx5/mlx5_txpp.c | 30 ++++++++++-----= ---------------
> >>   1 file changed, 10 insertions(+), 20 deletions(-)=
> >>
> >> diff --git a/drivers/net/mlx5/mlx5_txpp.c
> >> b/drivers/net/mlx5/mlx5_txpp.c index 73626f0e8f..af77e91e4c 1= 00644
> >> --- a/drivers/net/mlx5/mlx5_txpp.c
> >> +++ b/drivers/net/mlx5/mlx5_txpp.c
> >> @@ -890,7 +890,6 @@ mlx5_txpp_start(struct rte_eth_dev *dev)<= br> > >>     struct mlx5_priv *priv =3D dev->da= ta->dev_private;
> >>     struct mlx5_dev_ctx_shared *sh =3D pr= iv->sh;
> >>     int err =3D 0;
> >> -  int ret;
> >>
> >>     if (!priv->config.tx_pp) {
> >>          &n= bsp;  /* Packet pacing is not requested for the device. */ @@ -
> >> 903,14 +902,14 @@ mlx5_txpp_start(struct rte_eth_dev *dev) > >>          &n= bsp;  return 0;
> >>     }
> >>     if (priv->config.tx_pp > 0) { > >> -          ret = =3D rte_mbuf_dynflag_lookup
> >> -
> >>     (RTE_MBUF_DYNFLAG_TX_TIMESTAMP_NAME, = NULL);
> >> -          if (r= et < 0)
> >> +          err = =3D rte_mbuf_dynflag_lookup
> >> +          =         (RTE_MBUF_DYNFLAG_TX_TIMESTAMP_N= AME,
> >> NULL);
> >> +          /* No= flag registered means no service needed. */
> >> +          if (e= rr < 0)
> >>          &n= bsp;          return 0;
> >> +          err = =3D 0;
> >>     }
> >> -  ret =3D pthread_mutex_lock(&sh->txpp.mutex); > >> -  MLX5_ASSERT(!ret);
> >> -  RTE_SET_USED(ret);
> >> +  claim_zero(pthread_mutex_lock(&sh->txpp.mutex)= );
> >>     if (sh->txpp.refcnt) {
> >>          &n= bsp;  priv->txpp_en =3D 1;
> >>          &n= bsp;  ++sh->txpp.refcnt;
> >> @@ -924,9 +923,7 @@ mlx5_txpp_start(struct rte_eth_dev *dev)<= br> > >>          &n= bsp;          rte_errno =3D -e= rr;
> >>          &n= bsp;  }
> >>     }
> >> -  ret =3D pthread_mutex_unlock(&sh->txpp.mutex);=
> >> -  MLX5_ASSERT(!ret);
> >> -  RTE_SET_USED(ret);
> >> +  claim_zero(pthread_mutex_unlock(&sh->txpp.mute= x));
> >>     return err;
> >>   }
> >>
> >> @@ -944,28 +941,21 @@ mlx5_txpp_stop(struct rte_eth_dev *dev)=   {
> >>     struct mlx5_priv *priv =3D dev->da= ta->dev_private;
> >>     struct mlx5_dev_ctx_shared *sh =3D pr= iv->sh;
> >> -  int ret;
> >>
> >>     if (!priv->txpp_en) {
> >>          &n= bsp;  /* Packet pacing is already disabled for the device. */
> >>          &n= bsp;  return;
> >>     }
> >>     priv->txpp_en =3D 0;
> >> -  ret =3D pthread_mutex_lock(&sh->txpp.mutex); > >> -  MLX5_ASSERT(!ret);
> >> -  RTE_SET_USED(ret);
> >> +  claim_zero(pthread_mutex_lock(&sh->txpp.mutex)= );
> >>     MLX5_ASSERT(sh->txpp.refcnt);
> >>     if (!sh->txpp.refcnt || --sh->t= xpp.refcnt) {
> >> -          ret = =3D pthread_mutex_unlock(&sh->txpp.mutex);
> >> -          MLX5_= ASSERT(!ret);
> >> -          RTE_S= ET_USED(ret);
> >> +          claim= _zero(pthread_mutex_unlock(&sh->txpp.mutex));
> >>          &n= bsp;  return;
> >>     }
> >>     /* No references any more, do actual = destroy. */
> >>     mlx5_txpp_destroy(sh);
> >> -  ret =3D pthread_mutex_unlock(&sh->txpp.mutex);=
> >> -  MLX5_ASSERT(!ret);
> >> -  RTE_SET_USED(ret);
> >> +  claim_zero(pthread_mutex_unlock(&sh->txpp.mute= x));
> >>   }
> >>
> >>   /*
> >> --
> >> 2.18.1
> >

--_000_TYCP286MB1188C9B68B14429317F388398A949TYCP286MB1188JPNP_--