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 D4EF943672; Tue, 5 Dec 2023 09:04:14 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6D93542D45; Tue, 5 Dec 2023 09:04:14 +0100 (CET) Received: from dkmailrelay1.smartsharesystems.com (smartserver.smartsharesystems.com [77.243.40.215]) by mails.dpdk.org (Postfix) with ESMTP id 5DDA7402D2 for ; Tue, 5 Dec 2023 09:04:13 +0100 (CET) Received: from smartserver.smartsharesystems.com (smartserver.smartsharesys.local [192.168.4.10]) by dkmailrelay1.smartsharesystems.com (Postfix) with ESMTP id 1AB1A206C2; Tue, 5 Dec 2023 09:04:12 +0100 (CET) Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Subject: RE: [PATCH v1] mbuf: remove the redundant code for mbuf prefree X-MimeOLE: Produced By Microsoft Exchange V6.5 Date: Tue, 5 Dec 2023 09:04:08 +0100 Message-ID: <98CBD80474FA8B44BF855DF32C47DC35E9F08C@smartserver.smartshare.dk> In-Reply-To: <19ea4729-bdfa-4ee3-83d6-5922c1b42c0a@arm.com> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH v1] mbuf: remove the redundant code for mbuf prefree Thread-Index: AdonKQP5gIaDCUcXTlq8khzPFAtDIQAIkqbw References: <20231204023910.1714667-1-feifei.wang2@arm.com> <98CBD80474FA8B44BF855DF32C47DC35E9F081@smartserver.smartshare.dk> <19ea4729-bdfa-4ee3-83d6-5922c1b42c0a@arm.com> From: =?utf-8?B?TW9ydGVuIEJyw7hydXA=?= To: "Feifei Wang" Cc: , , "Ruifeng Wang" 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 PiBGcm9tOiBGZWlmZWkgV2FuZyBbbWFpbHRvOmZlaWZlaS53YW5nMkBhcm0uY29tXQ0KPiBTZW50 OiBUdWVzZGF5LCA1IERlY2VtYmVyIDIwMjMgMDQuMTMNCj4gDQo+IOWcqCAyMDIzLzEyLzQgMTU6 NDEsIE1vcnRlbiBCcsO4cnVwIOWGmemBkzoNCj4gPj4gRnJvbTogRmVpZmVpIFdhbmcgW21haWx0 bzpmZWlmZWkud2FuZzJAYXJtLmNvbV0NCj4gPj4gU2VudDogTW9uZGF5LCA0IERlY2VtYmVyIDIw MjMgMDMuMzkNCj4gPj4NCj4gPj4gRm9yICdydGVfcGt0bWJ1Zl9wcmVmcmVlX3NlZycgZnVuY3Rp b24sICdydGVfbWJ1Zl9yZWZjbnRfcmVhZChtKSA9PQ0KPiAxJw0KPiA+PiBhbmQgJ19fcnRlX21i dWZfcmVmY250X3VwZGF0ZShtLCAtMSkgPT0gMCcgYXJlIHRoZSBzYW1lIGNhc2VzIHdoZXJlDQo+ ID4+IG1idWYncyByZWZjbnQgdmFsdWUgc2hvdWxkIGJlIDEuIFRodXMgd2UgY2FuIHNpbXBsaWZ5 IHRoZSBjb2RlIGFuZA0KPiA+PiByZW1vdmUgdGhlIHJlZHVuZGFudCBwYXJ0Lg0KPiA+Pg0KPiA+ PiBGdXJ0aGVybW9yZSwgYWNjb3JkaW5nIHRvIFsxXSwgd2hlbiB0aGUgbWJ1ZiBpcyBzdG9yZWQg aW5zaWRlIHRoZQ0KPiA+PiBtZW1wb29sLCB0aGUgbS0+cmVmY250IHZhbHVlIHNob3VsZCBiZSAx LiBBbmQgdGhlbiBpdCBpcyBkZXRhY2hlZA0KPiA+PiBmcm9tIGl0cyBwYXJlbnQgZm9yIGFuIGlu ZGlyZWN0IG1idWYuIFRodXMgY2hhbmdlIHRoZSBkZXNjcmlwdGlvbiBvZg0KPiA+PiAncnRlX3Br dG1idWZfcHJlZnJlZV9zZWcnIGZ1bmN0aW9uLg0KPiA+Pg0KPiA+PiBbMV0NCj4gaHR0cHM6Ly9w YXRjaGVzLmRwZGsub3JnL3Byb2plY3QvZHBkay9wYXRjaC8yMDE3MDQwNDE2MjgwNy4yMDE1Ny00 LQ0KPiA+PiBvbGl2aWVyLm1hdHpANndpbmQuY29tLw0KPiA+Pg0KPiA+PiBTdWdnZXN0ZWQtYnk6 IFJ1aWZlbmcgV2FuZyA8cnVpZmVuZy53YW5nQGFybS5jb20+DQo+ID4+IFNpZ25lZC1vZmYtYnk6 IEZlaWZlaSBXYW5nIDxmZWlmZWkud2FuZzJAYXJtLmNvbT4NCj4gPj4gLS0tDQo+ID4+ICAgbGli L21idWYvcnRlX21idWYuaCB8IDIyICsrKy0tLS0tLS0tLS0tLS0tLS0tLS0NCj4gPj4gICAxIGZp bGUgY2hhbmdlZCwgMyBpbnNlcnRpb25zKCspLCAxOSBkZWxldGlvbnMoLSkNCj4gPj4NCj4gPj4g ZGlmZiAtLWdpdCBhL2xpYi9tYnVmL3J0ZV9tYnVmLmggYi9saWIvbWJ1Zi9ydGVfbWJ1Zi5oDQo+ ID4+IGluZGV4IDI4NmIzMmI3ODguLjQyZTliNTBkNTEgMTAwNjQ0DQo+ID4+IC0tLSBhL2xpYi9t YnVmL3J0ZV9tYnVmLmgNCj4gPj4gKysrIGIvbGliL21idWYvcnRlX21idWYuaA0KPiA+PiBAQCAt MTMyOCw3ICsxMzI4LDcgQEAgc3RhdGljIGlubGluZSBpbnQNCj4gPj4gX19ydGVfcGt0bWJ1Zl9w aW5uZWRfZXh0YnVmX2RlY3JlZihzdHJ1Y3QgcnRlX21idWYgKm0pDQo+ID4+ICAgICoNCj4gPj4g ICAgKiBUaGlzIGZ1bmN0aW9uIGRvZXMgdGhlIHNhbWUgdGhhbiBhIGZyZWUsIGV4Y2VwdCB0aGF0 IGl0IGRvZXMNCj4gbm90DQo+ID4+ICAgICogcmV0dXJuIHRoZSBzZWdtZW50IHRvIGl0cyBwb29s Lg0KPiA+PiAtICogSXQgZGVjcmVhc2VzIHRoZSByZWZlcmVuY2UgY291bnRlciwgYW5kIGlmIGl0 IHJlYWNoZXMgMCwgaXQgaXMNCj4gPj4gKyAqIEl0IGRlY3JlYXNlcyB0aGUgcmVmZXJlbmNlIGNv dW50ZXIsIGFuZCBpZiBpdCByZWFjaGVzIDEsIGl0IGlzDQo+ID4gTm8sIHRoZSBvcmlnaW5hbCBk ZXNjcmlwdGlvbiBpcyBjb3JyZWN0Lg0KPiA+IEhvd2V2ZXIsIHRoZSByZWZlcmVuY2UgY291bnRl ciBpcyBzZXQgdG8gMSB3aGVuIHB1dCBiYWNrIGluIHRoZSBwb29sLA0KPiBhcyBhIHNob3J0Y3V0 IHNvIGl0IGlzbid0IG5lZWRlZCB0byBiZSBzZXQgYmFjayB0byAxIHdoZW4gYWxsb2NhdGVkDQo+ IGZyb20gdGhlIHBvb2wuDQo+IA0KPiBPay4NCj4gDQo+IGZvciAnZWxzZSBpZiAoX19ydGVfbWJ1 Zl9yZWZjbnRfdXBkYXRlKG0sIC0xKSA9PSAwKScgY2FzZSwgaXQgaXMgZWFzeQ0KPiB0bw0KPiB1 bmRlcnN0YW5kLg0KPiANCj4gYnV0IGZvciAnKGxpa2VseShydGVfbWJ1Zl9yZWZjbnRfcmVhZCht KSA9PSAxKSknIGNhc2UsIEkgdGhpbmsgdGhpcw0KPiB3aWxsDQo+IGNyZWF0ZSBtaXNsZWFkaW5n LiBkcGRrIHVzZXJzIG1heWJlIGRpZmZpY3VsdCB0byB1bmRlcnN0YW5kIHdoeSB0aGUNCj4gY29k ZQ0KPiBjYW4gbm90IG1hdGNoIHRoZSBmdW5jdGlvbiBkZXNjcmlwdGlvbi4NCj4gDQo+IE1heWJl IHdlIG5lZWQgc29tZSBleHBsYW5hdGlvbiBoZXJlPw0KDQpBZ3JlZS4gSXQgaXMgcXVpdGUgY291 bnRlcmludHVpdGl2ZSAoYnV0IGEgY2xldmVyIG9wdGltaXphdGlvbiEpIHRoYXQgdGhlIHJlZmVy ZW5jZSBjb3VudGVyIGlzIDEgaW5zdGVhZCBvZiAwIHdoZW4gZnJlZS4NCg0KU29tZXRoaW5nIGxp a2U6DQoNCnN0YXRpYyBfX3J0ZV9hbHdheXNfaW5saW5lIHN0cnVjdCBydGVfbWJ1ZiAqDQpydGVf cGt0bWJ1Zl9wcmVmcmVlX3NlZyhzdHJ1Y3QgcnRlX21idWYgKm0pDQp7DQoJX19ydGVfbWJ1Zl9z YW5pdHlfY2hlY2sobSwgMCk7DQoNCglpZiAobGlrZWx5KHJ0ZV9tYnVmX3JlZmNudF9yZWFkKG0p ID09IDEpKSB7DQorCQkvKiBUaGlzIGJyYW5jaCBpcyBhIHBlcmZvcm1hbmNlIG9wdGltaXplZCB2 YXJpYW50IG9mIHRoZSBicmFuY2ggYmVsb3cuDQorCQkgKiBJZiB0aGUgcmVmZXJlbmNlIGNvdW50 ZXIgd291bGQgcmVhY2ggMCB3aGVuIGRlY3JlbWVudGluZyBpdCwNCisJCSAqIGRvIG5vdCBkZWNy ZW1lbnQgaXQgdG8gMCBhbmQgdGhlbiBpbml0aWFsaXplIGl0IHRvIDE7DQorCQkgKiBqdXN0IGxl YXZlIGl0IGF0IDEsIHRoZXJlYnkgYXZvaWRpbmcgd3JpdGluZyB0byBtZW1vcnkuDQorCQkgKi8N Cg0KCQlpZiAoIVJURV9NQlVGX0RJUkVDVChtKSkgew0KCQkJcnRlX3BrdG1idWZfZGV0YWNoKG0p Ow0KCQkJaWYgKFJURV9NQlVGX0hBU19FWFRCVUYobSkgJiYNCgkJCSAgICBSVEVfTUJVRl9IQVNf UElOTkVEX0VYVEJVRihtKSAmJg0KCQkJICAgIF9fcnRlX3BrdG1idWZfcGlubmVkX2V4dGJ1Zl9k ZWNyZWYobSkpDQoJCQkJcmV0dXJuIE5VTEw7DQoJCX0NCg0KCQlpZiAobS0+bmV4dCAhPSBOVUxM KQ0KCQkJbS0+bmV4dCA9IE5VTEw7DQoJCWlmIChtLT5uYl9zZWdzICE9IDEpDQoJCQltLT5uYl9z ZWdzID0gMTsNCisJCS8qIE5vIG5lZWQgdG8gaW5pdGlhbGl6ZSB0aGUgcmVmZXJlbmNlIGNvdW50 ZXI7IGl0IGlzIGFscmVhZHkgMS4gKi8NCg0KCQlyZXR1cm4gbTsNCg0KCX0gZWxzZSBpZiAoX19y dGVfbWJ1Zl9yZWZjbnRfdXBkYXRlKG0sIC0xKSA9PSAwKSB7DQoNCgkJaWYgKCFSVEVfTUJVRl9E SVJFQ1QobSkpIHsNCgkJCXJ0ZV9wa3RtYnVmX2RldGFjaChtKTsNCgkJCWlmIChSVEVfTUJVRl9I QVNfRVhUQlVGKG0pICYmDQoJCQkgICAgUlRFX01CVUZfSEFTX1BJTk5FRF9FWFRCVUYobSkgJiYN CgkJCSAgICBfX3J0ZV9wa3RtYnVmX3Bpbm5lZF9leHRidWZfZGVjcmVmKG0pKQ0KCQkJCXJldHVy biBOVUxMOw0KCQl9DQoNCgkJaWYgKG0tPm5leHQgIT0gTlVMTCkNCgkJCW0tPm5leHQgPSBOVUxM Ow0KCQlpZiAobS0+bmJfc2VncyAhPSAxKQ0KCQkJbS0+bmJfc2VncyA9IDE7DQorCQkvKiBJbml0 aWFsaXplIHRoZSByZWZlcmVuY2UgY291bnRlciB0byAxLCBzbw0KKwkJICogaW5jcmVtZW50aW5n IGl0IGlzIHVubmVjZXNzYXJ5IHdoZW4gYWxsb2NhdGluZyB0aGUgbWJ1Zi4NCisJCSAqLw0KCQly dGVfbWJ1Zl9yZWZjbnRfc2V0KG0sIDEpOw0KDQoJCXJldHVybiBtOw0KCX0NCglyZXR1cm4gTlVM TDsNCn0NCg0KDQpBbHRlcm5hdGl2ZWx5LCBhZGQgYSBmdW5jdGlvbiB0byBkbyB0aGUgaW5pdGlh bGl6YXRpb24gd29yazoNCg0Kc3RhdGljIF9fcnRlX2Fsd2F5c19pbmxpbmUgc3RydWN0IHJ0ZV9t YnVmICoNCnJ0ZV9wa3RtYnVmX3ByZWZyZWVfc2VnX2xhc3RfcmVmKHN0cnVjdCBydGVfbWJ1ZiAq bSwgY29uc3QgYm9vbCBpbml0X3JlZmNudCkNCnsNCglpZiAoIVJURV9NQlVGX0RJUkVDVChtKSkg ew0KCQlydGVfcGt0bWJ1Zl9kZXRhY2gobSk7DQoJCWlmIChSVEVfTUJVRl9IQVNfRVhUQlVGKG0p ICYmDQoJCSAgICBSVEVfTUJVRl9IQVNfUElOTkVEX0VYVEJVRihtKSAmJg0KCQkgICAgX19ydGVf cGt0bWJ1Zl9waW5uZWRfZXh0YnVmX2RlY3JlZihtKSkNCgkJCXJldHVybiBOVUxMOw0KCX0NCg0K CWlmIChtLT5uZXh0ICE9IE5VTEwpDQoJCW0tPm5leHQgPSBOVUxMOw0KCWlmIChtLT5uYl9zZWdz ICE9IDEpDQoJCW0tPm5iX3NlZ3MgPSAxOw0KDQorCS8qIFRoZSByZWZlcmVuY2UgY291bnRlciBt dXN0IGJlIGluaXRpYWxpemVkIHRvIDEgd2hlbiB0aGUgbWJ1ZiBpcyBmcmVlLA0KKwkgKiBzbyBp bmNyZW1lbnRpbmcgdG8gMSBpcyB1bm5lY2Vzc2FyeSB3aGVuIGFsbG9jYXRpbmcgdGhlIG1idWYu DQorCSAqLw0KCWlmIChpbml0X3JlZmNudCkNCgkJcnRlX21idWZfcmVmY250X3NldChtLCAxKTsN Cn0NCg0Kc3RhdGljIF9fcnRlX2Fsd2F5c19pbmxpbmUgc3RydWN0IHJ0ZV9tYnVmICoNCnJ0ZV9w a3RtYnVmX3ByZWZyZWVfc2VnKHN0cnVjdCBydGVfbWJ1ZiAqbSkNCnsNCglfX3J0ZV9tYnVmX3Nh bml0eV9jaGVjayhtLCAwKTsNCg0KCWlmIChsaWtlbHkocnRlX21idWZfcmVmY250X3JlYWQobSkg PT0gMSkpIHsNCisJCS8qIFRoaXMgYnJhbmNoIGlzIGEgcGVyZm9ybWFuY2Ugb3B0aW1pemVkIHZh cmlhbnQgb2YgdGhlIGJyYW5jaCBiZWxvdy4NCisJCSAqIElmIHRoZSByZWZlcmVuY2UgY291bnRl ciB3b3VsZCByZWFjaCAwIHdoZW4gZGVjcmVtZW50aW5nIGl0LA0KKwkJICogZG8gbm90IGRlY3Jl bWVudCBpdCB0byAwIGFuZCB0aGVuIGluaXRpYWxpemUgaXQgdG8gMTsNCisJCSAqIGp1c3QgbGVh dmUgaXQgYXQgMSwgdGhlcmVieSBhdm9pZGluZyB3cml0aW5nIHRvIG1lbW9yeS4NCisJCSAqLw0K CQlyZXR1cm4gcnRlX3BrdG1idWZfcHJlZnJlZV9zZWdfbGFzdF9yZWYobSwgZmFsc2UpOw0KCX0g ZWxzZSBpZiAoX19ydGVfbWJ1Zl9yZWZjbnRfdXBkYXRlKG0sIC0xKSA9PSAwKSB7DQoJCXJldHVy biBydGVfcGt0bWJ1Zl9wcmVmcmVlX3NlZ19sYXN0X3JlZihtLCB0cnVlKTsNCgl9DQoJcmV0dXJu IE5VTEw7DQp9DQoNCg0KQW5kIHdoaWxlIHdlJ3JlIGF0IGl0LCB3ZSBjb3VsZCBhZGQgdW5saWtl bHkoKSB0byB0aGUgc2Vjb25kIGNvbXBhcmlzb246DQppZiAodW5saWtlbHkoX19ydGVfbWJ1Zl9y ZWZjbnRfdXBkYXRlKG0sIC0xKSA9PSAwKSkNCg0KDQo=