From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f194.google.com (mail-wr0-f194.google.com [209.85.128.194]) by dpdk.org (Postfix) with ESMTP id 866031B1E6 for ; Mon, 21 May 2018 17:50:25 +0200 (CEST) Received: by mail-wr0-f194.google.com with SMTP id i12-v6so3334179wrc.4 for ; Mon, 21 May 2018 08:50:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=l1m+UUuMNNht5ziig5yz2xx/W+ej83ZbebCOXZPrgZc=; b=tsqPND29rsVkwTPo42eUvrPpXgQlZsmj+hHq5NmeXu7B/zdsIeGO0XyoRqMdyMmJuk T/s38K1ptSFB+v8S5sqZOEsqhYs/TDaHwgUWuEEP3iGwUppEgMRot+5PHw2f7EFEievo afYMQp0oo4olRnoLZWrxm0i6UnMAIY1D0ybALRJL5ctdtRAurZYMcinb45xpJiEr6KT2 e9uuW3PbL3oyRHi+MLLfP0YCUjYTGhsS8UMHFcq2kqJris75QP+rZtH5QeLrXLQe4sYf eWGtPcL0CkMT35Ao6/KvR6RDLn89gz6ZIPPbARG6sXgkvBZdmWLjAcRcp96pALA22s5a M7zw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=l1m+UUuMNNht5ziig5yz2xx/W+ej83ZbebCOXZPrgZc=; b=MRyjLzMWbDfrybpauDGL4CHr1xT+Xo2MtRSwBNZgt6l/66avos81kUamzvpmTdNxnR FgywCVJOMjwiWoSmygwWc+XBLsD4j1goqi6a6cecv6MdNpWSvUPX/7ggu18hqves0OEn HBquUc81Sp13JJwSNqiiAsEfjaaxgiBeR+u3dSaGX7Q7cYz5fdS1ngSDQ2L91kCKPFk0 eGQ4m5wjdH+QrTxx78Qac5nyS06q/Y2V8yxlPBXhu7Q+Jlvj/xORhP5mn3OLuMxPhN1w 1XfZwI36AwV+ON6vp8eBISImcwE07L6nD6d6VE00/fM5NnUxD8cUPAkfv8ovRZKTb3K/ N+uw== X-Gm-Message-State: ALKqPwfKb6fNKYQSsVTFqQ9jsW69pMlr0YfyjftPolQQeGmwafFNadWo TdxSheyta51tQWYsjFWA/ojvHg== X-Google-Smtp-Source: AB8JxZrbkH0HWY4JxKrDsPQfNni8Yvu7c8XKKqPX4UwozXnrtOC95b+DLWCrq81jE7Fm4K2s4aUgSA== X-Received: by 2002:adf:adf0:: with SMTP id w103-v6mr17045448wrc.101.1526917825288; Mon, 21 May 2018 08:50:25 -0700 (PDT) Received: from 6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id 42-v6sm30841386wrx.24.2018.05.21.08.50.24 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 21 May 2018 08:50:24 -0700 (PDT) Date: Mon, 21 May 2018 17:50:09 +0200 From: Adrien Mazarguil To: Shahaf Shuler Cc: dev@dpdk.org Message-ID: <20180521154829.6297-1-adrien.mazarguil@6wind.com> References: <20180515154853.6361-1-adrien.mazarguil@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180515154853.6361-1-adrien.mazarguil@6wind.com> X-Mailer: git-send-email 2.11.0 Subject: [dpdk-dev] [PATCH v2 1/2] net/mlx4: fix inadequate default in RSS converter 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: Mon, 21 May 2018 15:50:25 -0000 Below commit documents 0 as a value standing for a default set of RSS hash types, however the mlx4 PMD doesn't interpret it correctly and still uses its own internal special value for that (-1). Also, its function prototype was not updated. Fixes: ac8d22de2394 ("ethdev: flatten RSS configuration in flow API") Fixes: 1d173da83ef2 ("net/mlx4: fix default RSS hash fields") Signed-off-by: Adrien Mazarguil -- This is a rework of "net/mlx4: fix useless default in RSS converter" [1]. Not a candidate for stable anymore since DPDK 18.02 does not include the flow API RSS rework. [1] http://dpdk.org/ml/archives/dev/2018-May/100285.html --- drivers/net/mlx4/mlx4_flow.c | 6 +++--- drivers/net/mlx4/mlx4_flow.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/mlx4/mlx4_flow.c b/drivers/net/mlx4/mlx4_flow.c index 202779f7d..ebc9eeb8b 100644 --- a/drivers/net/mlx4/mlx4_flow.c +++ b/drivers/net/mlx4/mlx4_flow.c @@ -114,7 +114,7 @@ struct mlx4_drop { * Convert DPDK RSS hash types to their Verbs equivalent. * * This function returns the supported (default) set when @p types has - * special value (uint64_t)-1. + * special value 0. * * @param priv * Pointer to private structure. @@ -160,7 +160,7 @@ mlx4_conv_rss_types(struct priv *priv, uint64_t types) uint64_t conv = 0; unsigned int i; - if (types == (uint64_t)-1) + if (!types) return priv->hw_rss_sup; for (i = 0; i != RTE_DIM(in); ++i) if (types & in[i]) { @@ -1384,7 +1384,7 @@ mlx4_flow_internal(struct priv *priv, struct rte_flow_error *error) struct rte_flow_action_rss action_rss = { .func = RTE_ETH_HASH_FUNCTION_DEFAULT, .level = 0, - .types = -1, + .types = 0, .key_len = MLX4_RSS_HASH_KEY_SIZE, .queue_num = queues, .key = mlx4_rss_hash_key_default, diff --git a/drivers/net/mlx4/mlx4_flow.h b/drivers/net/mlx4/mlx4_flow.h index d1f1611eb..2e82903bd 100644 --- a/drivers/net/mlx4/mlx4_flow.h +++ b/drivers/net/mlx4/mlx4_flow.h @@ -48,7 +48,7 @@ struct rte_flow { /* mlx4_flow.c */ -uint64_t mlx4_conv_rss_types(struct priv *priv, uint64_t rss_hf); +uint64_t mlx4_conv_rss_types(struct priv *priv, uint64_t types); uint64_t mlx4_ibv_to_rss_types(uint64_t ibv_rss_types); int mlx4_flow_sync(struct priv *priv, struct rte_flow_error *error); void mlx4_flow_clean(struct priv *priv); -- 2.11.0