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 CC79C44096; Wed, 22 May 2024 09:26:39 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 4C3A0400D7; Wed, 22 May 2024 09:26:39 +0200 (CEST) Received: from dkmailrelay1.smartsharesystems.com (smartserver.smartsharesystems.com [77.243.40.215]) by mails.dpdk.org (Postfix) with ESMTP id B7A8C400D6 for ; Wed, 22 May 2024 09:26:37 +0200 (CEST) Received: from smartserver.smartsharesystems.com (smartserver.smartsharesys.local [192.168.4.10]) by dkmailrelay1.smartsharesystems.com (Postfix) with ESMTP id B538220568; Wed, 22 May 2024 09:26:36 +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/af_xdp: fix umem map size for zero copy X-MimeOLE: Produced By Microsoft Exchange V6.5 Date: Wed, 22 May 2024 09:26:34 +0200 Message-ID: <98CBD80474FA8B44BF855DF32C47DC35E9F487@smartserver.smartshare.dk> In-Reply-To: X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH v2] net/af_xdp: fix umem map size for zero copy Thread-Index: AQHao2YTmeB1E66cJUCYtzW336D4ZbGiCeoAgAB1C/CAAFvw0A== References: <20240426005128.148730-1-frank.du@intel.com> <20240511052618.1890677-1-frank.du@intel.com> <0d02e8c6-0ef4-44e3-9dd2-94685b46136a@amd.com> From: =?UTF-8?B?TW9ydGVuIEJyw7hydXA=?= To: "Du, Frank" , "Ferruh Yigit" , , "Andrew Rybchenko" , "Burakov, Anatoly" Cc: "Loftus, Ciara" 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 PiBGcm9tOiBEdSwgRnJhbmsgW21haWx0bzpmcmFuay5kdUBpbnRlbC5jb21dDQo+IFNlbnQ6IFdl ZG5lc2RheSwgMjIgTWF5IDIwMjQgMDMuMjUNCj4gDQo+ID4gRnJvbTogRmVycnVoIFlpZ2l0IDxm ZXJydWgueWlnaXRAYW1kLmNvbT4NCj4gPiBTZW50OiBXZWRuZXNkYXksIE1heSAyMiwgMjAyNCAx OjU4IEFNDQo+ID4NCj4gPiBPbiA1LzExLzIwMjQgNjoyNiBBTSwgRnJhbmsgRHUgd3JvdGU6DQo+ ID4gPiBUaGUgY3VycmVudCBjYWxjdWxhdGlvbiBhc3N1bWVzIHRoYXQgdGhlIG1idWZzIGFyZSBj b250aWd1b3VzLg0KPiA+ID4gSG93ZXZlciwgdGhpcyBhc3N1bXB0aW9uIGlzIGluY29ycmVjdCB3 aGVuIHRoZSBtZW1vcnkgc3BhbnMgYWNyb3NzIGEgaHVnZQ0KPiA+IHBhZ2UuDQo+ID4gPiBDb3Jy ZWN0IHRvIGRpcmVjdGx5IHJlYWQgdGhlIHNpemUgZnJvbSB0aGUgbWVtcG9vbCBtZW1vcnkgY2h1 bmtzLg0KPiA+ID4NCj4gPiA+IFNpZ25lZC1vZmYtYnk6IEZyYW5rIER1IDxmcmFuay5kdUBpbnRl bC5jb20+DQo+ID4gPg0KPiA+ID4gLS0tDQo+ID4gPiB2MjoNCj4gPiA+ICogQWRkIHZpcnR1YWwg Y29udGlndW91cyBkZXRlY3QgZm9yIGZvciBtdWx0aXBsZSBtZW1oZHJzLg0KPiA+ID4gLS0tDQo+ ID4gPiAgZHJpdmVycy9uZXQvYWZfeGRwL3J0ZV9ldGhfYWZfeGRwLmMgfCAzNA0KPiA+ID4gKysr KysrKysrKysrKysrKysrKysrKysrLS0tLS0NCj4gPiA+ICAxIGZpbGUgY2hhbmdlZCwgMjggaW5z ZXJ0aW9ucygrKSwgNiBkZWxldGlvbnMoLSkNCj4gPiA+DQo+ID4gPiBkaWZmIC0tZ2l0IGEvZHJp dmVycy9uZXQvYWZfeGRwL3J0ZV9ldGhfYWZfeGRwLmMNCj4gPiA+IGIvZHJpdmVycy9uZXQvYWZf eGRwL3J0ZV9ldGhfYWZfeGRwLmMNCj4gPiA+IGluZGV4IDI2OGExMzBjNDkuLjc0NTYxMDhkNmQg MTAwNjQ0DQo+ID4gPiAtLS0gYS9kcml2ZXJzL25ldC9hZl94ZHAvcnRlX2V0aF9hZl94ZHAuYw0K PiA+ID4gKysrIGIvZHJpdmVycy9uZXQvYWZfeGRwL3J0ZV9ldGhfYWZfeGRwLmMNCj4gPiA+IEBA IC0xMDM5LDE2ICsxMDM5LDM1IEBAIGV0aF9saW5rX3VwZGF0ZShzdHJ1Y3QgcnRlX2V0aF9kZXYg KmRldg0KPiA+ID4gX19ydGVfdW51c2VkLCAgfQ0KPiA+ID4NCj4gPiA+ICAjaWYgZGVmaW5lZChY RFBfVU1FTV9VTkFMSUdORURfQ0hVTktfRkxBRykNCj4gPiA+IC1zdGF0aWMgaW5saW5lIHVpbnRw dHJfdCBnZXRfYmFzZV9hZGRyKHN0cnVjdCBydGVfbWVtcG9vbCAqbXAsDQo+ID4gPiB1aW50NjRf dCAqYWxpZ24pDQo+ID4gPiArc3RhdGljIGlubGluZSB1aW50cHRyX3QgZ2V0X21lbWhkcl9pbmZv KHN0cnVjdCBydGVfbWVtcG9vbCAqbXAsDQo+ID4gPiArdWludDY0X3QgKmFsaWduLCBzaXplX3Qg KmxlbikNCj4gPiA+ICB7DQo+ID4gPiAtCXN0cnVjdCBydGVfbWVtcG9vbF9tZW1oZHIgKm1lbWhk cjsNCj4gPiA+ICsJc3RydWN0IHJ0ZV9tZW1wb29sX21lbWhkciAqbWVtaGRyLCAqbmV4dDsNCj4g PiA+ICAJdWludHB0cl90IG1lbWhkcl9hZGRyLCBhbGlnbmVkX2FkZHI7DQo+ID4gPiArCXNpemVf dCBtZW1oZHJfbGVuID0gMDsNCj4gPiA+DQo+ID4gPiArCS8qIGdldCB0aGUgbWVtcG9vbCBiYXNl IGFkZHIgYW5kIGFsaWduICovDQo+ID4gPiAgCW1lbWhkciA9IFNUQUlMUV9GSVJTVCgmbXAtPm1l bV9saXN0KTsNCj4gPiA+ICAJbWVtaGRyX2FkZHIgPSAodWludHB0cl90KW1lbWhkci0+YWRkcjsN Cg0KVGhpcyBpcyBub3QgYSBuZXcgYnVnOyBidXQgaWYgdGhlIG1lbXBvb2wgaXMgbm90IHBvcHVs YXRlZCwgbWVtaGRyIGlzIE5VTEwgaGVyZS4NCg0KPiA+ID4gIAlhbGlnbmVkX2FkZHIgPSBtZW1o ZHJfYWRkciAmIH4oZ2V0cGFnZXNpemUoKSAtIDEpOw0KPiA+ID4gIAkqYWxpZ24gPSBtZW1oZHJf YWRkciAtIGFsaWduZWRfYWRkcjsNCj4gPiA+DQo+ID4NCj4gPiBJIGFtIGF3YXJlIHRoaXMgaXMg bm90IHBhcnQgb2YgdGhpcyBwYXRjaCwgYnV0IGFzIG5vdGUsIGNhbid0IHdlIHVzZQ0KPiA+ICdS VEVfQUxJR05fRkxPT1InIHRvIGNhbGN1bGF0ZSBhbGlnbmVkIGFkZHJlc3MuDQo+IA0KPiBTdXJl LCB3aWxsIHVzZSBSVEVfQUxJR05fRkxPT1IgaW4gbmV4dCB2ZXJzaW9uLg0KPiANCj4gPg0KPiA+ DQo+ID4gPiArCW1lbWhkcl9sZW4gKz0gbWVtaGRyLT5sZW47DQo+ID4gPiArDQo+ID4gPiArCS8q IGNoZWNrIGlmIHZpcnR1YWwgY29udGlndW91cyBtZW1vcnkgZm9yIG11bHRpcGxlIG1lbWhkcnMg Ki8NCj4gPiA+ICsJbmV4dCA9IFNUQUlMUV9ORVhUKG1lbWhkciwgbmV4dCk7DQo+ID4gPiArCXdo aWxlIChuZXh0ICE9IE5VTEwpIHsNCj4gPiA+ICsJCWlmICgodWludHB0cl90KW5leHQtPmFkZHIg IT0gKHVpbnRwdHJfdCltZW1oZHItPmFkZHIgKyBtZW1oZHItDQo+ID4gPmxlbikgew0KPiA+ID4g KwkJCUFGX1hEUF9MT0coRVJSLCAibWVtb3J5IGNodW5rcyBub3QgdmlydHVhbA0KPiA+IGNvbnRp Z3VvdXMsICINCj4gPiA+ICsJCQkJCSJuZXh0OiAlcCwgY3VyOiAlcChsZW46ICUiIFBSSWQ2NA0K PiA+ICIgKVxuIiwNCj4gPiA+ICsJCQkJCW5leHQtPmFkZHIsIG1lbWhkci0+YWRkciwgbWVtaGRy LQ0KPiA+ID5sZW4pOw0KPiA+ID4gKwkJCXJldHVybiAwOw0KPiA+ID4gKwkJfQ0KPiA+ID4NCj4g Pg0KPiA+IElzbid0IHRoZXJlIGEgbWVtcG9vbCBmbGFnIHRoYXQgY2FuIGhlbHAgdXMgZmlndXJl IG91dCBtZW1wb29sIGlzIG5vdCBJT1ZBDQo+ID4gY29udGlndW91cz8gSXNuJ3QgaXQgc3VmZmlj aWVudCBvbiBpdHMgb3duPw0KPiANCj4gSW5kZWVkLCB3aGF0IHdlIG5lZWQgdG8gYXNjZXJ0YWlu IGlzIHdoZXRoZXIgaXQncyBjb250aWd1b3VzIGluIENQVSB2aXJ0dWFsDQo+IHNwYWNlLCBub3Qg SU9WQS4gSSBoYXZlbid0IGNvbWUgYWNyb3NzIGEgZmxhZyBzcGVjaWZpY2FsbHkgZm9yIENQVSB2 aXJ0dWFsDQo+IGNvbnRpZ3VpdHkuIFRoZSBtYWpvciBsaW1pdGF0aW9uIGluIFhEUCBpcyBYU0sg VU1FTSBvbmx5IHN1cHBvcnRzIHJlZ2lzdGVyaW5nDQo+IGEgc2luZ2xlIGNvbnRpZ3VvdXMgdmly dHVhbCBtZW1vcnkgYXJlYS4NCg0KSSB3b3VsZCBhc3N1bWUgdGhhdCB0aGUgRUFMIG1lbW9yeSBt YW5hZ2VyIG1lcmdlcyBmcmVlIG1lbW9yeSBpbnRvIGNvbnRpZ3VvdXMgY2h1bmtzIHdoZW5ldmVy IHBvc3NpYmxlLg0KQEFuYXRvbHksIHBsZWFzZSBjb25maXJtPw0KDQpJZiBteSBhc3N1bXB0aW9u IGlzIGNvcnJlY3QsIGl0IG1lYW5zIHRoYXQgaWYgbXAtPm5iX21lbV9jaHVua3MgIT0gMSwgdGhl biB0aGUgbWVtcG9vbCBpcyBub3QgdmlydHVhbCBjb250aWd1b3VzLiBBbmQgaWYgbXAtPm5iX21l bV9jaHVua3MgPT0gMSwgdGhlbiBpdCBpczsgdGhlcmUgaXMgbm8gbmVlZCB0byBpdGVyYXRlIHRo cm91Z2ggdGhlIG1lbWhkciBsaXN0Lg0KDQo+IA0KPiA+DQo+ID4NCj4gPiA+ICsJCS8qIHZpcnR1 YWwgY29udGlndW91cyAqLw0KPiA+ID4gKwkJbWVtaGRyID0gbmV4dDsNCj4gPiA+ICsJCW1lbWhk cl9sZW4gKz0gbWVtaGRyLT5sZW47DQo+ID4gPiArCQluZXh0ID0gU1RBSUxRX05FWFQobWVtaGRy LCBuZXh0KTsNCj4gPiA+ICsJfQ0KPiA+ID4NCj4gPiA+ICsJKmxlbiA9IG1lbWhkcl9sZW47DQo+ ID4gPiAgCXJldHVybiBhbGlnbmVkX2FkZHI7DQo+ID4gPiAgfQ0KPiA+ID4NCj4gPg0KPiA+IFRo aXMgZnVuY3Rpb24gZ29lcyB0b28gbXVjaCBkZXRhaWxzIG9mIHRoZSBtZW1wb29sIG9iamVjdCwg YW5kIGFueSBjaGFuZ2UgaW4NCj4gPiBtZW1wb29sIGRldGFpbHMgaGFzIHBvdGVudGlhbCB0byBi cmVhayB0aGlzIGNvZGUuDQo+ID4NCj4gPiBAQW5kcmV3LCBATW9ydGVuLCBkbyB5b3UgdGhpbmsg ZG9lcyBpdCBtYWtlIHNlbnNlIHRvIGhhdmUNCj4gPiAncnRlX21lbXBvb2xfaW5mb19nZXQoKScg a2luZCBvZiBmdW5jdGlvbiwgdGhhdCBwcm92aWRlcyBhdCBsZWFzdCBhZGRyZXNzDQo+IGFuZA0K PiA+IGxlbmd0aCBvZiB0aGUgbWVtcG9vbCwgYW5kIHVzZWQgaGVyZT8NCj4gPg0KPiA+IFRoaXMg aGVscHMgdG8gaGlkZSBpbnRlcm5hbCBkZXRhaWxzIGFuZCBjb21wbGV4aXR5IG9mIHRoZSBtZW1w b29sIGZvciB1c2Vycy4NCg0KSSB0aGluayBhbGwgdGhlIHJlbGV2YW50IGluZm9ybWF0aW9uIGlz IGF2YWlsYWJsZSBhcyAocHVibGljKSBmaWVsZHMgZGlyZWN0bHkgaW4gdGhlIHJ0ZV9tZW1wb29s Lg0KSSBhZ3JlZSBhYm91dCBoaWRpbmcgaW50ZXJuYWwgZGV0YWlscy4gRm9yIGRpc2NyaW1pbmF0 aW5nIGJldHdlZW4gcHJpdmF0ZSBhbmQgcHVibGljIGluZm9ybWF0aW9uLCBJIHdvdWxkIHByZWZl ciBtYXJraW5nIHRoZSAicHJpdmF0ZSIgZmllbGRzIGluIHRoZSBydGVfbWVtcG9vbCBzdHJ1Y3R1 cmUgYXMgc3VjaC4NCg0KT3B0aW1hbGx5IHdlIG5lZWQgYW4gcnRlX21lbXBvb2xfY3JlYXRlKCkg ZmxhZywgc3BlY2lmeWluZyB0aGF0IHRoZSBtZW1wb29sIG9iamVjdHMgbXVzdCBiZSBhbGxvY2F0 ZWQgYXMgb25lIGNodW5rIG9mIG1lbW9yeSB3aXRoIGNvbnRpZ3VvdXMgdmlydHVhbCBhZGRyZXNz ZXMgd2hlbiBwb3B1bGF0aW5nIHRoZSBtZW1wb29sLiBBcyBkaXNjdXNzZWQgaW4gYW5vdGhlciB0 aHJlYWQgWzFdLCB0aGUgcHJvcG9zZWQgcG9pbnRlciBjb21wcmVzc2lvbiBsaWJyYXJ5IHdvdWxk IGFsc28gYmVuZWZpdCBmcm9tIHN1Y2ggYSBtZW1wb29sIGZsYWcuDQoNClsxXSBodHRwczovL2lu Ym94LmRwZGsub3JnL2Rldi85OENCRDgwNDc0RkE4QjQ0QkY4NTVERjMyQzQ3REMzNUU5RjQ1Q0Bz bWFydHNlcnZlci5zbWFydHNoYXJlLmRrLw0KDQo+ID4NCj4gPg0KPiA+ID4NCj4gPiA+IEBAIC0x MTI1LDYgKzExNDQsNyBAQCB4c2tfdW1lbV9pbmZvICp4ZHBfdW1lbV9jb25maWd1cmUoc3RydWN0 DQo+ID4gcG1kX2ludGVybmFscyAqaW50ZXJuYWxzLA0KPiA+ID4gIAl2b2lkICpiYXNlX2FkZHIg PSBOVUxMOw0KPiA+ID4gIAlzdHJ1Y3QgcnRlX21lbXBvb2wgKm1iX3Bvb2wgPSByeHEtPm1iX3Bv b2w7DQo+ID4gPiAgCXVpbnQ2NF90IHVtZW1fc2l6ZSwgYWxpZ24gPSAwOw0KPiA+ID4gKwlzaXpl X3QgbGVuID0gMDsNCj4gPiA+DQo+ID4gPiAgCWlmIChpbnRlcm5hbHMtPnNoYXJlZF91bWVtKSB7 DQo+ID4gPiAgCQlpZiAoZ2V0X3NoYXJlZF91bWVtKHJ4cSwgaW50ZXJuYWxzLT5pZl9uYW1lLCAm dW1lbSkgPCAwKSBAQA0KPiA+ID4gLTExNTYsMTAgKzExNzYsMTIgQEAgeHNrX3VtZW1faW5mbyAq eGRwX3VtZW1fY29uZmlndXJlKHN0cnVjdA0KPiA+IHBtZF9pbnRlcm5hbHMgKmludGVybmFscywN Cj4gPiA+ICAJCX0NCj4gPiA+DQo+ID4gPiAgCQl1bWVtLT5tYl9wb29sID0gbWJfcG9vbDsNCj4g PiA+IC0JCWJhc2VfYWRkciA9ICh2b2lkICopZ2V0X2Jhc2VfYWRkcihtYl9wb29sLCAmYWxpZ24p Ow0KPiA+ID4gLQkJdW1lbV9zaXplID0gKHVpbnQ2NF90KW1iX3Bvb2wtPnBvcHVsYXRlZF9zaXpl ICoNCj4gPiA+IC0JCQkJKHVpbnQ2NF90KXVzcl9jb25maWcuZnJhbWVfc2l6ZSArDQo+ID4gPiAt CQkJCWFsaWduOw0KPiA+ID4gKwkJYmFzZV9hZGRyID0gKHZvaWQgKilnZXRfbWVtaGRyX2luZm8o bWJfcG9vbCwgJmFsaWduLCAmbGVuKTsNCj4gPiA+DQo+ID4NCj4gPiBJcyB0aGlzIGNhbGN1bGF0 aW9uIGNvcnJlY3QgaWYgbWVtcG9vbCBpcyBub3QgYWxyZWFkeSBhbGlnbmVkIHRvIHBhZ2Ugc2l6 ZT8NCg0KUGxlYXNlIG5vdGU6IFRoZSBtZW1wb29sIHVzZXMgb25lIG1lbXpvbmUgZm9yIHRoZSBt ZW1wb29sIHN0cnVjdHVyZSBpdHNlbGYuIFRoZSBvYmplY3RzIGluIHRoZSBtZW1wb29sIGFyZSBz dG9yZWQgaW4gYW5vdGhlciBtZW16b25lIChvciBtdWx0aXBsZSBvdGhlciBtZW16b25lcywgaWYg bmVjZXNzYXJ5KS4gSSB0aGluayB5b3UgYXJlIHRhbGtpbmcgYWJvdXQgdGhlIGFsaWdubWVudCBv ZiB0aGUgbWVtcG9vbCBvYmplY3QgY2h1bmssIG5vdCBvZiB0aGUgbWVtcG9vbCBzdHJ1Y3R1cmUg aXRzZWxmLg0KDQo+ID4NCj4gPiBMaWtlIGluIGFuIGV4YW1wbGUgcGFnZSBzaXplIGlzICcweDEw MDAnLCBhbmQgIm1lbWhkcl9hZGRyID0gMHgwMDBhMTA4MCINCj4gPiByZXR1cm5lZCBhbGlnbmVk IGFkZHJlc3MgaXMgJzB4MDAwYTEwMDAnLCAiYmFzZV9hZGRyID0gMHgwMDBhMTAwMCINCj4gPg0K PiA+IEFueSBhY2Nlc3MgYmV0d2VlbiAnMHgwMDBhMTAwMCcgJiAnMHgwMDBhMTA4MCcgaXMgaW52 YWxpZC4gSXMgdGhpcyBleHBlY3RlZD8NCj4gDQo+IFllcywgc2luY2UgdGhlIFhTSyBVTUVNIG1l bW9yeSBhcmVhIHJlcXVpcmVzIHBhZ2UgYWxpZ25tZW50LiBIb3dldmVyLCBubyBuZWVkDQo+IHRv IHdvcnJ5OyB0aGUgbWVtb3J5IHBvaW50ZXIgaW4gdGhlIFhTSyBUWC9SWCBkZXNjcmlwdG9yIGlz IG9idGFpbmVkIGZyb20gdGhlDQo+IG1idWYgZGF0YSBhcmVhLiBXZSBkb27igJl0IGhhdmUgYW55 IGNoYW5jZSB0byBhY2Nlc3MgdGhlIGludmFsaWQgcmFuZ2UNCj4gWzB4MDAwYTEwMDA6IDB4MDAw YTEwODBdIGhlcmUuDQo+IA0KPiA+DQo+ID4NCj4gPiA+ICsJCWlmICghYmFzZV9hZGRyKSB7DQo+ ID4gPiArCQkJQUZfWERQX0xPRyhFUlIsICJGYWlsZWQgdG8gcGFyc2UgbWVtaGRyIGluZm8gZnJv bQ0KPiA+IHBvb2xcbiIpOw0KPiA+ID4NCj4gPg0KPiA+IExvZyBtZXNzYWdlIGlzIG5vdCBhY2N1 cmF0ZSwgaXQgaXMgbm90IHBhcnNpbmcgbWVtaGRyIGluZm8gZmFpbGVkLCBidXQNCj4gbWVtcG9v bA0KPiA+IHdhcyBub3Qgc2F0aXNmeWluZyBleHBlY3RhdGlvbi4NCg0KTG9va2luZyBhdCBnZXRf bWVtaGRyX2luZm8oKSBhYm92ZSwgaXQgY291bGQgYmUgZWl0aGVyIG1lbXBvb2wgb3IgbWVtaGRy IGZhaWxpbmcgdG8gcGFyc2UuDQoNCj4gDQo+IFRoYW5rcywgd2lsbCBjb3JyZWN0IGl0IGluIG5l eHQgdmVyc2lvbi4NCj4gDQo+ID4NCj4gPiA+ICsJCQlnb3RvIGVycjsNCj4gPiA+ICsJCX0NCj4g PiA+ICsJCXVtZW1fc2l6ZSA9ICh1aW50NjRfdClsZW4gKyBhbGlnbjsNCj4gPiA+DQo+ID4gPiAg CQlyZXQgPSB4c2tfdW1lbV9fY3JlYXRlKCZ1bWVtLT51bWVtLCBiYXNlX2FkZHIsDQo+ID4gdW1l bV9zaXplLA0KPiA+ID4gIAkJCQkmcnhxLT5mcSwgJnJ4cS0+Y3EsICZ1c3JfY29uZmlnKTsNCg0K