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 6174EA0032 for ; Tue, 16 Nov 2021 15:52:15 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1325C41C3D; Tue, 16 Nov 2021 15:52:13 +0100 (CET) Received: from JPN01-TYC-obe.outbound.protection.outlook.com (mail-tycjpn01on2132.outbound.protection.outlook.com [40.107.114.132]) by mails.dpdk.org (Postfix) with ESMTP id C81D24115F; Tue, 16 Nov 2021 15:52:09 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Udl3LqRwDWFyIu9QvK78aFeuC8m2Y/FSlwMje7Ehbkdgg89hBoYMJb2qGYgYaXwncde97krD8In+M5fbJdDZB3hCfLxK9WrPnFjpzTt/mwZt/EL+dinzw3u2CjsbFB12/x68A4o/XlQgOakngUmwz/ceSU+6tmxWE1dL2nqM0Skd8lh3/ttRGf8RGobR7QHX/5APBvV+m5fV2/vEghSoOc7VRpldCaSdcghUY4WxCry/Tw2xSlxd0DF93jcWCKFvU4bMj2G2Cgb/7uvtKSAzGwFasBs3sdMTYSFax5HKLgAJKVXYQBZZza2IjUjCCQj0XuYPeLRfga0pJo9TN1qLxA== 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=SYX65yLfrkqzvQ6f8gRXSxvIctoRMD4xb5nNN2yMiHk=; b=QHsR0Ttm2To79EWAMPckEkDynsINj7PpDSb5x4xOhLknnUmr6ZRz523PMl6yuZRQNA6o/ZSEVePoyXwdQEtMPus2T/NXYWzJkdQvWnGeJc89rqfJnk5nIji6NBpmhf3NxzGBp2TAsH0qvibRIvQsZLVAuKotvsoYUZ8HE05gPD7JOeddBZSvh9Oc6dzqkTd846DXTJ/37b/o23oGCDCgihxf692iwviVT/cS1cAYANf8YIb234kIU5DyXLa6faqouEVdYdIOwTUUEWmsNXTgXnit27SZ39j0cYDQ+sU5o/LTYCOSe9Q5nHc4yBTA7o9lK6DpKMUyLEfx6UXKpeww7A== 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=SYX65yLfrkqzvQ6f8gRXSxvIctoRMD4xb5nNN2yMiHk=; b=EjuY660ASph92kHzFqo/Gy4xS598OKiWhgyHUQj1g2Ue7wfaexwZTChrMe3WERX9d5SIX5iJAW3TE2Hr3SDA4CMkWRtJcjq6uh5cVs0HzxscqPcQgVlkaKNYyWDltcfwuYsv3+1S2Drv3RkCx8n1Ah2oNWRA3+4I7INErqVMWro= Received: from TYCP286MB1188.JPNP286.PROD.OUTLOOK.COM (2603:1096:400:b7::8) by TYBP286MB0256.JPNP286.PROD.OUTLOOK.COM (2603:1096:404:802c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4690.17; Tue, 16 Nov 2021 14:52:07 +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.027; Tue, 16 Nov 2021 14:52:07 +0000 From: YE Chengfeng To: Slava Ovsiienko , Ferruh Yigit , "dev@dpdk.org" CC: Raslan Darawsheh , Matan Azrad , "stable@dpdk.org" Subject: =?gb2312?B?u9i4tDogW1BBVENIXSBuZXQvbWx4NTogcmVtb3ZlIHJlZHVuZGFudCAic2V0?= =?gb2312?Q?_used"?= Thread-Topic: [PATCH] net/mlx5: remove redundant "set used" Thread-Index: AQHX1tjhBIiFFw1jBUWmmrCXbIuNuKv+BxeAgAA0xYCAAAVvAIAAPBcpgAAMbwCAAB0J2YAHnlWc Date: Tue, 16 Nov 2021 14:52:07 +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: suggested_attachment_session_id: 5e8626d2-4d4e-fb9d-f864-229dec9eecfd authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=connect.ust.hk; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: f1ca89b4-3404-4a19-80e2-08d9a910a98a x-ms-traffictypediagnostic: TYBP286MB0256: 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: UU1ct2W4BOdVcZSUL93cU+DG5hbUwn2t4E4G/yAw8kmCQmFjPGsnGukDG+P5bmBYKPyswfQgvRPhlKFaKMoQFJqwCwqjmXV0hg9SIXM1G47caZc58f7XJJzqjrc0YK89eQHPOZK7OhzeulpDCzQgIk2CF7cGK4iN3MGhscngFnUytC3wFRGAfNW4+EiXlkc1jaOmxlYtABIW4q4r5Q0pIr1i2NhWFX7kW/JKf7b8GQC9N2sg3s6+5LwehTdlfXwuUQN8pgQbD3548El+BmB5R2HneuCDmB4PJ4+fhPsS2nqEHKr2nVRxa8ZABiQJ92ujS8wG4KtFXD8GjjLbO0sKPU4v9WpX4BAnDOIP0yCRPFdSkUL1YfhEvwpjeeURXkJjox4bYo9SMy61tjGySrRO5Rr1Rhh/SBSyw6+J/K7WPc3iHJ1TR64jOqOuFGHjSSuDGHr1Q0dkkKtKuzKk7PgUAnkHx3U8yhLqSq+tT8+4aL07DQQPKWT+K1GhFo20iRm9kfbgszFzCxo2wJu/j1Siq4chJos611b9kjXfOeuZr6BK/E0TEvCSJEqNgqq/Ekum8tGfnWY9g6R74kdGcOCoq8bhaArduTaBsMt+DIa9F0RdBy9nYitpJNttp+uREYx5y1PW8SZ0lxRBRlAmKBJV+thnxFQ2Rv0eS9QEBky4u6FmLnazUjWJP91v53T+A5qvEMuWkTwX6eYgZT11HrJ2WPy4genvwGPaMPDyLMUA35s0AKmDf24rax/RRYBvHquh+y7Hmo/neWdpCrdIwX2tZ2PMUFoqCTaFU4AB8DkHyOo= 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)(224303003)(86362001)(5660300002)(110136005)(38070700005)(786003)(83380400001)(54906003)(2906002)(316002)(166002)(66556008)(186003)(7696005)(52536014)(66476007)(53546011)(71200400001)(19627405001)(66446008)(66946007)(26005)(55236004)(76116006)(19627235002)(91956017)(64756008)(33656002)(6506007)(4326008)(8936002)(45080400002)(122000001)(38100700002)(966005)(508600001)(9686003)(55016002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?gb2312?B?YXRCKzY2Mm9seDQ4bEFUTndtbVlXMHRPbC9kWm1vNlRwaENRUk4zbk9hSkpL?= =?gb2312?B?MjNWZmlTN3RDR21uTUZML0RtRndlRTV2MlF5RWR4azRvTUtJalN5QmwrcTlz?= =?gb2312?B?SjdSVEU3bTNLd3hqdXplblFvNDVTZFZOOGV2U2dJMXBYMnRxZHNNZjBUbVhl?= =?gb2312?B?OFNDZ0lad1NIdzdhYjY0MTI5SGZJUXVjb2tPbkVjZ01QeWx1QlBiVlc1c3ph?= =?gb2312?B?b1dFeHBvRXoyUHEySCs2cU9rWUdQb2RVN2V2ZFc3N1hkckVZRmtubGFleG5v?= =?gb2312?B?RE1SdVhjcUJScHdRTHF4VWRRUHhJSFNaNHNHSnBubThhTUtWWk9xWHUxUVEv?= =?gb2312?B?VlYxQUY0eXl6YmxkMGVyczJzaHdnaldDWUpQbXVpTVFJeGNGYW9COVAxUEQv?= =?gb2312?B?UFRnWDhrR2Q5aWIzMjZtcUN5OUVBU0lZeE1aRkJhTVBOT3p5QUQ5Qm83ZnZF?= =?gb2312?B?Y093aTgrL05lYmlCSDhDMThRYTdjUlVKR0ZSNTBVNkxXcDlHNkxNTUJjbWZm?= =?gb2312?B?cVdSbmJNaDZPbFNiS2t5ejdYdVNsZEhBejNqdU1qUFVoUUJTKzh4VEVSUWtG?= =?gb2312?B?WUtXUUxmN0YvMDJxT3FsMTBwVXJnbVpPamlFVUpZTTdIblhPWUc3QmNZdzdY?= =?gb2312?B?eFVZSytob1FIYnIvQXBoQXZ2Y09RVzhjWGd1bXhoZEZXRlVWTkc1WktMKzA4?= =?gb2312?B?T3o3Y3A3SUVNWnNtQ3VWMXdENWlaWGo5bGZSSnp1TTdDUzY3c3ZQZ2xTQ2VG?= =?gb2312?B?SVNKRkpXdlVOQ3pPY3ZEdUVmYzN4KzRmSmxGckZuZVd6YW9ZWG9acmpwcnZC?= =?gb2312?B?UmVHU2hsWm1kSVFZUGhOKzY1S3pJQk9rbWlSZ1pSQSthZGdTcGVmNzN2ankv?= =?gb2312?B?Q2t4amppS0JraDI3TVNjcnlsRDF0eXVtdG9xK0d3MU5nVE4rWjZDY0RzSFcz?= =?gb2312?B?L2NldFQ0QUI0UkwvRjVlcHZrR1ZzaVdSRWdVYXp1eXpzMVZscHdaeStoRTRN?= =?gb2312?B?akgvSWFidHZ4NUxvYlViM3lIbU9tV3RTS2ZBQUlUejg0dkhPdHI4RWU4YWIv?= =?gb2312?B?M2N4clk1dGFyemZPZHpWYkp3eTB2MldnTmdSQU5WSUo5VWNIODJrTllHbjNo?= =?gb2312?B?Q1c4bjdqVmZyZUYvckpacVhUZERWSmltaDU4eUlvRDM1UmhxV25XUHBOU010?= =?gb2312?B?aFpYUnR2dW0vd0VaclI1M2I2U3hNVzdyNVMzWnJucWprRXNuWWZRYnZtQVpm?= =?gb2312?B?Z0dOYXpPUlFTVzFTVCtlUkFqMW5jUkQrODVNVVNwVnhSSWJpN0Mya1JKdE9q?= =?gb2312?B?TGNIajVRUHZra090TEE1MXQ5QWwzOXBidFliQUR3WjlSbEhVcFNnckdpQ3c5?= =?gb2312?B?RHlYN0Y1NHpEMGFxT1EzQ0NMeWR2ckJGaExoa0ZKTVBZYlZsNUR4ZmRwazdh?= =?gb2312?B?cjV2bFNseVhqR05UVUtOOWxnangyTHp4bVpUUEJUcUgvL1JwcDdNd3pKendM?= =?gb2312?B?UHZ1dVJkY3hrVFlwc3NvTTdCTjdDb0VDcS9RT1ZvM3pJbjdvYnozWmhLTmdO?= =?gb2312?B?OCs0UUduWGdETnJ2NVpFdkwwVEhLeTN2MnNuWHh1RmJoR3dHN0JzcFRXWG9n?= =?gb2312?B?ZU9GUHl0YzRubDBPNlR3ZEtCSVJsN050eXEwU25sZVBBVUFyZUk2YVN1K2tI?= =?gb2312?B?OHZJeU1hWnJTdzJjVmszSVNyT2RjK2tpbzlZNVUrYitFOG0rWnMya2xCMHV3?= =?gb2312?B?K2NuUHlsUUZQeGxsdjR0TGNBU2J5a0srMXQyOGwxY1JNMlA4RWxCMEpDbWlj?= =?gb2312?B?QVN0Y2JCeElTTnRMZFVjejhIeTVBOHRLQ21keDMwdjFpZ2d2azgyODhEWHZZ?= =?gb2312?B?dlFqcStTRWd1MUJuM05haGtsMUF4T1BHVEUzVWREYldzU0R0REVjZnFyaVRz?= =?gb2312?B?SU5rRmllVDNmekgwYTRoUHdEclpvQVhJZ2xwZU5oeENIUlFJWlJ6SGIzNG1u?= =?gb2312?B?WGtPdU0raWZMUnlYcExJenNaRjVyRnA0RDhpN3hnbVdkN2wxckdtbUkyN3JC?= =?gb2312?B?ZUlUYlVJN3k0bjR3Zno0c2djMGxwcEk3anZia3N6S1BhTWVnQTVOQXRCMVho?= =?gb2312?B?VmZ1TVRNam9Dc1hRSm03Zk9iZmdueDMrTmhRalFIZ081KzhTamRzREFIcTI5?= =?gb2312?B?NUE9PQ==?= Content-Type: multipart/alternative; boundary="_000_TYCP286MB11889A108D70EDA12087BD9F8A999TYCP286MB1188JPNP_" 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: f1ca89b4-3404-4a19-80e2-08d9a910a98a X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Nov 2021 14:52:07.7070 (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: o3PUpDZ7QtORSC37Ud0v9kmaCfK5QO553QP5C6CgHTa2ZCcpyR+3SXj/SvQNE4SuNjzKXI+uFek4hTRK85Rbfg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYBP286MB0256 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org --_000_TYCP286MB11889A108D70EDA12087BD9F8A999TYCP286MB1188JPNP_ Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: base64 SGksDQoNCkkgc2VuZCB0aGUgdjYgcGF0Y2guDQoNCkJlc3QgcmVnYXJkcywNCkNoZW5nZmVuZw0K X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18NCreivP7IyzogWUUgQ2hlbmdmZW5nIDxj eWVhYUBjb25uZWN0LnVzdC5oaz4NCreiy83KsbzkOiAyMDIxxOoxMdTCMTLI1SAyOjMxDQrK1bz+ yMs6IFNsYXZhIE92c2lpZW5rbyA8dmlhY2hlc2xhdm9AbnZpZGlhLmNvbT47IEZlcnJ1aCBZaWdp dCA8ZmVycnVoLnlpZ2l0QGludGVsLmNvbT47IGRldkBkcGRrLm9yZyA8ZGV2QGRwZGsub3JnPg0K s63LzTogUmFzbGFuIERhcmF3c2hlaCA8cmFzbGFuZEBudmlkaWEuY29tPjsgTWF0YW4gQXpyYWQg PG1hdGFuQG52aWRpYS5jb20+OyBzdGFibGVAZHBkay5vcmcgPHN0YWJsZUBkcGRrLm9yZz4NCtb3 zOI6IFJlOiBbUEFUQ0hdIG5ldC9tbHg1OiByZW1vdmUgcmVkdW5kYW50ICJzZXQgdXNlZCINCg0K 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_TYCP286MB11889A108D70EDA12087BD9F8A999TYCP286MB1188JPNP_ Content-Type: text/html; charset="gb2312" Content-Transfer-Encoding: quoted-printable
Hi,

I send the v6 patch.

Best regards,
Chengfeng

=B7=A2=BC=FE=C8=CB: YE Chen= gfeng <cyeaa@connect.ust.hk>
=B7=A2=CB=CD=CA=B1=BC=E4: 2021=C4=EA11=D4=C212=C8=D5 2:31
=CA=D5=BC=FE=C8=CB: Slava Ovsiienko <viacheslavo@nvidia.com>; = Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org <dev@dpdk.org&= gt;
=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;
 
Got it.

=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: 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@c= onnect.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 previo= us 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_TYCP286MB11889A108D70EDA12087BD9F8A999TYCP286MB1188JPNP_--