[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v8 5/6] AMD/IOMMU: pull ATS disabling earlier

To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
From: "Durrant, Paul" <xadimgnik@xxxxxxxxx>
Date: Tue, 28 Sep 2021 08:36:52 +0100
Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>
Delivery-date: Tue, 28 Sep 2021 07:36:59 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 22/09/2021 15:38, Jan Beulich wrote:

Disabling should be done in the opposite order of enabling: ATS wants to
be turned off before adjusting the DTE, just like it gets enabled only
after the DTE was suitably prepared. Note that we want ATS to be
disabled as soon as any of the DTEs involved in the handling of a device
(including phantom devices) gets adjusted respectively. For this reason
the "devfn == pdev->devfn" of the original conditional gets dropped.

Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>


Reviewed-by: Paul Durrant <paul@xxxxxxx>

References:
- [PATCH v8 0/6] AMD/IOMMU: further work split from XSA-378
  - From: Jan Beulich
- [PATCH v8 5/6] AMD/IOMMU: pull ATS disabling earlier
  - From: Jan Beulich

Prev by Date: [PATCH v5] xen-pciback: allow compiling on other archs than x86
Next by Date: Re: [PATCH v2 09/11] vpci/header: Reset the command register when adding devices
Previous by thread: [PATCH v8 5/6] AMD/IOMMU: pull ATS disabling earlier
Next by thread: [PATCH v8 6/6] AMD/IOMMU: expose errors and warnings unconditionally
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.