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 03ECE42A5D; Thu, 4 May 2023 15:21:49 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9644841144; Thu, 4 May 2023 15:21:49 +0200 (CEST) Received: from dkmailrelay1.smartsharesystems.com (smartserver.smartsharesystems.com [77.243.40.215]) by mails.dpdk.org (Postfix) with ESMTP id 65333410DC for ; Thu, 4 May 2023 15:21:48 +0200 (CEST) Received: from smartserver.smartsharesystems.com (smartserver.smartsharesys.local [192.168.4.10]) by dkmailrelay1.smartsharesystems.com (Postfix) with ESMTP id 5C8AB2032C; Thu, 4 May 2023 15:21:47 +0200 (CEST) Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: base64 Subject: RE: [PATCH v2] net/ixgbe: add proper memory barriers for some Rx functions X-MimeOLE: Produced By Microsoft Exchange V6.5 Date: Thu, 4 May 2023 15:21:45 +0200 Message-ID: <98CBD80474FA8B44BF855DF32C47DC35D878E4@smartserver.smartshare.dk> In-Reply-To: <43fb4e62-63b5-7076-327e-dc30cd561fff@loongson.cn> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH v2] net/ixgbe: add proper memory barriers for some Rx functions Thread-Index: Adl+iq5TUSKdWb8eRFa4ViK7NCEKagAAHHHQ References: <20230424090532.367194-1-zhoumin@loongson.cn> <20671e5e-8e86-4bc3-2d95-4cec014b0539@yandex.ru> <43fb4e62-63b5-7076-327e-dc30cd561fff@loongson.cn> From: =?UTF-8?B?TW9ydGVuIEJyw7hydXA=?= To: "zhoumin" , "Konstantin Ananyev" Cc: , , , , , , "Tyler Retzlaff" 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 PiBGcm9tOiB6aG91bWluIFttYWlsdG86emhvdW1pbkBsb29uZ3Nvbi5jbl0NCj4gU2VudDogVGh1 cnNkYXksIDQgTWF5IDIwMjMgMTUuMTcNCj4gDQo+IEhpIEtvbnN0YW50aW4sDQo+IA0KPiBUaGFu a3MgZm9yIHlvdXLCoCBjb21tZW50cy4NCj4gDQo+IE9uIDIwMjMvNS8xIOS4i+WNiDk6MjksIEtv bnN0YW50aW4gQW5hbnlldiB3cm90ZToNCj4gPj4gU2VnbWVudGF0aW9uIGZhdWx0IGhhcyBiZWVu IG9ic2VydmVkIHdoaWxlIHJ1bm5pbmcgdGhlDQo+ID4+IGl4Z2JlX3JlY3ZfcGt0c19scm8oKSBm dW5jdGlvbiB0byByZWNlaXZlIHBhY2tldHMgb24gdGhlIExvb25nc29uIDNDNTAwMA0KPiA+PiBw cm9jZXNzb3Igd2hpY2ggaGFzIDY0IGNvcmVzIGFuZCA0IE5VTUEgbm9kZXMuDQo+ID4+DQo+ID4+ IEZyb20gdGhlIGl4Z2JlX3JlY3ZfcGt0c19scm8oKSBmdW5jdGlvbiwgd2UgZm91bmQgdGhhdCBh cyBsb25nIGFzIHRoZQ0KPiA+PiBmaXJzdA0KPiA+PiBwYWNrZXQgaGFzIHRoZSBFT1AgYml0IHNl dCwgYW5kIHRoZSBsZW5ndGggb2YgdGhpcyBwYWNrZXQgaXMgbGVzcw0KPiA+PiB0aGFuIG9yDQo+ ID4+IGVxdWFsIHRvIHJ4cS0+Y3JjX2xlbiwgdGhlIHNlZ21lbnRhdGlvbiBmYXVsdCB3aWxsIGRl ZmluaXRlbHkgaGFwcGVuDQo+ID4+IGV2ZW4NCj4gPj4gdGhvdWdoIG9uIHRoZSBvdGhlciBwbGF0 Zm9ybXMsIHN1Y2ggYXMgWDg2Lg0KPiA+Pg0KPiA+PiBCZWNhdXNlIHdoZW4gcHJvY2Vzc2QgdGhl IGZpcnN0IHBhY2tldCB0aGUgZmlyc3Rfc2VnLT5uZXh0IHdpbGwgYmUNCj4gPj4gTlVMTCwgaWYN Cj4gPj4gYXQgdGhlIHNhbWUgdGltZSB0aGlzIHBhY2tldCBoYXMgdGhlIEVPUCBiaXQgc2V0IGFu ZCBpdHMgbGVuZ3RoIGlzIGxlc3MNCj4gPj4gdGhhbiBvciBlcXVhbCB0byByeHEtPmNyY19sZW4s IHRoZSBmb2xsb3dpbmcgbG9vcCB3aWxsIGJlIGV4Y2VjdXRlZDoNCj4gPj4NCj4gPj4gwqDCoMKg IGZvciAobHAgPSBmaXJzdF9zZWc7IGxwLT5uZXh0ICE9IHJ4bTsgbHAgPSBscC0+bmV4dCkNCj4g Pj4gwqDCoMKgwqDCoMKgwqAgOw0KPiA+Pg0KPiA+PiBXZSBrbm93IHRoYXQgdGhlIGZpcnN0X3Nl Zy0+bmV4dCB3aWxsIGJlIE5VTEwgdW5kZXIgdGhpcyBjb25kaXRpb24uDQo+ID4+IFNvIHRoZQ0K PiA+PiBleHByZXNzaW9uIG9mIGxwLT5uZXh0LT5uZXh0IHdpbGwgY2F1c2UgdGhlIHNlZ21lbnRh dGlvbiBmYXVsdC4NCj4gPj4NCj4gPj4gTm9ybWFsbHksIHRoZSBsZW5ndGggb2YgdGhlIGZpcnN0 IHBhY2tldCB3aXRoIEVPUCBiaXQgc2V0IHdpbGwgYmUNCj4gPj4gZ3JlYXRlcg0KPiA+PiB0aGFu IHJ4cS0+Y3JjX2xlbi4gSG93ZXZlciwgdGhlIG91dC1vZi1vcmRlciBleGVjdXRpb24gb2YgQ1BV IG1heQ0KPiA+PiBtYWtlIHRoZQ0KPiA+PiByZWFkIG9yZGVyaW5nIG9mIHRoZSBzdGF0dXMgYW5k IHRoZSByZXN0IG9mIHRoZSBkZXNjcmlwdG9yIGZpZWxkcyBpbg0KPiA+PiB0aGlzDQo+ID4+IGZ1 bmN0aW9uIG5vdCBiZSBjb3JyZWN0LiBUaGUgcmVsYXRlZCBjb2RlcyBhcmUgYXMgZm9sbG93aW5n Og0KPiA+Pg0KPiA+PiDCoMKgwqDCoMKgwqDCoCByeGRwID0gJnJ4X3JpbmdbcnhfaWRdOw0KPiA+ PiDCoCMxwqDCoMKgwqAgc3RhdGVyciA9IHJ0ZV9sZV90b19jcHVfMzIocnhkcC0+d2IudXBwZXIu c3RhdHVzX2Vycm9yKTsNCj4gPj4NCj4gPj4gwqDCoMKgwqDCoMKgwqAgaWYgKCEoc3RhdGVyciAm IElYR0JFX1JYREFEVl9TVEFUX0REKSkNCj4gPj4gwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBicmVh azsNCj4gPj4NCj4gPj4gwqAjMsKgwqDCoMKgIHJ4ZCA9ICpyeGRwOw0KPiA+Pg0KPiA+PiBUaGUg c2VudGVuY2UgIzIgbWF5IGJlIGV4ZWN1dGVkIGJlZm9yZSBzZW50ZW5jZSAjMS4gVGhpcyBhY3Rp b24gaXMNCj4gPj4gbGlrZWx5DQo+ID4+IHRvIG1ha2UgdGhlIHJlYWR5IHBhY2tldCB6ZXJvIGxl bmd0aC4gSWYgdGhlIHBhY2tldCBpcyB0aGUgZmlyc3QNCj4gPj4gcGFja2V0IGFuZA0KPiA+PiBo YXMgdGhlIEVPUCBiaXQgc2V0LCB0aGUgYWJvdmUgc2VnbWVudGF0aW9uIGZhdWx0IHdpbGwgaGFw cGVuLg0KPiA+Pg0KPiA+PiBTbywgd2Ugc2hvdWxkIGFkZCBydGVfcm1iKCkgdG8gZW5zdXJlIHRo ZSByZWFkIG9yZGVyaW5nIGJlIGNvcnJlY3QuDQo+ID4+IFdlIGFsc28NCj4gPj4gZGlkIHRoZSBz YW1lIHRoaW5nIGluIHRoZSBpeGdiZV9yZWN2X3BrdHMoKSBmdW5jdGlvbiB0byBtYWtlIHRoZSBy eGQNCj4gPj4gZGF0YQ0KPiA+PiBiZSB2YWxpZCBldmVuIHRob3VndGggd2UgZGlkIG5vdCBmaW5k IHNlZ21lbnRhdGlvbiBmYXVsdCBpbiB0aGlzDQo+ID4+IGZ1bmN0aW9uLg0KPiA+Pg0KPiA+PiBT aWduZWQtb2ZmLWJ5OiBNaW4gWmhvdSA8emhvdW1pbkBsb29uZ3Nvbi5jbj4NCj4gPj4gLS0tDQo+ ID4+IHYyOg0KPiA+PiAtIE1ha2UgdGhlIGNhbGxpbmcgb2YgcnRlX3JtYigpIGZvciBhbGwgcGxh dGZvcm1zDQo+ID4+IC0tLQ0KPiA+PiDCoGRyaXZlcnMvbmV0L2l4Z2JlL2l4Z2JlX3J4dHguYyB8 IDMgKysrDQo+ID4+IMKgMSBmaWxlIGNoYW5nZWQsIDMgaW5zZXJ0aW9ucygrKQ0KPiA+Pg0KPiA+ PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy9uZXQvaXhnYmUvaXhnYmVfcnh0eC5jDQo+ID4+IGIvZHJp dmVycy9uZXQvaXhnYmUvaXhnYmVfcnh0eC5jDQo+ID4+IGluZGV4IGM5ZDZjYTllZmUuLjMwMmE1 YWI3ZmYgMTAwNjQ0DQo+ID4+IC0tLSBhL2RyaXZlcnMvbmV0L2l4Z2JlL2l4Z2JlX3J4dHguYw0K PiA+PiArKysgYi9kcml2ZXJzL25ldC9peGdiZS9peGdiZV9yeHR4LmMNCj4gPj4gQEAgLTE4MjMs NiArMTgyMyw4IEBAIGl4Z2JlX3JlY3ZfcGt0cyh2b2lkICpyeF9xdWV1ZSwgc3RydWN0IHJ0ZV9t YnVmDQo+ID4+ICoqcnhfcGt0cywNCj4gPj4gwqDCoMKgwqDCoMKgwqDCoCBzdGF0ZXJyID0gcnhk cC0+d2IudXBwZXIuc3RhdHVzX2Vycm9yOw0KPiA+PiDCoMKgwqDCoMKgwqDCoMKgIGlmICghKHN0 YXRlcnIgJiBydGVfY3B1X3RvX2xlXzMyKElYR0JFX1JYREFEVl9TVEFUX0REKSkpDQo+ID4+IMKg wqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBicmVhazsNCj4gPj4gKw0KPiA+PiArwqDCoMKgwqDCoMKg wqAgcnRlX3JtYigpOw0KPiA+PiDCoMKgwqDCoMKgwqDCoMKgIHJ4ZCA9ICpyeGRwOw0KPiA+DQo+ ID4NCj4gPg0KPiA+IEluZGVlZCwgbG9va3MgbGlrZSBhIHByb2JsZW0gdG8gbWUgb24gc3lzdGVt cyB3aXRoIHJlbGF4ZWQgTU8uDQo+ID4gU3RyYW5nZSB0aGF0IGl0IHdhcyBuZXZlciBoaXQgb24g YXJtIG9yIHBwYyAtIGNjLWluZyBBUk0vUFBDIG1haW50YWluZXJzLg0KPiBUaGUgTG9vbmdBcmNo IGFyY2hpdGVjdHVyZSB1c2VzIHRoZSBXZWFrIENvbnNpc3RlbmN5IG1vZGVsIHdoaWNoIGNhbg0K PiBjYXVzZSB0aGUgcHJvYmxlbSwgZXNwZWNpYWxseSBpbiBzY2VuYXJpbyB3aXRoIG1hbnkgY29y ZXMsIHN1Y2ggYXMNCj4gTG9vbmdzb24gM0M1MDAwIHdpdGggZm91ciBOVU1BIG5vZGUsIHdoaWNo IGhhcyA2NCBjb3Jlcy4gSSBjYW5ub3QNCj4gcmVwcm9kdWNlIGl0IG9uIExvb25nc29uIDNDNTAw MCB3aXRoIG9uZSBOVU1BIG5vZGUsIHdoaWNoIGp1c3QgaGFzIDE2IGNvcmVzLg0KPiA+IEFib3V0 IGEgZml4IC0gbG9va3MgcmlnaHQsIGJ1dCBhIGJpdCBleGNlc3NpdmUgdG8gbWUgLQ0KPiA+IGFz IEkgdW5kZXJzdGFuZCBhbGwgd2UgbmVlZCBoZXJlIGlzIHRvIHByZXZlbnQgcmUtb3JkZXJpbmcg YnkgQ1BVIGl0c2VsZi4NCj4gWWVzLCB0aGFua3MgZm9yIGNjLWluZy4NCj4gPiBTbyBydGVfc21w X3JtYigpIHNlZW1zIGVub3VnaCBoZXJlLg0KPiA+IE9yIG1pZ2h0IGJlIGp1c3Q6DQo+ID4gc3Rh dGVyciA9IF9fYXRvbWljX2xvYWRfbigmcnhkcC0+d2IudXBwZXIuc3RhdHVzX2Vycm9yLA0KPiA+ IF9fQVRPTUlDX0FDUVVJUkUpOw0KPiA+DQo+IERvZXMgX19hdG9taWNfbG9hZF9uKCkgd29yayBv biBXaW5kb3dzIGlmIHdlIHVzZSBpdCB0byBzb2x2ZSB0aGlzIHByb2JsZW0gPw0KDQpZZXMsIF9f YXRvbWljX2xvYWRfbigpIHdvcmtzIG9uIFdpbmRvd3MgdG9vLg0KDQoNCg==