[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] VT-d: Tylersburg errata apply to further steppings
- To: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Tue, 3 Aug 2021 15:16:14 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=M+MCmpB3bu8w8TPPkXRleFzoMRNNuIJB3/HiR289Sf0=; b=Iy8TwLML6HCIvuDa/ja/n31uaeVTG4lMQ9nrBC2gfxcBVyKQF4zErAq5eLzA08s+OvcidW9SeceoXYb2XYLb3I+5dyuvji74hDv1albqmvCcbv5klK9XgVSgg8yp5GTO2Rz0zF0/czrAAjg+gEo0gxMOa9TUiFwLUrB6D8PjWnW58oUf3G+6DxUGoqGXxZvE6t+0E1p7bDsY+ohx/YXDmvKObeyoaUVxx/l23PDS5+659RUTthr71c1fYtZJklODzme8mJtYUYA2l1btthpzUQjFr9PCp7P6ZzY9v0Aj0PF+k6H3Gm0gGm2wmV15Bfh/n3TGJIDMtNO7PPaDUXRdXQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nVWrZpTm1OSoEW9wmGP5A3Ry1LxVD8clAMXJnYNcFse/ARVXFqrYPjpNvMXKp7H3mRmlTyWHQci+cKu9Z3xhPCKQVnNc4H6tt+S/kotsi61iE59eGHaLlaSvhOnbDfx1vYuZQRf9ICEcADcda81/qu3JfQPsiZ2VcMVnbbY6m47yMx17v4OTGYhOPwe6xtRcK3DSRPZ0sHPP2TPPvZDEqIZ9aJN4gv4agzM5T/Lxg/iEHMhebhhlsgWlyCB5zI/u89ShwxVyeROslTrRaJ2tiEEvdtlRbJfn26Oh2zUSLS2+BlYyR988gJV4r/4ysPJKSj4QWRUsTrZp8KciAL81NA==
- Authentication-results: citrix.com; dkim=none (message not signed) header.d=none;citrix.com; dmarc=none action=none header.from=suse.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Kevin Tian <kevin.tian@xxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- Delivery-date: Tue, 03 Aug 2021 13:16:28 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 03.08.2021 15:12, Marek Marczykowski-Górecki wrote:
> On Tue, Aug 03, 2021 at 03:06:50PM +0200, Jan Beulich wrote:
>> On 03.08.2021 15:01, Marek Marczykowski-Górecki wrote:
>>> Ok, then, just setting iommu_intremap=false should do the right thing,
>>
>> ... if "iommu=force" is in use (but not otherwise), ...
>
> But that's the purpose of iommu=force, no?
> With "iommu=force": strictly require IOMMU
> Without "iommu=force": use IOMMU on best-effort basis
>
> It makes sense to refuse the boot if intremap is broken in the first
> case. But also, it makes sense to allow using IOMMU without intremp in
> the second case.
I agree with both statements. What I disagree with is that the latter
happens by default (instead of only upon admin override), including
the case of intremap being unavailable in the first place.
Jan
|