From patchwork Tue Nov 10 18:04:09 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Amber, Kumar" X-Patchwork-Id: 83915 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 66F46A04DD; Tue, 10 Nov 2020 19:09:34 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 49AA8323C; Tue, 10 Nov 2020 19:09:33 +0100 (CET) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id E83992BAA for ; Tue, 10 Nov 2020 19:09:31 +0100 (CET) IronPort-SDR: nTpy8xqg4AeDi7cI5qLNToJJx6/E9QJHZdIWITkjsyhYyZJOjotPLt6FpSlCTbnvbmOS0ZZM8l +NUV7H0dBMJA== X-IronPort-AV: E=McAfee;i="6000,8403,9801"; a="170131586" X-IronPort-AV: E=Sophos;i="5.77,467,1596524400"; d="scan'208";a="170131586" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Nov 2020 10:09:28 -0800 IronPort-SDR: 3rksX/aq8ZbwGq54ftKVad3t+OK8IfFi7Juk5pHDYphqC560LJ+VqUssOPmj7BfCafGdU5vTVB kaHAdmzPotHA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,467,1596524400"; d="scan'208";a="322967705" Received: from unknown (HELO localhost.localdomain) ([10.190.212.185]) by orsmga003.jf.intel.com with ESMTP; 10 Nov 2020 10:09:27 -0800 From: Kumar Amber To: dev@dpdk.org Date: Tue, 10 Nov 2020 23:34:09 +0530 Message-Id: <20201110180409.42133-1-kumar.amber@intel.com> X-Mailer: git-send-email 2.17.1 Subject: [dpdk-dev] [PATCH v1] net/i40e: argument in RSS action should have queue 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" The driver must check for the queue number in the RSS action list and if not should return with a proper error message to user. Signed-off-by: Kumar Amber --- drivers/net/i40e/i40e_flow.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/drivers/net/i40e/i40e_flow.c b/drivers/net/i40e/i40e_flow.c index 5bec0c7a84..975340cb1a 100644 --- a/drivers/net/i40e/i40e_flow.c +++ b/drivers/net/i40e/i40e_flow.c @@ -4917,6 +4917,17 @@ i40e_flow_parse_rss_action(struct rte_eth_dev *dev, NEXT_ITEM_OF_ACTION(act, actions, index); rss = act->conf; + /** + * Check if Queue number is specified + * in argument else throw an error. + */ + if (rss->queue == NULL) { + rte_flow_error_set(error, EINVAL, + RTE_FLOW_ERROR_TYPE_ACTION, + act, "Queue numbers not given."); + return -rte_errno; + } + /** * RSS only supports forwarding, * check if the first not void action is RSS.