From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id C44422C0C for ; Mon, 28 Mar 2016 04:31:00 +0200 (CEST) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga104.fm.intel.com with ESMTP; 27 Mar 2016 19:30:59 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,404,1455004800"; d="scan'208";a="772794073" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga003.jf.intel.com with ESMTP; 27 Mar 2016 19:30:59 -0700 Received: from FMSMSX109.amr.corp.intel.com (10.18.116.9) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.248.2; Sun, 27 Mar 2016 19:30:53 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx109.amr.corp.intel.com (10.18.116.9) with Microsoft SMTP Server (TLS) id 14.3.248.2; Sun, 27 Mar 2016 19:30:52 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.232]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.42]) with mapi id 14.03.0248.002; Mon, 28 Mar 2016 10:30:50 +0800 From: "Xu, Qian Q" To: Jianbo Liu , "Ananyev, Konstantin" CC: "Richardson, Bruce" , "Lu, Wenzhuo" , "Zhang, Helin" , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH] ixgbe: avoid unnessary break when checking at the tail of rx hwring Thread-Index: AQHRf1jIyW4XguksHkekVEsbzhfThp9bZVMAgAD844CAAhP1AIAENz6AgAJb1gCABFl6gIAE0WFg Date: Mon, 28 Mar 2016 02:30:50 +0000 Message-ID: <82F45D86ADE5454A95A89742C8D1410E032143C2@shsmsx102.ccr.corp.intel.com> References: <1457965558-15331-1-git-send-email-jianbo.liu@linaro.org> <6A0DE07E22DDAD4C9103DF62FEBC09090343BBF2@shsmsx102.ccr.corp.intel.com> <20160316111454.GB24668@bricha3-MOBL3> <20160318100358.GA4848@bricha3-MOBL3> <2601191342CEEE43887BDE71AB97725836B1FAE1@irsmsx105.ger.corp.intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiN2RkZTJjZDMtN2QwZS00ZDM5LTg2NjMtNzBlOTA3YmNkM2I2IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IlZ5MVFDVUh0VWtOS1ZiK0x5YlhSbFNEU0FyTE8rQUVaU0JcL3c5a0FITGdRPSJ9 x-ctpclassification: CTP_IC 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] ixgbe: avoid unnessary break when checking at the tail of rx hwring X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 28 Mar 2016 02:31:01 -0000 SmlhbmJvDQpDb3VsZCB5b3UgdGVsbCBtZSB0aGUgY2FzZSB0aGF0IGNhbiByZXByb2R1Y2UgdGhl IGlzc3VlPyBXZSBjYW4gaGVscCBldmFsdWF0ZSB0aGUgaW1wYWN0IG9mIHBlcmZvcm1hbmNlIG9u IGl4Z2JlLCBidXQgSSdtIG5vdCBzdXJlIGhvdyB0byBjaGVjayBpZiB5b3VyIHBhdGNoIHJlYWxs eSBmaXggYSBwcm9ibGVtIGJlY2F1c2UgSSBkb27igJl0IGtub3cgaG93IHRvIHJlcHJvZHVjZSB0 aGUgcHJvYmxlbSEgQ291bGQgeW91IGZpcnN0IHRlYWNoIG1lIG9uIGhvdyB0byByZXByb2R1Y2Ug eW91ciBpc3N1ZT8gT3IgeW91IG1heSBub3QgcmVwcm9kdWNlIGl0IGJ5IHlvdXJzZWxmPyANCg0K VGhhbmtzDQpRaWFuDQoNCg0KLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCkZyb206IGRldiBb bWFpbHRvOmRldi1ib3VuY2VzQGRwZGsub3JnXSBPbiBCZWhhbGYgT2YgSmlhbmJvIExpdQ0KU2Vu dDogRnJpZGF5LCBNYXJjaCAyNSwgMjAxNiA0OjUzIFBNDQpUbzogQW5hbnlldiwgS29uc3RhbnRp bg0KQ2M6IFJpY2hhcmRzb24sIEJydWNlOyBMdSwgV2Vuemh1bzsgWmhhbmcsIEhlbGluOyBkZXZA ZHBkay5vcmcNClN1YmplY3Q6IFJlOiBbZHBkay1kZXZdIFtQQVRDSF0gaXhnYmU6IGF2b2lkIHVu bmVzc2FyeSBicmVhayB3aGVuIGNoZWNraW5nIGF0IHRoZSB0YWlsIG9mIHJ4IGh3cmluZw0KDQpP biAyMiBNYXJjaCAyMDE2IGF0IDIyOjI3LCBBbmFueWV2LCBLb25zdGFudGluIDxrb25zdGFudGlu LmFuYW55ZXZAaW50ZWwuY29tPiB3cm90ZToNCj4NCj4NCj4+IC0tLS0tT3JpZ2luYWwgTWVzc2Fn ZS0tLS0tDQo+PiBGcm9tOiBKaWFuYm8gTGl1IFttYWlsdG86amlhbmJvLmxpdUBsaW5hcm8ub3Jn XQ0KPj4gU2VudDogTW9uZGF5LCBNYXJjaCAyMSwgMjAxNiAyOjI3IEFNDQo+PiBUbzogUmljaGFy ZHNvbiwgQnJ1Y2UNCj4+IENjOiBMdSwgV2Vuemh1bzsgWmhhbmcsIEhlbGluOyBBbmFueWV2LCBL b25zdGFudGluOyBkZXZAZHBkay5vcmcNCj4+IFN1YmplY3Q6IFJlOiBbZHBkay1kZXZdIFtQQVRD SF0gaXhnYmU6IGF2b2lkIHVubmVzc2FyeSBicmVhayB3aGVuIA0KPj4gY2hlY2tpbmcgYXQgdGhl IHRhaWwgb2YgcnggaHdyaW5nDQo+Pg0KPj4gT24gMTggTWFyY2ggMjAxNiBhdCAxODowMywgQnJ1 Y2UgUmljaGFyZHNvbiA8YnJ1Y2UucmljaGFyZHNvbkBpbnRlbC5jb20+IHdyb3RlOg0KPj4gPiBP biBUaHUsIE1hciAxNywgMjAxNiBhdCAxMDoyMDowMUFNICswODAwLCBKaWFuYm8gTGl1IHdyb3Rl Og0KPj4gPj4gT24gMTYgTWFyY2ggMjAxNiBhdCAxOToxNCwgQnJ1Y2UgUmljaGFyZHNvbiA8YnJ1 Y2UucmljaGFyZHNvbkBpbnRlbC5jb20+IHdyb3RlOg0KPj4gPj4gPiBPbiBXZWQsIE1hciAxNiwg MjAxNiBhdCAwMzo1MTo1M1BNICswODAwLCBKaWFuYm8gTGl1IHdyb3RlOg0KPj4gPj4gPj4gSGkg V2Vuemh1bywNCj4+ID4+ID4+DQo+PiA+PiA+PiBPbiAxNiBNYXJjaCAyMDE2IGF0IDE0OjA2LCBM dSwgV2Vuemh1byA8d2Vuemh1by5sdUBpbnRlbC5jb20+IHdyb3RlOg0KPj4gPj4gPj4gPiBISSBK aWFuYm8sDQo+PiA+PiA+PiA+DQo+PiA+PiA+PiA+DQo+PiA+PiA+PiA+PiAtLS0tLU9yaWdpbmFs IE1lc3NhZ2UtLS0tLQ0KPj4gPj4gPj4gPj4gRnJvbTogZGV2IFttYWlsdG86ZGV2LWJvdW5jZXNA ZHBkay5vcmddIE9uIEJlaGFsZiBPZiBKaWFuYm8gDQo+PiA+PiA+PiA+PiBMaXUNCj4+ID4+ID4+ ID4+IFNlbnQ6IE1vbmRheSwgTWFyY2ggMTQsIDIwMTYgMTA6MjYgUE0NCj4+ID4+ID4+ID4+IFRv OiBaaGFuZywgSGVsaW47IEFuYW55ZXYsIEtvbnN0YW50aW47IGRldkBkcGRrLm9yZw0KPj4gPj4g Pj4gPj4gQ2M6IEppYW5ibyBMaXUNCj4+ID4+ID4+ID4+IFN1YmplY3Q6IFtkcGRrLWRldl0gW1BB VENIXSBpeGdiZTogYXZvaWQgdW5uZXNzYXJ5IGJyZWFrIA0KPj4gPj4gPj4gPj4gd2hlbiBjaGVj a2luZyBhdCB0aGUgdGFpbCBvZiByeCBod3JpbmcNCj4+ID4+ID4+ID4+DQo+PiA+PiA+PiA+PiBX aGVuIGNoZWNraW5nIHJ4IHJpbmcgcXVldWUsIGl0J3MgcG9zc2libGUgdGhhdCBsb29wIHdpbGwg DQo+PiA+PiA+PiA+PiBicmVhayBhdCB0aGUgdGFpbCB3aGlsZSB0aGVyZSBhcmUgcGFja2V0cyBz dGlsbCBpbiB0aGUgcXVldWUgaGVhZGVyLg0KPj4gPj4gPj4gPiBXb3VsZCB5b3UgbGlrZSB0byBn aXZlIG1vcmUgZGV0YWlscyBhYm91dCBpbiB3aGF0IHNjZW5hcmlvIHRoaXMgaXNzdWUgd2lsbCBi ZSBoaXQ/IFRoYW5rcy4NCj4+ID4+ID4+ID4NCj4+ID4+ID4+DQo+PiA+PiA+PiB2UE1EIHdpbGwg cGxhY2UgZXh0cmEgUlRFX0lYR0JFX0RFU0NTX1BFUl9MT09QIC0gMSBudW1iZXIgb2YgDQo+PiA+ PiA+PiBlbXB0eSBkZXNjcmlwdGlvcnMgYXQgdGhlIGVuZCBvZiBod3JpbmcgdG8gYXZvaWQgb3Zl cmZsb3cgd2hlbiANCj4+ID4+ID4+IGRvIGNoZWNraW5nIG9uIHJ4IHNpZGUuDQo+PiA+PiA+Pg0K Pj4gPj4gPj4gRm9yIHRoZSBsb29wIGluIF9yZWN2X3Jhd19wa3RzX3ZlYygpLCB3ZSBjaGVjayA0 IGRlc2NyaXB0b3JzIA0KPj4gPj4gPj4gZWFjaCB0aW1lLiBJZiBhbGwgNCBERCBhcmUgc2V0LCBh bmQgYWxsIDQgcGFja2V0cyBhcmUgDQo+PiA+PiA+PiByZWNlaXZlZC5UaGF0J3MgT0sgaW4gdGhl IG1pZGRsZS4NCj4+ID4+ID4+IEJ1dCBpZiBjb21lIHRvIHRoZSBlbmQgb2YgaHdyaW5nLCBhbmQg bGVzcyB0aGFuIDQgZGVzY3JpcHRvcnMgDQo+PiA+PiA+PiBsZWZ0LCB3ZSBzdGlsbCBuZWVkIHRv IGNoZWNrIDQgZGVzY3JpcHRvcnMgYXQgdGhlIHNhbWUgdGltZSwgc28gDQo+PiA+PiA+PiB0aGUg ZXh0cmEgZW1wdHkgZGVzY3JpcHRvcnMgYXJlIGNoZWNrZWQgd2l0aCB0aGVtLg0KPj4gPj4gPj4g VGhpcyB0aW1lLCB0aGUgbnVtYmVyIG9mIHJlY2VpdmVkIHBhY2tldHMgaXMgYXBwYXJlbnRseSBs ZXNzIA0KPj4gPj4gPj4gdGhhbiA0LCBhbmQgd2UgYnJlYWsgb3V0IG9mIHRoZSBsb29wIGJlY2F1 c2Ugb2YgdGhlIGNvbmRpdGlvbiANCj4+ID4+ID4+ICJ2YXIgIT0gUlRFX0lYR0JFX0RFU0NTX1BF Ul9MT09QIi4NCj4+ID4+ID4+IFNvIHRoZSBwcm9ibGVtIGFyaXNlcy4gSXQgaXMgcG9zc2libGUg dGhhdCB0aGVyZSBjb3VsZCBiZSBtb3JlIA0KPj4gPj4gPj4gcGFja2V0cyBhdCB0aGUgaHdyaW5n IGJlZ2lubmluZyB0aGF0IHN0aWxsIHdhaXRpbmcgZm9yIGJlaW5nIHJlY2VpdmVkLg0KPj4gPj4g Pj4gSSB0aGluayB0aGlzIGZpeCBjYW4gYXZvaWQgdGhpcyBzaXR1YXRpb24sIGFuZCBhdCBsZWFz dCByZWR1Y2UgDQo+PiA+PiA+PiB0aGUgbGF0ZW5jeSBmb3IgdGhlIHBhY2tldHMgaW4gdGhlIGhl YWRlci4NCj4+ID4+ID4+DQo+PiA+PiA+IFBhY2tldHMgYXJlIGFsd2F5cyByZWNlaXZlZCBpbiBv cmRlciBmcm9tIHRoZSBOSUMsIHNvIG5vIHBhY2tldHMgDQo+PiA+PiA+IGV2ZXIgZ2V0IGxlZnQg YmVoaW5kIG9yIHNraXBwZWQgb24gYW4gUlggYnVyc3QgY2FsbC4NCj4+ID4+ID4NCj4+ID4+ID4g L0JydWNlDQo+PiA+PiA+DQo+PiA+Pg0KPj4gPj4gSSBrbmV3IHBhY2tldHMgYXJlIHJlY2VpdmVk IGluIG9yZGVyLCBhbmQgbm8gcGFja2V0cyB3aWxsIGJlIA0KPj4gPj4gc2tpcHBlZCwgYnV0IHNv bWUgd2lsbCBiZSBsZWZ0IGJlaGluZCBhcyBJIGV4cGxhaW5lZCBhYm92ZS4NCj4+ID4+IHZQTUQg d2lsbCBub3QgcmVjZWl2ZWQgbmJfcGt0cyByZXF1aXJlZCBieSBvbmUgUlggYnVyc3QgY2FsbCwg YW5kIA0KPj4gPj4gdGhvc2UgYXQgdGhlIGJlZ2lubmluZyBvZiBod3JpbmcgYXJlIHN0aWxsIHdh aXRpbmcgdG8gYmUgcmVjZWl2ZWQgDQo+PiA+PiB0aWxsIHRoZSBuZXh0IGNhbGwuDQo+PiA+Pg0K Pj4gPj4gVGhhbmtzIQ0KPj4gPj4gSmlhbmJvDQo+PiA+IEhJIEppYW5ibywNCj4+ID4NCj4+ID4g b2ssIEkgdW5kZXJzdGFuZCBub3cuIEknbSBub3Qgc3VyZSB0aGF0IHRoaXMgaXMgYSBzaWduaWZp Y2FudCANCj4+ID4gcHJvYmxlbSB0aG91Z2gsIHNpbmNlIHdlIGFyZSB3b3JraW5nIGluIHBvbGxp bmcgbW9kZS4gSXMgdGhlcmUgYSANCj4+ID4gcGVyZm9ybWFuY2UgaW1wYWN0IHRvIHlvdXIgY2hh bmdlLCBiZWNhdXNlIEkgZG9uJ3QgdGhpbmsgdGhhdCB3ZSBjYW4gcmVkdWNlIHBlcmZvcm1hbmNl IGp1c3QgdG8gZml4IHRoaXM/DQo+PiA+DQo+PiA+IFJlZ2FyZHMsDQo+PiA+IC9CcnVjZQ0KPj4g SXQgd2lsbCBiZSBhIHByb2JsZW0gYmVjYXVzZSB0aGUgcG9zc2liaWxpdHkgY291bGQgYmUgaGln aC4NCj4+IENvbnNpZGVyaW5nIHJ4IGh3cmluZyBzaXplIGlzIDEyOCBhbmQgcnggYnVyc3QgaXMg MzIsIHRoZSBwb3NzaWJsaXR5IA0KPj4gY2FuIGJlIDMyLzEyOC4NCj4+IEkga25vdyB0aGlzIGNo YW5nZSBpcyBjcml0aWNhbCwgc28gSSB3YW50IHlvdSAoYW5kIG1haW50YWluZXJzKSB0byBkbyAN Cj4+IGZ1bGwgZXZhbHVhdGlvbnMgYWJvdXQgdGhyb3VnaHB1dC9sYXRlbmN5Li5iZWZvcmUgbWFr aW5nIGNvbmNsdXNpb24uDQo+DQo+IEkgYW0gc3RpbGwgbm90IHN1cmUgd2hhdCBpcyBhIHByb2Js ZW0geW91IGFyZSB0cnlpbmcgdG8gc29sdmUgaGVyZS4NCj4gWWVzIHJlY3ZfcmF3X3BrdHNfdmVj KCkgY2FsbCB3b3VsZG4ndCB3cmFwIGFyb3VuZCBIVyByaW5nIGJvdW5kYXJ5LCANCj4gYW5kIHll cyBjYW4gcmV0dXJuIGxlc3MgcGFja2V0cyB0aGF0IGFyZSBhY3R1YWxseSBhdmFpbGFibGUgYnkg dGhlIEhXLg0KPiBUaG91Z2ggYXMgQnJ1Y2UgcG9pbnRlZCwgdGhleSdsbCBiZSByZXR1cm5lZCB0 byB0aGUgdXNlciBieSBuZXh0IGNhbGwuDQpIYXZlIHlvdSB0aG91Z2h0IG9mIHRoZSBpbnRlcnZh bCBiZXR3ZWVuIHRoZXNlIHR3byBjYWxsLCBob3cgbG9uZyBjb3VsZCBpdCBiZT8NCklmIGFwcGxp Y2F0aW9uIGlzIGEgc2ltcGxlIG9uZSBsaWtlIGwyZndkL3Rlc3RwbWQsIHRoYXQncyBmaW5lLg0K QnV0IGlmIHRoZSBpbnRlcnZhbCBpcyBsb25nIGJlY2F1c2UgYXBwbGljYXRpb24gaGFzIG1vcmUg d29yayB0byBkbywgdGhleSBhcmUgZGlmZmVyZW50Lg0KDQo+IEFjdHVhbGx5IHJlY3ZfcGt0c19i dWxrX2FsbG9jKCkgd29ya3MgaW4gYSBzaW1pbGFyIHdheS4NCj4gV2h5IGRvIHlvdSBjb25zaWRl ciB0aGF0IGFzIGEgcHJvYmxlbT8NCkRyaXZlciBzaG91bGQgcHVsbCBwYWNrZXRzIG91dCBvZiBo YXJkd2FyZSBhbmQgZ2l2ZSB0aGVtIHRvIEFQUCBhcyBmYXN0IGFzIHBvc3NpYmxlLg0KSWYgbm90 LCB0aGVyZSBpcyBhIHBvc3NpYmlsaXR5IHRoYXQgb3ZlcmZsb3cgdGhlIGhhcmR3YXJlIHF1ZXVl IGJ5IG1vcmUgaW5jb21pbmcgcGFja2V0cy4NCg0KSSBkaWQgc29tZSB0ZXN0aW5ncyB3aXRoIHBr dGdlbi1kcGRrLCBhbmQgaXQgYmVoYXZlcyBhIGxpdHRsZSBiZXR0ZXIgd2l0aCB0aGlzIHBhdGNo IChhdCBsZWFzdCBub3Qgd29yc2UpLg0KU29ycnkgSSBjYW4ndCBwcm92aWRlIG1vcmUgY29uY3Jl YXRlIGV2aWRlbmNlcyBiZWNhdXNlIEkgZG9uJ3QgaGF2ZSBpeGlhL3NwcmludCBlcXVpcG1lbnQg YXQgaGFuZC4NClRoYXQncyB3aHkgSSBhc2tlZCB5b3UgdG8gZG8gZnVsbCBldmFsdWF0aW9ucyBi ZWZvcmUgcmVqZWN0IHRoaXMgcGF0Y2guIDotKQ0KDQpUaGFua3MhDQoNCj4gS29uc3RhbnRpbg0K Pg0KPj4NCj4+IEppYW5ibw0K