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 5E127A0597; Wed, 8 Apr 2020 03:19:15 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id A6C661C0C0; Wed, 8 Apr 2020 03:18:38 +0200 (CEST) Received: from huawei.com (szxga03-in.huawei.com [45.249.212.189]) by dpdk.org (Postfix) with ESMTP id 2C97D1C023; Wed, 8 Apr 2020 03:18:37 +0200 (CEST) Received: from DGGEMM404-HUB.china.huawei.com (unknown [172.30.72.55]) by Forcepoint Email with ESMTP id BE6CB1BE5111392CBDF1; Wed, 8 Apr 2020 09:18:34 +0800 (CST) Received: from DGGEMM533-MBX.china.huawei.com ([169.254.5.250]) by DGGEMM404-HUB.china.huawei.com ([10.3.20.212]) with mapi id 14.03.0487.000; Wed, 8 Apr 2020 09:18:26 +0800 From: wangyunjian To: Ferruh Yigit , "dev@dpdk.org" CC: "keith.wiles@intel.com" , "Lilijun (Jerry)" , xudingke , "stable@dpdk.org" Thread-Topic: [dpdk-stable] [dpdk-dev] [PATCH v3 2/5] net/tap: fix mbuf and mem leak during queue release Thread-Index: AQHWDJQ6PqgvXOFNWEu/xWhTRe7ATahtGFeAgAFVJWA= Date: Wed, 8 Apr 2020 01:18:25 +0000 Message-ID: <34EFBCA9F01B0748BEB6B629CE643AE60CF5C5A8@DGGEMM533-MBX.china.huawei.com> References: <1586233370-13856-1-git-send-email-wangyunjian@huawei.com> <9589a267-c438-d01c-c2e2-61ac7c47876d@intel.com> In-Reply-To: <9589a267-c438-d01c-c2e2-61ac7c47876d@intel.com> Accept-Language: en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.173.251.152] Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 MIME-Version: 1.0 X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [dpdk-stable] [PATCH v3 2/5] net/tap: fix mbuf and mem leak during queue release 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" DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogRmVycnVoIFlpZ2l0IFtt YWlsdG86ZmVycnVoLnlpZ2l0QGludGVsLmNvbV0NCj4gU2VudDogVHVlc2RheSwgQXByaWwgNywg MjAyMCA4OjU1IFBNDQo+IFRvOiB3YW5neXVuamlhbiA8d2FuZ3l1bmppYW5AaHVhd2VpLmNvbT47 IGRldkBkcGRrLm9yZw0KPiBDYzoga2VpdGgud2lsZXNAaW50ZWwuY29tOyBMaWxpanVuIChKZXJy eSkgPGplcnJ5LmxpbGlqdW5AaHVhd2VpLmNvbT47IHh1ZGluZ2tlDQo+IDx4dWRpbmdrZUBodWF3 ZWkuY29tPjsgc3RhYmxlQGRwZGsub3JnDQo+IFN1YmplY3Q6IFJlOiBbZHBkay1zdGFibGVdIFtk cGRrLWRldl0gW1BBVENIIHYzIDIvNV0gbmV0L3RhcDogZml4IG1idWYgYW5kIG1lbQ0KPiBsZWFr IGR1cmluZyBxdWV1ZSByZWxlYXNlDQo+IA0KPiBPbiA0LzcvMjAyMCA1OjIyIEFNLCB3YW5neXVu amlhbiB3cm90ZToNCj4gPiBGcm9tOiBZdW5qaWFuIFdhbmcgPHdhbmd5dW5qaWFuQGh1YXdlaS5j b20+DQo+ID4NCj4gPiBGb3IgdGhlIHRhcCBQTUQsIHdlIHNob3VsZCByZWxlYXNlIG1idWZzIGFu ZCBpb3ZlY3MgZnJvbSB0aGUgUnggcXVldWUNCj4gPiB3aGVuIGNsb3NlIG9yIHJlbW92ZSBkZXZp Y2UuDQo+ID4NCj4gPiBGaXhlczogMDc4MWY1NzYyY2ZlICgibmV0L3RhcDogc3VwcG9ydCBzZWdt ZW50ZWQgbWJ1ZnMiKQ0KPiA+IENDOiBzdGFibGVAZHBkay5vcmcNCj4gPg0KPiA+IFNpZ25lZC1v ZmYtYnk6IFl1bmppYW4gV2FuZyA8d2FuZ3l1bmppYW5AaHVhd2VpLmNvbT4NCj4gPiAtLS0NCj4g PiAgZHJpdmVycy9uZXQvdGFwL3J0ZV9ldGhfdGFwLmMgfCAxMiArKysrKysrKysrKysNCj4gPiAg MSBmaWxlIGNoYW5nZWQsIDEyIGluc2VydGlvbnMoKykNCj4gPg0KPiA+IGRpZmYgLS1naXQgYS9k cml2ZXJzL25ldC90YXAvcnRlX2V0aF90YXAuYw0KPiA+IGIvZHJpdmVycy9uZXQvdGFwL3J0ZV9l dGhfdGFwLmMgaW5kZXggNGM0YjZiMGIyLi5hOWJhMGNhNjggMTAwNjQ0DQo+ID4gLS0tIGEvZHJp dmVycy9uZXQvdGFwL3J0ZV9ldGhfdGFwLmMNCj4gPiArKysgYi9kcml2ZXJzL25ldC90YXAvcnRl X2V0aF90YXAuYw0KPiA+IEBAIC0xMDIyLDYgKzEwMjIsNyBAQCB0YXBfZGV2X2Nsb3NlKHN0cnVj dCBydGVfZXRoX2RldiAqZGV2KQ0KPiA+ICAJaW50IGk7DQo+ID4gIAlzdHJ1Y3QgcG1kX2ludGVy bmFscyAqaW50ZXJuYWxzID0gZGV2LT5kYXRhLT5kZXZfcHJpdmF0ZTsNCj4gPiAgCXN0cnVjdCBw bWRfcHJvY2Vzc19wcml2YXRlICpwcm9jZXNzX3ByaXZhdGUgPSBkZXYtPnByb2Nlc3NfcHJpdmF0 ZTsNCj4gPiArCXN0cnVjdCByeF9xdWV1ZSAqcnhxOw0KPiA+DQo+ID4gIAl0YXBfbGlua19zZXRf ZG93bihkZXYpOw0KPiA+ICAJdGFwX2Zsb3dfZmx1c2goZGV2LCBOVUxMKTsNCj4gPiBAQCAtMTAy OSw4ICsxMDMwLDEzIEBAIHRhcF9kZXZfY2xvc2Uoc3RydWN0IHJ0ZV9ldGhfZGV2ICpkZXYpDQo+ ID4NCj4gPiAgCWZvciAoaSA9IDA7IGkgPCBSVEVfUE1EX1RBUF9NQVhfUVVFVUVTOyBpKyspIHsN Cj4gPiAgCQlpZiAocHJvY2Vzc19wcml2YXRlLT5yeHFfZmRzW2ldICE9IC0xKSB7DQo+ID4gKwkJ CXJ4cSA9ICZpbnRlcm5hbHMtPnJ4cVtpXTsNCj4gPiAgCQkJY2xvc2UocHJvY2Vzc19wcml2YXRl LT5yeHFfZmRzW2ldKTsNCj4gPiAgCQkJcHJvY2Vzc19wcml2YXRlLT5yeHFfZmRzW2ldID0gLTE7 DQo+ID4gKwkJCXJ0ZV9wa3RtYnVmX2ZyZWUocnhxLT5wb29sKTsNCj4gPiArCQkJcnRlX2ZyZWUo cnhxLT5pb3ZlY3MpOw0KPiA+ICsJCQlyeHEtPnBvb2wgPSBOVUxMOw0KPiA+ICsJCQlyeHEtPmlv dmVjcyA9IE5VTEw7DQo+ID4gIAkJfQ0KPiA+ICAJCWlmIChwcm9jZXNzX3ByaXZhdGUtPnR4cV9m ZHNbaV0gIT0gLTEpIHsNCj4gPiAgCQkJY2xvc2UocHJvY2Vzc19wcml2YXRlLT50eHFfZmRzW2ld KTsNCj4gPiBAQCAtMjM5OSw2ICsyNDA1LDcgQEAgcnRlX3BtZF90YXBfcmVtb3ZlKHN0cnVjdCBy dGVfdmRldl9kZXZpY2UNCj4gKmRldikNCj4gPiAgCXN0cnVjdCBydGVfZXRoX2RldiAqZXRoX2Rl diA9IE5VTEw7DQo+ID4gIAlzdHJ1Y3QgcG1kX2ludGVybmFscyAqaW50ZXJuYWxzOw0KPiA+ICAJ c3RydWN0IHBtZF9wcm9jZXNzX3ByaXZhdGUgKnByb2Nlc3NfcHJpdmF0ZTsNCj4gPiArCXN0cnVj dCByeF9xdWV1ZSAqcnhxOw0KPiA+ICAJaW50IGk7DQo+ID4NCj4gPiAgCS8qIGZpbmQgdGhlIGV0 aGRldiBlbnRyeSAqLw0KPiA+IEBAIC0yNDI1LDggKzI0MzIsMTMgQEAgcnRlX3BtZF90YXBfcmVt b3ZlKHN0cnVjdCBydGVfdmRldl9kZXZpY2UNCj4gKmRldikNCj4gPiAgCX0NCj4gPiAgCWZvciAo aSA9IDA7IGkgPCBSVEVfUE1EX1RBUF9NQVhfUVVFVUVTOyBpKyspIHsNCj4gPiAgCQlpZiAocHJv Y2Vzc19wcml2YXRlLT5yeHFfZmRzW2ldICE9IC0xKSB7DQo+ID4gKwkJCXJ4cSA9ICZpbnRlcm5h bHMtPnJ4cVtpXTsNCj4gPiAgCQkJY2xvc2UocHJvY2Vzc19wcml2YXRlLT5yeHFfZmRzW2ldKTsN Cj4gPiAgCQkJcHJvY2Vzc19wcml2YXRlLT5yeHFfZmRzW2ldID0gLTE7DQo+ID4gKwkJCXJ0ZV9w a3RtYnVmX2ZyZWUocnhxLT5wb29sKTsNCj4gPiArCQkJcnRlX2ZyZWUocnhxLT5pb3ZlY3MpOw0K PiA+ICsJCQlyeHEtPnBvb2wgPSBOVUxMOw0KPiA+ICsJCQlyeHEtPmlvdmVjcyA9IE5VTEw7DQo+ ID4gIAkJfQ0KPiA+ICAJCWlmIChwcm9jZXNzX3ByaXZhdGUtPnR4cV9mZHNbaV0gIT0gLTEpIHsN Cj4gPiAgCQkJY2xvc2UocHJvY2Vzc19wcml2YXRlLT50eHFfZmRzW2ldKTsNCj4gPg0KPiANCj4g VGhhbmtzIGZvciB0aGUgZml4LCBidXQgaW5zdGVhZCBvZiBkdXBsaWNhdGluZyB0aGlzIGZvciAn Y2xvc2UoKScgJiAncmVtb3ZlKCknLCBjYW4NCj4gJ3JlbW92ZSgpJyBjYWxsIHRoZSAnY2xvc2Uo KSc/DQo+IFRoZXkgc2hvdWxkIGJlIGRvaW5nIGFsbW9zdCBzYW1lIHRoaW5nIGJ1dCBJIGNhbiBz ZWUgdGhlcmUgaXMgZGlmZmVyZW5jZQ0KPiBiZXR3ZWVuIHR3bywgd2hpY2ggbWF5IG1lYW4gc29t ZXRoaW5nIGlzIG1pc3NlZCwgdW5pZnlpbmcgdGhlbSBmaXhlcyB0aG9zZQ0KPiBtaXNzZWQgcGFy dHMgdG9vLg0KPiBKdXN0IGEgcmVtaW5kZXIgdGhhdCB0aGVyZSBjYW4gYmUgdHJlZSB2YWxpZCBw YXRoIGFuZCBzaG91bGQgd29yayBmaW5lOg0KPiAxLSB1c2VyICdjbG9zZSgpJyB0aGUgUE1EDQo+ IDItIHVzZXIgZGlyZWN0bHkgJ3JlbW92ZSgpJyB0aGUgUE1EDQo+IDMtIHVzZXIgZmlyc3QgJ2Ns b3NlKCknLCBsYXRlciAncmVtb3ZlKCknIHRoZSBQTUQNCj4gDQo+IFRoYW5rcywNCj4gRmVycnVo DQoNCk9LLCBJIGdvdCB5b3VyIHBvaW50LCB3aWxsIGRvIHRoaXMgaW4gdjQuDQoNClRoYW5rcw0K WXVuamlhbg0K