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 6FA25A04B5; Wed, 30 Sep 2020 15:37:04 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 6B5731D6DE; Wed, 30 Sep 2020 15:37:02 +0200 (CEST) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 2EF4A1D669 for ; Wed, 30 Sep 2020 15:37:00 +0200 (CEST) IronPort-SDR: e9e8Lm/yoIPRWm/LFkEqUVRSBSsvuPAmbZsERxv9LZ9TbDut+CC6nsvEBGaUDOedq8XymWQ5J5 34HVyT1ZCvFA== X-IronPort-AV: E=McAfee;i="6000,8403,9759"; a="226594255" X-IronPort-AV: E=Sophos;i="5.77,322,1596524400"; d="scan'208";a="226594255" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Sep 2020 06:36:58 -0700 IronPort-SDR: 6MG6Lm4HeTcN2yhv+eKA61djUVaPWD2AyzpNnNU3Y0oK9e3Il3JAvOV99042mywJDszFm+3ZkE 8lXbldfu+2dQ== X-IronPort-AV: E=Sophos;i="5.77,322,1596524400"; d="scan'208";a="312594484" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.229.39]) ([10.213.229.39]) by orsmga006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Sep 2020 06:36:56 -0700 To: Lijun Ou , wenzhuo.lu@intel.com, beilei.xing@intel.com, adrien.mazarguil@6wind.com Cc: dev@dpdk.org, linuxarm@huawei.com References: <1599702678-11142-1-git-send-email-oulijun@huawei.com> <1600955105-53176-1-git-send-email-oulijun@huawei.com> <1600955105-53176-2-git-send-email-oulijun@huawei.com> From: Ferruh Yigit Message-ID: Date: Wed, 30 Sep 2020 14:36:53 +0100 MIME-Version: 1.0 In-Reply-To: <1600955105-53176-2-git-send-email-oulijun@huawei.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v4] app/testpmd: fix the default RSS key configuration 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" On 9/24/2020 2:45 PM, Lijun Ou wrote: > It use the NIC valid default RSS key instead of the testpmd > dummy RSS key in the flow configuration when the RSS key is > not specified in the flow rule. If the NIC RSS key is > invalid, it will use testpmd dummy RSS key as the default > key. > > Fixes: ac8d22de2394 ("ethdev: flatten RSS configuration in flow API") > Cc: stable@dpdk.org > > Signed-off-by: Lijun Ou > --- > V3->V4: > -fix checkpatch warning and shorter commit content. > > V2->V3: > -fix checkpatch warning. > > V1->V2: > -fix the commit. > --- > app/test-pmd/cmdline_flow.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c > index 6263d30..e6648da 100644 > --- a/app/test-pmd/cmdline_flow.c > +++ b/app/test-pmd/cmdline_flow.c > @@ -4312,6 +4312,7 @@ parse_vc_action_rss(struct context *ctx, const struct token *token, > action_rss_data->queue[i] = i; > if (!port_id_is_invalid(ctx->port, DISABLED_WARN) && > ctx->port != (portid_t)RTE_PORT_ALL) { > + struct rte_eth_rss_conf rss_conf = {0}; > struct rte_eth_dev_info info; > int ret2; > > @@ -4322,6 +4323,13 @@ parse_vc_action_rss(struct context *ctx, const struct token *token, > action_rss_data->conf.key_len = > RTE_MIN(sizeof(action_rss_data->key), > info.hash_key_size); > + > + rss_conf.rss_key_len = sizeof(action_rss_data->key); > + rss_conf.rss_key = action_rss_data->key; > + ret2 = rte_eth_dev_rss_hash_conf_get(ctx->port, &rss_conf); > + if (ret2 != 0) > + return ret2; > + action_rss_data->conf.key = rss_conf.rss_key; Do you need this last assignment at all? 'rss_conf.rss_key' point to 'action_rss_data->key' 'action_rss_data->conf.key' already point to 'action_rss_data->key' so it just assigns same value back, no?