From: <shaibran@amazon.com>
To: <ferruh.yigit@amd.com>
Cc: <dev@dpdk.org>, Shai Brandes <shaibran@amazon.com>
Subject: [PATCH 2/3] net/ena: support malformed Rx descriptor error
Date: Wed, 23 Oct 2024 12:41:18 +0300 [thread overview]
Message-ID: <20241023094119.4299-3-shaibran@amazon.com> (raw)
In-Reply-To: <20241023094119.4299-1-shaibran@amazon.com>
From: Shai Brandes <shaibran@amazon.com>
Add validation check for packet descriptor consistency
and improve reset logic and stats tracking.
Signed-off-by: Shai Brandes <shaibran@amazon.com>
Reviewed-by: Amit Bernstein <amitbern@amazon.com>
---
doc/guides/rel_notes/release_24_11.rst | 1 +
drivers/net/ena/ena_ethdev.c | 29 +++++++++++++++++++-------
drivers/net/ena/ena_ethdev.h | 2 ++
3 files changed, 25 insertions(+), 7 deletions(-)
diff --git a/doc/guides/rel_notes/release_24_11.rst b/doc/guides/rel_notes/release_24_11.rst
index fa4822d928..e94564b8b7 100644
--- a/doc/guides/rel_notes/release_24_11.rst
+++ b/doc/guides/rel_notes/release_24_11.rst
@@ -142,6 +142,7 @@ New Features
* Modified the PMD API that controls the LLQ header policy.
* Replaced ``enable_llq``, ``normal_llq_hdr`` and ``large_llq_hdr`` devargs
with a new shared devarg ``llq_policy`` that keeps the same logic.
+ * Added validation check for Rx packet descriptor consistency.
* **Updated Cisco enic driver.**
diff --git a/drivers/net/ena/ena_ethdev.c b/drivers/net/ena/ena_ethdev.c
index 1a652b09ac..bba48ea1cc 100644
--- a/drivers/net/ena/ena_ethdev.c
+++ b/drivers/net/ena/ena_ethdev.c
@@ -162,6 +162,8 @@ static const struct ena_stats ena_stats_rx_strings[] = {
ENA_STAT_RX_ENTRY(mbuf_alloc_fail),
ENA_STAT_RX_ENTRY(bad_desc_num),
ENA_STAT_RX_ENTRY(bad_req_id),
+ ENA_STAT_RX_ENTRY(bad_desc),
+ ENA_STAT_RX_ENTRY(unknown_error),
};
#define ENA_STATS_ARRAY_GLOBAL ARRAY_SIZE(ena_stats_global_strings)
@@ -1262,7 +1264,9 @@ static int ena_stats_get(struct rte_eth_dev *dev,
stats->q_ibytes[i] = rx_stats->bytes;
stats->q_ipackets[i] = rx_stats->cnt;
stats->q_errors[i] = rx_stats->bad_desc_num +
- rx_stats->bad_req_id;
+ rx_stats->bad_req_id +
+ rx_stats->bad_desc +
+ rx_stats->unknown_error;
}
max_rings_stats = RTE_MIN(dev->data->nb_tx_queues,
@@ -2772,6 +2776,7 @@ static uint16_t eth_ena_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
struct ena_ring *rx_ring = (struct ena_ring *)(rx_queue);
unsigned int free_queue_entries;
uint16_t next_to_clean = rx_ring->next_to_clean;
+ enum ena_regs_reset_reason_types reset_reason;
uint16_t descs_in_use;
struct rte_mbuf *mbuf;
uint16_t completed;
@@ -2804,15 +2809,25 @@ static uint16_t eth_ena_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
PMD_RX_LOG_LINE(ERR,
"Failed to get the packet from the device, rc: %d",
rc);
- if (rc == ENA_COM_NO_SPACE) {
+ switch (rc) {
+ case ENA_COM_NO_SPACE:
++rx_ring->rx_stats.bad_desc_num;
- ena_trigger_reset(rx_ring->adapter,
- ENA_REGS_RESET_TOO_MANY_RX_DESCS);
- } else {
+ reset_reason = ENA_REGS_RESET_TOO_MANY_RX_DESCS;
+ break;
+ case ENA_COM_FAULT:
+ ++rx_ring->rx_stats.bad_desc;
+ reset_reason = ENA_REGS_RESET_RX_DESCRIPTOR_MALFORMED;
+ break;
+ case ENA_COM_EIO:
++rx_ring->rx_stats.bad_req_id;
- ena_trigger_reset(rx_ring->adapter,
- ENA_REGS_RESET_INV_RX_REQ_ID);
+ reset_reason = ENA_REGS_RESET_INV_RX_REQ_ID;
+ break;
+ default:
+ ++rx_ring->rx_stats.unknown_error;
+ reset_reason = ENA_REGS_RESET_DRIVER_INVALID_STATE;
+ break;
}
+ ena_trigger_reset(rx_ring->adapter, reset_reason);
return 0;
}
diff --git a/drivers/net/ena/ena_ethdev.h b/drivers/net/ena/ena_ethdev.h
index fe7d4a2d65..b8aead8f46 100644
--- a/drivers/net/ena/ena_ethdev.h
+++ b/drivers/net/ena/ena_ethdev.h
@@ -140,6 +140,8 @@ struct ena_stats_rx {
u64 mbuf_alloc_fail;
u64 bad_desc_num;
u64 bad_req_id;
+ u64 bad_desc;
+ u64 unknown_error;
};
struct __rte_cache_aligned ena_ring {
--
2.17.1
next prev parent reply other threads:[~2024-10-23 9:41 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-23 9:41 [PATCH 0/3] net/ena: ena release 2.11.0 shaibran
2024-10-23 9:41 ` [PATCH 1/3] net/ena: upgrade driver version to 2.11.0 shaibran
2024-10-23 9:41 ` shaibran [this message]
2024-10-23 9:41 ` [PATCH 3/3] net/ena: remove outdated devarg documentation shaibran
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241023094119.4299-3-shaibran@amazon.com \
--to=shaibran@amazon.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@amd.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).