From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id 32095C5CFDB for ; Fri, 14 Aug 2026 12:54:56 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8E91B402D1; Fri, 14 Aug 2026 14:54:50 +0200 (CEST) Received: from agw.arknetworks.am (agw.arknetworks.am [79.141.165.80]) by mails.dpdk.org (Postfix) with ESMTP id F159E4013F; Fri, 14 Aug 2026 14:54:47 +0200 (CEST) Received: from localhost.localdomain (unknown [78.109.78.39]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by agw.arknetworks.am (Postfix) with ESMTPSA id 2DB7BE0ADB; Fri, 14 Aug 2026 16:54:47 +0400 (+04) DKIM-Filter: OpenDKIM Filter v2.11.0 agw.arknetworks.am 2DB7BE0ADB DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arknetworks.am; s=default; t=1786712087; bh=Hz7B+ADWUhmpSvHSyBoSxqIpGRQSLLIOMP5WMZr1FVY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=kE3bT7vXqO6WIKhsYYtsQwh9SRfq+C46oKsp6A8jbnkrpSAdwtEZPScV5c83H1DUq TFX2/NMMSBzCBGUzSc2C2Ic+4HDNVPwAos6Tf9Wl3jctiRtZ78qSDdg7aJVFaJdU+0 Q7pUnWQ7N/jNKLyVV5vheB+iUYXeDhtC1br/WXAuihGkaKO+5u7Y2kOQHLqz0K6W1I sN8m1n2phSmN2w0u44H0lPL//r/cQaTfXZm+d3ya5ghM1WlpRtrM9XBqWP3i/v4K31 EQkRUAgHW7lkLntKDDsfYY+Qawy6CvPDBQlOL8WdVLv521vmgPbHJASVcbExMahyu6 6DKey3o5ytydA== From: Ivan Malov To: dev@dpdk.org Cc: Andy Moreton , Viacheslav Galaktionov , Roman Zhukov , Pieter Jansen van Vuuren , Stephen Hemminger , Andrew Rybchenko , stable@dpdk.org Subject: [PATCH v3 01/14] common/sfc_efx/base: reduce stack in RSS context table write Date: Fri, 14 Aug 2026 16:54:25 +0400 Message-ID: <20260814125438.13283-2-ivan.malov@arknetworks.am> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260814125438.13283-1-ivan.malov@arknetworks.am> References: <20260811174821.8930-1-ivan.malov@arknetworks.am> <20260814125438.13283-1-ivan.malov@arknetworks.am> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 From: Andy Moreton Code analysis reports an error for excessive stack consumption (over 1KB). Use a heap allocated payload buffer instead. Fixes: e7ea5f304f0f ("common/sfc_efx/base: support selecting RSS table entry count") Cc: stable@dpdk.org Signed-off-by: Andy Moreton Reviewed-by: Ivan Malov Reviewed-by: Viacheslav Galaktionov --- drivers/common/sfc_efx/base/ef10_rx.c | 23 ++++++++++++++++++----- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/drivers/common/sfc_efx/base/ef10_rx.c b/drivers/common/sfc_efx/base/ef10_rx.c index afc9cf025f..bb38da991e 100644 --- a/drivers/common/sfc_efx/base/ef10_rx.c +++ b/drivers/common/sfc_efx/base/ef10_rx.c @@ -394,11 +394,10 @@ efx_mcdi_rss_context_write_table( __in unsigned int nentries) { const efx_nic_cfg_t *encp = efx_nic_cfg_get(enp); + uint8_t *payload = NULL; efx_mcdi_req_t req; - EFX_MCDI_DECLARE_BUF(payload, - MC_CMD_RSS_CONTEXT_WRITE_TABLE_IN_LENMAX_MCDI2, - MC_CMD_RSS_CONTEXT_WRITE_TABLE_OUT_LEN); unsigned int i; + size_t size; int rc; if (nentries > @@ -413,6 +412,16 @@ efx_mcdi_rss_context_write_table( goto fail2; } + size = EFX_MCDI_BUF_SIZE( + MC_CMD_RSS_CONTEXT_WRITE_TABLE_IN_LEN(nentries), + MC_CMD_RSS_CONTEXT_WRITE_TABLE_OUT_LEN); + + EFSYS_KMEM_ALLOC(enp->en_esip, size, payload); + if (payload == NULL) { + rc = ENOMEM; + goto fail3; + } + req.emr_cmd = MC_CMD_RSS_CONTEXT_WRITE_TABLE; req.emr_in_buf = payload; req.emr_in_length = MC_CMD_RSS_CONTEXT_WRITE_TABLE_IN_LEN(nentries); @@ -425,7 +434,7 @@ efx_mcdi_rss_context_write_table( for (i = 0; i < nentries; ++i) { if (table[i] >= encp->enc_rx_scale_indirection_max_nqueues) { rc = EINVAL; - goto fail3; + goto fail4; } MCDI_IN_POPULATE_INDEXED_DWORD_2(req, @@ -437,13 +446,17 @@ efx_mcdi_rss_context_write_table( efx_mcdi_execute(enp, &req); if (req.emr_rc != 0) { rc = req.emr_rc; - goto fail4; + goto fail5; } + EFSYS_KMEM_FREE(enp->en_esip, size, payload); return (0); +fail5: + EFSYS_PROBE(fail5); fail4: EFSYS_PROBE(fail4); + EFSYS_KMEM_FREE(enp->en_esip, size, payload); fail3: EFSYS_PROBE(fail3); fail2: -- 2.47.3