From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-eopbgr00069.outbound.protection.outlook.com [40.107.0.69]) by dpdk.org (Postfix) with ESMTP id B413D2904 for ; Thu, 1 Nov 2018 09:59:11 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Z0zPXkfNGewsEuNdh1ZM68ZbODwQ9023ArxtAIqr8Ew=; b=pWhmp4NyFFkAqfbU6XeTwsNXu+HPzF16rGT7TrzBeN/q9HYzuer8ZnSkhxvltpCj1oNYM8PKptg+wQaeHPvik1li25LYdgySGZJlph9qLp2Dy8/13YoC8QR4Zud+6u8SIwdLapMB28UWN931vMOcXqW+1QGnzeU/mkvMEiDhPGs= Received: from VI1PR0502MB3743.eurprd05.prod.outlook.com (52.134.8.154) by VI1PR0502MB2909.eurprd05.prod.outlook.com (10.175.24.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1294.24; Thu, 1 Nov 2018 08:59:10 +0000 Received: from VI1PR0502MB3743.eurprd05.prod.outlook.com ([fe80::84d3:8812:d45c:98e4]) by VI1PR0502MB3743.eurprd05.prod.outlook.com ([fe80::84d3:8812:d45c:98e4%4]) with mapi id 15.20.1294.023; Thu, 1 Nov 2018 08:59:10 +0000 From: Ophir Munk To: "dev@dpdk.org" , Yongseok Koh CC: Thomas Monjalon , Olga Shern , Ophir Munk , Shahaf Shuler , Asaf Penso Thread-Topic: [PATCH v2] net/mlx5: set RSS key to NULL to indicate default RSS Thread-Index: AQHUccEnLoFSrTYmp0WASea6Fnjxlw== Date: Thu, 1 Nov 2018 08:59:10 +0000 Message-ID: <1541062744-25491-1-git-send-email-ophirmu@mellanox.com> References: <1538588246-21436-1-git-send-email-ophirmu@mellanox.com> In-Reply-To: <1538588246-21436-1-git-send-email-ophirmu@mellanox.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: LO2P265CA0207.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:9e::27) To VI1PR0502MB3743.eurprd05.prod.outlook.com (2603:10a6:803:11::26) x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [37.142.13.130] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; VI1PR0502MB2909; 6:6h7ABHnvb3/0SiRN64ZNv+oqaP4kGoXD5UiqhpIwGaT0ZY1L3pisrSxbSExUnHhywkzcB0/7TTPYu8EneQXqk7etlVP9fGrCD74BLYtYj/RvA2QA64QG7bl8PgI1TrA2vpLmiIcxcDYw9DeGzMyWfxBr+W8OTWC562WCP7lPHjaYudHicuiZB8SOB6NBgaGHHDG63OHy8SDyaIeu5J7eH472wS62uwJxGcE6DkNTImqSM4jDSe4rMGo5n9TNHZtdx5wILaXa4paowxmBrequrT5wHKsj8cxShE9/kiFKNIe3Jzs56UqrHWqgnBVkwH1sKxI6BSGSYLnZSZCtUcPCGLZ+HWmJUp7TYkrpBWehx6C96p0dOaDPDeiYTHrQBWV50ClsiZEO36JbdYAmlbqublUGKYCyOCRyem1CfhBQHwmxB3F/mpOLmbF3PA4ECXSYVrHoJhywPmOrQY43ggYhxg==; 5:2viR1fB6OqwLxAriaWZo96jJJZvbELXsx7BKbTQVQ1wYYrSLlsB1cBkWnMJY1RblE2iDdHWKshKTMa9WxFPGY9nk5yXGy2Mgd84nMiRpbsyrPMu7gUT/qdgen0sb7iIWpJVCWt0EaFt2eSsTDBEjDXlRnisp+NckFe6tIyhn9RU=; 7:fMxls2ng2Wyxgv7eFAfCpTipUq6MPiF5wq3sv5X0k8c3kFJL8ZKRe95vYE5Z6ynp3c14Gs6gbzCbUprquY9vWavFA8BQlR+fEVgaHnyIqPhe7CXUCUCeB9R0WhkGVw7Wy6saAG1CJHIHJE5/jf4QjA== x-ms-office365-filtering-correlation-id: ed5301ba-ce98-4715-b40c-08d63fd84986 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:VI1PR0502MB2909; x-ms-traffictypediagnostic: VI1PR0502MB2909: authentication-results: spf=none (sender IP is ) smtp.mailfrom=ophirmu@mellanox.com; x-ld-processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(278428928389397); x-ms-exchange-senderadcheck: 1 x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3002001)(10201501046)(93006095)(93001095)(3231382)(944501410)(52105095)(6055026)(148016)(149066)(150057)(6041310)(20161123560045)(20161123564045)(20161123562045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(201708071742011)(7699051)(76991095); SRVR:VI1PR0502MB2909; BCL:0; PCL:0; RULEID:; SRVR:VI1PR0502MB2909; x-forefront-prvs: 0843C17679 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(346002)(366004)(39860400002)(376002)(136003)(396003)(199004)(189003)(26005)(186003)(68736007)(36756003)(7736002)(107886003)(6636002)(575784001)(86362001)(2900100001)(66066001)(2501003)(476003)(2616005)(5250100002)(256004)(6512007)(97736004)(53936002)(5660300001)(305945005)(446003)(11346002)(486006)(2906002)(6436002)(25786009)(99286004)(54906003)(105586002)(106356001)(110136005)(102836004)(3846002)(4326008)(6116002)(6486002)(52116002)(8936002)(71200400001)(71190400001)(316002)(76176011)(14454004)(81166006)(81156014)(6506007)(386003)(8676002)(478600001); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR0502MB2909; H:VI1PR0502MB3743.eurprd05.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: T+u1YC15fsBYPAMGT8nT+QwUOJ1ChmR1xO4rSPqouISCTKPgbEfGM4rtGa5XTptjj5F12IX6IaniOraziWnhnHBwXzcDH3OGECsndoTgzPm/6L7ZKdh5tFkXkZlhRtsWk6uE5zrLsiBc0j0PP1V6mVzPsJYimej4SPhOv9irfCPCL16YOVxuwWMfJ9X9B1X4HL6eOsaHmysp9yYbsAz5YJXbsOJzFabKmVoSRkZKOJ4Cfxa252edKibJeJv7y7JwJnLAkrjfQlhgnkhZisrjnPcovv7/655tGKzq2N4w5al0exM5lWowJe1DKIrM+hCNZNTzQP0d/KtIAV9Xl25NuM0yqc6T6dd3Vf5FXLMBwY4= spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-Network-Message-Id: ed5301ba-ce98-4715-b40c-08d63fd84986 X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Nov 2018 08:59:10.1054 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0502MB2909 Subject: [dpdk-dev] [PATCH v2] net/mlx5: set RSS key to NULL to indicate default RSS 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: Thu, 01 Nov 2018 08:59:11 -0000 Applications which add RSS rules must supply an RSS key and length. If an application is only interested in default RSS operation it should not care about the exact RSS key. By setting the key to NULL - the PMD will use the default RSS key. In addition if the application does not care about the RSS type it can set it to 0 and the PMD will use the default type (ETH_RSS_IP). Signed-off-by: Ophir Munk --- doc/guides/nics/mlx5.rst | 1 + drivers/net/mlx5/mlx5_flow.c | 8 +++++++- drivers/net/mlx5/mlx5_flow_dv.c | 8 ++++++-- drivers/net/mlx5/mlx5_flow_verbs.c | 9 +++++++-- drivers/net/mlx5/mlx5_rxq.c | 14 ++++++-------- lib/librte_ethdev/rte_flow.h | 2 +- 6 files changed, 28 insertions(+), 14 deletions(-) diff --git a/doc/guides/nics/mlx5.rst b/doc/guides/nics/mlx5.rst index 6769628..de8f15b 100644 --- a/doc/guides/nics/mlx5.rst +++ b/doc/guides/nics/mlx5.rst @@ -54,6 +54,7 @@ Features - Support for scattered TX and RX frames. - IPv4, IPv6, TCPv4, TCPv6, UDPv4 and UDPv6 RSS on any number of queues. - Several RSS hash keys, one for each flow type. +- NULL RSS key indicates default RSS key - Configurable RETA table. - Support for multiple MAC addresses. - VLAN filtering. diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 280af0a..5742989 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -912,7 +912,13 @@ uint32_t mlx5_flow_adjust_priority(struct rte_eth_dev = *dev, int32_t priority, RTE_FLOW_ERROR_TYPE_ACTION_CONF, &rss->level, "tunnel RSS is not supported"); - if (rss->key_len < MLX5_RSS_HASH_KEY_LEN) + /* allow RSS key_len 0 in case of NULL (default) RSS key */ + if (rss->key_len =3D=3D 0 && rss->key !=3D NULL) + return rte_flow_error_set(error, ENOTSUP, + RTE_FLOW_ERROR_TYPE_ACTION_CONF, + &rss->key_len, + "RSS hash key length 0"); + if (rss->key_len > 0 && rss->key_len < MLX5_RSS_HASH_KEY_LEN) return rte_flow_error_set(error, ENOTSUP, RTE_FLOW_ERROR_TYPE_ACTION_CONF, &rss->key_len, diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_d= v.c index 8f729f4..fd52bf4 100644 --- a/drivers/net/mlx5/mlx5_flow_dv.c +++ b/drivers/net/mlx5/mlx5_flow_dv.c @@ -1056,6 +1056,7 @@ { const struct rte_flow_action_queue *queue; const struct rte_flow_action_rss *rss; + uint8_t *rss_key; int actions_n =3D dev_flow->dv.actions_n; struct rte_flow *flow =3D dev_flow->flow; =20 @@ -1094,8 +1095,11 @@ memcpy((*flow->queue), rss->queue, rss->queue_num * sizeof(uint16_t)); flow->rss.queue_num =3D rss->queue_num; - memcpy(flow->key, rss->key, MLX5_RSS_HASH_KEY_LEN); - flow->rss.types =3D rss->types; + /* NULL RSS key indicates default RSS key */ + rss_key =3D (!rss->key) ? rss_hash_default_key : rss->key; + memcpy(flow->key, rss_key, MLX5_RSS_HASH_KEY_LEN); + /* RSS type 0 indicates default RSS type ETH_RSS_IP */ + flow->rss.types =3D !rss->types ? ETH_RSS_IP : rss->types; flow->rss.level =3D rss->level; /* Added to array only in apply since we need the QP */ flow->actions |=3D MLX5_FLOW_ACTION_RSS; diff --git a/drivers/net/mlx5/mlx5_flow_verbs.c b/drivers/net/mlx5/mlx5_flo= w_verbs.c index 81bc39f..ae9042f 100644 --- a/drivers/net/mlx5/mlx5_flow_verbs.c +++ b/drivers/net/mlx5/mlx5_flow_verbs.c @@ -925,14 +925,19 @@ struct mlx5_flow *dev_flow) { const struct rte_flow_action_rss *rss =3D action->conf; + const uint8_t *rss_key; struct rte_flow *flow =3D dev_flow->flow; =20 if (flow->queue) memcpy((*flow->queue), rss->queue, rss->queue_num * sizeof(uint16_t)); flow->rss.queue_num =3D rss->queue_num; - memcpy(flow->key, rss->key, MLX5_RSS_HASH_KEY_LEN); - flow->rss.types =3D rss->types; + + /* NULL RSS key indicates default RSS key */ + rss_key =3D !rss->key ? rss_hash_default_key : rss->key; + memcpy(flow->key, rss_key, MLX5_RSS_HASH_KEY_LEN); + /* RSS type 0 indicates default RSS type (ETH_RSS_IP) */ + flow->rss.types =3D !rss->types ? ETH_RSS_IP : rss->types; flow->rss.level =3D rss->level; *action_flags |=3D MLX5_FLOW_ACTION_RSS; } diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c index ed993ea..36fc202 100644 --- a/drivers/net/mlx5/mlx5_rxq.c +++ b/drivers/net/mlx5/mlx5_rxq.c @@ -1783,10 +1783,6 @@ struct mlx5_hrxq * rte_errno =3D ENOMEM; return NULL; } - if (!rss_key_len) { - rss_key_len =3D MLX5_RSS_HASH_KEY_LEN; - rss_key =3D rss_hash_default_key; - } #ifdef HAVE_IBV_DEVICE_TUNNEL_SUPPORT qp =3D mlx5_glue->dv_create_qp (priv->ctx, @@ -1798,8 +1794,7 @@ struct mlx5_hrxq * IBV_QP_INIT_ATTR_RX_HASH, .rx_hash_conf =3D (struct ibv_rx_hash_conf){ .rx_hash_function =3D IBV_RX_HASH_FUNC_TOEPLITZ, - .rx_hash_key_len =3D rss_key_len ? rss_key_len : - MLX5_RSS_HASH_KEY_LEN, + .rx_hash_key_len =3D rss_key_len, .rx_hash_key =3D rss_key ? (void *)(uintptr_t)rss_key : rss_hash_default_key, @@ -1824,8 +1819,7 @@ struct mlx5_hrxq * IBV_QP_INIT_ATTR_RX_HASH, .rx_hash_conf =3D (struct ibv_rx_hash_conf){ .rx_hash_function =3D IBV_RX_HASH_FUNC_TOEPLITZ, - .rx_hash_key_len =3D rss_key_len ? rss_key_len : - MLX5_RSS_HASH_KEY_LEN, + .rx_hash_key_len =3D rss_key_len, .rx_hash_key =3D rss_key ? (void *)(uintptr_t)rss_key : rss_hash_default_key, @@ -1888,8 +1882,12 @@ struct mlx5_hrxq * LIST_FOREACH(hrxq, &priv->hrxqs, next) { struct mlx5_ind_table_ibv *ind_tbl; =20 + if (!rss_key_len) + rss_key_len =3D MLX5_RSS_HASH_KEY_LEN; if (hrxq->rss_key_len !=3D rss_key_len) continue; + if (!rss_key) + rss_key =3D rss_hash_default_key; if (memcmp(hrxq->rss_key, rss_key, rss_key_len)) continue; if (hrxq->hash_fields !=3D hash_fields) diff --git a/lib/librte_ethdev/rte_flow.h b/lib/librte_ethdev/rte_flow.h index c0fe879..6cc72ab 100644 --- a/lib/librte_ethdev/rte_flow.h +++ b/lib/librte_ethdev/rte_flow.h @@ -1785,7 +1785,7 @@ struct rte_flow_action_rss { uint64_t types; /**< Specific RSS hash types (see ETH_RSS_*). */ uint32_t key_len; /**< Hash key length in bytes. */ uint32_t queue_num; /**< Number of entries in @p queue. */ - const uint8_t *key; /**< Hash key. */ + const uint8_t *key; /**< Hash key. NULL should indicate default key */ const uint16_t *queue; /**< Queue indices to use. */ }; =20 --=20 1.8.3.1