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 88BD3489EF for ; Mon, 27 Oct 2025 17:21:35 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 810EA40668; Mon, 27 Oct 2025 17:21:35 +0100 (CET) Received: from mail-wr1-f54.google.com (mail-wr1-f54.google.com [209.85.221.54]) by mails.dpdk.org (Postfix) with ESMTP id 8EA0B40662 for ; Mon, 27 Oct 2025 17:21:34 +0100 (CET) Received: by mail-wr1-f54.google.com with SMTP id ffacd0b85a97d-4270a3464caso2334000f8f.1 for ; Mon, 27 Oct 2025 09:21:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1761582094; x=1762186894; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=4dIZ+UBSgwfq9dX+bRTxcAdrSh4LhF7l+vi/w+rHccA=; b=AegeFWmxmTs6jPhUnrmboqrmXUbP4o68r+yJXGENWo2sfoyor2ihT/PgnoZG1VCp6v 5cvsegEj1dZPboHcmjolVm/C5MPoR2puSPsZMX/CLEdneLhs6fkKXSY2I3XOOO/Grmq6 LHC/acmwt+A3zSiFtOFcRQNsJUgPB6AvZKye/7gvmZVsgu5xO6NHHbfTD1n6TKuIhbsj W/o5h7MB0midCVoIM/vLG2xU+teCUH8Ro5dbG3yJiKlj1Puwd01j1epu2E+KuYVPS4dX Ugf/YIoHt/QfohQ7EhfScXUKrgaLb7epMWDdZ9yCCHnemZ3+2SRV8XxJsgefWyvZV54W 24Ag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761582094; x=1762186894; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=4dIZ+UBSgwfq9dX+bRTxcAdrSh4LhF7l+vi/w+rHccA=; b=Pa7Z8KetvWPOPPNgBI8Vbi4VfL4xAo3cPO0M4mVgE15nzUMo/58IBGT0UOy1+bbOlU +cb12hLp1/cBQXUaUWHMJlAZfSDEBTa7Dep3726BRgIK+l1zIWHIdb4PLrFBGVAaBBTM qDQWjLEIKekQgpPO+KztZJwDXhhWZMsXjXHdwnlj0J6GUEwZwkHW28OZX4zMaqOZcGZt gs7x0v4dzY7QH8Q1z+8ks1g+lmoUbCvomFT+KMSiHqrPfE3wzaWXdu7YvnMtXmE+Bbui sALp3pSatWIjc/5PvWJEPNvE0ZQSRyNvhoo+myxVAQpToIRhRoSIZTvGWpX4cERAkRW3 0TDQ== X-Forwarded-Encrypted: i=1; AJvYcCU31c2uDKqt6g53Mj2L+At3ohY5lBV3S9fOYZMvzJIfxYWlX70S+p3LbwpfNXQ2TdZoZmUoxtc=@dpdk.org X-Gm-Message-State: AOJu0Ywslux+hZ7zVNPpit6SV5aJBNFN6BClI8NJo/mc70reomt5p/M/ hi88ArAHQ6/26TCucFxuIDJp8Ki3XkzThrPaWmDtIpkGVmx7aKXo05lp X-Gm-Gg: ASbGnctu9sehNPCn1gfSBAa3aDZ2GZxQTItI8eM1/ddl1rmm+DMmYI1qoZHsWnKd2SP epHAbHTtTZRrTJKMoiXR/QQfHlDkWvCKtTtTxBCAlR/6mlGX9Pp4+9iD2p77RNNH29jaC6KTPP8 afmDNABNbAakzVbYNb+2YxociEsm9ErkcWDeYiTr9VhEBgmNRH2hliYHtxgEDKJgWmUzxhKq4dM RBt/NRrzJQ0XRzlRR44DRrA2tDyX7vYD5IFk5T9U20svadUSiDiRt7x23r6DncNID+BSm/r8nVN UJTUTf5AzfIZEhAub7b+yOGoj4TsHLWCK5HoTjKcrMYXMS5FzSGI3Xt+ar/gDqCEoFxNgO3nSNr 09x4a4QQOR2VpZW/8mirOE9MK58q5PLWysbVQPAYOhsxyIQt6D5UDfcuoMvIRMzVIdJnQZFXcKV 5ZruHYSXH4zzCYIT1F X-Google-Smtp-Source: AGHT+IGRyRXqcoF0uMTURd0L6Grk8ztNuCcx51v1zv2ticFmAsMeBiIMsGE15iZjyB+uycgYSNx78A== X-Received: by 2002:a05:6000:250d:b0:427:7ac:528f with SMTP id ffacd0b85a97d-429a7e75f8emr277508f8f.33.1761582093966; Mon, 27 Oct 2025 09:21:33 -0700 (PDT) Received: from localhost ([2a01:4b00:d036:ae00:6fc5:c3bc:147e:832c]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-429952d9e80sm15145993f8f.28.2025.10.27.09.21.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 27 Oct 2025 09:21:33 -0700 (PDT) From: luca.boccassi@gmail.com To: Shani Peretz Cc: Anatoly Burakov , dpdk stable Subject: patch 'eal: fix DMA mask validation with IOVA mode option' has been queued to stable release 22.11.11 Date: Mon, 27 Oct 2025 16:19:06 +0000 Message-ID: <20251027162001.3710450-28-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20251027162001.3710450-1-luca.boccassi@gmail.com> References: <20251027162001.3710450-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Hi, FYI, your patch has been queued to stable release 22.11.11 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 10/29/25. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/0980930c32a05807f48c2cace72e5e2e5dabf7a7 Thanks. Luca Boccassi --- >From 0980930c32a05807f48c2cace72e5e2e5dabf7a7 Mon Sep 17 00:00:00 2001 From: Shani Peretz Date: Thu, 18 Sep 2025 09:47:54 +0300 Subject: [PATCH] eal: fix DMA mask validation with IOVA mode option [ upstream commit e37ff4ef296f33e4c3a0e9306241c4f8fcae5061 ] When --iova-mode is explicitly specified in command line, DMA mask constraints were not being validated, leading to potential runtime failures when device DMA capabilities are exceeded. The issue occurred because rte_bus_get_iommu_class() was only called during IOVA mode auto-detection, but this function has the important side effect of triggering DMA mask detection (e.g., Intel IOMMU address width checking via pci_device_iommu_support_va()). This created an inconsistency, when choosing explicit mode, the DMA checks are bypassed, but when choosing auto-detection mode, the constraints are checked and enforced. The fix moves rte_bus_get_iommu_class() outside the conditional logic to ensure it's always called during EAL initialization. Fixes: 4374ebc24bc1 ("malloc: modify error message for DMA mask check") Signed-off-by: Shani Peretz Acked-by: Anatoly Burakov --- lib/eal/freebsd/eal.c | 6 +++++- lib/eal/linux/eal.c | 5 ++++- lib/eal/windows/eal.c | 5 ++++- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/lib/eal/freebsd/eal.c b/lib/eal/freebsd/eal.c index 008a6c68e4..aa9f85c647 100644 --- a/lib/eal/freebsd/eal.c +++ b/lib/eal/freebsd/eal.c @@ -692,6 +692,10 @@ rte_eal_init(int argc, char **argv) * with a message describing the cause. */ has_phys_addr = internal_conf->no_hugetlbfs == 0; + + /* Always call rte_bus_get_iommu_class() to trigger DMA mask detection and validation */ + enum rte_iova_mode bus_iova_mode = rte_bus_get_iommu_class(); + iova_mode = internal_conf->iova_mode; if (iova_mode == RTE_IOVA_PA && !has_phys_addr) { rte_eal_init_alert("Cannot use IOVA as 'PA' since physical addresses are not available"); @@ -702,7 +706,7 @@ rte_eal_init(int argc, char **argv) RTE_LOG(DEBUG, EAL, "Specific IOVA mode is not requested, autodetecting\n"); if (has_phys_addr) { RTE_LOG(DEBUG, EAL, "Selecting IOVA mode according to bus requests\n"); - iova_mode = rte_bus_get_iommu_class(); + iova_mode = bus_iova_mode; if (iova_mode == RTE_IOVA_DC) iova_mode = RTE_IOVA_PA; } else { diff --git a/lib/eal/linux/eal.c b/lib/eal/linux/eal.c index f6ac970ed9..9f9a03bcf7 100644 --- a/lib/eal/linux/eal.c +++ b/lib/eal/linux/eal.c @@ -1065,10 +1065,13 @@ rte_eal_init(int argc, char **argv) phys_addrs = rte_eal_using_phys_addrs() != 0; + /* Always call rte_bus_get_iommu_class() to trigger DMA mask detection and validation */ + enum rte_iova_mode bus_iova_mode = rte_bus_get_iommu_class(); + /* if no EAL option "--iova-mode=", use bus IOVA scheme */ if (internal_conf->iova_mode == RTE_IOVA_DC) { /* autodetect the IOVA mapping mode */ - enum rte_iova_mode iova_mode = rte_bus_get_iommu_class(); + enum rte_iova_mode iova_mode = bus_iova_mode; if (iova_mode == RTE_IOVA_DC) { RTE_LOG(DEBUG, EAL, "Buses did not request a specific IOVA mode.\n"); diff --git a/lib/eal/windows/eal.c b/lib/eal/windows/eal.c index 56fadc7afe..3a459172f9 100644 --- a/lib/eal/windows/eal.c +++ b/lib/eal/windows/eal.c @@ -359,6 +359,9 @@ rte_eal_init(int argc, char **argv) has_phys_addr = false; } + /* Always call rte_bus_get_iommu_class() to trigger DMA mask detection and validation */ + enum rte_iova_mode bus_iova_mode = rte_bus_get_iommu_class(); + iova_mode = internal_conf->iova_mode; if (iova_mode == RTE_IOVA_PA && !has_phys_addr) { rte_eal_init_alert("Cannot use IOVA as 'PA' since physical addresses are not available"); @@ -369,7 +372,7 @@ rte_eal_init(int argc, char **argv) RTE_LOG(DEBUG, EAL, "Specific IOVA mode is not requested, autodetecting\n"); if (has_phys_addr) { RTE_LOG(DEBUG, EAL, "Selecting IOVA mode according to bus requests\n"); - iova_mode = rte_bus_get_iommu_class(); + iova_mode = bus_iova_mode; if (iova_mode == RTE_IOVA_DC) iova_mode = RTE_IOVA_PA; } else { -- 2.47.3 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2025-10-27 15:54:35.909480984 +0000 +++ 0028-eal-fix-DMA-mask-validation-with-IOVA-mode-option.patch 2025-10-27 15:54:34.803949751 +0000 @@ -1 +1 @@ -From e37ff4ef296f33e4c3a0e9306241c4f8fcae5061 Mon Sep 17 00:00:00 2001 +From 0980930c32a05807f48c2cace72e5e2e5dabf7a7 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit e37ff4ef296f33e4c3a0e9306241c4f8fcae5061 ] + @@ -23 +24,0 @@ -Cc: stable@dpdk.org @@ -34 +35 @@ -index c1ab8d86d2..0f957919d3 100644 +index 008a6c68e4..aa9f85c647 100644 @@ -37 +38 @@ -@@ -670,12 +670,16 @@ rte_eal_init(int argc, char **argv) +@@ -692,6 +692,10 @@ rte_eal_init(int argc, char **argv) @@ -46,2 +47,4 @@ - if (iova_mode == RTE_IOVA_DC) { - EAL_LOG(DEBUG, "Specific IOVA mode is not requested, autodetecting"); + if (iova_mode == RTE_IOVA_PA && !has_phys_addr) { + rte_eal_init_alert("Cannot use IOVA as 'PA' since physical addresses are not available"); +@@ -702,7 +706,7 @@ rte_eal_init(int argc, char **argv) + RTE_LOG(DEBUG, EAL, "Specific IOVA mode is not requested, autodetecting\n"); @@ -49 +52 @@ - EAL_LOG(DEBUG, "Selecting IOVA mode according to bus requests"); + RTE_LOG(DEBUG, EAL, "Selecting IOVA mode according to bus requests\n"); @@ -52,3 +55,3 @@ - if (iova_mode == RTE_IOVA_DC) { - if (!RTE_IOVA_IN_MBUF) { - iova_mode = RTE_IOVA_VA; + if (iova_mode == RTE_IOVA_DC) + iova_mode = RTE_IOVA_PA; + } else { @@ -56 +59 @@ -index 52efb8626b..3a0c9c9db6 100644 +index f6ac970ed9..9f9a03bcf7 100644 @@ -59 +62 @@ -@@ -1042,10 +1042,13 @@ rte_eal_init(int argc, char **argv) +@@ -1065,10 +1065,13 @@ rte_eal_init(int argc, char **argv) @@ -73 +76 @@ - EAL_LOG(DEBUG, "Buses did not request a specific IOVA mode."); + RTE_LOG(DEBUG, EAL, "Buses did not request a specific IOVA mode.\n"); @@ -75 +78 @@ -index 4f0a164d9b..2502ec3c3d 100644 +index 56fadc7afe..3a459172f9 100644 @@ -78 +81 @@ -@@ -348,12 +348,15 @@ rte_eal_init(int argc, char **argv) +@@ -359,6 +359,9 @@ rte_eal_init(int argc, char **argv) @@ -86,2 +89,4 @@ - if (iova_mode == RTE_IOVA_DC) { - EAL_LOG(DEBUG, "Specific IOVA mode is not requested, autodetecting"); + if (iova_mode == RTE_IOVA_PA && !has_phys_addr) { + rte_eal_init_alert("Cannot use IOVA as 'PA' since physical addresses are not available"); +@@ -369,7 +372,7 @@ rte_eal_init(int argc, char **argv) + RTE_LOG(DEBUG, EAL, "Specific IOVA mode is not requested, autodetecting\n"); @@ -89 +94 @@ - EAL_LOG(DEBUG, "Selecting IOVA mode according to bus requests"); + RTE_LOG(DEBUG, EAL, "Selecting IOVA mode according to bus requests\n"); @@ -92,3 +97,3 @@ - if (iova_mode == RTE_IOVA_DC) { - if (!RTE_IOVA_IN_MBUF) { - iova_mode = RTE_IOVA_VA; + if (iova_mode == RTE_IOVA_DC) + iova_mode = RTE_IOVA_PA; + } else {