From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id C1076A0547 for ; Mon, 21 Jun 2021 10:19:15 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8697540040; Mon, 21 Jun 2021 10:19:15 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mails.dpdk.org (Postfix) with ESMTP id 7908D40040 for ; Mon, 21 Jun 2021 10:19:13 +0200 (CEST) IronPort-SDR: jEQAFkEXMPjn/zA9/BokIfD1Jdu2z/oDGgKuh7zPFr5QUZmuU/eAt6sy4AZiWUTlb4Zsb1OAK/ I0O8VWF0PkPQ== X-IronPort-AV: E=McAfee;i="6200,9189,10021"; a="206620182" X-IronPort-AV: E=Sophos;i="5.83,289,1616482800"; d="scan'208";a="206620182" Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Jun 2021 01:19:12 -0700 IronPort-SDR: 2hEN+HqkQ7jGjY4ch2ApSeYukaCEU5bXMzGFKH8TVrBdExKvfdbdJIhHthSLEb879101P6pwYN cKeG7TpOczYw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.83,289,1616482800"; d="scan'208";a="473355266" Received: from orsmsx605.amr.corp.intel.com ([10.22.229.18]) by fmsmga004.fm.intel.com with ESMTP; 21 Jun 2021 01:19:12 -0700 Received: from orsmsx605.amr.corp.intel.com (10.22.229.18) by ORSMSX605.amr.corp.intel.com (10.22.229.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.4; Mon, 21 Jun 2021 01:19:10 -0700 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx605.amr.corp.intel.com (10.22.229.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.4 via Frontend Transport; Mon, 21 Jun 2021 01:19:10 -0700 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (104.47.58.100) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.4; Mon, 21 Jun 2021 01:19:10 -0700 Received: from MN2PR11MB3807.namprd11.prod.outlook.com (2603:10b6:208:f0::24) by MN2PR11MB3550.namprd11.prod.outlook.com (2603:10b6:208:ee::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4242.21; Mon, 21 Jun 2021 08:19:09 +0000 Received: from MN2PR11MB3807.namprd11.prod.outlook.com ([fe80::55a3:215a:f054:6b52]) by MN2PR11MB3807.namprd11.prod.outlook.com ([fe80::55a3:215a:f054:6b52%5]) with mapi id 15.20.4242.023; Mon, 21 Jun 2021 08:19:09 +0000 From: "Xing, Beilei" To: "Yang, SteveX" , "dev@dpdk.org" CC: "Yang, SteveX" , "stable@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v3] net/i40e: fix set rss hash function invalid Thread-Index: AQHXZnReyb4VOVA52U+DF/UrvnZ+GqseH0Bg Date: Mon, 21 Jun 2021 08:19:09 +0000 Message-ID: References: <20210611065457.988344-1-stevex.yang@intel.com> <20210621080342.184889-1-stevex.yang@intel.com> In-Reply-To: <20210621080342.184889-1-stevex.yang@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.55.46.52] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: fca62ca5-f355-4573-251d-08d9348d3eac x-ms-traffictypediagnostic: MN2PR11MB3550: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:1079; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: Mo0HV4nos1ZssRIqzVP3ahZcLHHiSr2u/PIOu9JvtrquXm1WvDMLyP8Z9UPsg4cK/YUk3BazHUB4E9ROBcBtAFUqd9hCvsRJu8Uvt9vSIEXDHkMwGyV1zjr7zi/i0JUU/pBn+3lemQJrSp9Ln17GRkeEnJeEmT+y4+yUQY5yvrGrXwJRMF5J/6eiOUgNdJFaCmEZUONyqdaw7m5F2NUhkfysEIRSQUxDIE/5Cq723wn9egGZ9v3W64pZsBgyHmGgWOnrsuiHIK72nuvx8tmIcRMAhSkgYAevnOKLbcwa3pK4JdqspCJ4qD9etx2L2PFC7pC11aFtf9jzpDdIDefJI5ciDFnlfklS3ZxTUNwr/F5lIqdh2r4EkpRERB+lV0RsdEWQOSe+unTmggjzo9U4isILNcEoz2N2V1xIYQUMN4/eU6wzmFk/enOrQBqD6chzKvdWl8cwT3wgkqtn1rXYSk2OGvz49jsbM4ksW0m5zf3mtTlGfbU/LPtywFNMSah9U5xh6XrrWs4uwJ4A4g4Ptx5JPB15AV1YsYUWlMLuTA3iFEEpRZ9qcetKopg2zwXQIY/KrQs6BFNm8FT+T+2Zo/8WzsdhCbsJTYmL1U8EfH8= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MN2PR11MB3807.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(136003)(396003)(39860400002)(346002)(376002)(366004)(186003)(71200400001)(478600001)(26005)(86362001)(52536014)(122000001)(38100700002)(7696005)(33656002)(8676002)(66946007)(8936002)(5660300002)(6506007)(53546011)(76116006)(64756008)(66556008)(66476007)(55016002)(450100002)(9686003)(66446008)(54906003)(2906002)(4326008)(110136005)(83380400001)(316002); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Q6VNlk7+UvuLP7zat+L0qEsztQwgLE+ely97oBGwBDw8rgMUJkDONEfdHEov?= =?us-ascii?Q?UlBUzWrFpXbb8kNAEbX06O+HORSC+T1/Fc4JJ05i86+TUSb2K+NtfDab2vqd?= =?us-ascii?Q?AxkpmmXYfgzmI5bKfhQx5t0QuE6xG07Q5hXT8Q0LfQgyK3TNQVgwvu+81YBF?= =?us-ascii?Q?vZnv0mtlnS0pQGuMRXgYf2A+nIQqUNRPxNZ6EzNCvAFWiVHzV8Yx+hTpmTb6?= =?us-ascii?Q?4YY4F7o1+npK4BWybAYNRRHVTIsaVCNfxfcB4E8XCEGFn67EPmtHmgBU4e20?= =?us-ascii?Q?l8JpqL1vaMp8pLEA6SNxNf4/BfNx2ZPEB7lgOZagfG/eyfmRVb/kMnsda3X1?= =?us-ascii?Q?Qkr8+W0PusSw9cDfJU7ADZhFnzeA0DjG2AF4kSleVqptcL3eqO5UjiUfJ/u/?= =?us-ascii?Q?laFtkK7WeLXKoz9XOlCQqQat3nnqHm2HjHVKhYXgf9mmwAtHPU9BCSGnHkyz?= =?us-ascii?Q?HSJcwd0WzfvzmT3sIiyYLkAEwSAaHSmqdDhoU9dIS3U/IZj20nFTnTYGYCG+?= =?us-ascii?Q?6RIRWvfST7YrOyT7VzxHwkwqTwlsSeNKOt8YhyIPnEYFTjq51zbJgazg9Hhu?= =?us-ascii?Q?3YIw1QXBTUFkgrn6AO6FloFfb4VWyOwQCdXQMj0h4awSf5NCpdsZZrtqExPH?= =?us-ascii?Q?UdgO6WCy03TtVfFa+tkmqRE2Wrvd6sx36C44CqS2gkZ597SvbTdLwpL8nHMS?= =?us-ascii?Q?S0GJSd7sDQEE49CgBcBqIsxb+TntGNVMs8XUrBARTpjj532Iaa2gZRTm/I7T?= =?us-ascii?Q?Y6eGY8VN+mLC7NaXCSq/Mu1/XVZ3sX0Wtu0xh0pHVLVp64o1dxgPeIbbShVy?= =?us-ascii?Q?3E/bQHTZ6JaoZ5Ris8ouYRqCIQAtqqIxSjb0G6ijPplx7xCcP9O0VJWxZ8ne?= =?us-ascii?Q?5dCfJePhLxvcTsvxLaZfn3ITjE5cBznEyYamkGWnp3wiy5AbS1sWTOzNejwH?= =?us-ascii?Q?lbEA5idiorxBON1WcjlqDDEPITmuplQxmCNOqHxnFvElJihs3lb7MUA9oz7K?= =?us-ascii?Q?7cm1lWEZzOj8bRDh5g9MH+8ZsgcftdOjD1DOtl/qQqz4GqY0DQc/T72voOBv?= =?us-ascii?Q?zPfW/f1KFYoAjo31OB8yCwMGl3y8rtjKpx5bGyZ8uMmezwOWatUzCyisLLlj?= =?us-ascii?Q?tE5SnnN3kw+MYe1e2HnlKU3F0gzyByWKcrAR8QiHBPNXrrptIC8LVOAgOYS+?= =?us-ascii?Q?TIHxwyVkJxUCluUNJ45c6obMHYQBOrwJFOPq1OMMCrbMC4wq+XUOXmgGFJLv?= =?us-ascii?Q?+N0RVG5XvsXUhlJaY8qY2VbuEOgYEnfkYaOTiqSJcvTJA0yR58BzX282HQLg?= =?us-ascii?Q?Ugw=3D?= arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZBwm9eMNVcYo+llyM1zPt8HE5647QoGlPYY++o93MxZFy51MkevcUdYLcARravrwSPD1YDtHDKDsqg3zk4GtjKjm+Upn7qgcReSNo/s/jlo/3kkWoXkly51j6vFAhRSkzDIcaQcLaWm6rTvQv43KJ0q5HNfN+NAawW1r7MVsYVGmh3iBBM7wjSIKZYzN1oJqYDWGu6okB/rcQBPTsicwWGhD50YSx9f9v9aML+sAGH3XPd0QRi4SSvDpWWwYrGI2/N37XjBntXR0LE8YFmGFiHfKNf4asrgyx8ih9i23aw3P8euyHcdMI5+ijwhxPANSQ4KeURbfsDw4pxr04BQYRA== arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=3xwH2YMCvwQ86Om6jxNgr9mnlA+QhHkgnZWhmdtebNk=; b=U4jMrUaAjTB2gIFxyqikpxjhRcsVIk8Utm7de0dkHv1fTJZZFADfCs08NyDqEyWz9kVDZQXGYf9NLh0hMZq765nxnokLqalvwfrR3FWbq9oUJDFl8qt2JadyJhHc4JKIgt5FOZk+1zYGSYc3EdMxyvPIJbMDvieSUnSODqwjOaIO9VLwg5RPpd8n2jPE/4Y3JO+NfTn48eBuZwUYQRVSyc/Q/fhSwZbBVj5VVAW3Vt4ksKN0tJaWPrQxxeBhtPX8asVZ0q1v2nQq7r+HZNX1AkXk5kfggJJbIlVP2462NxS+0Id78+NhfqE3U7BBu8Wns5OnfJI76sw4IKsj+nsF7g== arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=3xwH2YMCvwQ86Om6jxNgr9mnlA+QhHkgnZWhmdtebNk=; b=YIIdGThUZcpQze1o40IA71gMJAd1f/NhSBHZI5oP3dvyRDY0VvCP7Nov2xMaRDSCqKShcSnKHDHsYiZW9EBAnUrmeYWNBrSuax3GYkM8C4tc4kyysJFZKgp22I1rPszxnKLQm4ZPe6V7M/gCqFdFdOraZ9gfi3lD8ZJiPWyhSuA= x-ms-exchange-crosstenant-authas: Internal x-ms-exchange-crosstenant-authsource: MN2PR11MB3807.namprd11.prod.outlook.com x-ms-exchange-crosstenant-network-message-id: fca62ca5-f355-4573-251d-08d9348d3eac x-ms-exchange-crosstenant-originalarrivaltime: 21 Jun 2021 08:19:09.4968 (UTC) x-ms-exchange-crosstenant-fromentityheader: Hosted x-ms-exchange-crosstenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d x-ms-exchange-crosstenant-mailboxtype: HOSTED x-ms-exchange-crosstenant-userprincipalname: Di6croCqv/LDrK1sj1mdSwb7FLtBfrkQgXi6pLOD4ymx+0eHmG42DSC52hV2MlMFS/MjAKk4jn0K6AVCMC3S+g== x-ms-exchange-transport-crosstenantheadersstamped: MN2PR11MB3550 x-originatororg: intel.com Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH v3] net/i40e: fix set rss hash function invalid X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" > -----Original Message----- > From: dev On Behalf Of Steve Yang > Sent: Monday, June 21, 2021 4:04 PM > To: dev@dpdk.org > Cc: Xing, Beilei ; Yang, SteveX > ; stable@dpdk.org > Subject: [dpdk-dev] [PATCH v3] net/i40e: fix set rss hash function invali= d > > i40e can support following rss hash function types: default/toeplitz, > symmetric toeplitz, and simple_xor. However, when filter engine parses > pattern action, it only supports symmetric toeplitz & default. > > Add simple_xor and toeplitz hash functions support when parsing pattern > action. > > Fixes: ef4c16fd9148 ("net/i40e: refactor RSS flow") > Cc: stable@dpdk.org > > Signed-off-by: Steve Yang > --- > v3: > - add Cc stable line. > v2: > - add the fix line. > - support simple_xor and toeplitz hash functions explicitly. > > drivers/net/i40e/i40e_hash.c | 20 ++++++++++++++------ > 1 file changed, 14 insertions(+), 6 deletions(-) > > diff --git a/drivers/net/i40e/i40e_hash.c b/drivers/net/i40e/i40e_hash.c > index b1cb24f437..0cef21c88f 100644 > --- a/drivers/net/i40e/i40e_hash.c > +++ b/drivers/net/i40e/i40e_hash.c > @@ -1105,13 +1105,21 @@ i40e_hash_parse_pattern_act(const struct > rte_eth_dev *dev, > NULL, > "RSS Queues not supported when > pattern specified"); > > - if (rss_act->func =3D=3D > RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ) > + switch (rss_act->func) { > + case RTE_ETH_HASH_FUNCTION_SYMMETRIC_TOEPLITZ: > rss_conf->symmetric_enable =3D true; > - else if (rss_act->func !=3D RTE_ETH_HASH_FUNCTION_DEFAULT) > - return rte_flow_error_set(error, -EINVAL, > - > RTE_FLOW_ERROR_TYPE_ACTION_CONF, > - NULL, > - "Only symmetric TOEPLITZ > supported when pattern specified"); > + break; > + case RTE_ETH_HASH_FUNCTION_DEFAULT: > + case RTE_ETH_HASH_FUNCTION_TOEPLITZ: > + case RTE_ETH_HASH_FUNCTION_SIMPLE_XOR: > + break; > + default: > + return rte_flow_error_set(error, EINVAL, > + RTE_FLOW_ERROR_TYPE_ACTION_CONF, > + NULL, > + "RSS hash function not supported " > + "when pattern specified"); > + } > > if (!i40e_hash_validate_rss_types(rss_act->types)) > return rte_flow_error_set(error, EINVAL, > -- > 2.27.0 Acked-by: Beilei Xing