From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 28F7B1B274; Tue, 8 Jan 2019 14:43:49 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Jan 2019 05:43:49 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,454,1539673200"; d="scan'208";a="115117479" Received: from irsmsx105.ger.corp.intel.com ([163.33.3.28]) by fmsmga008.fm.intel.com with ESMTP; 08 Jan 2019 05:43:48 -0800 Received: from irsmsx155.ger.corp.intel.com (163.33.192.3) by irsmsx105.ger.corp.intel.com (163.33.3.28) with Microsoft SMTP Server (TLS) id 14.3.408.0; Tue, 8 Jan 2019 13:43:47 +0000 Received: from irsmsx105.ger.corp.intel.com ([169.254.7.116]) by irsmsx155.ger.corp.intel.com ([169.254.14.157]) with mapi id 14.03.0415.000; Tue, 8 Jan 2019 13:43:46 +0000 From: "Ananyev, Konstantin" To: =?utf-8?B?TW9ydGVuIEJyw7hydXA=?= , "Hu, Jiayu" , Stephen Hemminger CC: "dev@dpdk.org" , "Bie, Tiwei" , "Richardson, Bruce" , "stable@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH] gro: add missing invalid packet checks Thread-Index: AQHUpxi6gYDMrmZp9k2BAzziHTP876Wk6XyAgAAclYCAACVRUIAADnMwgAAnmQA= Date: Tue, 8 Jan 2019 13:43:45 +0000 Message-ID: <2601191342CEEE43887BDE71AB977258010D900A85@irsmsx105.ger.corp.intel.com> References: <1546567036-29444-1-git-send-email-jiayu.hu@intel.com> <1546927725-68831-1-git-send-email-jiayu.hu@intel.com> <20190107223151.18b185b7@hermes.lan> <2601191342CEEE43887BDE71AB977258010D900817@irsmsx105.ger.corp.intel.com> <98CBD80474FA8B44BF855DF32C47DC35B425AD@smartserver.smartshare.dk> In-Reply-To: <98CBD80474FA8B44BF855DF32C47DC35B425AD@smartserver.smartshare.dk> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiOWFhZGI0MmEtZjcyNy00YmQ5LWExNjktN2MxOWE2ZWIyNDE3IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiZE1ISWV6aDUwS2xJZEVVQ1l1dDRqXC9FVXo2UEZcL0hkQXBLY0J3TWNFXC8zakVhbDN5UUFMZGtTZGY2TERHRkplQiJ9 x-ctpclassification: CTP_NT dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] gro: add missing invalid packet checks 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: , X-List-Received-Date: Tue, 08 Jan 2019 13:43:51 -0000 DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogTW9ydGVuIEJyw7hydXAg W21haWx0bzptYkBzbWFydHNoYXJlc3lzdGVtcy5jb21dDQo+IFNlbnQ6IFR1ZXNkYXksIEphbnVh cnkgOCwgMjAxOSAxMTozNCBBTQ0KPiBUbzogQW5hbnlldiwgS29uc3RhbnRpbiA8a29uc3RhbnRp bi5hbmFueWV2QGludGVsLmNvbT47IEh1LCBKaWF5dSA8amlheXUuaHVAaW50ZWwuY29tPjsgU3Rl cGhlbiBIZW1taW5nZXINCj4gPHN0ZXBoZW5AbmV0d29ya3BsdW1iZXIub3JnPg0KPiBDYzogZGV2 QGRwZGsub3JnOyBCaWUsIFRpd2VpIDx0aXdlaS5iaWVAaW50ZWwuY29tPjsgUmljaGFyZHNvbiwg QnJ1Y2UgPGJydWNlLnJpY2hhcmRzb25AaW50ZWwuY29tPjsgc3RhYmxlQGRwZGsub3JnDQo+IFN1 YmplY3Q6IFJFOiBbZHBkay1kZXZdIFtQQVRDSF0gZ3JvOiBhZGQgbWlzc2luZyBpbnZhbGlkIHBh Y2tldCBjaGVja3MNCj4gDQo+ID4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gPiBGcm9t OiBkZXYgW21haWx0bzpkZXYtYm91bmNlc0BkcGRrLm9yZ10gT24gQmVoYWxmIE9mIEFuYW55ZXYs DQo+ID4gS29uc3RhbnRpbg0KPiA+IFNlbnQ6IFR1ZXNkYXksIEphbnVhcnkgOCwgMjAxOSAxMToz OSBBTQ0KPiA+IFRvOiBIdSwgSmlheXU7IFN0ZXBoZW4gSGVtbWluZ2VyDQo+ID4gQ2M6IGRldkBk cGRrLm9yZzsgQmllLCBUaXdlaTsgUmljaGFyZHNvbiwgQnJ1Y2U7IHN0YWJsZUBkcGRrLm9yZw0K PiA+IFN1YmplY3Q6IFJlOiBbZHBkay1kZXZdIFtQQVRDSF0gZ3JvOiBhZGQgbWlzc2luZyBpbnZh bGlkIHBhY2tldCBjaGVja3MNCj4gPg0KPiA+DQo+ID4NCj4gPiA+DQo+ID4gPg0KPiA+ID4gPiAt LS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiA+ID4gPiBGcm9tOiBTdGVwaGVuIEhlbW1pbmdl ciBbbWFpbHRvOnN0ZXBoZW5AbmV0d29ya3BsdW1iZXIub3JnXQ0KPiA+ID4gPiBTZW50OiBUdWVz ZGF5LCBKYW51YXJ5IDgsIDIwMTkgMjozMiBQTQ0KPiA+ID4gPiBUbzogSHUsIEppYXl1IDxqaWF5 dS5odUBpbnRlbC5jb20+DQo+ID4gPiA+IENjOiBkZXZAZHBkay5vcmc7IEJpZSwgVGl3ZWkgPHRp d2VpLmJpZUBpbnRlbC5jb20+OyBSaWNoYXJkc29uLA0KPiA+IEJydWNlDQo+ID4gPiA+IDxicnVj ZS5yaWNoYXJkc29uQGludGVsLmNvbT47IHN0YWJsZUBkcGRrLm9yZw0KPiA+ID4gPiBTdWJqZWN0 OiBSZTogW2RwZGstZGV2XSBbUEFUQ0hdIGdybzogYWRkIG1pc3NpbmcgaW52YWxpZCBwYWNrZXQN Cj4gPiBjaGVja3MNCj4gPiA+ID4NCj4gPiA+ID4gT24gVHVlLCAgOCBKYW4gMjAxOSAxNDowODo0 NSArMDgwMA0KPiA+ID4gPiBKaWF5dSBIdSA8amlheXUuaHVAaW50ZWwuY29tPiB3cm90ZToNCj4g PiA+ID4NCj4gPiA+ID4gPiArCS8qDQo+ID4gPiA+ID4gKwkgKiBEb24ndCBwcm9jZXNzIHRoZSBw YWNrZXQgd2hvc2UgRXRoZXJuZXQsIElQdjQgYW5kIFRDUA0KPiA+IGhlYWRlcg0KPiA+ID4gPiA+ ICsJICogbGVuZ3RocyBhcmUgaW52YWxpZC4gSW4gYWRkaXRpb24sIGlmIHRoZSBJUHY0IGhlYWRl cg0KPiA+IGNvbnRhaW5zDQo+ID4gPiA+ID4gKwkgKiBPcHRpb25zLCB0aGUgcGFja2V0IHNob3Vs ZG4ndCBiZSBwcm9jZXNzZWQuDQo+ID4gPiA+ID4gKwkgKi8NCj4gPiA+ID4gPiArCWlmICh1bmxp a2VseShJTExFR0FMX0VUSEVSX0hEUkxFTihwa3QtPmwyX2xlbikgfHwNCj4gPiA+ID4gPiArCQkJ SUxMRUdBTF9JUFY0X0hEUkxFTihwa3QtPmwzX2xlbikgfHwNCj4gPiA+ID4gPiArCQkJSUxMRUdB TF9UQ1BfSERSTEVOKHBrdC0+bDRfbGVuKSkpDQo+ID4gPiA+ID4gKwkJcmV0dXJuIC0xOw0KPiA+ ID4NCj4gPiA+IEluIHRoZSBHUk8gZGVzaWduLCB3ZSBhc3N1bWUgYXBwbGljYXRpb25zIGdpdmUg Y29ycmVjdA0KPiA+ID4gTUJVRi0+bDJfbGVuLy4uIGZvciBpbnB1dCBwYWNrZXRzIG9mIEdSTy4g U3BlY2lmaWNhbGx5LCBHUk8NCj4gPiA+IGxpYnJhcnkgYXNzdW1lcyBhcHBsaWNhdGlvbnMgd2ls bCBzZXQgdmFsdWVzIHRvIE1CVUYtPmwyX2xlbi8uLi4NCj4gPiA+IGFuZCBndWFyYW50ZWUgdGhl IHZhbHVlcyBhcmUgdGhlIHNhbWUgYXMgdGhlIHZhbHVlcyBpbiB0aGUgcGFja2V0DQo+ID4gPiBo ZWFkZXJzLiBUaGUgcmVhc29uIGZvciB0aGlzIGFzc3VtcHRpb24gaXMgdG8gcHJvY2VzcyBoZWFk ZXIgZmFzdGVyLg0KPiANCj4gPiA+IFRoaXMgaXMgYWxzbyB3aHkgSSB3YW50IHRvIGFkZCB0aGlz IGFzc3VtcHRpb24gaW4gdGhlIHByb2dyYW1tZXINCj4gPiA+IGd1aWRlLg0KPiANCj4gKzEgdG8g bW9yZSBkZXRhaWxlZCBkb2N1bWVudGF0aW9uIGFib3V0IGFzc3VtcHRpb25zIGFuZCBwcmVjb25k aXRpb25zLg0KPiANCj4gDQo+ID4gPg0KPiA+ID4gVGhlIGFib3ZlIGNvZGUgaXMgdG8gZm9yYmlk IEdSTyB0byBwcm9jZXNzIGludmFsaWQgcGFja2V0cywgd2hpY2gNCj4gPiA+IGhhdmUgaW52YWxp ZCBwYWNrZXQgaGVhZGVyIGxlbmd0aHMsIGxpa2UgVENQIGhlYWRlciBsZW5ndGggaXMgbGVzcw0K PiA+IHRoYW4NCj4gPiA+IDIwIGJ5dGVzLg0KPiA+ID4NCj4gPiA+ID4NCj4gPiA+ID4gSSBsaWtl IGl0IHdoZW4gY29kZSBpcyBhcyBwaWNreSBhcyBwb3NzaWJsZSB3aGVuIGRvaW5nDQo+ID4gb3B0 aW1pemF0aW9ucyBiZWNhdXNlDQo+ID4gPiA+IGl0IHJlZHVjZXMgcG9zc2libGUgc2VjdXJpdHkg cmlza2cuDQo+ID4gPiA+DQo+ID4gPiA+IFRvIG1lIHRoaXMgbG9va3MgbW9yZSBjb25mdXNpbmcg YW5kIG5vdCBhcyBjYXJlZnVsIGFzIGRvaW5nIGl0DQo+ID4gbGlrZToNCj4gPiA+ID4NCj4gPiA+ ID4gCWlmICh1bmxpa2VseShwa3QtPmwyX2xlbiAhPSBFVEhFUl9IRFJfTEVOKSkNCj4gPiA+ID4g CQlyZXR1cm4gLTE7DQo+ID4gPiA+IAlldGhfaGRyID0gcnRlX3BrdG1idWZfbXRvZChwa3QsIHN0 cnVjdCBldGhlcl9oZHIgKik7DQo+ID4gPiA+IAlpcHY0X2hkciA9IChzdHJ1Y3QgaXB2NF9oZHIg KikoKGNoYXIgKilldGhfaGRyICsgRVRIRVJfSERSX0xFTik7DQo+ID4gPiA+DQo+ID4gPiA+IAlp ZiAocGt0LT5sM19sZW4gIT0gKGlwdjQtPnZlcnNpb25faWhsICYgSVBWNF9IRFJfSUhMX01BU0sp IDw8IDQpDQo+ID4gPiA+IAkJcmV0dXJuIC0xOw0KPiA+ID4gPg0KPiA+ID4gPiAJaWYgKHBrdC0+ bDRfbGVuIDwgc2l6ZW9mKHN0cnVjdCB0Y3BfaGRyKSkNCj4gPiA+ID4gCQlyZXR1cm4gLTE7DQo+ ID4gPiA+DQo+ID4gPiA+IFlvdSBzaG91bGQgYWxzbyBjaGVjayBmb3IgVENQIG9wdGlvbnMgYXMg d2VsbC4NCj4gPiA+DQo+ID4gPiBUaGVyZSBhcmUgdHdvIHdheXMgdG8gZ2V0IGV0aGVyLCBpcHY0 IGFuZCB0Y3AgaGVhZGVyczoNCj4gPiA+IDEpLiBVc2UgTUJVRi0+bDJfbGVuL2wzX2xlbi4uLjsN Cj4gPiA+IDIpLiBQYXJzZSBwYWNrZXQgYW5kIGlnbm9yZSBNQlVGLT5sMl9sZW4vLi4uLg0KPiA+ ID4NCj4gPiA+IElmIHdlIGZvbGxvdyB0aGUgY2hvaWNlIDEsIHdlIGRvbid0IG5lZWQgdG8gcGFy c2UgcGFja2V0IGFuZA0KPiA+ID4gZG9uJ3QgbmVlZCB0byBjaGVjayBpZiB2YWx1ZXMgb2YgTUJV Ri0+bDJfbGVuLy4uLiBhcmUgY29ycmVjdCwNCj4gPiA+IHNpbmNlIHdlIGFzc3VtZSBhcHBsaWNh dGlvbnMgd2lsbCBzZXQgY29ycmVjdCB2YWx1ZXMuIElmIHdlIGZvbGxvdw0KPiA+ID4gdGhlIGNo b2ljZSAyLCB3ZSBkb24ndCBuZWVkIHRvIGNhcmUgYWJvdXQgdGhlIHZhbHVlcyBvZiBNQlVGLQ0K PiA+ID5sMl9sZW4vLi4uDQo+ID4gPg0KPiA+ID4gSSBhbSBhIGxpdHRsZSBjb25mdXNlZCBhYm91 dCB5b3VyIGNvZGUsIHNpbmNlIGl0IHBhcnNlcyBwYWNrZXQgYW5kDQo+ID4gPiBjaGVja3MgaWYg dGhlIHZhbHVlcyBvZiBNQlVGLT5sMl9sZW4vLi4uIGFyZSBjb3JyZWN0LiBJZiB3ZSBkb24ndCB1 c2UNCj4gPiA+IE1CVUYtPmwyX2xlbi8uLi4gdG8gZ2V0IGV0aGVyL2lwdjQvdGNwIGhlYWRlcnMs IHdoeSBzaG91bGQgd2UgY2hlY2sNCj4gPiA+IHRoZSB2YWx1ZXMgb2YgTUJVRi0+bDJfbGVuLy4u Lj8NCj4gPiA+DQo+ID4NCj4gPiBBZ3JlZSB0aGF0IHdlIGRvbid0IG5lZWQgYm90aC4NCj4gPiBN eSBwcmVmZXJlbmNlIHdvdWxkIGJlIHRvIHN0aWNrIHdpdGggMSkuDQo+ID4gSW4gbWFueSBjYXNl cyB1c2VyIHdvdWxkIGhhdmUgYWxyZWFkeSBkZXRlcm1pbmVkIGwyL2wzL2w0IGxlbg0KPiA+IGJ5 IHRoaXMgc3RhZ2UuDQo+ID4gS29uc3RhbnRpbg0KPiANCj4gRG8gd2UgaGF2ZSBhIGdlbmVyaWMg cGFja2V0IGhlYWRlciB2YWxpZGF0aW9uIGxpYnJhcnk/IE90aGVyd2lzZSwgdGhhdCB3b3VsZCBw ZXJoYXBzIGJlIGEgYmV0dGVyIHBhdGguIFN1Y2ggYSBsaWJyYXJ5IGNvdWxkIHByb2JhYmx5IHVz ZQ0KPiBzb21lIG9mIHRoZSBmbGFncyBmcm9tIHRoZSBQTUQgdG8gZGV0ZXJtaW5lIGhvdyBtdWNo IHRvIHZhbGlkYXRlIGluIHNvZnR3YXJlLg0KDQpBRkFJSyAtIHdlIGRvbid0IGhhdmUgYSBnZW5l cmljIGhlYWRlciBwYXJzaW5nIGxpYnJhcnkuDQpZZXMsIGl0IHdvdWxkIGJlIGdvb2QgdG8gaGF2 ZSBzdWNoIGFiaWxpdHksIGJ1dCBJIHRoaW5rIHRoYXQncyBvdXQgb2Ygc2NvcGUgZm9yIHRoYXQg cGF0Y2guDQpCVFcsIHZvbHVudGVlcnMgYXJlIHdlbGNvbWUgOikNCg0KPiANCj4gQW5kIGlmIGl0 IGlzIGEgZG9jdW1lbnRlZCBwcmVjb25kaXRpb24gb2YgdGhlIEdSTyBsaWJyYXJ5IHRoYXQgbS0+ bDJfbGVuL2wzX2xlbi4uLiBtdXN0IGJlIHNldCBhbmQgc2Vuc2libGUsIHBlcmhhcHMgYW4gUlRF X0FTU0VSVCgpIGNvdWxkDQo+IGJlIGNvbnNpZGVyZWQgaW5zdGVhZCBvZiBncmFjZWZ1bGx5IHJl dHVybmluZyAtMT8NCg0KSSBzdXBwb3NlIHRoYXQncyB0b28gZXh0cmVtZS4NCldoYXQncyB3cm9u ZyB3aXRoIGNoZWNraW5nIGlucHV0IHBhcmFtZXRlcnMgYW5kIHJldHVybiBhbiBlcnJvciBpZiB0 aGV5IGFyZSBpbnZhbGlkPw0KS29uc3RhbnRpbg0KDQoNCg==