From nobody Sat May 30 11:15:10 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=citrix.com); dmarc=pass(p=reject dis=none) header.from=citrix.com ARC-Seal: i=2; a=rsa-sha256; t=1778053088; cv=pass; d=zohomail.com; s=zohoarc; b=b9eSSbCUE0eaYlKO53gpZ4/Ez8yLxw+wboxuJZwzoZf7IqabbYEV7R/78VopxwLm/WdDNrXCoy3Unhfc9DOHWHhNgjaufVgArqR4eTeJc0WDzTAr28307SaLWNIzuMeSYZb2wXsUmeIGoKhs7QzF8DU71Ctz7Y/M5KvMP3vAQ8w= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778053088; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=eJTGyzoNCBosYYOm3qJmbE2HE4BnkvRapJ+78U4LS4U=; b=mciV0IXgjDVpX4mlzcw5wdlfsRiyDqE+kHCxvq7hGVoqUmf3x26s/1nR7r51e7mGE5YnWvKeqmzvN8KBdmxqBjw7aRf1O+jRh1gFc2gGNlMy9+TYWOOS0puQYoWb6xDyLnfqOx85YVeb9nFHz+YSRlCN2yzkCN0vZrqkRmjT0I8= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=citrix.com); dmarc=pass header.from= (p=reject dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 17780530883061011.7019905289716; Wed, 6 May 2026 00:38:08 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1301210.1575517 (Exim 4.92) (envelope-from ) id 1wKWpD-00055L-Sp; Wed, 06 May 2026 07:37:43 +0000 Received: by outflank-mailman (output) from mailman id 1301210.1575517; Wed, 06 May 2026 07:37:43 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wKWpD-00055E-Pm; Wed, 06 May 2026 07:37:43 +0000 Received: by outflank-mailman (input) for mailman id 1301210; Wed, 06 May 2026 07:37:42 +0000 Received: from mx.expurgate.net ([195.190.135.10]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wKWpC-00054i-Hd for xen-devel@lists.xenproject.org; Wed, 06 May 2026 07:37:42 +0000 Received: from mx.expurgate.net (helo=localhost) by mx.expurgate.net with esmtp id 1wKWpA-006Cfl-Cp for xen-devel@lists.xenproject.org; Wed, 06 May 2026 09:37:41 +0200 Received: from [10.42.69.7] (helo=localhost) by localhost with ESMTP (eXpurgate MTA 0.9.1) (envelope-from ) id 69faefc5-5cb7-0a2a0a5109dd-0a2a4507dd94-0 for ; Wed, 06 May 2026 09:37:41 +0200 Received: from [40.93.201.29] (helo=CY3PR05CU001.outbound.protection.outlook.com) by tlsNG-ef75cf.mxtls.expurgate.net with ESMTPS (eXpurgate 4.56.1) (envelope-from ) id 69faefc4-229c-0a2a45070019-285dc91dc8f5-3 for ; Wed, 06 May 2026 09:37:41 +0200 Received: from CH7PR03MB7860.namprd03.prod.outlook.com (2603:10b6:610:24e::14) by DS7PR03MB5397.namprd03.prod.outlook.com (2603:10b6:5:2c0::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9870.27; Wed, 6 May 2026 07:37:39 +0000 Received: from CH7PR03MB7860.namprd03.prod.outlook.com ([fe80::f5ba:35df:1c9f:b343]) by CH7PR03MB7860.namprd03.prod.outlook.com ([fe80::f5ba:35df:1c9f:b343%4]) with mapi id 15.20.9891.008; Wed, 6 May 2026 07:37:39 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" Authentication-Results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=citrix.com header.i="@citrix.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=npkXec1ooaw6zg78Dg0dA4tKQHjOWNxKGn5JzoC2Rycr1dbkswhGsDwLaM+dPBPgMVtKOLR1LIzNEHhUrLS+lPHvrsx+J16eJsBj5pK0xnj9nOehCzapxqjkyxcNJN9NzJME7muY/WQND02I9gdBSdwdcjFO4x4FiXjVjd6K8M/kNlkdFiwSNidLOO8vPACY7/kGWs2W6YHJpnCX53f7DtUwA/fErAOpLG/064sPbhNHtBvvPFlFeZM5fMYji/jBN+MSoKZ2UQlmxshTYmpDqRdY6BSAXOYbPC1w4mUrPrNEQ6Ah0lU7Y2KwTkCgLRfpJC/LWALRAJp/YNhES6hBnw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=eJTGyzoNCBosYYOm3qJmbE2HE4BnkvRapJ+78U4LS4U=; b=ex9zNBfrsBG4nFn5W6CKsa5ZAzBkvse4XaPInDSukEPvACBqpdQ6t15gbkineph/thzbCycnSTL5nq559Wq3ANV4O6Altnm+vMgEIOSuqDg3SDddQUTfmwLNQdRJ8X2G7oF7lUGorzREbjaQyPqR7Em5nC4LOAA8AkwsxL76laEOO4vhPRTCJkBqAoWItWGNc5ekt09zT54hPa4JL+EdEJc3Ov4w7dLUkOPyRbfwPSBdTr9YVM8mNUI+zUGZBuofxFuO3S2VnzYIbDEJRgGToPy+o5tyhOxOb1O/CcKKg3GjVrwz4echDuzTZjjuNXtFN64/1GXdwm6tfxyui9ss4g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=citrix.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=eJTGyzoNCBosYYOm3qJmbE2HE4BnkvRapJ+78U4LS4U=; b=qDCYSJ2KMfeLZABsroktcnn1bIGEj4zSbvHBqQA/ObnJvalSXZ9G61FfB8woe26SXwrq6WsVXxSkRi15mMsyAReyUekI6GE79pAjfH6nTaBsEFDUDs5gtLNqGY6X+KI4BN+qUfrP94PTCIcrNzrNzTFl4ujgTKvjf5QzKT7SF38= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com; From: Roger Pau Monne To: xen-devel@lists.xenproject.org Cc: Roger Pau Monne , Jan Beulich , Andrew Cooper , Jason Andryuk , Teddy Astie Subject: [PATCH 1/2] iommu/amd-vi: allow disable_iommu() against non-initialized IOMMUs Date: Wed, 6 May 2026 09:37:18 +0200 Message-ID: <20260506073719.40075-2-roger.pau@citrix.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260506073719.40075-1-roger.pau@citrix.com> References: <20260506073719.40075-1-roger.pau@citrix.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: MR1P264CA0182.FRAP264.PROD.OUTLOOK.COM (2603:10a6:501:58::18) To CH7PR03MB7860.namprd03.prod.outlook.com (2603:10b6:610:24e::14) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH7PR03MB7860:EE_|DS7PR03MB5397:EE_ X-MS-Office365-Filtering-Correlation-Id: bbf90afd-edc0-4e97-eb6b-08deab425967 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|366016|376014|22082099003|18002099003|56012099003; X-Microsoft-Antispam-Message-Info: gN7aOcWAzKQxwiZNxH6DAa7bkTMs4zRbkT5Hk3EpMP9+y6vQomCwOC72sPT5ZMV3QPt3GlhnZ8K5le071Veo2Mv6XAUoKNTdP6MpFfddyFmkDzaH08duKumvl0Ep6JkWC7ittBYhX1JclMIVKYGl6XggIAUg0WbKeuiGmLgd6uNw9jKlViuLkfHgG312MYkZodAV5RVuhVBzOucx8VTdc3bsT4YqydOncOxrVO7WJksoCRTbWHxZR5l/FngReEEwLm5VVc0dNpT0zVRBa0Fr8Z3TazXtbnb5RvC0nq9NR3TYYDfukzqApLzcx9zK3qMpQtqnhNxuMXfs/aLQj9cKieSFPhX6i2mrXrYw6dTVXLX3gqlQDDJCtkhYwnGo8LsEZSuCau7Xy0QYKTDfZUJZtlWeAvuu4OZnAxC5ha2vhLkefRagmTfoeDxLrQdwX9cUrc+Nn9aouD9m0tTybMDE/avp8Yw/sTaaI8LxHXgwCCJp/WaFFZGqsuLTXkAZH8L2Inl4qU7V16esxonlLakrPizyNsNJa95CIPYWb5/KxfyJ34ZuRMaKh82IrzEK7RiDPl8xiD6wjra5aFgryrGj5TCldC4vWV3NsDvbi23Ms1ol9nixJyAqiWEM7WDFHvBAN1qN7yH3B9A7DGaXiTzwcwUW0waSgkfqQqcLfVXU100mgAWlnoaDDfbaAbfdRXE7 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CH7PR03MB7860.namprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(366016)(376014)(22082099003)(18002099003)(56012099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dW01Vng0cG9sbmRUd0dQbjc0cHZVZUlvcHRvcU1oajRtemVKK2RxY3M4MHhx?= =?utf-8?B?MWVYTVhmbzAzNnJwQ2VGSzFtK3U3dUErSmJhV05qZWRqSkdVTkM5dDFJd0V3?= =?utf-8?B?bUhiYzB5Q1gyaUFPZHpYa3lScURiUnZrK0ZxVXl3bmVZSi9xdGhWVmpnUlFp?= =?utf-8?B?NzVpRkRRTUZTejRGckV5Z0JDaUdtb1JEaDNadVNHZFd2ZEFpTEpFVGNDYXhR?= =?utf-8?B?RGxaeGU3YXZpQmxobjE0VndQVzVEMHZzZ013Tm9MazV4L21LNmhlczVWdXJN?= =?utf-8?B?WWFoYWZnZzZKWlNVVnVoci84ZExCc1B3WWVGdWVPSEhuYW11SzBXVXBXeDdh?= =?utf-8?B?aWNCWnp0OEhpK0h6TG84MktqaU1wWmVOS3FSQ3ppU1pmVEhBNVFKcW54a3NH?= =?utf-8?B?dThid1NHRDFuWEdvNHJ2KzlzL1dtaHVsSC9Pa3Z6S2llczFWY3pMVklPajA2?= =?utf-8?B?TTVFZ1hMeWt6OHZtZ3h3MktrUWdKZlorTmwrQ2lpbHZKL0ZsaFh0cHRBanpY?= =?utf-8?B?QWdUYWtaRllUSHdvaDI3Vy9kSldsVnFtVjY3Y0R3K1d2NVV1SWViSzNyK2E2?= =?utf-8?B?OFFTTFpVTG9JRDVFR2xtN0Z4YmtmMTlMN2drMFlwUDVONkJZd1l4d3RQRzBH?= =?utf-8?B?MXI4WWNHdzJGRzYvZ2FkampvdGRaRFhiVG5JZjhxTkVJQ0gvWWViQkcvcTJv?= =?utf-8?B?cFdzREt4d3FXN3RQWEN4SUdXMTJPRjByL214MWM5VVREeG1LSHRkeVo2MXh6?= =?utf-8?B?MDFLOXowL29Ub2krMnFXL3FSbW1pTFRkUTdEUnpEbCs2b0Npakl4dU10M0xP?= =?utf-8?B?ck9Db1dnam5HRjhQeDJQVjJ1SDdkV3ZEZUsyTTdMRG43ME93S2MxYWRLZGpa?= =?utf-8?B?dWtMUGE5NFBLNGYvTEJpVjgzamovblA5eDJmbTcwZFpiMWJYZjFwazdSVzRC?= =?utf-8?B?WVQyVmtUcTNUV0JDYVVKTUQ0Sk9ZcklYZ3lFdUo5WlZaTmhyM1RZUEJnbzlp?= =?utf-8?B?TC9PeHQwN1YxS0pjaGpMNjFYWms0THhYWkJNMDBycVZpS1ZlMnl3QWVncThz?= =?utf-8?B?d1BBNmcrRzdHSlgwSUdFeW1Uck1iWGhZbENvbDFNSUo3cUhZVVZWVm1mLzA0?= =?utf-8?B?Y3hveTdOaGtld1VHNktWekNQbFJ2RjBWdUZPbW03bnhIRjlONk5ocWUvb2dz?= =?utf-8?B?b1llNkw0YzA1N0JZcS9oendQQm9uRmE0NEZjZFMvVmdWdldSS0FnUUxQeXFk?= =?utf-8?B?RjVyTTRSMnhPVU1ielhTNVNOaXJWYXJlcHpuWHQySFllUXE2V1JMaXZkMWJC?= =?utf-8?B?YXh1L3dTNGpMNFI2TUVwQVdiU285OWRUR05NakF2bVVOWXBXS3llNkFnTjR5?= =?utf-8?B?RWptR3NOeldaeWRhajdGeTdLaUlzbmVZYnovYVhyTDdvRU8ycytYb2J1WUhN?= =?utf-8?B?Q2NMRXBObXo4OWNMTVo1NzgraWxWY3lHU0pORnVqY3Y5WDU0SjZwdE5KaUpS?= =?utf-8?B?NkdrOEVkN3NVWitiSUVzKys3aElCelN1a1l3MDJSZ2Z1eExlQzNHcGVkOU5v?= =?utf-8?B?MmpmV1phdWZCdDlub244aXJSTktLZnYzM0NIWTJZNTZDWkFnbGJaZkxZamJS?= =?utf-8?B?bnB6d1pvRU8vTnFXT3NtNVpDRnl0Qm5VY0tZYVR5WEEvTVlGT2FXL3FqVGdv?= =?utf-8?B?TC9HbnZFOXl0SzJQS1FQM1UyWjA3azhXMVhwQ2JYQ3MySDVGREdwdnBPM2dG?= =?utf-8?B?RUxmL2JOT0c5cUU1bG9RK016YjZiZDhudXRKTTBacG5ia1g3NXBvOWVEdUJP?= =?utf-8?B?REZsWk5OMGcwaVljRE90S0tQdFV1eG10dVZJc296TFBTRFU5WFhMS2VZdlVJ?= =?utf-8?B?aEYrUGdCZEZ6eTUrWWNaMERPUTYwUjN2SVcxb1VmNWkvTkZZbm5wNGdwc3pE?= =?utf-8?B?bVFFVjVoL1RDU0JoSUVlMEpBRWdTNFZFOW5mMVFTTTlzNVBWeEcvQmRUaTRG?= =?utf-8?B?Vlppekx4d2xpTlFRZ0RWTXFqcXNPVTRvQUVHbGRqeFBUdUhDSUxKV1RZSEhm?= =?utf-8?B?bmora3lGaStaYnhRcnRKVXpxUEJwWm5rMGJuTUk5eHdVZk4wQlN0TTJmZjgw?= =?utf-8?B?U3RQL3A0Y3A0MXZLcFpEc2xoeW1zdkZPSE44OFhUYVZHZUZQenY4TEl3NktO?= =?utf-8?B?eFMzTnJWN2NWS1BxWW05RjduSzRGRUtWMDRCcWlScnZZMDgzaUF2UFluZEdH?= =?utf-8?B?MG1GeDYvWGpoMWxCL2VSZlVYSm9CL01vamJ0YTRLaUpSdDlrTkJBRE1qOEQ2?= =?utf-8?B?anJPSktYSVk3ZVNTb0YwN3AxNkFTNGxhRDI3cmhTdWE3MmptYVVGZz09?= X-OriginatorOrg: citrix.com X-MS-Exchange-CrossTenant-Network-Message-Id: bbf90afd-edc0-4e97-eb6b-08deab425967 X-MS-Exchange-CrossTenant-AuthSource: CH7PR03MB7860.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 May 2026 07:37:39.1016 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 335836de-42ef-43a2-b145-348c2ee9ca5b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: yrb+cqEDO+WYKb3Qrd0A7PFTvabu/2i8dSxIvFNO+V6233V24KjWLs5wW23m/O57dyFvv6MfVn/px/nAuNWlaA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR03MB5397 X-purgate-ID: tlsNG-ef75cf/1778053061-2B979C48-944672F3/0/0 X-purgate-type: clean X-purgate-size: 2134 X-ZohoMail-DKIM: pass (identity @citrix.com) X-ZM-MESSAGEID: 1778053090423158501 Introduce a force option to disable_iommu() that allows it to disable the IOMMU, even when ->enabled is not set. While there remove the unlikely(), this is not a hot path anyway. No functional change, as there are no current callers that pass force =3D= =3D true. Signed-off-by: Roger Pau Monn=C3=A9 Acked-by: Jan Beulich --- xen/drivers/passthrough/amd/iommu_init.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/xen/drivers/passthrough/amd/iommu_init.c b/xen/drivers/passthr= ough/amd/iommu_init.c index d77dd8511288..76ae78e5ea53 100644 --- a/xen/drivers/passthrough/amd/iommu_init.c +++ b/xen/drivers/passthrough/amd/iommu_init.c @@ -929,13 +929,13 @@ static void enable_iommu(struct amd_iommu *iommu) amd_iommu_flush_all_caches(iommu); } =20 -static void disable_iommu(struct amd_iommu *iommu) +static void disable_iommu(struct amd_iommu *iommu, bool force) { unsigned long flags; =20 spin_lock_irqsave(&iommu->lock, flags); =20 - if ( unlikely(!iommu->enabled) ) + if ( !iommu->enabled && !force ) goto out; =20 if ( !iommu->ctrl.int_cap_xt_en ) @@ -1139,7 +1139,7 @@ static void __init amd_iommu_init_cleanup(void) iommu->ctrl.int_cap_xt_en =3D 0; =20 if ( iommu->enabled ) - disable_iommu(iommu); + disable_iommu(iommu, false); else if ( iommu->mmio_base ) writeq(iommu->ctrl.raw, iommu->mmio_base + IOMMU_CONTROL_MMIO_OFFSET); @@ -1584,7 +1584,7 @@ void cf_check amd_iommu_crash_shutdown(void) struct amd_iommu *iommu; =20 for_each_amd_iommu ( iommu ) - disable_iommu(iommu); + disable_iommu(iommu, false); } =20 void cf_check amd_iommu_resume(void) @@ -1598,7 +1598,7 @@ void cf_check amd_iommu_resume(void) * To make sure that iommus have not been touched=20 * before re-enablement */ - disable_iommu(iommu); + disable_iommu(iommu, false); enable_iommu(iommu); if ( !iommu->features.flds.ia_sup ) invalidate_all =3D false; --=20 2.53.0 From nobody Sat May 30 11:15:10 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=citrix.com); dmarc=pass(p=reject dis=none) header.from=citrix.com ARC-Seal: i=2; a=rsa-sha256; t=1778053112; cv=pass; d=zohomail.com; s=zohoarc; b=oDutE0KQxPScT9i+kTA5jhroe1pBA/hFxbB1u4nCrBH4PKtXAcTSZez1j2JtHomutWLU2Z1/RfU6O9i2iBVN9oY4xPRzdrxpubvI67S9KTBKJ+D9aDMiAW2hd9+I5XISGmW4gZs+S+ID9k/LszOeM6F++a39XCXkTJ/6LaakFPA= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1778053112; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=GOqAXfM7Rlhy7EjMS2OGtDITdoCH4UGlxIzZkMc1Lfg=; b=KpJJaqKU6n/XKBCPOteTCCqn6bh4dykxTQUxgKXx0uqC5d5SyyvA14t6lqHElG8suvFhJYA2JoHHMHG70hilALEllrgyG1yE3CAwk7COFSsjWwaJL0bQGE8TRGBTrAznoxF7pd4FR1d5uUeZoVvoXvDE0v29QzkasUh22wdakUM= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=citrix.com); dmarc=pass header.from= (p=reject dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1778053112380185.98678450309308; Wed, 6 May 2026 00:38:32 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1301223.1575525 (Exim 4.92) (envelope-from ) id 1wKWpn-0005kD-4H; Wed, 06 May 2026 07:38:19 +0000 Received: by outflank-mailman (output) from mailman id 1301223.1575525; Wed, 06 May 2026 07:38:19 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wKWpn-0005k6-1b; Wed, 06 May 2026 07:38:19 +0000 Received: by outflank-mailman (input) for mailman id 1301223; Wed, 06 May 2026 07:38:17 +0000 Received: from mx.expurgate.net ([195.190.135.10]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1wKWpl-0005jU-MH for xen-devel@lists.xenproject.org; Wed, 06 May 2026 07:38:17 +0000 Received: from mx.expurgate.net (helo=localhost) by mx.expurgate.net with esmtp id 1wKWpk-008hD3-VR for xen-devel@lists.xenproject.org; Wed, 06 May 2026 09:38:16 +0200 Received: from [10.42.69.11] (helo=localhost) by localhost with ESMTP (eXpurgate MTA 0.9.1) (envelope-from ) id 69faefe0-e002-0a2a0a5209dd-0a2a450bb674-26 for ; Wed, 06 May 2026 09:38:16 +0200 Received: from [40.93.195.24] (helo=SN4PR2101CU001.outbound.protection.outlook.com) by tlsNG-42698a.mxtls.expurgate.net with ESMTPS (eXpurgate 4.56.1) (envelope-from ) id 69faefe7-212f-0a2a450b0019-285dc318861d-3 for ; Wed, 06 May 2026 09:38:16 +0200 Received: from CH7PR03MB7860.namprd03.prod.outlook.com (2603:10b6:610:24e::14) by DS7PR03MB5397.namprd03.prod.outlook.com (2603:10b6:5:2c0::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9870.27; Wed, 6 May 2026 07:38:13 +0000 Received: from CH7PR03MB7860.namprd03.prod.outlook.com ([fe80::f5ba:35df:1c9f:b343]) by CH7PR03MB7860.namprd03.prod.outlook.com ([fe80::f5ba:35df:1c9f:b343%4]) with mapi id 15.20.9891.008; Wed, 6 May 2026 07:38:13 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" Authentication-Results: eu.smtp.expurgate.cloud; dkim=pass header.s=selector1 header.d=citrix.com header.i="@citrix.com" header.h="From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck" ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=WYTqF1l4alcMATFZ2XVqBtJDL9iYDuAP2/OoFma0KtvRQ8vPccmQjeweI6EIS6xLqt+WfJCaBPbQlFPK85Fzf7WqHB4vpcEFsFG6ObHQ+5lhvMjZy0FT6d3lASdU9GjJhcvouMvxyFEib8o4PV811heBB9qrqZ4AWqAjNjgAtEoIvIfl9+d3sihwUVOj6f3Cx0nivYdpcwQI6y8H+DiFsXZqsgH7ZwKnC7IOjExp17dkaBgVlZhMeBJP4YQPMnmrbvEE9c6AGIfVsbS4lJcLaLwU8Z3HykXeZB+qTsSixfLoB9irqkYRU/ei0J7RspEo6nm1amV6pd0GIkXpkxVvIQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=GOqAXfM7Rlhy7EjMS2OGtDITdoCH4UGlxIzZkMc1Lfg=; b=NCVAyAGba/zu/SsFiW5xtxn6IGjnBBz2bIZ/I9/jlQ45HVU5iSRrzzeUHL7J9+nTvBP2Nz+y33ARSLgVNZaPmK6Byk9a2W3ql/D+qXyCiKsCJ3RGfgJZ+mOAeloPHigjvFULz5KOVAZkqKUlVhTDklBX8NTEVRyYFDENUEvlw/kP2LxVBdhkC8qFYzf0GMlLCPgqVxEjTcRRq5y4GLbuliUQIfQbaiAk3VqpXSX2MSMNo9zW6ktWq8GachtPCPk/fhqhRWcd61bJ5wkkIGuerjkY/fbDLdUZsPFS8CoqC4YTUp6mmZoRd2j1zaIw9s1LkqUNHbGl/iZfnut7z3whXQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=citrix.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=GOqAXfM7Rlhy7EjMS2OGtDITdoCH4UGlxIzZkMc1Lfg=; b=Xl7w26QzyeIMXs9DoJH1eDpCF6QLbQH6Fw+3CvxOmG4hsECShroLleJlxE+Sja8nig9i6yrU6BgtBx0kZ+574xbj3DHvIcebIid9F5Tfkdnu9QPAitXPIObC5FicLoBq+4glftGcCH+bakc+NfPSycIKEbpRD4qoTYLKEg9yedg= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com; From: Roger Pau Monne To: xen-devel@lists.xenproject.org Cc: Roger Pau Monne , Jan Beulich , Andrew Cooper , Jason Andryuk , Teddy Astie Subject: [PATCH 2/2] iommu/amd-vi: do not zero IOMMU MMIO region Date: Wed, 6 May 2026 09:37:19 +0200 Message-ID: <20260506073719.40075-3-roger.pau@citrix.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260506073719.40075-1-roger.pau@citrix.com> References: <20260506073719.40075-1-roger.pau@citrix.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: MR1P264CA0217.FRAP264.PROD.OUTLOOK.COM (2603:10a6:501:56::16) To CH7PR03MB7860.namprd03.prod.outlook.com (2603:10b6:610:24e::14) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH7PR03MB7860:EE_|DS7PR03MB5397:EE_ X-MS-Office365-Filtering-Correlation-Id: 69905622-d2ca-495e-dbcf-08deab426e01 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|366016|376014|22082099003|18002099003|56012099003; X-Microsoft-Antispam-Message-Info: 9FZ2K3lkjWnLCOUXHd/XZfy1Wc/G+YDP6DQ/o0M79JR3Als0YGs6vDdVz3gz2/F4BICUUah5e2ySc0d4CGC3XrobsBfup9liHa6MBJ32o1wgvFNRd+5IPrpDC/RJzadH2ZViAay+mtCpLnjNrDgYEVmXgXNQPBMQviXxgxcnrwGdkm6Eja3Vmip6dbUWjfkfOS3g+A/PjJuPlqVcyjIFZmisACrhrs9pRMnIjMuocpHopTStHUmLM5OZnKhj1r85s0yxZia/FK86qQvyePYvMKXQ3vKzixcNyTk4NGL7Vq+IDV05vfjzpyIzZbvXULq9q3gfpP897MG3VPgJcm/bPZAhDhq5LbC58VAUgJIDZOv32vrNlXkbRpKQu1LnxbcNY9tC74ChJBtYoP8zerK/pH0ai7SChxOjV0QZ0I5hPNpa9mwYdCSrMCi0yT7LXT5w/MJzow4eoZ1aa1xH2inKlewMun7rGGhVngaCbUGjqo9Ra1AA94xjoJXAZE7V5r18UQc6vR3OjSoFkaeE9PbTwyumWHIxCdzcwGdnYsiC/xAyD+jpG/my1IutqsO2pwF0K9IKi97kAnNpXC8fmV1zy1YnfarpqS1KemHWefEHgnqcw5cjSVngHdyUTTrHHi0eMuo7HTHgntPyXtn/sa8AStdviAP6Rz2tgBIe+9WDKt1gUESaB0oiDngHZtFShJD8 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CH7PR03MB7860.namprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(366016)(376014)(22082099003)(18002099003)(56012099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MFBWTW1EWlJ6enF5d0R3ZnhmeG9KbUhLZ0RqMWtGbVAvSm10bEsxQ3ZDVWxs?= =?utf-8?B?VGtQRnJ2czNsRlkzdVMzY3VmSm5reDFWTTZaK0Y1TUI3UHhrNzFyVlk2TklT?= =?utf-8?B?VDV6VkJFRXhYOHkveVVDOHZWdDd2WWpWU29FbC9VdTdnWWFobU96NFBlejlI?= =?utf-8?B?cnFsb2tTYUIxM1BueVFTYjFtYjlGOTV1bzJFQWxzWmFrQ3ZwMFJsczZWYXow?= =?utf-8?B?Wk1kRkI3enFRNDdKS1hyK1pRM01ZREd4QXhJOHY0dWNmSkdYQXRSSlVXZkdz?= =?utf-8?B?dlNML1gyWXdvaXdzMzdoa1RPMjl4MUxNVVE5UjJLY2ZUOExDYjM1YnlnemdT?= =?utf-8?B?bWhQVDJWazd0dUpLM3hxRWh5MmhCcDd2bjYzQzFxSzBDSjJpM214Z2UvbVor?= =?utf-8?B?RTVnM0kvS0Uzei9SZ1Qzd2ZzSnJnQmtGOE5KOHhsUms1NTFUZVB5MEw2OUVC?= =?utf-8?B?OEhDZU4yNjlGN045aEEzMHYzWWdEbFdaYTdNd1M0UExIaVBiZHZVZW44V00x?= =?utf-8?B?Sk5jZlByTXJSMEllSm1IbCtvckFPdnFxbXlqVys4Vzk2dklXUkpLQTlzc1Bi?= =?utf-8?B?d2dFWEdKUnZJc0dXb3JqcmJjSk5xdU5xaXdNUTFVWS9sSmE2ZU9lUU41RVRD?= =?utf-8?B?RmhRMS9aK1BsNHNGVHpoYWxLSmJzME1ua1ZiUzZRUDVBMXF3ZTFab2xWQmlx?= =?utf-8?B?d0NmM3hXb3M5V1dXek5BY2RISTdDd3pxVE1IYktYNGZKempIbXowM3pGNEI5?= =?utf-8?B?YnBhc0FCcVBpMktrUDdhS2RuVmxHbWltSFhzZ29rWDRHN0JTeC9ZeFFITmtD?= =?utf-8?B?MDZma2dqM0doM094ekFjOEE1WDJtdHpETm1sdU1ZMDI5R21Wa2VNTlNTVGR5?= =?utf-8?B?Y2xaUVhPRm9XelI1ZTJvdERyektxbWF6SFQ0TjcrRkFqVWtUamFrYUZOZGVa?= =?utf-8?B?dGZqVjI1emZiejkveVZnR1dhWEZjMklHZWhLNWdrcS9SbEhlcnRhSWx0SCtT?= =?utf-8?B?b3dhQTQ0RHdsQ0FoZGNNOFhScTdkWVZ6MlBnUnZzY3dTMXVkaVJNeDlEYW83?= =?utf-8?B?eGU4Rlp2RER5aDJQcEF3dmxkVUNwSit6SjZvVjhKaXVYcmZ6ekpnL1grM2Y4?= =?utf-8?B?aE0xWUxVcER0czBLNy8wRTFxcEFhcVR6UFByWW02OTVnUXBuS0ZYbUovakor?= =?utf-8?B?LzZtdm0ybG03ZzcrWHRhZG5aQnViOWVVOVJESUZOS2FDMHgvb2w2SU13REt2?= =?utf-8?B?NnBxS3dpQ0pTNi9jVmdFbll4UUhrQ0Vab3BraGZqbVcxYnZ1S01jOE5sYUFq?= =?utf-8?B?VGJOeS9tSUMyYWRaamRBMEhzbW1iVVh6b1dXNkhZOWx2UTFZMnFTa01BVVNI?= =?utf-8?B?TlF5ODdMOUNncXBWb2hwRnZ4L2dKTE1WS3lpWnVYMFMzc0tFQTZneDZrZUJK?= =?utf-8?B?YUNQNEVjZzdvaTdBV2wzd0pJMGdlbnNubzdCT2E0bWNhK3dMZmJqcVdQVVBm?= =?utf-8?B?OWE2U1NLRWRnUXVRQkljNEpZRUlzV2RzZ3VKNjFvL3VrTG00RVUyQzdTWlp4?= =?utf-8?B?a1BWNC9xYTJoSVJtanJiSU1ZOEluV3FvcWJhOFhxcXZ3REdmSUc3eWI0VGZG?= =?utf-8?B?b2JGbTNjYWMwQ0F2V0NwUkdEc1h5Y0JrbW5kTG9HMzAxNW82UW5pbUtuOHZM?= =?utf-8?B?c3YxNUx6dWVwSFVUMFlBZE1UTGZjNUhBU0duZTZHbjB3cTlnUlVhclArUXZF?= =?utf-8?B?a2R3ajRkaHU5SnlIWFJ4MnpRbWFxekpvOVRRZGQ2aGlJY3dPYWpUSEtveDBr?= =?utf-8?B?R3NkK21JUGtGam5SbE43N0N3NTV2cHIrWFc3RnY1cWs0a2VSdE84RG4xWk5z?= =?utf-8?B?S0t1TkxFUnlYK05EdWNyRkduR0dReCtjTlZLQ29DcitjQS9JSmIra1JUTVVt?= =?utf-8?B?cFAwcFViRk5DZlphejhTL2FtR05zMWlTSDhCRHBtZ2xNRGl0MkpndnY5V3dR?= =?utf-8?B?R1ZvVHkrTjNZNWVxdlpzTkpSTzBqRndjQUsrbVRBS1VjTWF4Y3dLNVlRU2dZ?= =?utf-8?B?RlFlMTJGRytPaktoRmREY3hhMis4ZE50bWFOL1RyZ3pmUHNwNU9nNld3eUNv?= =?utf-8?B?c3Vhdnoxc2h4d3E4WXBqTEEwb3JVV3hqei9pb3JSVkd2QkcySzZWNVlRYW9n?= =?utf-8?B?L2pIMVRIUEEzME1GMEVPQU9jNWZBL25KMUg1REtDR0hzc29ORnRPMnlFS3Jp?= =?utf-8?B?SjV4bDcya0Y5REZXV0M5Q3hkRllnZDkxdHRmcjZ3Q3MydzV1SGhhNzloeHo1?= =?utf-8?B?UVZNYXhDWE1PVGh4QWtsS3B5OFpYSjNKUW45RjRSV1BodEJtMU1wdz09?= X-OriginatorOrg: citrix.com X-MS-Exchange-CrossTenant-Network-Message-Id: 69905622-d2ca-495e-dbcf-08deab426e01 X-MS-Exchange-CrossTenant-AuthSource: CH7PR03MB7860.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 May 2026 07:38:13.7111 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 335836de-42ef-43a2-b145-348c2ee9ca5b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: wGsFo1vduHKW89EzdUZEPKgjUUs1HT+eVL3ypb+TaMVj6C7PvZgswkOrbq1iqMdWPL/idvZIeY/B/FCLzYYnOQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR03MB5397 X-purgate-ID: tlsNG-42698a/1778053096-7E573F3B-912873B5/0/0 X-purgate-type: clean X-purgate-size: 2541 X-ZohoMail-DKIM: pass (identity @citrix.com) X-ZM-MESSAGEID: 1778053115423154100 Attempting to memset the whole IOMMU MMIO region to zero is dangerous to say the least. We don't know what registers might be there, neither what values might be safe for those registers. On a forthcoming platform doing the zeroing of the MMIO region can put the IOMMU in a broken state, which is not recovered by the IOMMU initialization procedure in Xen. Instead attempt to forcefully disable the IOMMU ahead of enabling it. Fold map_iommu_mmio_region() into it's only caller, as the function body is just an ioremap() call after the removal of the memset(). Fixes: 0700c962ac2d ("Add AMD IOMMU support into hypervisor") Signed-off-by: Roger Pau Monn=C3=A9 --- xen/drivers/passthrough/amd/iommu_init.c | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/xen/drivers/passthrough/amd/iommu_init.c b/xen/drivers/passthr= ough/amd/iommu_init.c index 76ae78e5ea53..8bf5ca4de18f 100644 --- a/xen/drivers/passthrough/amd/iommu_init.c +++ b/xen/drivers/passthrough/amd/iommu_init.c @@ -42,18 +42,6 @@ static bool iommu_has_ht_flag(struct amd_iommu *iommu, u= 8 mask) return iommu->ht_flags & mask; } =20 -static int __init map_iommu_mmio_region(struct amd_iommu *iommu) -{ - iommu->mmio_base =3D ioremap(iommu->mmio_base_phys, - IOMMU_MMIO_REGION_LENGTH); - if ( !iommu->mmio_base ) - return -ENOMEM; - - memset(iommu->mmio_base, 0, IOMMU_MMIO_REGION_LENGTH); - - return 0; -} - static void __init unmap_iommu_mmio_region(struct amd_iommu *iommu) { if ( iommu->mmio_base ) @@ -1367,11 +1355,14 @@ static int __init amd_iommu_prepare_one(struct amd_= iommu *iommu) { int rc =3D alloc_ivrs_mappings(iommu->sbdf.seg); =20 - if ( !rc ) - rc =3D map_iommu_mmio_region(iommu); if ( rc ) return rc; =20 + iommu->mmio_base =3D ioremap(iommu->mmio_base_phys, + IOMMU_MMIO_REGION_LENGTH); + if ( !iommu->mmio_base ) + return -ENOMEM; + get_iommu_features(iommu); =20 /* @@ -1381,6 +1372,11 @@ static int __init amd_iommu_prepare_one(struct amd_i= ommu *iommu) if ( amd_iommu_max_paging_mode < amd_iommu_min_paging_mode ) return -ERANGE; =20 + /* Read current control register and forcefully disable the IOMMU. */ + iommu->ctrl.raw =3D readq(iommu->mmio_base + IOMMU_CONTROL_MMIO_OFFSET= ); + disable_iommu(iommu, true); + iommu->ctrl.raw =3D 0; + return 0; } =20 --=20 2.53.0