linux-stable.git/drivers/iommu/intel/nested.c, branch linux-6.9.y

iommu/vt-d: Fix constant-out-of-range warning

2024-02-21T09:28:48+00:00

On 32-bit builds, the vt-d driver causes a warning with clang:

drivers/iommu/intel/nested.c:112:13: error: result of comparison of constant 18446744073709551615 with expression of type 'unsigned long' is always false [-Werror,-Wtautological-constant-out-of-range-compare]
  112 |         if (npages == U64_MAX)
      |             ~~~~~~ ^  ~~~~~~~

Make the variable a 64-bit type, which matches both the caller and the
use anyway.

Fixes: f6f3721244a8 ("iommu/vt-d: Add iotlb flush for nested domain")
Signed-off-by: Arnd Bergmann 
Link: https://lore.kernel.org/r/20240213095832.455245-1-arnd@kernel.org
Signed-off-by: Lu Baolu 
Signed-off-by: Joerg Roedel

iommu/vt-d: Update iotlb in nested domain attach

2024-02-21T09:28:44+00:00

Should call domain_update_iotlb() to update the has_iotlb_device flag
of the domain after attaching device to nested domain. Without it, this
flag is not set properly and would result in missing device TLB flush.

Fixes: 9838f2bb6b6b ("iommu/vt-d: Set the nested domain to a device")
Signed-off-by: Yi Liu 
Reviewed-by: Kevin Tian 
Link: https://lore.kernel.org/r/20240208082307.15759-5-yi.l.liu@intel.com
Signed-off-by: Lu Baolu 
Signed-off-by: Joerg Roedel

iommu/vt-d: Track nested domains in parent

2024-02-21T09:28:42+00:00

Today the parent domain (s2_domain) is unaware of which DID's are
used by and which devices are attached to nested domains (s1_domain)
nested on it. This leads to a problem that some operations (flush
iotlb/devtlb and enable dirty tracking) on parent domain only apply to
DID's and devices directly tracked in the parent domain hence are
incomplete.

This tracks the nested domains in list in parent domain. With this,
operations on parent domain can loop the nested domains and refer to
the devices and iommu_array to ensure the operations on parent domain
take effect on all the affected devices and iommus.

Signed-off-by: Yi Liu 
Reviewed-by: Kevin Tian 
Link: https://lore.kernel.org/r/20240208082307.15759-2-yi.l.liu@intel.com
Signed-off-by: Lu Baolu 
Signed-off-by: Joerg Roedel

iommu/vt-d: Add iotlb flush for nested domain

2024-01-11T19:53:28+00:00

This implements the .cache_invalidate_user() callback to support iotlb
flush for nested domain.

Link: https://lore.kernel.org/r/20240111041015.47920-9-yi.l.liu@intel.com
Reviewed-by: Kevin Tian 
Signed-off-by: Lu Baolu 
Co-developed-by: Yi Liu 
Signed-off-by: Yi Liu 
Signed-off-by: Jason Gunthorpe

iommu/vt-d: Add nested domain allocation

2023-10-26T14:16:34+00:00

This adds the support for IOMMU_HWPT_DATA_VTD_S1 type. And 'nested_parent'
is added to mark the nested parent domain to sanitize the input parent domain.

Link: https://lore.kernel.org/r/20231026044216.64964-8-yi.l.liu@intel.com
Signed-off-by: Lu Baolu 
Signed-off-by: Yi Liu 
Reviewed-by: Kevin Tian 
Signed-off-by: Jason Gunthorpe

iommu/vt-d: Set the nested domain to a device

2023-10-26T14:16:34+00:00

This adds the helper for setting the nested domain to a device hence
enable nested domain usage on Intel VT-d.

Link: https://lore.kernel.org/r/20231026044216.64964-7-yi.l.liu@intel.com
Signed-off-by: Jacob Pan 
Signed-off-by: Lu Baolu 
Signed-off-by: Yi Liu 
Reviewed-by: Kevin Tian 
Signed-off-by: Jason Gunthorpe

iommu/vt-d: Add helper for nested domain allocation

2023-10-26T14:16:33+00:00

This adds helper for accepting user parameters and allocate a nested
domain.

Link: https://lore.kernel.org/r/20231026044216.64964-4-yi.l.liu@intel.com
Reviewed-by: Kevin Tian 
Signed-off-by: Jacob Pan 
Signed-off-by: Lu Baolu 
Signed-off-by: Yi Liu 
Signed-off-by: Jason Gunthorpe