From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3E6744893F; Wed, 15 Oct 2025 09:09:54 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 29F2840651; Wed, 15 Oct 2025 09:09:54 +0200 (CEST) Received: from pdx-out-007.esa.us-west-2.outbound.mail-perimeter.amazon.com (pdx-out-007.esa.us-west-2.outbound.mail-perimeter.amazon.com [52.34.181.151]) by mails.dpdk.org (Postfix) with ESMTP id E512840615 for ; Wed, 15 Oct 2025 09:09:51 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amazon.com; i=@amazon.com; q=dns/txt; s=amazoncorp2; t=1760512192; x=1792048192; h=from:to:cc:subject:date:message-id:mime-version; bh=I7dr1fP8oTQm2QuYPYryA6cnYlJTJDzbinDWYq4aQ+o=; b=rtOiN7OrV1RXtpo4IDqXjhdCCCMrFs5bRPzQUXY4ZRiOOdGXNt/MfMKY JbckwEJ9cY3lnFXGpXFSPfQsRfMNEA0np5mpaFjOh/4zuxjm/KzJfER0R akWnolcaA6wtDfAY7KqG70xAxTtlmiOdMxnXIUCrREMq9/oaHmtK1kjC1 /PtH8RqYGiK959Jx15wwBK4Y/TdoVEHfDsUi0IUTgiQeD15ctacSM+HG5 77jkzEJvi5FY9TKbnf8B5HvSa0hnuNQP1AbixpkEA/E8fX75Csan0NOq9 i9LuNnZD+8ApWGETSd5CGC54MWwMOZDgYZyPARBDTNv4mq0Er+XsQQvrW Q==; X-CSE-ConnectionGUID: QI969hMLTCGaAX76PtavHA== X-CSE-MsgGUID: Gowgw0rTQky1vcDEgamQaA== X-IronPort-AV: E=Sophos;i="6.18,263,1751241600"; d="scan'208";a="4917059" Received: from ip-10-5-6-203.us-west-2.compute.internal (HELO smtpout.naws.us-west-2.prod.farcaster.email.amazon.dev) ([10.5.6.203]) by internal-pdx-out-007.esa.us-west-2.outbound.mail-perimeter.amazon.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Oct 2025 07:09:49 +0000 Received: from EX19MTAUWB002.ant.amazon.com [205.251.233.111:23548] by smtpin.naws.us-west-2.prod.farcaster.email.amazon.dev [10.0.35.225:2525] with esmtp (Farcaster) id c289e2e5-66f4-45ae-90d9-83b8221cf497; Wed, 15 Oct 2025 07:09:49 +0000 (UTC) X-Farcaster-Flow-ID: c289e2e5-66f4-45ae-90d9-83b8221cf497 Received: from EX19D001UWA001.ant.amazon.com (10.13.138.214) by EX19MTAUWB002.ant.amazon.com (10.250.64.231) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA) id 15.2.2562.20; Wed, 15 Oct 2025 07:09:47 +0000 Received: from HFA15-CG15235BS.amazon.com (10.1.213.14) by EX19D001UWA001.ant.amazon.com (10.13.138.214) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA) id 15.2.2562.20; Wed, 15 Oct 2025 07:09:46 +0000 From: Shai Brandes To: CC: , Shai Brandes Subject: [PATCH 06/21] net/ena/base: add Rx cdesc dump on bad request ID Date: Wed, 15 Oct 2025 10:09:32 +0300 Message-ID: <20251015070932.504-1-shaibran@amazon.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [10.1.213.14] X-ClientProxiedBy: EX19D043UWC003.ant.amazon.com (10.13.139.240) To EX19D001UWA001.ant.amazon.com (10.13.138.214) X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org This patch moves the check of bad req_id from ena_com_rx_pkt() to ena_com_cdesc_rx_pkt_get(), such that on bad req_id the head will point to it and later the descriptor will be dumped. Signed-off-by: Shai Brandes Reviewed-by: Amit Bernstein Reviewed-by: Yosef Raisman --- drivers/net/ena/base/ena_eth_com.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/drivers/net/ena/base/ena_eth_com.c b/drivers/net/ena/base/ena_eth_com.c index fd98e5ae67..212adc7d1f 100644 --- a/drivers/net/ena/base/ena_eth_com.c +++ b/drivers/net/ena/base/ena_eth_com.c @@ -322,6 +322,13 @@ static int ena_com_cdesc_rx_pkt_get(struct ena_com_io_cq *io_cq, return ENA_COM_FAULT; } + if (unlikely(cdesc->base.req_id >= io_cq->q_depth)) { + ena_trc_err(dev, + "Bad req_id in descriptor #%u on q_id: %u, req_id: %u\n", + count, io_cq->qid, cdesc->base.req_id); + return ENA_COM_EIO; + } + ena_com_cq_inc_head(io_cq); count++; last = ENA_FIELD_GET(status, @@ -624,7 +631,6 @@ int ena_com_rx_pkt(struct ena_com_io_cq *io_cq, { struct ena_com_rx_buf_info *ena_buf = &ena_rx_ctx->ena_bufs[0]; struct ena_eth_io_rx_cdesc_ext *cdesc = NULL; - u16 q_depth = io_cq->q_depth; u16 cdesc_idx = 0; u16 nb_hw_desc; u16 i = 0; @@ -635,7 +641,7 @@ int ena_com_rx_pkt(struct ena_com_io_cq *io_cq, rc = ena_com_cdesc_rx_pkt_get(io_cq, &cdesc_idx, &nb_hw_desc); if (unlikely(rc != ENA_COM_OK)) - return ENA_COM_FAULT; + return rc; if (nb_hw_desc == 0) { ena_rx_ctx->descs = nb_hw_desc; @@ -659,8 +665,6 @@ int ena_com_rx_pkt(struct ena_com_io_cq *io_cq, do { ena_buf[i].len = cdesc->base.length; ena_buf[i].req_id = cdesc->base.req_id; - if (unlikely(ena_buf[i].req_id >= q_depth)) - return ENA_COM_EIO; if (++i >= nb_hw_desc) break; -- 2.17.1