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 B3B80A2EFC for ; Wed, 18 Sep 2019 05:23:16 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 5EA141BF3B; Wed, 18 Sep 2019 05:23:15 +0200 (CEST) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 6D22D1BF32 for ; Wed, 18 Sep 2019 05:23:13 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 17 Sep 2019 20:23:12 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,519,1559545200"; d="scan'208";a="177587324" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga007.jf.intel.com with ESMTP; 17 Sep 2019 20:23:12 -0700 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 17 Sep 2019 20:23:12 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.140]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.53]) with mapi id 14.03.0439.000; Wed, 18 Sep 2019 11:23:10 +0800 From: "Liu, Yong" To: "Bie, Tiwei" CC: "maxime.coquelin@redhat.com" , "dev@dpdk.org" Thread-Topic: [PATCH v2 2/2] net/virtio: on demand cleanup when doing in order xmit Thread-Index: AQHVZ7JtQUXzusXJgESf+TfYAHtWwacwQDqAgACQEdA= Date: Wed, 18 Sep 2019 03:23:09 +0000 Message-ID: <86228AFD5BCD8E4EBFD2B90117B5E81E633A4DD8@SHSMSX103.ccr.corp.intel.com> References: <20190827102407.65106-1-yong.liu@intel.com> <20190910161446.36361-1-yong.liu@intel.com> <20190910161446.36361-2-yong.liu@intel.com> <20190918024335.GB20995@___> In-Reply-To: <20190918024335.GB20995@___> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZTU3NDY3ZjUtMmU5Ny00NmQ1LTk2OTQtZTUzMTczYzQ1M2Q5IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiQXVpYmFuUVF5TTRhREIyb0xic21qM2N1Uno3SzJ6aVAxMHNIMzVtZWgyRHhZRHNoaUJDNm4yb0N1RjlGK2RPUCJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.2.0.6 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 v2 2/2] net/virtio: on demand cleanup when doing in order xmit 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" VGhhbmtzIGZvciBjb21tZW50cywgd2lsbCB1cGRhdGUgaW4gbmV4dCBwYXRjaC4NCg0KPiAtLS0t LU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBCaWUsIFRpd2VpDQo+IFNlbnQ6IFdlZG5l c2RheSwgU2VwdGVtYmVyIDE4LCAyMDE5IDEwOjQ0IEFNDQo+IFRvOiBMaXUsIFlvbmcgPHlvbmcu bGl1QGludGVsLmNvbT4NCj4gQ2M6IG1heGltZS5jb3F1ZWxpbkByZWRoYXQuY29tOyBkZXZAZHBk ay5vcmcNCj4gU3ViamVjdDogUmU6IFtQQVRDSCB2MiAyLzJdIG5ldC92aXJ0aW86IG9uIGRlbWFu ZCBjbGVhbnVwIHdoZW4gZG9pbmcgaW4NCj4gb3JkZXIgeG1pdA0KPiANCj4gT24gV2VkLCBTZXAg MTEsIDIwMTkgYXQgMTI6MTQ6NDZBTSArMDgwMCwgTWFydmluIExpdSB3cm90ZToNCj4gPiBDaGVj ayB3aGV0aGVyIHNwYWNlIGFyZSBlbm91Z2ggYmVmb3JlIGJ1cnN0IGVucXVldWUgb3BlcmF0aW9u LiBJZiBtb3JlDQo+ID4gc3BhY2UgaXMgbmVlZGVkLCB3aWxsIHRyeSB0byBjbGVhbnVwIHVzZWQg ZGVzY3JpcHRvcnMgZm9yIHNwYWNlIG9uDQo+ID4gZGVtYW5kLiBJdCBjYW4gZ2l2ZSBtb3JlIGNo YW5jZXMgdG8gZnJlZSB1c2VkIGRlc2NyaXB0b3JzLCB0aHVzIHdpbGwNCj4gPiBoZWxwIFJGQzI1 NDQgcGVyZm9ybWFuY2UuDQo+ID4NCj4gPiBTaWduZWQtb2ZmLWJ5OiBNYXJ2aW4gTGl1IDx5b25n LmxpdUBpbnRlbC5jb20+DQo+ID4gLS0tDQo+ID4gIGRyaXZlcnMvbmV0L3ZpcnRpby92aXJ0aW9f cnh0eC5jIHwgNzMgKysrKysrKysrKysrKysrKysrKysrKystLS0tLS0tLS0NCj4gPiAgMSBmaWxl IGNoYW5nZWQsIDU0IGluc2VydGlvbnMoKyksIDE5IGRlbGV0aW9ucygtKQ0KPiA+DQo+ID4gZGlm ZiAtLWdpdCBhL2RyaXZlcnMvbmV0L3ZpcnRpby92aXJ0aW9fcnh0eC5jDQo+IGIvZHJpdmVycy9u ZXQvdmlydGlvL3ZpcnRpb19yeHR4LmMNCj4gPiBpbmRleCBkM2NhMzY4MzEuLjg0MmI2MDBjMyAx MDA2NDQNCj4gPiAtLS0gYS9kcml2ZXJzL25ldC92aXJ0aW8vdmlydGlvX3J4dHguYw0KPiA+ICsr KyBiL2RyaXZlcnMvbmV0L3ZpcnRpby92aXJ0aW9fcnh0eC5jDQo+ID4gQEAgLTIxNTIsNiArMjE1 MiwyMiBAQCB2aXJ0aW9feG1pdF9wa3RzKHZvaWQgKnR4X3F1ZXVlLCBzdHJ1Y3QgcnRlX21idWYN Cj4gKip0eF9wa3RzLCB1aW50MTZfdCBuYl9wa3RzKQ0KPiA+ICAJcmV0dXJuIG5iX3R4Ow0KPiA+ ICB9DQo+ID4NCj4gPiArc3RhdGljIF9fcnRlX2Fsd2F5c19pbmxpbmUgaW50DQo+ID4gK3ZpcnRp b194bWl0X3RyeV9jbGVhbnVwX2lub3JkZXIoc3RydWN0IHZpcnRxdWV1ZSAqdnEsIHVpbnQxNl90 IG5lZWQpDQo+ID4gK3sNCj4gPiArCXVpbnQxNl90IG5iX3VzZWQsIG5iX2NsZWFuLCBuYl9kZXNj czsNCj4gPiArCXN0cnVjdCB2aXJ0aW9faHcgKmh3ID0gdnEtPmh3Ow0KPiA+ICsNCj4gPiArCW5i X2Rlc2NzID0gdnEtPnZxX2ZyZWVfY250ICsgbmVlZDsNCj4gPiArCW5iX3VzZWQgPSBWSVJUUVVF VUVfTlVTRUQodnEpOw0KPiA+ICsJdmlydGlvX3JtYihody0+d2Vha19iYXJyaWVycyk7DQo+ID4g KwluYl9jbGVhbiA9IFJURV9NSU4obmVlZCwgKGludCluYl91c2VkKTsNCj4gPiArDQo+ID4gKwl2 aXJ0aW9feG1pdF9jbGVhbnVwX2lub3JkZXIodnEsIG5iX2NsZWFuKTsNCj4gPiArDQo+ID4gKwly ZXR1cm4gKG5iX2Rlc2NzIC0gdnEtPnZxX2ZyZWVfY250KTsNCj4gPiArfQ0KPiA+ICsNCj4gPiAg dWludDE2X3QNCj4gPiAgdmlydGlvX3htaXRfcGt0c19pbm9yZGVyKHZvaWQgKnR4X3F1ZXVlLA0K PiA+ICAJCQlzdHJ1Y3QgcnRlX21idWYgKip0eF9wa3RzLA0KPiA+IEBAIC0yMTYxLDggKzIxNzcs OSBAQCB2aXJ0aW9feG1pdF9wa3RzX2lub3JkZXIodm9pZCAqdHhfcXVldWUsDQo+ID4gIAlzdHJ1 Y3QgdmlydHF1ZXVlICp2cSA9IHR4dnEtPnZxOw0KPiA+ICAJc3RydWN0IHZpcnRpb19odyAqaHcg PSB2cS0+aHc7DQo+ID4gIAl1aW50MTZfdCBoZHJfc2l6ZSA9IGh3LT52dG5ldF9oZHJfc2l6ZTsN Cj4gPiAtCXVpbnQxNl90IG5iX3VzZWQsIG5iX2F2YWlsLCBuYl90eCA9IDAsIG5iX2lub3JkZXJf cGt0cyA9IDA7DQo+ID4gKwl1aW50MTZfdCBuYl91c2VkLCBuYl90eCA9IDAsIG5iX2lub3JkZXJf cGt0cyA9IDA7DQo+ID4gIAlzdHJ1Y3QgcnRlX21idWYgKmlub3JkZXJfcGt0c1tuYl9wa3RzXTsN Cj4gPiArCWludCBuZWVkLCBuYl9sZWZ0Ow0KPiA+DQo+ID4gIAlpZiAodW5saWtlbHkoaHctPnN0 YXJ0ZWQgPT0gMCAmJiB0eF9wa3RzICE9IGh3LT5pbmplY3RfcGt0cykpDQo+ID4gIAkJcmV0dXJu IG5iX3R4Ow0KPiA+IEBAIC0yMTc1LDE3ICsyMTkyLDEyIEBAIHZpcnRpb194bWl0X3BrdHNfaW5v cmRlcih2b2lkICp0eF9xdWV1ZSwNCj4gPiAgCW5iX3VzZWQgPSBWSVJUUVVFVUVfTlVTRUQodnEp Ow0KPiA+DQo+ID4gIAl2aXJ0aW9fcm1iKGh3LT53ZWFrX2JhcnJpZXJzKTsNCj4gPiAtCWlmIChs aWtlbHkobmJfdXNlZCA+IHZxLT52cV9uZW50cmllcyAtIHZxLT52cV9mcmVlX3RocmVzaCkpDQo+ ID4gLQkJdmlydGlvX3htaXRfY2xlYW51cF9pbm9yZGVyKHZxLCBuYl91c2VkKTsNCj4gPiAtDQo+ ID4gLQlpZiAodW5saWtlbHkoIXZxLT52cV9mcmVlX2NudCkpDQo+ID4gKwlpZiAobGlrZWx5KG5i X3VzZWQgPiAodnEtPnZxX25lbnRyaWVzIC0gdnEtPnZxX2ZyZWVfdGhyZXNoKSkpDQo+ID4gIAkJ dmlydGlvX3htaXRfY2xlYW51cF9pbm9yZGVyKHZxLCBuYl91c2VkKTsNCj4gPg0KPiA+IC0JbmJf YXZhaWwgPSBSVEVfTUlOKHZxLT52cV9mcmVlX2NudCwgbmJfcGt0cyk7DQo+ID4gLQ0KPiA+IC0J Zm9yIChuYl90eCA9IDA7IG5iX3R4IDwgbmJfYXZhaWw7IG5iX3R4KyspIHsNCj4gPiArCWZvciAo bmJfdHggPSAwOyBuYl90eCA8IG5iX3BrdHM7IG5iX3R4KyspIHsNCj4gPiAgCQlzdHJ1Y3QgcnRl X21idWYgKnR4bSA9IHR4X3BrdHNbbmJfdHhdOw0KPiA+IC0JCWludCBzbG90cywgbmVlZDsNCj4g PiArCQlpbnQgc2xvdHM7DQo+ID4NCj4gPiAgCQkvKiBvcHRpbWl6ZSByaW5nIHVzYWdlICovDQo+ ID4gIAkJaWYgKCh2dHBjaV93aXRoX2ZlYXR1cmUoaHcsIFZJUlRJT19GX0FOWV9MQVlPVVQpIHx8 DQo+ID4gQEAgLTIxOTksMTEgKzIyMTEsMjUgQEAgdmlydGlvX3htaXRfcGt0c19pbm9yZGVyKHZv aWQgKnR4X3F1ZXVlLA0KPiA+ICAJCQlpbm9yZGVyX3BrdHNbbmJfaW5vcmRlcl9wa3RzXSA9IHR4 bTsNCj4gPiAgCQkJbmJfaW5vcmRlcl9wa3RzKys7DQo+ID4NCj4gPiAtCQkJdmlydGlvX3VwZGF0 ZV9wYWNrZXRfc3RhdHMoJnR4dnEtPnN0YXRzLCB0eG0pOw0KPiA+ICAJCQljb250aW51ZTsNCj4g PiAgCQl9DQo+ID4NCj4gPiAgCQlpZiAobmJfaW5vcmRlcl9wa3RzKSB7DQo+ID4gKwkJCW5lZWQg PSBuYl9pbm9yZGVyX3BrdHMgLSB2cS0+dnFfZnJlZV9jbnQ7DQo+ID4gKw0KPiA+ICsJCQlpZiAo dW5saWtlbHkobmVlZCA+IDApKSB7DQo+ID4gKwkJCQluYl9sZWZ0ID0gdmlydGlvX3htaXRfdHJ5 X2NsZWFudXBfaW5vcmRlcih2cSwNCj4gPiArCQkJCQkJCQkJbmVlZCk7DQo+IA0KPiBUaGVyZSBp cyBubyBuZWVkIHRvIGludHJvZHVjZSBgbmJfbGVmdGAuIExvb2tzIGJldHRlcg0KPiB0byBqdXN0 IHJldXNlIGBuZWVkYC4NCj4gDQo+ID4gKw0KPiA+ICsJCQkJaWYgKHVubGlrZWx5KG5iX2xlZnQg PiAwKSkgew0KPiA+ICsJCQkJCVBNRF9UWF9MT0coRVJSLA0KPiA+ICsJCQkJCQkiTm8gZnJlZSB0 eCBkZXNjcmlwdG9ycyB0byAiDQo+ID4gKwkJCQkJCSJ0cmFuc21pdCIpOw0KPiA+ICsJCQkJCW5i X2lub3JkZXJfcGt0cyA9IHZxLT52cV9mcmVlX2NudDsNCj4gDQo+IFlvdSBuZWVkIHRvIGhhbmRs ZSBuYl90eCBhcyB3ZWxsLCBvdGhlcndpc2UgbWJ1ZnMgd2lsbCBsZWFrLg0KPiBPciBtYXliZSBq dXN0IGxlYXZlIG5iX2lub3JkZXJfcGt0cyB1bmNoYW5nZWQsIGFuZCBsZXQgdGhlIGNvZGUNCj4g b3V0c2lkZSB0aGUgbG9vcCBoYW5kbGUgaXQuDQo+IA0KPiANCj4gPiArCQkJCQlicmVhazsNCj4g PiArCQkJCX0NCj4gPiArCQkJfQ0KPiA+ICsNCj4gPiAgCQkJdmlydHF1ZXVlX2VucXVldWVfeG1p dF9pbm9yZGVyKHR4dnEsIGlub3JkZXJfcGt0cywNCj4gPiAgCQkJCQkJCW5iX2lub3JkZXJfcGt0 cyk7DQo+ID4gIAkJCW5iX2lub3JkZXJfcGt0cyA9IDA7DQo+ID4gQEAgLTIyMTIsMTUgKzIyMzgs OSBAQCB2aXJ0aW9feG1pdF9wa3RzX2lub3JkZXIodm9pZCAqdHhfcXVldWUsDQo+ID4gIAkJc2xv dHMgPSB0eG0tPm5iX3NlZ3MgKyAxOw0KPiA+ICAJCW5lZWQgPSBzbG90cyAtIHZxLT52cV9mcmVl X2NudDsNCj4gPiAgCQlpZiAodW5saWtlbHkobmVlZCA+IDApKSB7DQo+ID4gLQkJCW5iX3VzZWQg PSBWSVJUUVVFVUVfTlVTRUQodnEpOw0KPiA+IC0JCQl2aXJ0aW9fcm1iKGh3LT53ZWFrX2JhcnJp ZXJzKTsNCj4gPiAtCQkJbmVlZCA9IFJURV9NSU4obmVlZCwgKGludCluYl91c2VkKTsNCj4gPiAt DQo+ID4gLQkJCXZpcnRpb194bWl0X2NsZWFudXBfaW5vcmRlcih2cSwgbmVlZCk7DQo+ID4gKwkJ CW5iX2xlZnQgPSB2aXJ0aW9feG1pdF90cnlfY2xlYW51cF9pbm9yZGVyKHZxLCBzbG90cyk7DQo+ ID4NCj4gPiAtCQkJbmVlZCA9IHNsb3RzIC0gdnEtPnZxX2ZyZWVfY250Ow0KPiA+IC0NCj4gPiAt CQkJaWYgKHVubGlrZWx5KG5lZWQgPiAwKSkgew0KPiA+ICsJCQlpZiAodW5saWtlbHkobmJfbGVm dCA+IDApKSB7DQo+ID4gIAkJCQlQTURfVFhfTE9HKEVSUiwNCj4gPiAgCQkJCQkiTm8gZnJlZSB0 eCBkZXNjcmlwdG9ycyB0byB0cmFuc21pdCIpOw0KPiA+ICAJCQkJYnJlYWs7DQo+ID4gQEAgLTIy MzMsOSArMjI1MywyNCBAQCB2aXJ0aW9feG1pdF9wa3RzX2lub3JkZXIodm9pZCAqdHhfcXVldWUs DQo+ID4gIAl9DQo+ID4NCj4gPiAgCS8qIFRyYW5zbWl0IGFsbCBpbm9yZGVyIHBhY2tldHMgKi8N Cj4gPiAtCWlmIChuYl9pbm9yZGVyX3BrdHMpDQo+ID4gKwlpZiAobmJfaW5vcmRlcl9wa3RzKSB7 DQo+ID4gKwkJbmVlZCA9IG5iX2lub3JkZXJfcGt0cyAtIHZxLT52cV9mcmVlX2NudDsNCj4gPiAr DQo+ID4gKwkJaWYgKHVubGlrZWx5KG5lZWQgPiAwKSkgew0KPiA+ICsJCQluYl9sZWZ0ID0gdmly dGlvX3htaXRfdHJ5X2NsZWFudXBfaW5vcmRlcih2cSwNCj4gPiArCQkJCQkJCQkgIG5lZWQpOw0K PiA+ICsNCj4gPiArCQkJaWYgKHVubGlrZWx5KG5iX2xlZnQgPiAwKSkgew0KPiA+ICsJCQkJUE1E X1RYX0xPRyhFUlIsDQo+ID4gKwkJCQkJIk5vIGZyZWUgdHggZGVzY3JpcHRvcnMgdG8gdHJhbnNt aXQiKTsNCj4gPiArCQkJCW5iX2lub3JkZXJfcGt0cyA9IHZxLT52cV9mcmVlX2NudDsNCj4gPiAr CQkJCW5iX3R4IC09IG5iX2xlZnQ7DQo+ID4gKwkJCX0NCj4gPiArCQl9DQo+ID4gKw0KPiA+ICAJ CXZpcnRxdWV1ZV9lbnF1ZXVlX3htaXRfaW5vcmRlcih0eHZxLCBpbm9yZGVyX3BrdHMsDQo+ID4g IAkJCQkJCW5iX2lub3JkZXJfcGt0cyk7DQo+ID4gKwl9DQo+ID4NCj4gPiAgCXR4dnEtPnN0YXRz LnBhY2tldHMgKz0gbmJfdHg7DQo+ID4NCj4gPiAtLQ0KPiA+IDIuMTcuMQ0KPiA+DQo=