DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 0/2] qede: 2020-02 minor fixes
@ 2021-02-21 11:16 Igor Russkikh
  2021-02-21 11:16 ` [dpdk-dev] [PATCH 1/2] qede: reduce trace verbosity level Igor Russkikh
  2021-02-21 11:16 ` [dpdk-dev] [PATCH 2/2] qede: make driver accept bigger rss tables Igor Russkikh
  0 siblings, 2 replies; 3+ messages in thread
From: Igor Russkikh @ 2021-02-21 11:16 UTC (permalink / raw)
  To: dev; +Cc: Rasesh Mody, Devendra Singh Rawat, Igor Russkikh

Two small fixes after customer feedback.

Igor Russkikh (2):
  qede: reduce trace verbosity level
  qede: make driver accept bigger rss tables

 drivers/net/qede/base/ecore_int.c | 2 +-
 drivers/net/qede/qede_ethdev.c    | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [dpdk-dev] [PATCH 1/2] qede: reduce trace verbosity level
  2021-02-21 11:16 [dpdk-dev] [PATCH 0/2] qede: 2020-02 minor fixes Igor Russkikh
@ 2021-02-21 11:16 ` Igor Russkikh
  2021-02-21 11:16 ` [dpdk-dev] [PATCH 2/2] qede: make driver accept bigger rss tables Igor Russkikh
  1 sibling, 0 replies; 3+ messages in thread
From: Igor Russkikh @ 2021-02-21 11:16 UTC (permalink / raw)
  To: dev; +Cc: Rasesh Mody, Devendra Singh Rawat, Igor Russkikh, stable

On some hardware units it was found this trace is flooding the output,
making any dpdk interactive usage kind of problematic.

It is only informational, without any consequences handling, so reducing
it to verbose from explicit notice level.

CC: stable@dpdk.org
Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
---
 drivers/net/qede/base/ecore_int.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/qede/base/ecore_int.c b/drivers/net/qede/base/ecore_int.c
index 4207b1853..2c4aac941 100644
--- a/drivers/net/qede/base/ecore_int.c
+++ b/drivers/net/qede/base/ecore_int.c
@@ -928,7 +928,7 @@ static void ecore_int_attn_print(struct ecore_hwfn *p_hwfn,
 				 bool b_clear)
 {
 	/* @DPDK */
-	DP_NOTICE(p_hwfn->p_dev, false, "[block_id %d type %d]\n", id, type);
+	DP_VERBOSE(p_hwfn, ECORE_MSG_INTR, "[block_id %d type %d]\n", id, type);
 }
 
 /**
-- 
2.25.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [dpdk-dev] [PATCH 2/2] qede: make driver accept bigger rss tables
  2021-02-21 11:16 [dpdk-dev] [PATCH 0/2] qede: 2020-02 minor fixes Igor Russkikh
  2021-02-21 11:16 ` [dpdk-dev] [PATCH 1/2] qede: reduce trace verbosity level Igor Russkikh
@ 2021-02-21 11:16 ` Igor Russkikh
  1 sibling, 0 replies; 3+ messages in thread
From: Igor Russkikh @ 2021-02-21 11:16 UTC (permalink / raw)
  To: dev; +Cc: Rasesh Mody, Devendra Singh Rawat, Igor Russkikh, stable

We found some dpdk applications blindly pass fixed side rss hash tables,
and do not check driver/device capabilities.

Moreover, many other drivers do not do such a strong check as well.

So here we fix it, making qede accept any size rss_key. For larger key
tables we just crop it with notice trace message.

CC: stable@dpdk.org
Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
---
 drivers/net/qede/qede_ethdev.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/qede/qede_ethdev.c b/drivers/net/qede/qede_ethdev.c
index ab5f5b106..7363d98f2 100644
--- a/drivers/net/qede/qede_ethdev.c
+++ b/drivers/net/qede/qede_ethdev.c
@@ -2139,8 +2139,8 @@ int qede_rss_hash_update(struct rte_eth_dev *eth_dev,
 		/* RSS hash key */
 		if (key) {
 			if (len > (ECORE_RSS_KEY_SIZE * sizeof(uint32_t))) {
-				DP_ERR(edev, "RSS key length exceeds limit\n");
-				return -EINVAL;
+				len = ECORE_RSS_KEY_SIZE * sizeof(uint32_t);
+				DP_NOTICE(edev, false, "RSS key length exceeds limit\n");
 			}
 			DP_INFO(edev, "Applying user supplied hash key\n");
 			rss_params.update_rss_key = 1;
-- 
2.25.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-02-21 11:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-21 11:16 [dpdk-dev] [PATCH 0/2] qede: 2020-02 minor fixes Igor Russkikh
2021-02-21 11:16 ` [dpdk-dev] [PATCH 1/2] qede: reduce trace verbosity level Igor Russkikh
2021-02-21 11:16 ` [dpdk-dev] [PATCH 2/2] qede: make driver accept bigger rss tables Igor Russkikh

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ http://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git