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 D363BA0526; Tue, 21 Jul 2020 08:41:56 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 78BA91BFFF; Tue, 21 Jul 2020 08:41:55 +0200 (CEST) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 072CB1BFFE; Tue, 21 Jul 2020 08:41:51 +0200 (CEST) IronPort-SDR: MAu8ZQUkPw2kukxR3j5/TVc3JMMnwCyPsmS/ISs1slU1rpclUC1C8kpfduEbGUskEN9p/ZyF6f 4W1E/YKDJJZw== X-IronPort-AV: E=McAfee;i="6000,8403,9688"; a="211623482" X-IronPort-AV: E=Sophos;i="5.75,377,1589266800"; d="scan'208";a="211623482" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Jul 2020 23:41:50 -0700 IronPort-SDR: colI+XRXKPtZNP1hw1B7XGL+dgWTo77lNQklrRzVshzsgiFEPBzNyD2+IPPsYKPZIqaOZaevBu 7A1NB1mdqv3Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,377,1589266800"; d="scan'208";a="319799674" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by fmsmga002.fm.intel.com with ESMTP; 20 Jul 2020 23:41:50 -0700 Received: from fmsmsx114.amr.corp.intel.com (10.18.116.8) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 20 Jul 2020 23:41:50 -0700 Received: from cdsmsx103.ccr.corp.intel.com (172.17.3.37) by FMSMSX114.amr.corp.intel.com (10.18.116.8) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 20 Jul 2020 23:41:49 -0700 Received: from cdsmsx102.ccr.corp.intel.com ([169.254.2.201]) by CDSMSX103.ccr.corp.intel.com ([169.254.5.225]) with mapi id 14.03.0439.000; Tue, 21 Jul 2020 14:41:46 +0800 From: "Xie, WeiX" To: "Wang, ShougangX" , "dev@dpdk.org" CC: "Xing, Beilei" , "Guo, Jia" , "Wang, ShougangX" , "stable@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v2] net/i40e: fix incorrect hash look up table Thread-Index: AQHWXySrAslY/DhCGU2jcPdEARPwAakRlUHA Date: Tue, 21 Jul 2020 06:41:45 +0000 Message-ID: <6FD6A7610D20924F885A4ECF34E8AC910463D264@CDSMSX102.ccr.corp.intel.com> References: <20200715063515.9262-1-shougangx.wang@intel.com> <20200721054920.29749-1-shougangx.wang@intel.com> In-Reply-To: <20200721054920.29749-1-shougangx.wang@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.17.6.105] Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH v2] net/i40e: fix incorrect hash look up table 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" VGVzdGVkLWJ5OiBaaGFuZywgWGlYIDx4aXguemhhbmdAaW50ZWwuY29tPg0KDQpSZWdhcmRzLA0K WGllIFdlaQ0KDQoNCi0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tDQpGcm9tOiBkZXYgW21haWx0 bzpkZXYtYm91bmNlc0BkcGRrLm9yZ10gT24gQmVoYWxmIE9mIFNob3VnYW5nIFdhbmcNClNlbnQ6 IFR1ZXNkYXksIEp1bHkgMjEsIDIwMjAgMTo0OSBQTQ0KVG86IGRldkBkcGRrLm9yZw0KQ2M6IFhp bmcsIEJlaWxlaSA8YmVpbGVpLnhpbmdAaW50ZWwuY29tPjsgR3VvLCBKaWEgPGppYS5ndW9AaW50 ZWwuY29tPjsgV2FuZywgU2hvdWdhbmdYIDxzaG91Z2FuZ3gud2FuZ0BpbnRlbC5jb20+OyBzdGFi bGVAZHBkay5vcmcNClN1YmplY3Q6IFtkcGRrLWRldl0gW1BBVENIIHYyXSBuZXQvaTQwZTogZml4 IGluY29ycmVjdCBoYXNoIGxvb2sgdXAgdGFibGUNCg0KVGhlIGhhc2ggbG9vayB1cCB0YWJsZShM VVQpIHdpbGwgbm90IGJlIGluaXRpYWxpemluZyB3aGVuIHN0YXJ0aW5nIHRlc3RwbWQgd2l0aCAt LWRpc2FibGUtcnNzLiBTbyB0aGF0IHNvbWUgaW52YWxpZCBxdWV1ZSBpbmRleGVzIG1heSBzdGls bCBpbiB0aGUgTFVULiBXaGVuIGVuYWJsZSBSU1MgYnkgY3JlYXRpbmcgUlNTIHJ1bGUsIHNvbWUg cGFja2V0cyB3aWxsIG5vdCBiZSBpbnRvIHRoZSB2YWxpZCBxdWV1ZXMuDQpUaGlzIHBhdGNoIGZp eGVzIHRoaXMgaXNzdWUgYnkgaW5pdGlhbGl6aW5nIHRoZSBMVVQgd2hlbiBjcmVhdGluZyBhbiBS U1MgcnVsZS4NCg0KRml4ZXM6IGZlYWFlMjg1YjM0MiAoIm5ldC9pNDBlOiBzdXBwb3J0IGhhc2gg Y29uZmlndXJhdGlvbiBpbiBSU1MgZmxvdyIpDQpDYzogc3RhYmxlQGRwZGsub3JnDQoNClNpZ25l ZC1vZmYtYnk6IFNob3VnYW5nIFdhbmcgPHNob3VnYW5neC53YW5nQGludGVsLmNvbT4NCi0tLQ0K IGRyaXZlcnMvbmV0L2k0MGUvaTQwZV9ldGhkZXYuYyB8IDEzNCArKysrKysrKysrKysrKysrLS0t LS0tLS0tLS0tLS0tLS0NCiAxIGZpbGUgY2hhbmdlZCwgNjMgaW5zZXJ0aW9ucygrKSwgNzEgZGVs ZXRpb25zKC0pDQoNCmRpZmYgLS1naXQgYS9kcml2ZXJzL25ldC9pNDBlL2k0MGVfZXRoZGV2LmMg Yi9kcml2ZXJzL25ldC9pNDBlL2k0MGVfZXRoZGV2LmMgaW5kZXggMzkzYjUzMjBmLi5lNTY1NDMz OTMgMTAwNjQ0DQotLS0gYS9kcml2ZXJzL25ldC9pNDBlL2k0MGVfZXRoZGV2LmMNCisrKyBiL2Ry aXZlcnMvbmV0L2k0MGUvaTQwZV9ldGhkZXYuYw0KQEAgLTEzMDcwLDYgKzEzMDcwLDU1IEBAIGk0 MGVfcnNzX2NvbmZfaW5pdChzdHJ1Y3QgaTQwZV9ydGVfZmxvd19yc3NfY29uZiAqb3V0LA0KIAly ZXR1cm4gMDsNCiB9DQogDQorLyogSWYgY29uZiBpcyBOVUxMLCBmdW5jdGlvbiB3aWxsIGluaXQg aGFzaCBMVVQgd2l0aCBkZWZhdWx0IA0KK2NvbmZpZ3JhdGlvbiovIHN0YXRpYyBpbnQgaTQwZV9y c3Nfc2V0X2x1dChzdHJ1Y3QgaTQwZV9wZiAqcGYsDQorCQkgc3RydWN0IGk0MGVfcnRlX2Zsb3df cnNzX2NvbmYgKmNvbmYpIHsNCisJc3RydWN0IGk0MGVfaHcgKmh3ID0gSTQwRV9QRl9UT19IVyhw Zik7DQorCXVpbnQzMl90IGx1dCA9IDA7DQorCXVpbnQxNl90IGosIG51bTsNCisJdWludDMyX3Qg aTsNCisNCisJLyogSWYgYm90aCBWTURRIGFuZCBSU1MgZW5hYmxlZCwgbm90IGFsbCBvZiBQRiBx dWV1ZXMgYXJlIGNvbmZpZ3VyZWQuDQorCSAqIEl0J3MgbmVjZXNzYXJ5IHRvIGNhbGN1bGF0ZSB0 aGUgYWN0dWFsIFBGIHF1ZXVlcyB0aGF0IGFyZSBjb25maWd1cmVkLg0KKwkgKi8NCisJaWYgKHBm LT5kZXZfZGF0YS0+ZGV2X2NvbmYucnhtb2RlLm1xX21vZGUgJiBFVEhfTVFfUlhfVk1EUV9GTEFH KQ0KKwkJbnVtID0gaTQwZV9wZl9jYWxjX2NvbmZpZ3VyZWRfcXVldWVzX251bShwZik7DQorCWVs c2UNCisJCW51bSA9IHBmLT5kZXZfZGF0YS0+bmJfcnhfcXVldWVzOw0KKw0KKwlpZiAoY29uZiA9 PSBOVUxMKQ0KKwkJbnVtID0gUlRFX01JTihudW0sIEk0MEVfTUFYX1FfUEVSX1RDKTsNCisJZWxz ZQ0KKwkJbnVtID0gUlRFX01JTihudW0sIGNvbmYtPmNvbmYucXVldWVfbnVtKTsNCisJUE1EX0RS Vl9MT0coSU5GTywgIk1heCBvZiBjb250aWd1b3VzICV1IFBGIHF1ZXVlcyBhcmUgY29uZmlndXJl ZCIsDQorCQkJbnVtKTsNCisNCisJaWYgKG51bSA9PSAwKSB7DQorCQlQTURfRFJWX0xPRyhFUlIs DQorCQkJIk5vIFBGIHF1ZXVlcyBhcmUgY29uZmlndXJlZCB0byBlbmFibGUgUlNTIGZvciBwb3J0 ICV1IiwNCisJCQlwZi0+ZGV2X2RhdGEtPnBvcnRfaWQpOw0KKwkJcmV0dXJuIC1FTk9UU1VQOw0K Kwl9DQorDQorCS8qIEZpbGwgaW4gcmVkaXJlY3Rpb24gdGFibGUgKi8NCisJZm9yIChpID0gMCwg aiA9IDA7IGkgPCBody0+ZnVuY19jYXBzLnJzc190YWJsZV9zaXplOyBpKyssIGorKykgew0KKwkJ aWYgKGogPT0gbnVtKQ0KKwkJCWogPSAwOw0KKwkJaWYgKGNvbmYgPT0gTlVMTCkNCisJCQlsdXQg PSAobHV0IDw8IDgpIHwgKGogJiAoKDB4MSA8PA0KKwkJCQlody0+ZnVuY19jYXBzLnJzc190YWJs ZV9lbnRyeV93aWR0aCkgLSAxKSk7DQorCQllbHNlDQorCQkJbHV0ID0gKGx1dCA8PCA4KSB8IChj b25mLT5jb25mLnF1ZXVlW2pdICYgKCgweDEgPDwNCisJCQlody0+ZnVuY19jYXBzLnJzc190YWJs ZV9lbnRyeV93aWR0aCkgLSAxKSk7DQorCQlpZiAoKGkgJiAzKSA9PSAzKQ0KKwkJCUk0MEVfV1JJ VEVfUkVHKGh3LCBJNDBFX1BGUUZfSExVVChpID4+IDIpLCBsdXQpOw0KKwl9DQorDQorCXJldHVy biAwOw0KK30NCisNCiAvKiBXcml0ZSBIRU5BIHJlZ2lzdGVyIHRvIGVuYWJsZSBoYXNoICovICBz dGF0aWMgaW50ICBpNDBlX3Jzc19oYXNoX3NldChzdHJ1Y3QgaTQwZV9wZiAqcGYsIHN0cnVjdCBp NDBlX3J0ZV9mbG93X3Jzc19jb25mICpyc3NfY29uZikgQEAgLTEzMzE4LDEyICsxMzM2NywyNCBA QCBzdGF0aWMgaW50ICBpNDBlX3Jzc19lbmFibGVfaGFzaChzdHJ1Y3QgaTQwZV9wZiAqcGYsDQog CQlzdHJ1Y3QgaTQwZV9ydGVfZmxvd19yc3NfY29uZiAqY29uZikNCiB7DQorCWVudW0gcnRlX2V0 aF9yeF9tcV9tb2RlIG1xX21vZGUgPSANCitwZi0+ZGV2X2RhdGEtPmRldl9jb25mLnJ4bW9kZS5t cV9tb2RlOw0KIAlzdHJ1Y3QgaTQwZV9ydGVfZmxvd19yc3NfY29uZiAqcnNzX2luZm8gPSAmcGYt PnJzc19pbmZvOw0KIAlzdHJ1Y3QgaTQwZV9ydGVfZmxvd19yc3NfY29uZiByc3NfY29uZjsNCisJ aW50IHJldDsNCiANCiAJaWYgKCEoY29uZi0+Y29uZi50eXBlcyAmIHBmLT5hZGFwdGVyLT5mbG93 X3R5cGVzX21hc2spKQ0KIAkJcmV0dXJuIC1FTk9UU1VQOw0KIA0KKwkvKiBJZiB0aGUgUlNTIGlz IGRpc2FibGVkIGJlZm9yZSB0aGlzLCB0aGUgTFVUIGlzIHVuaW5pdGlhbGl6ZWQuwqANCisJICog U28gaXQgaXMgbmVjZXNzYXJ5IHRvIGluaXRpYWxpemUgaXQgaGVyZS4NCisJICovDQorCWlmICgh KG1xX21vZGUgJiBFVEhfTVFfUlhfUlNTX0ZMQUcpICYmICFwZi0+cnNzX2luZm8uY29uZi5xdWV1 ZV9udW0gJiYNCisJICAgICFwZi0+YWRhcHRlci0+cnNzX3JldGFfdXBkYXRlZCkgew0KKwkJcmV0 ID0gaTQwZV9yc3Nfc2V0X2x1dChwZiwgTlVMTCk7DQorCQlpZiAocmV0KQ0KKwkJCXJldHVybiBy ZXQ7DQorCX0NCisNCiAJbWVtc2V0KCZyc3NfY29uZiwgMCwgc2l6ZW9mKHJzc19jb25mKSk7DQog CXJ0ZV9tZW1jcHkoJnJzc19jb25mLCBjb25mLCBzaXplb2YocnNzX2NvbmYpKTsNCiANCkBAIC0x MzM2MiwzOSArMTM0MjMsNyBAQCBzdGF0aWMgaW50DQogaTQwZV9yc3NfY29uZmlnX3F1ZXVlX3Jl Z2lvbihzdHJ1Y3QgaTQwZV9wZiAqcGYsDQogCQlzdHJ1Y3QgaTQwZV9ydGVfZmxvd19yc3NfY29u ZiAqY29uZikNCiB7DQotCXN0cnVjdCBpNDBlX2h3ICpodyA9IEk0MEVfUEZfVE9fSFcocGYpOw0K LQl1aW50MzJfdCBsdXQgPSAwOw0KLQl1aW50MTZfdCBqLCBudW07DQotCXVpbnQzMl90IGk7DQot DQotCS8qIElmIGJvdGggVk1EUSBhbmQgUlNTIGVuYWJsZWQsIG5vdCBhbGwgb2YgUEYgcXVldWVz IGFyZSBjb25maWd1cmVkLg0KLQkgKiBJdCdzIG5lY2Vzc2FyeSB0byBjYWxjdWxhdGUgdGhlIGFj dHVhbCBQRiBxdWV1ZXMgdGhhdCBhcmUgY29uZmlndXJlZC4NCi0JICovDQotCWlmIChwZi0+ZGV2 X2RhdGEtPmRldl9jb25mLnJ4bW9kZS5tcV9tb2RlICYgRVRIX01RX1JYX1ZNRFFfRkxBRykNCi0J CW51bSA9IGk0MGVfcGZfY2FsY19jb25maWd1cmVkX3F1ZXVlc19udW0ocGYpOw0KLQllbHNlDQot CQludW0gPSBwZi0+ZGV2X2RhdGEtPm5iX3J4X3F1ZXVlczsNCi0NCi0JbnVtID0gUlRFX01JTihu dW0sIGNvbmYtPmNvbmYucXVldWVfbnVtKTsNCi0JUE1EX0RSVl9MT0coSU5GTywgIk1heCBvZiBj b250aWd1b3VzICV1IFBGIHF1ZXVlcyBhcmUgY29uZmlndXJlZCIsDQotCQkJbnVtKTsNCi0NCi0J aWYgKG51bSA9PSAwKSB7DQotCQlQTURfRFJWX0xPRyhFUlIsDQotCQkJIk5vIFBGIHF1ZXVlcyBh cmUgY29uZmlndXJlZCB0byBlbmFibGUgUlNTIGZvciBwb3J0ICV1IiwNCi0JCQlwZi0+ZGV2X2Rh dGEtPnBvcnRfaWQpOw0KLQkJcmV0dXJuIC1FTk9UU1VQOw0KLQl9DQotDQotCS8qIEZpbGwgaW4g cmVkaXJlY3Rpb24gdGFibGUgKi8NCi0JZm9yIChpID0gMCwgaiA9IDA7IGkgPCBody0+ZnVuY19j YXBzLnJzc190YWJsZV9zaXplOyBpKyssIGorKykgew0KLQkJaWYgKGogPT0gbnVtKQ0KLQkJCWog PSAwOw0KLQkJbHV0ID0gKGx1dCA8PCA4KSB8IChjb25mLT5jb25mLnF1ZXVlW2pdICYgKCgweDEg PDwNCi0JCQlody0+ZnVuY19jYXBzLnJzc190YWJsZV9lbnRyeV93aWR0aCkgLSAxKSk7DQotCQlp ZiAoKGkgJiAzKSA9PSAzKQ0KLQkJCUk0MEVfV1JJVEVfUkVHKGh3LCBJNDBFX1BGUUZfSExVVChp ID4+IDIpLCBsdXQpOw0KLQl9DQorCWk0MGVfcnNzX3NldF9sdXQocGYsIGNvbmYpOw0KIA0KIAlp NDBlX3Jzc19tYXJrX2ludmFsaWRfcnVsZShwZiwgY29uZik7DQogDQpAQCAtMTM0OTEsNDYgKzEz NTIwLDkgQEAgaTQwZV9yc3NfZGlzYWJsZV9oYXNoKHN0cnVjdCBpNDBlX3BmICpwZiwgIHN0YXRp YyBpbnQgIGk0MGVfcnNzX2NsZWFyX3F1ZXVlX3JlZ2lvbihzdHJ1Y3QgaTQwZV9wZiAqcGYpICB7 DQotCXN0cnVjdCBpNDBlX2h3ICpodyA9IEk0MEVfUEZfVE9fSFcocGYpOw0KIAlzdHJ1Y3QgaTQw ZV9ydGVfZmxvd19yc3NfY29uZiAqcnNzX2luZm8gPSAmcGYtPnJzc19pbmZvOw0KLQl1aW50MTZf dCBxdWV1ZVtJNDBFX01BWF9RX1BFUl9UQ107DQotCXVpbnQzMl90IG51bV9yeHEsIGk7DQotCXVp bnQzMl90IGx1dCA9IDA7DQotCXVpbnQxNl90IGosIG51bTsNCi0NCi0JbnVtX3J4cSA9IFJURV9N SU4ocGYtPmRldl9kYXRhLT5uYl9yeF9xdWV1ZXMsIEk0MEVfTUFYX1FfUEVSX1RDKTsNCiANCi0J Zm9yIChqID0gMDsgaiA8IG51bV9yeHE7IGorKykNCi0JCXF1ZXVlW2pdID0gajsNCi0NCi0JLyog SWYgYm90aCBWTURRIGFuZCBSU1MgZW5hYmxlZCwgbm90IGFsbCBvZiBQRiBxdWV1ZXMgYXJlIGNv bmZpZ3VyZWQuDQotCSAqIEl0J3MgbmVjZXNzYXJ5IHRvIGNhbGN1bGF0ZSB0aGUgYWN0dWFsIFBG IHF1ZXVlcyB0aGF0IGFyZSBjb25maWd1cmVkLg0KLQkgKi8NCi0JaWYgKHBmLT5kZXZfZGF0YS0+ ZGV2X2NvbmYucnhtb2RlLm1xX21vZGUgJiBFVEhfTVFfUlhfVk1EUV9GTEFHKQ0KLQkJbnVtID0g aTQwZV9wZl9jYWxjX2NvbmZpZ3VyZWRfcXVldWVzX251bShwZik7DQotCWVsc2UNCi0JCW51bSA9 IHBmLT5kZXZfZGF0YS0+bmJfcnhfcXVldWVzOw0KLQ0KLQludW0gPSBSVEVfTUlOKG51bSwgbnVt X3J4cSk7DQotCVBNRF9EUlZfTE9HKElORk8sICJNYXggb2YgY29udGlndW91cyAldSBQRiBxdWV1 ZXMgYXJlIGNvbmZpZ3VyZWQiLA0KLQkJCW51bSk7DQotDQotCWlmIChudW0gPT0gMCkgew0KLQkJ UE1EX0RSVl9MT0coRVJSLA0KLQkJCSJObyBQRiBxdWV1ZXMgYXJlIGNvbmZpZ3VyZWQgdG8gZW5h YmxlIFJTUyBmb3IgcG9ydCAldSIsDQotCQkJcGYtPmRldl9kYXRhLT5wb3J0X2lkKTsNCi0JCXJl dHVybiAtRU5PVFNVUDsNCi0JfQ0KLQ0KLQkvKiBGaWxsIGluIHJlZGlyZWN0aW9uIHRhYmxlICov DQotCWZvciAoaSA9IDAsIGogPSAwOyBpIDwgaHctPmZ1bmNfY2Fwcy5yc3NfdGFibGVfc2l6ZTsg aSsrLCBqKyspIHsNCi0JCWlmIChqID09IG51bSkNCi0JCQlqID0gMDsNCi0JCWx1dCA9IChsdXQg PDwgOCkgfCAocXVldWVbal0gJiAoKDB4MSA8PA0KLQkJCWh3LT5mdW5jX2NhcHMucnNzX3RhYmxl X2VudHJ5X3dpZHRoKSAtIDEpKTsNCi0JCWlmICgoaSAmIDMpID09IDMpDQotCQkJSTQwRV9XUklU RV9SRUcoaHcsIEk0MEVfUEZRRl9ITFVUKGkgPj4gMiksIGx1dCk7DQotCX0NCisJaTQwZV9yc3Nf c2V0X2x1dChwZiwgTlVMTCk7DQogDQogCXJzc19pbmZvLT5jb25mLnF1ZXVlX251bSA9IDA7DQog CW1lbXNldCgmcnNzX2luZm8tPmNvbmYucXVldWUsIDAsIHNpemVvZih1aW50MTZfdCkpOw0KLS0N CjIuMTcuMQ0KDQo=