From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id D2F4BA04B4; Fri, 8 Nov 2019 15:35:18 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id A693A1C1E5; Fri, 8 Nov 2019 15:35:18 +0100 (CET) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id A138F1C1C7; Fri, 8 Nov 2019 15:35:16 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga104.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Nov 2019 06:35:15 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,281,1569308400"; d="scan'208";a="201367306" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by fmsmga008.fm.intel.com with ESMTP; 08 Nov 2019 06:35:15 -0800 Received: from fmsmsx120.amr.corp.intel.com (10.18.124.208) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 8 Nov 2019 06:35:14 -0800 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by fmsmsx120.amr.corp.intel.com (10.18.124.208) with Microsoft SMTP Server (TLS) id 14.3.439.0; Fri, 8 Nov 2019 06:35:14 -0800 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.127]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.60]) with mapi id 14.03.0439.000; Fri, 8 Nov 2019 22:35:11 +0800 From: "Xu, Rosen" To: David Marchand CC: dev , Kevin Traynor , dpdk stable , "Ye, Xiaolong" Thread-Topic: [dpdk-dev] [PATCH 4/9] net/ipn3ke: fix incorrect commit check logic Thread-Index: AQHVjvgK3KyHU1UyI0qblBiPQpnraaeBZSZg Date: Fri, 8 Nov 2019 14:35:10 +0000 Message-ID: <0E78D399C70DA940A335608C6ED296D73AB39165@SHSMSX104.ccr.corp.intel.com> References: <20191001125315.6191-1-ktraynor@redhat.com> <20191001130405.7076-1-ktraynor@redhat.com> <20191001130405.7076-3-ktraynor@redhat.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMDE4NTJiNmYtYTc4ZS00YzllLTg2MzYtZjEwMDQ1MTdlMmE5IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiSXFVc1ZwWDduMFpDSm1aazdlMDFVaE5leTFESmdscEZhbnlKeUF3WUhiV3QzYk1VYWFXbFpwdXZTK0VieUdzTCJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH 4/9] net/ipn3ke: fix incorrect commit check logic 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" SGkgRGF2aWQsDQoNClRvbyBtYW55IHRoaW5ncyBpbiB0aGVzZSBkYXlzLiBJIGhhdmUgcmV2aWV3 ZWQgaXQuIFRoYW5rcyBhIGxvdC4NCg0KPiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBG cm9tOiBEYXZpZCBNYXJjaGFuZCBbbWFpbHRvOmRhdmlkLm1hcmNoYW5kQHJlZGhhdC5jb21dDQo+ IFNlbnQ6IFdlZG5lc2RheSwgT2N0b2JlciAzMCwgMjAxOSAxNjowMA0KPiBUbzogWHUsIFJvc2Vu IDxyb3Nlbi54dUBpbnRlbC5jb20+DQo+IENjOiBkZXYgPGRldkBkcGRrLm9yZz47IEtldmluIFRy YXlub3IgPGt0cmF5bm9yQHJlZGhhdC5jb20+OyBkcGRrDQo+IHN0YWJsZSA8c3RhYmxlQGRwZGsu b3JnPjsgWWUsIFhpYW9sb25nIDx4aWFvbG9uZy55ZUBpbnRlbC5jb20+DQo+IFN1YmplY3Q6IFJl OiBbZHBkay1kZXZdIFtQQVRDSCA0LzldIG5ldC9pcG4za2U6IGZpeCBpbmNvcnJlY3QgY29tbWl0 IGNoZWNrDQo+IGxvZ2ljDQo+IA0KPiBIZWxsbyBSb3NlbiwNCj4gDQo+IFJldmlldyBwbGVhc2Uu DQo+IA0KPiBPbiBUdWUsIE9jdCAxLCAyMDE5IGF0IDM6MDQgUE0gS2V2aW4gVHJheW5vciA8a3Ry YXlub3JAcmVkaGF0LmNvbT4gd3JvdGU6DQo+ID4NCj4gPiBDb3Zlcml0eSBpcyBjb21wbGFpbmlu ZyBhYm91dCBpZGVudGljYWwgY29kZSByZWdhcmRsZXNzIG9mIHdoaWNoDQo+ID4gYnJhbmNoIG9m IHRoZSBpZiBlbHNlIGlzIHRha2VuLiBGdW5jdGlvbmFsbHkgaXQgbWVhbnMgYW4gZXJyb3Igd2ls bA0KPiA+IGFsd2F5cyBiZSByZXR1cm5lZCBpZiB0aGlzIGlmIGVsc2UgaXMgaGl0LiBSZW1vdmUg dGhlIGVsc2UgYnJhbmNoLg0KPiA+DQo+ID4gICAgIENJRCAzMzc5MjggKCMxIG9mIDEpOiBJZGVu dGljYWwgY29kZSBmb3IgZGlmZmVyZW50IGJyYW5jaGVzDQo+ID4gICAgIChJREVOVElDQUxfQlJB TkNIRVMpaWRlbnRpY2FsX2JyYW5jaGVzOiBUaGUgc2FtZSBjb2RlIGlzIGV4ZWN1dGVkDQo+ID4g ICAgIHJlZ2FyZGxlc3Mgb2Ygd2hldGhlciBuLT5sZXZlbCAhPSBJUE4zS0VfVE1fTk9ERV9MRVZF TF9DT1MgfHwNCj4gPiAgICAgbi0+bl9jaGlsZHJlbiAhPSAwVSBpcyB0cnVlLCBiZWNhdXNlIHRo ZSAndGhlbicgYW5kICdlbHNlJyBicmFuY2hlcw0KPiA+ICAgICBhcmUgaWRlbnRpY2FsLiBTaG91 bGQgb25lIG9mIHRoZSBicmFuY2hlcyBiZSBtb2RpZmllZCwgb3IgdGhlIGVudGlyZQ0KPiA+ICAg ICAnaWYnIHN0YXRlbWVudCByZXBsYWNlZD8NCj4gPiAxNTA2ICBpZiAobi0+bGV2ZWwgIT0gSVBO M0tFX1RNX05PREVfTEVWRUxfQ09TIHx8DQo+ID4gMTUwNyAgICAgICAgICBuLT5uX2NoaWxkcmVu ICE9IDApIHsNCj4gPiAxNTA4ICAgICAgICAgIHJldHVybiAtcnRlX3RtX2Vycm9yX3NldChlcnJv ciwNCj4gPiAxNTA5ICAgICAgICAgICAgICAgICAgRUlOVkFMLA0KPiA+IDE1MTAgICAgICAgICAg ICAgICAgICBSVEVfVE1fRVJST1JfVFlQRV9VTlNQRUNJRklFRCwNCj4gPiAxNTExICAgICAgICAg ICAgICAgICAgTlVMTCwNCj4gPiAxNTEyICAgICAgICAgICAgICAgICAgcnRlX3N0cmVycm9yKEVJ TlZBTCkpOw0KPiA+ICAgICBlbHNlX2JyYW5jaDogVGhlIGVsc2UgYnJhbmNoLCBpZGVudGljYWwg dG8gdGhlIHRoZW4gYnJhbmNoLg0KPiA+IDE1MTMgIH0gZWxzZSB7DQo+ID4gMTUxNCAgICAgICAg ICByZXR1cm4gLXJ0ZV90bV9lcnJvcl9zZXQoZXJyb3IsDQo+ID4gMTUxNSAgICAgICAgICAgICAg ICAgIEVJTlZBTCwNCj4gPiAxNTE2ICAgICAgICAgICAgICAgICAgUlRFX1RNX0VSUk9SX1RZUEVf VU5TUEVDSUZJRUQsDQo+ID4gMTUxNyAgICAgICAgICAgICAgICAgIE5VTEwsDQo+ID4gMTUxOCAg ICAgICAgICAgICAgICAgIHJ0ZV9zdHJlcnJvcihFSU5WQUwpKTsNCj4gPiAxNTE5ICB9DQo+ID4N Cj4gPiBDb3Zlcml0eSBpc3N1ZTogMzM3OTI4DQo+ID4gRml4ZXM6IGM4MjA0NjhhYzk5YyAoIm5l dC9pcG4za2U6IHN1cHBvcnQgVE0iKQ0KPiA+IENjOiByb3Nlbi54dUBpbnRlbC5jb20NCj4gPiBD Yzogc3RhYmxlQGRwZGsub3JnDQo+ID4NCj4gPiBTaWduZWQtb2ZmLWJ5OiBLZXZpbiBUcmF5bm9y IDxrdHJheW5vckByZWRoYXQuY29tPg0KPiA+IC0tLQ0KPiA+ICBkcml2ZXJzL25ldC9pcG4za2Uv aXBuM2tlX3RtLmMgfCA2IC0tLS0tLQ0KPiA+ICAxIGZpbGUgY2hhbmdlZCwgNiBkZWxldGlvbnMo LSkNCj4gPg0KPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL25ldC9pcG4za2UvaXBuM2tlX3RtLmMN Cj4gPiBiL2RyaXZlcnMvbmV0L2lwbjNrZS9pcG4za2VfdG0uYyBpbmRleCBhZGYwMmMxNTcuLmE5 MzE0NWQ1OSAxMDA2NDQNCj4gPiAtLS0gYS9kcml2ZXJzL25ldC9pcG4za2UvaXBuM2tlX3RtLmMN Cj4gPiArKysgYi9kcml2ZXJzL25ldC9pcG4za2UvaXBuM2tlX3RtLmMNCj4gPiBAQCAtMTUxMSwx MCArMTUxMSw0IEBAIGlwbjNrZV90bV9oaWVyYXJjaHlfY29tbWl0X2NoZWNrKHN0cnVjdA0KPiBy dGVfZXRoX2RldiAqZGV2LA0KPiA+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgIE5VTEwsDQo+ID4gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgcnRlX3N0cmVycm9yKEVJTlZBTCkpOw0KPiA+IC0gICAgICAgICAgICAg ICAgICAgICAgIH0gZWxzZSB7DQo+ID4gLSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBy ZXR1cm4gLXJ0ZV90bV9lcnJvcl9zZXQoZXJyb3IsDQo+ID4gLSAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgRUlOVkFMLA0KPiA+IC0gICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIFJURV9UTV9FUlJPUl9UWVBFX1VOU1BFQ0lG SUVELA0KPiA+IC0gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg IE5VTEwsDQo+ID4gLSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgcnRlX3N0cmVycm9yKEVJTlZBTCkpOw0KPiA+ICAgICAgICAgICAgICAgICAgICAgICAgIH0N Cj4gPiAgICAgICAgICAgICAgICAgfQ0KPiA+IC0tDQo+ID4gMi4yMS4wDQo+ID4NCj4gDQo+IC0t DQo+IERhdmlkIE1hcmNoYW5kDQoNClJldmlld2VkLWJ5OiBSb3NlbiBYdSA8cm9zZW4ueHVAaW50 ZWwuY29tPg0K