iommu/vt-d: Add scalable mode support
Subsystems (2)
v6 cover iommu/vt-d: Add scalable mode support
Hi,
Intel vt-d rev3.0 [1] introduces a new translation mode called 'scalable mode', which enables PASID-granular translations for first level, second level, nested and pass-through modes. The vt-d scalable mode is the key ingredient to enable Scalable I/O Virtualization (Scalable IOV) [2] [3], which allows sharing a device in minimal possible granularity (ADI - Assignable Device Interface). It also includes all the capabilities required to enable Shared Virtual Addressing (SVA). As a result, previous Extended Context (ECS) mode is deprecated (no production ever implements ECS).
Each scalable mode pasid table entry is 64 bytes in length, with fields point to the first level page table and the second level page table. The PGTT (Pasid Granular Translation Type) field is used by hardware to determine the translation type.
A Scalable Mode .-------------.
PASID Entry .-| |
.------------------. .-| | 1st Level |
7| | | | | Page Table |
.------------------. | | | |
6| | | | | |
'------------------' | | '-------------'
5| | | '-------------'
'------------------' '-------------'
4| | ^
'------------------' /
3| | / .-------------.
.----.-------.-----. / .-| |
2| | FLPTR | |/ .-| | 2nd Level |
.----'-------'-----. | | | Page Table |
1| | | | | |
.-.-------..------.. | | | |
0| | SLPTR || PGTT ||----> | | '-------------'
'-'-------''------'' | '-------------'
6 | 0 '-------------'
3 v
.------------------------------------.
| PASID Granular Translation Type |
| |
| 001b: 1st level translation only |
| 101b: 2nd level translation only |
| 011b: Nested translation |
| 100b: Pass through |
'------------------------------------'This patch series adds the scalable mode support in the Intel IOMMU driver. It will make all the Intel IOMMU features work in scalable mode. The changes are all constrained within the Intel IOMMU driver, as it's purely internal format change.
References:
[1] https://software.intel.com/en-us/download/intel-virtualization-technology-for-directed-io-architecture-specification
[2] https://software.intel.com/en-us/download/intel-scalable-io-virtualization-technical-specification
[3] https://schd.ws/hosted_files/lc32018/00/LC3-SIOV-final.pdf
Change log:v5->v6:
- [02/12] Move pasid table allocation out of lock range and replace GFP_ATOMIC with GFP_KERNEL when allocating the pasid memory.
v4->v5:
- [04/12] Add a comment to explain why we only print two QWORDs of an invalid descriptor.
- [06/12] Fix domain agaw being less than iommu agaw.
- [09/12] Move data type to local branch.
- [11/12] Hold iommu lock when setup pasid entry for SVM.
- All rebased on top of vt-d branch of iommu tree.
v3->v4:
- Rebase all patches to 4.20-rc1.
- Use the right PASID directory table size.
- Add a function for pass through translation setting up.
- Refine the parameters passed to first level translation mode interface.
v2->v3:
- Rebase all patches on top of vt-d branch of iommu repo.
- Set the pasid directory table size to 1 page for devices which have no pasid support.
- Fix various comments received during v2 review period. All were code style related.
v1->v2:
- Rebase all patches on top of v4.19-rc1;
- Add 256-bit invalidation descriptor support;
- Reserve a domain id for first level and pass-through
usage to make hardware cache entries more efficiently;
- Various code refinements.
Lu Baolu (12):
iommu/vt-d: Enumerate the scalable mode capability iommu/vt-d: Manage scalalble mode PASID tables iommu/vt-d: Move page table helpers into header iommu/vt-d: Add 256-bit invalidation descriptor support iommu/vt-d: Reserve a domain id for FL and PT modes iommu/vt-d: Add second level page table interface iommu/vt-d: Setup pasid entry for RID2PASID support iommu/vt-d: Pass pasid table to context mapping iommu/vt-d: Setup context and enable RID2PASID support iommu/vt-d: Add first level page table interface iommu/vt-d: Shared virtual address in scalable mode iommu/vt-d: Remove deferred invalidation
.../admin-guide/kernel-parameters.txt | 12 +- drivers/iommu/dmar.c | 91 ++-- drivers/iommu/intel-iommu.c | 351 +++++++------- drivers/iommu/intel-pasid.c | 449 +++++++++++++++++- drivers/iommu/intel-pasid.h | 40 +- drivers/iommu/intel-svm.c | 171 +++---- drivers/iommu/intel_irq_remapping.c | 6 +- include/linux/intel-iommu.h | 75 ++- 8 files changed, 838 insertions(+), 357 deletions(-)
Signature · 1 line
2.17.1
PATCH v6 1/12 iommu/vt-d: Enumerate the scalable mode capability
The Intel vt-d spec rev3.0 introduces a new translation mode called scalable mode, which enables PASID-granular translations for first level, second level, nested and pass-through modes. At the same time, the previous Extended Context (ECS) mode is deprecated (no production ever implements ECS).
This patch adds enumeration for Scalable Mode and removes the deprecated ECS enumeration. It provides a boot time option to disable scalable mode even hardware claims to support it.
--- .../admin-guide/kernel-parameters.txt | 12 ++-- drivers/iommu/intel-iommu.c | 64 +++++-------------- include/linux/intel-iommu.h | 1 + 3 files changed, 24 insertions(+), 53 deletions(-)
Documentation/admin-guide/kernel-parameters.txt +6 -6
@@ -1683,12 +1683,12 @@ By default, super page will be supported if Intel IOMMU has the capability. With this option, super page will not be supported. - ecs_off [Default Off] - By default, extended context tables will be supported if - the hardware advertises that it has support both for the - extended tables themselves, and also PASID support. With - this option set, extended tables will not be used even - on hardware which claims to support them. + sm_off [Default Off] + By default, scalable mode will be supported if the + hardware advertises that it has support for the scalable + mode translation. With this option set, scalable mode + will not be used even on hardware which claims to support + it. tboot_noforce [Default Off] Do not force the Intel IOMMU enabled under tboot. By default, tboot will force Intel IOMMU on, which
drivers/iommu/intel-iommu.c +17 -47
142 lines not on this page · raw message
include/linux/intel-iommu.h +1 -0
@@ -177,6 +177,7 @@* Extended Capability Register */ +#define ecap_smts (e) (((e) >> 43) & 0x1)#define ecap_dit (e) ((e >> 41) & 0x1)#define ecap_pasid (e) ((e >> 40) & 0x1)#define ecap_pss (e) ((e >> 35) & 0x1f)
Signature · 6 lines
2.17.1 _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
PATCH v6 2/12 iommu/vt-d: Manage scalalble mode PASID tables
In reply to Lu BaoluIn scalable mode, pasid structure is a two level table with a pasid directory table and a pasid table. Any pasid entry can be identified by a pasid value in below way.
1
9 6 5 0
.-----------------------.-------.
| PASID | |
'-----------------------'-------' .-------------.
| | | |
| | | |
| | | |
| .-----------. | .-------------.
| | | |----->| PASID Entry |
| | | | '-------------'
| | | |Plus | |
| .-----------. | | |
|---->| DIR Entry |-------->| |
| '-----------' '-------------'
.---------. |Plus | |
| Context | | | |
| Entry |------->| |
'---------' '-----------'This changes the pasid table APIs to support scalable mode PASID directory and PASID table. It also adds a helper to get the PASID table entry according to the pasid value.
--- drivers/iommu/intel-iommu.c | 23 ++++++---- drivers/iommu/intel-pasid.c | 87 ++++++++++++++++++++++++++++++------- drivers/iommu/intel-pasid.h | 12 ++++- drivers/iommu/intel-svm.c | 6 +-- 4 files changed, 97 insertions(+), 31 deletions(-)
drivers/iommu/intel-iommu.c +14 -9
@@ -425,21 +425,24 @@ static LIST_HEAD(device_domain_list);/* * Iterate over elements in device_domain_list and call the specified -* callback @fn against each element. This helper should only be used -* in the context where the device_domain_lock has already been holden. +* callback @fn against each element. */ int for_each_device_domain (int (*fn)(struct device_domain_info *info,void *data),void *data) {int ret =0 ; +unsigned long flags;struct device_domain_info *info; -assert_spin_locked (&device_domain_lock); +spin_lock_irqsave (&device_domain_lock, flags);list_for_each_entry (info, &device_domain_list, global) { ret =fn (info, data); -if (ret) +if (ret) { +spin_unlock_irqrestore (&device_domain_lock, flags);return ret; + } } +spin_unlock_irqrestore (&device_domain_lock, flags);return 0 ; }
@@ -2481,16 +2484,18 @@ static struct dmar_domain *dmar_insert_one_dev_info(struct intel_iommu *iommu,list_add (&info->global, &device_domain_list);if (dev) dev->archdata.iommu = info; +spin_unlock_irqrestore (&device_domain_lock, flags); -if (dev &&dev_is_pci (dev) && info->pasid_supported) { +/* PASID table is mandatory for a PCI device in scalable mode. */ +if (dev &&dev_is_pci (dev) &&sm_supported (iommu)) { ret =intel_pasid_alloc_table (dev);if (ret) { -pr_warn ("No pasid table for %s, pasid disabled\n" , -dev_name (dev)); - info->pasid_supported =0 ; +pr_err ("PASID table allocation for %s failed\n" , +dev_name (dev)); +dmar_remove_one_dev_info (domain, dev); +return NULL ; } } -spin_unlock_irqrestore (&device_domain_lock, flags);if (dev &&domain_context_mapping (domain, dev)) {pr_err ("Domain context map for %s failed\n" ,dev_name (dev));
drivers/iommu/intel-pasid.c +72 -15
151 lines not on this page · raw message
drivers/iommu/intel-pasid.h +10 -2
@@ -11,12 +11,20 @@#define __INTEL_PASID_H#define PASID_MIN 0x1 -#define PASID_MAX 0x20000 +#define PASID_MAX 0x100000 +#define PASID_PTE_MASK 0x3F +#define PASID_PTE_PRESENT 1 +#define PDE_PFN_MASK PAGE_MASK +#define PASID_PDE_SHIFT 6 -struct pasid_entry { +struct pasid_dir_entry {u64 val; }; +struct pasid_entry { +u64 val[8 ]; +}; +/* The representative of a PASID table */ struct pasid_table {void *table;/* pasid table pointer */
drivers/iommu/intel-svm.c +1 -5
@@ -65,8 +65,6 @@ int intel_svm_init(struct intel_iommu *iommu) order =get_order (sizeof (struct pasid_entry ) * iommu->pasid_max);if (ecap_dis (iommu->ecap)) { -/* Just making it explicit... */ -BUILD_BUG_ON (sizeof (struct pasid_entry ) !=sizeof (struct pasid_state_entry )); pages =alloc_pages (GFP_KERNEL | __GFP_ZERO, order);if (pages) iommu->pasid_state_table =page_address (pages);
@@ -405,9 +403,7 @@ int intel_svm_bind_mm(struct device *dev, int *pasid, int flags, struct svm_dev_ pasid_entry_val |=PASID_ENTRY_FLPM_5LP ; entry =intel_pasid_get_entry (dev, svm->pasid); - entry->val = pasid_entry_val; - -wmb (); +WRITE_ONCE (entry->val[0 ], pasid_entry_val);/* * Flush PASID cache when a PASID table entry becomes
Signature · 1 line
2.17.1
PATCH v6 3/12 iommu/vt-d: Move page table helpers into header
In reply to Lu BaoluSo that they could also be used in other source files.
--- drivers/iommu/intel-iommu.c | 43 ------------------------------------- include/linux/intel-iommu.h | 43 +++++++++++++++++++++++++++++++++++++ 2 files changed, 43 insertions(+), 43 deletions(-)
drivers/iommu/intel-iommu.c +0 -43
@@ -290,49 +290,6 @@ static inline void context_clear_entry(struct context_entry *context) context->hi =0 ; } -/* -* 0: readable -* 1: writable -* 2-6: reserved -* 7: super page -* 8-10: available -* 11: snoop behavior -* 12-63: Host physcial address -*/ -struct dma_pte { -u64 val; -}; - -static inline void dma_clear_pte (struct dma_pte *pte) -{ - pte->val =0 ; -} - -static inline u64 dma_pte_addr (struct dma_pte *pte) -{ -#ifdef CONFIG_64BIT -return pte->val &VTD_PAGE_MASK ; -#else -/* Must have a full atomic 64-bit read */ -return __cmpxchg64 (&pte->val,0ULL ,0ULL ) &VTD_PAGE_MASK ; -#endif -} - -static inline bool dma_pte_present (struct dma_pte *pte) -{ -return (pte->val &3 ) !=0 ; -} - -static inline bool dma_pte_superpage (struct dma_pte *pte) -{ -return (pte->val &DMA_PTE_LARGE_PAGE ); -} - -static inline int first_pte_in_page (struct dma_pte *pte) -{ -return !((unsigned long )pte & ~VTD_PAGE_MASK ); -} -/* * This domain is a statically identity mapping domain. * 1. This domain creats a static 1:1 mapping to all usable memory.
include/linux/intel-iommu.h +43 -0
49 lines not on this page · raw message
Signature · 1 line
2.17.1
PATCH v6 4/12 iommu/vt-d: Add 256-bit invalidation descriptor support
In reply to Lu BaoluIntel vt-d spec rev3.0 requires software to use 256-bit descriptors in invalidation queue. As the spec reads in section 6.5.2:
Remapping hardware supporting Scalable Mode Translations (ECAP_REG.SMTS=1) allow software to additionally program the width of the descriptors (128-bits or 256-bits) that will be written into the Queue. Software should setup the Invalidation Queue for 256-bit descriptors before progra- mming remapping hardware for scalable-mode translation as 128-bit descriptors are treated as invalid descriptors (see Table 21 in Section 6.5.2.10) in scalable-mode.
This patch adds 256-bit invalidation descriptor support if the hardware presents scalable mode capability.
--- drivers/iommu/dmar.c | 91 +++++++++++++++++++---------- drivers/iommu/intel-svm.c | 76 +++++++++++++++--------- drivers/iommu/intel_irq_remapping.c | 6 +- include/linux/intel-iommu.h | 9 ++- 4 files changed, 121 insertions(+), 61 deletions(-)
drivers/iommu/dmar.c +61 -30
@@ -1160,6 +1160,7 @@ static int qi_check_fault(struct intel_iommu *iommu, int index)int head, tail;struct q_inval *qi = iommu->qi;int wait_index = (index +1 ) %QI_LENGTH ; +int shift =qi_shift (iommu);if (qi->desc_status[wait_index] ==QI_ABORT )return -EAGAIN ;
@@ -1173,13 +1174,19 @@ static int qi_check_fault(struct intel_iommu *iommu, int index)*/ if (fault &DMA_FSTS_IQE ) { head =readl (iommu->reg +DMAR_IQH_REG ); -if ((head >>DMAR_IQ_SHIFT ) == index) { -pr_err ("VT-d detected invalid descriptor: " -"low=%llx, high=%llx\n" , - (unsigned long long )qi->desc[index].low, - (unsigned long long )qi->desc[index].high); -memcpy (&qi->desc[index], &qi->desc[wait_index], -sizeof (struct qi_desc )); +if ((head >> shift) == index) { +struct qi_desc *desc = qi->desc + head; + +/* +* desc->qw2 and desc->qw3 are either reserved or +* used by software as private data. We won't print +* out these two qw's for security consideration. +*/ +pr_err ("VT-d detected invalid descriptor: qw0 = %llx, qw1 = %llx\n" , + (unsigned long long )desc->qw0, + (unsigned long long )desc->qw1); +memcpy (desc, qi->desc + (wait_index << shift), +1 << shift);writel (DMA_FSTS_IQE , iommu->reg +DMAR_FSTS_REG );return -EINVAL ; }
@@ -1191,10 +1198,10 @@ static int qi_check_fault(struct intel_iommu *iommu, int index)*/ if (fault &DMA_FSTS_ITE ) { head =readl (iommu->reg +DMAR_IQH_REG ); - head = ((head >>DMAR_IQ_SHIFT ) -1 +QI_LENGTH ) %QI_LENGTH ; + head = ((head >> shift) -1 +QI_LENGTH ) %QI_LENGTH ; head |=1 ; tail =readl (iommu->reg +DMAR_IQT_REG ); - tail = ((tail >>DMAR_IQ_SHIFT ) -1 +QI_LENGTH ) %QI_LENGTH ; + tail = ((tail >> shift) -1 +QI_LENGTH ) %QI_LENGTH ;writel (DMA_FSTS_ITE , iommu->reg +DMAR_FSTS_REG );
@@ -1222,15 +1229,14 @@ int qi_submit_sync(struct qi_desc *desc, struct intel_iommu *iommu) {int rc;struct q_inval *qi = iommu->qi; -struct qi_desc *hw, wait_desc; +int offset, shift, length; +struct qi_desc wait_desc;int wait_index, index;unsigned long flags;if (!qi)return 0 ; - hw = qi->desc; - restart: rc =0 ;
@@ -1243,16 +1249,21 @@ int qi_submit_sync(struct qi_desc *desc, struct intel_iommu *iommu) index = qi->free_head; wait_index = (index +1 ) %QI_LENGTH ; + shift =qi_shift (iommu); + length =1 << shift; qi->desc_status[index] = qi->desc_status[wait_index] =QI_IN_USE ; - hw[index] = *desc; - - wait_desc.low =QI_IWD_STATUS_DATA (QI_DONE ) | + offset = index << shift; +memcpy (qi->desc + offset, desc, length); + wait_desc.qw0 =QI_IWD_STATUS_DATA (QI_DONE ) |QI_IWD_STATUS_WRITE |QI_IWD_TYPE ; - wait_desc.high =virt_to_phys (&qi->desc_status[wait_index]); + wait_desc.qw1 =virt_to_phys (&qi->desc_status[wait_index]); + wait_desc.qw2 =0 ; + wait_desc.qw3 =0 ; - hw[wait_index] = wait_desc; + offset = wait_index << shift; +memcpy (qi->desc + offset, &wait_desc, length); qi->free_head = (qi->free_head +2 ) %QI_LENGTH ; qi->free_cnt -=2 ;
@@ -1261,7 +1272,7 @@ int qi_submit_sync(struct qi_desc *desc, struct intel_iommu *iommu)* update the HW tail register indicating the presence of * new descriptors. */ -writel (qi->free_head <<DMAR_IQ_SHIFT , iommu->reg +DMAR_IQT_REG ); +writel (qi->free_head << shift, iommu->reg +DMAR_IQT_REG );while (qi->desc_status[wait_index] !=QI_DONE ) {/*
@@ -1298,8 +1309,10 @@ void qi_global_iec(struct intel_iommu *iommu) {struct qi_desc desc; - desc.low =QI_IEC_TYPE ; - desc.high =0 ; + desc.qw0 =QI_IEC_TYPE ; + desc.qw1 =0 ; + desc.qw2 =0 ; + desc.qw3 =0 ;/* should never fail */ qi_submit_sync (&desc, iommu);
@@ -1310,9 +1323,11 @@ void qi_flush_context(struct intel_iommu *iommu, u16 did, u16 sid, u8 fm, {struct qi_desc desc; - desc.low =QI_CC_FM (fm) |QI_CC_SID (sid) |QI_CC_DID (did) + desc.qw0 =QI_CC_FM (fm) |QI_CC_SID (sid) |QI_CC_DID (did) |QI_CC_GRAN (type) |QI_CC_TYPE ; - desc.high =0 ; + desc.qw1 =0 ; + desc.qw2 =0 ; + desc.qw3 =0 ;qi_submit_sync (&desc, iommu); }
@@ -1331,10 +1346,12 @@ void qi_flush_iotlb(struct intel_iommu *iommu, u16 did, u64 addr,if (cap_read_drain (iommu->cap)) dr =1 ; - desc.low =QI_IOTLB_DID (did) |QI_IOTLB_DR (dr) |QI_IOTLB_DW (dw) + desc.qw0 =QI_IOTLB_DID (did) |QI_IOTLB_DR (dr) |QI_IOTLB_DW (dw) |QI_IOTLB_GRAN (type) |QI_IOTLB_TYPE ; - desc.high =QI_IOTLB_ADDR (addr) |QI_IOTLB_IH (ih) + desc.qw1 =QI_IOTLB_ADDR (addr) |QI_IOTLB_IH (ih) |QI_IOTLB_AM (size_order); + desc.qw2 =0 ; + desc.qw3 =0 ;qi_submit_sync (&desc, iommu); }
@@ -1347,15 +1364,17 @@ void qi_flush_dev_iotlb(struct intel_iommu *iommu, u16 sid, u16 pfsid,if (mask) {WARN_ON_ONCE (addr & ((1ULL << (VTD_PAGE_SHIFT + mask)) -1 )); addr |= (1ULL << (VTD_PAGE_SHIFT + mask -1 )) -1 ; - desc.high =QI_DEV_IOTLB_ADDR (addr) |QI_DEV_IOTLB_SIZE ; + desc.qw1 =QI_DEV_IOTLB_ADDR (addr) |QI_DEV_IOTLB_SIZE ; }else - desc.high =QI_DEV_IOTLB_ADDR (addr); + desc.qw1 =QI_DEV_IOTLB_ADDR (addr);if (qdep >=QI_DEV_IOTLB_MAX_INVS ) qdep =0 ; - desc.low =QI_DEV_IOTLB_SID (sid) |QI_DEV_IOTLB_QDEP (qdep) | + desc.qw0 =QI_DEV_IOTLB_SID (sid) |QI_DEV_IOTLB_QDEP (qdep) |QI_DIOTLB_TYPE |QI_DEV_IOTLB_PFSID (pfsid); + desc.qw2 =0 ; + desc.qw3 =0 ;qi_submit_sync (&desc, iommu); }
@@ -1403,16 +1422,24 @@ static void __dmar_enable_qi(struct intel_iommu *iommu)u32 sts;unsigned long flags;struct q_inval *qi = iommu->qi; +u64 val =virt_to_phys (qi->desc); qi->free_head = qi->free_tail =0 ; qi->free_cnt =QI_LENGTH ; +/* +* Set DW=1 and QS=1 in IQA_REG when Scalable Mode capability +* is present. +*/ +if (ecap_smts (iommu->ecap)) + val |= (1 <<11 ) |1 ; +raw_spin_lock_irqsave (&iommu->register_lock, flags);/* write zero to the tail reg */ writel (0 , iommu->reg +DMAR_IQT_REG ); -dmar_writeq (iommu->reg +DMAR_IQA_REG ,virt_to_phys (qi->desc)); +dmar_writeq (iommu->reg +DMAR_IQA_REG , val); iommu->gcmd |=DMA_GCMD_QIE ;writel (iommu->gcmd, iommu->reg +DMAR_GCMD_REG );
@@ -1448,8 +1475,12 @@ int dmar_enable_qi(struct intel_iommu *iommu) qi = iommu->qi; - - desc_page =alloc_pages_node (iommu->node,GFP_ATOMIC | __GFP_ZERO,0 ); +/* +* Need two pages to accommodate 256 descriptors of 256 bits each +* if the remapping hardware supports scalable mode translation. +*/ + desc_page =alloc_pages_node (iommu->node,GFP_ATOMIC | __GFP_ZERO, + !!ecap_smts (iommu->ecap));if (!desc_page) {kfree (qi); iommu->qi =NULL ;
drivers/iommu/intel-svm.c +49 -27
118 lines not on this page · raw message
drivers/iommu/intel_irq_remapping.c +4 -2
@@ -145,9 +145,11 @@ static int qi_flush_iec(struct intel_iommu *iommu, int index, int mask) {struct qi_desc desc; - desc.low =QI_IEC_IIDEX (index) |QI_IEC_TYPE |QI_IEC_IM (mask) + desc.qw0 =QI_IEC_IIDEX (index) |QI_IEC_TYPE |QI_IEC_IM (mask) |QI_IEC_SELECTIVE ; - desc.high =0 ; + desc.qw1 =0 ; + desc.qw2 =0 ; + desc.qw3 =0 ;return qi_submit_sync (&desc, iommu); }
include/linux/intel-iommu.h +7 -2
@@ -401,13 +401,18 @@ enum {#define QI_GRAN_NONG_PASID 2#define QI_GRAN_PSI_PASID 3 +#define qi_shift (iommu) (DMAR_IQ_SHIFT + !!ecap_smts((iommu)->ecap)) +struct qi_desc { -u64 low, high; +u64 qw0; +u64 qw1; +u64 qw2; +u64 qw3; };struct q_inval {raw_spinlock_t q_lock; -struct qi_desc *desc;/* invalidation queue */ +void *desc;/* invalidation queue */ int *desc_status;/* desc status */ int free_head;/* first free entry */ int free_tail;/* last free entry */
Signature · 1 line
2.17.1
PATCH v6 5/12 iommu/vt-d: Reserve a domain id for FL and PT modes
In reply to Lu BaoluVt-d spec rev3.0 (section 6.2.3.1) requires that each pasid entry for first-level or pass-through translation should be programmed with a domain id different from those used for second-level or nested translation. It is recommended that software could use a same domain id for all first-only and pass-through translations.
This reserves a domain id for first-level and pass-through translations.
--- drivers/iommu/intel-iommu.c | 10 ++++++++++ drivers/iommu/intel-pasid.h | 6 ++++++ 2 files changed, 16 insertions(+)
drivers/iommu/intel-iommu.c +10 -0
@@ -1621,6 +1621,16 @@ static int iommu_init_domains(struct intel_iommu *iommu)*/ set_bit (0 , iommu->domain_ids); +/* +* Vt-d spec rev3.0 (section 6.2.3.1) requires that each pasid +* entry for first-level or pass-through translation modes should +* be programmed with a domain id different from those used for +* second-level or nested translation. We reserve a domain id for +* this purpose. +*/ +if (sm_supported (iommu)) +set_bit (FLPT_DEFAULT_DID , iommu->domain_ids); +return 0 ; }
drivers/iommu/intel-pasid.h +6 -0
@@ -17,6 +17,12 @@#define PDE_PFN_MASK PAGE_MASK#define PASID_PDE_SHIFT 6 +/* +* Domain ID reserved for pasid entries programmed for first-level +* only and pass-through transfer modes. +*/ +#define FLPT_DEFAULT_DID 1 +struct pasid_dir_entry {u64 val; };
Signature · 1 line
2.17.1
PATCH v6 6/12 iommu/vt-d: Add second level page table interface
In reply to Lu BaoluThis adds the interfaces to setup or tear down the structures for second level page table translations. This includes types of second level only translation and pass through.
--- drivers/iommu/intel-iommu.c | 2 +- drivers/iommu/intel-pasid.c | 280 ++++++++++++++++++++++++++++++++++++ drivers/iommu/intel-pasid.h | 8 ++ include/linux/intel-iommu.h | 3 + 4 files changed, 292 insertions(+), 1 deletion(-)
drivers/iommu/intel-iommu.c +1 -1
@@ -1210,7 +1210,7 @@ static void iommu_set_root_entry(struct intel_iommu *iommu)raw_spin_unlock_irqrestore (&iommu->register_lock, flag); } -static void iommu_flush_write_buffer (struct intel_iommu *iommu) +void iommu_flush_write_buffer (struct intel_iommu *iommu) {u32 val;unsigned long flag;
drivers/iommu/intel-pasid.c +280 -0
289 lines not on this page · raw message
drivers/iommu/intel-pasid.h +8 -0
@@ -49,5 +49,13 @@ struct pasid_table *intel_pasid_get_table(struct device *dev);int intel_pasid_get_dev_max_id (struct device *dev);struct pasid_entry *intel_pasid_get_entry (struct device *dev,int pasid);void intel_pasid_clear_entry (struct device *dev,int pasid); +int intel_pasid_setup_second_level (struct intel_iommu *iommu, +struct dmar_domain *domain, +struct device *dev,int pasid); +int intel_pasid_setup_pass_through (struct intel_iommu *iommu, +struct dmar_domain *domain, +struct device *dev,int pasid); +void intel_pasid_tear_down_entry (struct intel_iommu *iommu, +struct device *dev,int pasid);#endif /* __INTEL_PASID_H */
include/linux/intel-iommu.h +3 -0
@@ -177,6 +177,8 @@* Extended Capability Register */ +#define ecap_smpwc (e) (((e) >> 48) & 0x1) +#define ecap_slts (e) (((e) >> 46) & 0x1)#define ecap_smts (e) (((e) >> 43) & 0x1)#define ecap_dit (e) ((e >> 41) & 0x1)#define ecap_pasid (e) ((e >> 40) & 0x1)
@@ -662,6 +664,7 @@ void free_pgtable_page(void *vaddr);struct intel_iommu *domain_get_iommu (struct dmar_domain *domain);int for_each_device_domain (int (*fn)(struct device_domain_info *info,void *data),void *data); +void iommu_flush_write_buffer (struct intel_iommu *iommu);#ifdef CONFIG_INTEL_IOMMU_SVM int intel_svm_init (struct intel_iommu *iommu);
Signature · 1 line
2.17.1
PATCH v6 7/12 iommu/vt-d: Setup pasid entry for RID2PASID support
In reply to Lu Baoluwhen the scalable mode is enabled, there is no second level page translation pointer in the context entry any more (for DMA request without PASID). Instead, a new RID2PASID field is introduced in the context entry. Software can choose any PASID value to set RID2PASID and then setup the translation in the corresponding PASID entry. Upon receiving a DMA request without PASID, hardware will firstly look at this RID2PASID field and then treat this request as a request with a pasid value specified in RID2PASID field.
Though software is allowed to use any PASID for the RID2PASID, we will always use the PASID 0 as a sort of design decision.
--- drivers/iommu/intel-iommu.c | 20 ++++++++++++++++++++ drivers/iommu/intel-pasid.h | 1 + 2 files changed, 21 insertions(+)
drivers/iommu/intel-iommu.c +20 -0
@@ -2462,6 +2462,22 @@ static struct dmar_domain *dmar_insert_one_dev_info(struct intel_iommu *iommu,dmar_remove_one_dev_info (domain, dev);return NULL ; } + +/* Setup the PASID entry for requests without PASID: */ +spin_lock (&iommu->lock); +if (hw_pass_through &&domain_type_is_si (domain)) + ret =intel_pasid_setup_pass_through (iommu, domain, + dev,PASID_RID2PASID ); +else + ret =intel_pasid_setup_second_level (iommu, domain, + dev,PASID_RID2PASID ); +spin_unlock (&iommu->lock); +if (ret) { +pr_err ("Setup RID2PASID for %s failed\n" , +dev_name (dev)); +dmar_remove_one_dev_info (domain, dev); +return NULL ; + } }if (dev &&domain_context_mapping (domain, dev)) {
@@ -4825,6 +4841,10 @@ static void __dmar_remove_one_dev_info(struct device_domain_info *info) iommu = info->iommu;if (info->dev) { +if (dev_is_pci (info->dev) &&sm_supported (iommu)) +intel_pasid_tear_down_entry (iommu, info->dev, +PASID_RID2PASID ); +iommu_disable_dev_iotlb (info);domain_context_clear (iommu, info->dev);intel_pasid_free_table (info->dev);
drivers/iommu/intel-pasid.h +1 -0
@@ -10,6 +10,7 @@#ifndef __INTEL_PASID_H#define __INTEL_PASID_H +#define PASID_RID2PASID 0x0#define PASID_MIN 0x1#define PASID_MAX 0x100000#define PASID_PTE_MASK 0x3F
Signature · 1 line
2.17.1
PATCH v6 8/12 iommu/vt-d: Pass pasid table to context mapping
In reply to Lu BaoluSo that the pasid related info, such as the pasid table and the maximum of pasid could be used during setting up scalable mode context.
--- drivers/iommu/intel-iommu.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-)
drivers/iommu/intel-iommu.c +11 -3
@@ -1920,6 +1920,7 @@ static void domain_exit(struct dmar_domain *domain)static int domain_context_mapping_one (struct dmar_domain *domain,struct intel_iommu *iommu, +struct pasid_table *table,u8 bus,u8 devfn) {u16 did = domain->iommu_did[iommu->seq_id];
@@ -2042,6 +2043,7 @@ static int domain_context_mapping_one(struct dmar_domain *domain,struct domain_context_mapping_data {struct dmar_domain *domain;struct intel_iommu *iommu; +struct pasid_table *table; }; static int domain_context_mapping_cb(struct pci_dev *pdev,
@@ -2050,25 +2052,31 @@ static int domain_context_mapping_cb(struct pci_dev *pdev,struct domain_context_mapping_data *data = opaque;return domain_context_mapping_one (data->domain, data->iommu, -PCI_BUS_NUM (alias), alias &0xff ); + data->table,PCI_BUS_NUM (alias), + alias &0xff ); }static int domain_context_mapping (struct dmar_domain *domain,struct device *dev) { +struct domain_context_mapping_data data; +struct pasid_table *table;struct intel_iommu *iommu;u8 bus, devfn; -struct domain_context_mapping_data data; iommu =device_to_iommu (dev, &bus, &devfn);if (!iommu)return -ENODEV ; + table =intel_pasid_get_table (dev); +if (!dev_is_pci (dev)) -return domain_context_mapping_one (domain, iommu, bus, devfn); +return domain_context_mapping_one (domain, iommu, table, + bus, devfn); data.domain = domain; data.iommu = iommu; + data.table = table;return pci_for_each_dma_alias (to_pci_dev (dev), &domain_context_mapping_cb, &data);
Signature · 1 line
2.17.1
PATCH v6 9/12 iommu/vt-d: Setup context and enable RID2PASID support
In reply to Lu BaoluThis patch enables the translation for requests without PASID in the scalable mode by setting up the root and context entries.
--- drivers/iommu/intel-iommu.c | 136 ++++++++++++++++++++++++++++-------- drivers/iommu/intel-pasid.h | 1 + include/linux/intel-iommu.h | 1 + 3 files changed, 108 insertions(+), 30 deletions(-)
drivers/iommu/intel-iommu.c +106 -30
@@ -1197,6 +1197,8 @@ static void iommu_set_root_entry(struct intel_iommu *iommu)unsigned long flag; addr =virt_to_phys (iommu->root_entry); +if (sm_supported (iommu)) + addr |=DMA_RTADDR_SMT ;raw_spin_lock_irqsave (&iommu->register_lock, flag);dmar_writeq (iommu->reg +DMAR_RTADDR_REG , addr);
@@ -1918,6 +1920,56 @@ static void domain_exit(struct dmar_domain *domain)free_domain_mem (domain); } +/* +* Get the PASID directory size for scalable mode context entry. +* Value of X in the PDTS field of a scalable mode context entry +* indicates PASID directory with 2^(X + 7) entries. +*/ +static inline unsigned long context_get_sm_pds (struct pasid_table *table) +{ +int pds, max_pde; + + max_pde = table->max_pasid >>PASID_PDE_SHIFT ; + pds =find_first_bit ((unsigned long *)&max_pde,MAX_NR_PASID_BITS ); +if (pds <7 ) +return 0 ; + +return pds -7 ; +} + +/* +* Set the RID_PASID field of a scalable mode context entry. The +* IOMMU hardware will use the PASID value set in this field for +* DMA translations of DMA requests without PASID. +*/ +static inline void +context_set_sm_rid2pasid (struct context_entry *context,unsigned long pasid) +{ + context->hi |= pasid & ((1 <<20 ) -1 ); + context->hi |= (1 <<20 ); +} + +/* +* Set the DTE(Device-TLB Enable) field of a scalable mode context +* entry. +*/ +static inline void context_set_sm_dte (struct context_entry *context) +{ + context->lo |= (1 <<2 ); +} + +/* +* Set the PRE(Page Request Enable) field of a scalable mode context +* entry. +*/ +static inline void context_set_sm_pre (struct context_entry *context) +{ + context->lo |= (1 <<4 ); +} + +/* Convert value to context PASID directory size field coding. */ +#define context_pdts (pds) (((pds) & 0x7) << 9) + static int domain_context_mapping_one(struct dmar_domain *domain, struct intel_iommu *iommu, struct pasid_table *table,
@@ -1928,8 +1980,7 @@ static int domain_context_mapping_one(struct dmar_domain *domain,struct device_domain_info *info =NULL ;struct context_entry *context;unsigned long flags; -struct dma_pte *pgd; -int ret, agaw; +int ret;WARN_ON (did ==0 );
@@ -1975,41 +2026,67 @@ static int domain_context_mapping_one(struct dmar_domain *domain, } } - pgd = domain->pgd; -context_clear_entry (context); -context_set_domain_id (context, did); -/* -* Skip top levels of page tables for iommu which has less agaw -* than default. Unnecessary for PT mode. -*/ -if (translation !=CONTEXT_TT_PASS_THROUGH ) { -for (agaw = domain->agaw; agaw > iommu->agaw; agaw--) { - ret = -ENOMEM ; - pgd =phys_to_virt (dma_pte_addr (pgd)); -if (!dma_pte_present (pgd)) - goto out_unlock; - } +if (sm_supported (iommu)) { +unsigned long pds; - info =iommu_support_dev_iotlb (domain, iommu, bus, devfn); -if (info && info->ats_supported) - translation =CONTEXT_TT_DEV_IOTLB ; -else - translation =CONTEXT_TT_MULTI_LEVEL ; +WARN_ON (!table); + +/* Setup the PASID DIR pointer: */ + pds =context_get_sm_pds (table); + context->lo = (u64 )virt_to_phys (table->table) | +context_pdts (pds); + +/* Setup the RID_PASID field: */ +context_set_sm_rid2pasid (context,PASID_RID2PASID ); -context_set_address_root (context,virt_to_phys (pgd)); -context_set_address_width (context, agaw); - }else {/* -* In pass through mode, AW must be programmed to -* indicate the largest AGAW value supported by -* hardware. And ASR is ignored by hardware. +* Setup the Device-TLB enable bit and Page request +* Enable bit: */ -context_set_address_width (context, iommu->msagaw); + info =iommu_support_dev_iotlb (domain, iommu, bus, devfn); +if (info && info->ats_supported) +context_set_sm_dte (context); +if (info && info->pri_supported) +context_set_sm_pre (context); + }else { +struct dma_pte *pgd = domain->pgd; +int agaw; + +context_set_domain_id (context, did); +context_set_translation_type (context, translation); + +if (translation !=CONTEXT_TT_PASS_THROUGH ) { +/* +* Skip top levels of page tables for iommu which has +* less agaw than default. Unnecessary for PT mode. +*/ +for (agaw = domain->agaw; agaw > iommu->agaw; agaw--) { + ret = -ENOMEM ; + pgd =phys_to_virt (dma_pte_addr (pgd)); +if (!dma_pte_present (pgd)) + goto out_unlock; + } + + info =iommu_support_dev_iotlb (domain, iommu, bus, devfn); +if (info && info->ats_supported) + translation =CONTEXT_TT_DEV_IOTLB ; +else + translation =CONTEXT_TT_MULTI_LEVEL ; + +context_set_address_root (context,virt_to_phys (pgd)); +context_set_address_width (context, agaw); + }else { +/* +* In pass through mode, AW must be programmed to +* indicate the largest AGAW value supported by +* hardware. And ASR is ignored by hardware. +*/ +context_set_address_width (context, iommu->msagaw); + } } -context_set_translation_type (context, translation);context_set_fault_enable (context);context_set_present (context);domain_flush_cache (domain, context,sizeof (*context));
@@ -5180,7 +5257,6 @@ static void intel_iommu_put_resv_regions(struct device *dev, }#ifdef CONFIG_INTEL_IOMMU_SVM -#define MAX_NR_PASID_BITS (20)static inline unsigned long intel_iommu_get_pts (struct device *dev) {int pts, max_pasid;
drivers/iommu/intel-pasid.h +1 -0
@@ -17,6 +17,7 @@#define PASID_PTE_PRESENT 1#define PDE_PFN_MASK PAGE_MASK#define PASID_PDE_SHIFT 6 +#define MAX_NR_PASID_BITS 20/* * Domain ID reserved for pasid entries programmed for first-level
include/linux/intel-iommu.h +1 -0
@@ -258,6 +258,7 @@/* DMA_RTADDR_REG */ #define DMA_RTADDR_RTT (((u64)1) << 11) +#define DMA_RTADDR_SMT (((u64)1) << 10)/* CCMD_REG */ #define DMA_CCMD_ICC (((u64)1) << 63)
Signature · 1 line
2.17.1
PATCH v6 10/12 iommu/vt-d: Add first level page table interface
In reply to Lu BaoluThis adds an interface to setup the PASID entries for first level page table translation.
--- drivers/iommu/intel-pasid.c | 80 +++++++++++++++++++++++++++++++++++++ drivers/iommu/intel-pasid.h | 11 +++++ include/linux/intel-iommu.h | 1 + 3 files changed, 92 insertions(+)
drivers/iommu/intel-pasid.c +80 -0
@@ -10,6 +10,7 @@#define pr_fmt (fmt) "DMAR: " fmt#include <linux/bitops.h> +#include <linux/cpufeature.h> #include <linux/dmar.h> #include <linux/intel-iommu.h> #include <linux/iommu.h>
@@ -389,6 +390,26 @@ static inline void pasid_set_page_snoop(struct pasid_entry *pe, bool value)pasid_set_bits (&pe->val[1 ],1 <<23 , value); } +/* +* Setup the First Level Page table Pointer field (Bit 140~191) +* of a scalable mode PASID entry. +*/ +static inline void +pasid_set_flptr (struct pasid_entry *pe,u64 value) +{ +pasid_set_bits (&pe->val[2 ],VTD_PAGE_MASK , value); +} + +/* +* Setup the First Level Paging Mode field (Bit 130~131) of a +* scalable mode PASID entry. +*/ +static inline void +pasid_set_flpm (struct pasid_entry *pe,u64 value) +{ +pasid_set_bits (&pe->val[2 ],GENMASK_ULL (3 ,2 ), value <<2 ); +} +static void pasid_cache_invalidation_with_pasid (struct intel_iommu *iommu,u16 did,int pasid)
@@ -459,6 +480,65 @@ void intel_pasid_tear_down_entry(struct intel_iommu *iommu,devtlb_invalidation_with_pasid (iommu, dev, pasid); } +/* +* Set up the scalable mode pasid table entry for first only +* translation type. +*/ +int intel_pasid_setup_first_level (struct intel_iommu *iommu, +struct device *dev,pgd_t *pgd, +int pasid,u16 did,int flags) +{ +struct pasid_entry *pte; + +if (!ecap_flts (iommu->ecap)) { +pr_err ("No first level translation support on %s\n" , + iommu->name); +return -EINVAL ; + } + + pte =intel_pasid_get_entry (dev, pasid); +if (WARN_ON (!pte)) +return -EINVAL ; + +pasid_clear_entry (pte); + +/* Setup the first level page table pointer: */ +pasid_set_flptr (pte, (u64 )__pa (pgd)); +if (flags &PASID_FLAG_SUPERVISOR_MODE ) { +if (!ecap_srs (iommu->ecap)) { +pr_err ("No supervisor request support on %s\n" , + iommu->name); +return -EINVAL ; + } +pasid_set_sre (pte); + } + +#ifdef CONFIG_X86 +if (cpu_feature_enabled (X86_FEATURE_LA57 )) +pasid_set_flpm (pte,1 ); +#endif /* CONFIG_X86 */ + +pasid_set_domain_id (pte, did); +pasid_set_address_width (pte, iommu->agaw); +pasid_set_page_snoop (pte, !!ecap_smpwc (iommu->ecap)); + +/* Setup Present and PASID Granular Transfer Type: */ +pasid_set_translation_type (pte,1 ); +pasid_set_present (pte); + +if (!ecap_coherent (iommu->ecap)) +clflush_cache_range (pte,sizeof (*pte)); + +if (cap_caching_mode (iommu->cap)) { +pasid_cache_invalidation_with_pasid (iommu, did, pasid); +iotlb_invalidation_with_pasid (iommu, did, pasid); + }else { +iommu_flush_write_buffer (iommu); + } + +return 0 ; +} +/* * Set up the scalable mode pasid entry for second only translation type. */
drivers/iommu/intel-pasid.h +11 -0
@@ -25,6 +25,14 @@*/ #define FLPT_DEFAULT_DID 1 +/* +* The SUPERVISOR_MODE flag indicates a first level translation which +* can be used for access to kernel addresses. It is valid only for +* access to the kernel's static 1:1 mapping of physical memory — not +* to vmalloc or even module mappings. +*/ +#define PASID_FLAG_SUPERVISOR_MODE BIT(0) +struct pasid_dir_entry {u64 val; };
@@ -51,6 +59,9 @@ struct pasid_table *intel_pasid_get_table(struct device *dev);int intel_pasid_get_dev_max_id (struct device *dev);struct pasid_entry *intel_pasid_get_entry (struct device *dev,int pasid);void intel_pasid_clear_entry (struct device *dev,int pasid); +int intel_pasid_setup_first_level (struct intel_iommu *iommu, +struct device *dev,pgd_t *pgd, +int pasid,u16 did,int flags);int intel_pasid_setup_second_level (struct intel_iommu *iommu,struct dmar_domain *domain,struct device *dev,int pasid);
include/linux/intel-iommu.h +1 -0
@@ -178,6 +178,7 @@*/ #define ecap_smpwc (e) (((e) >> 48) & 0x1) +#define ecap_flts (e) (((e) >> 47) & 0x1)#define ecap_slts (e) (((e) >> 46) & 0x1)#define ecap_smts (e) (((e) >> 43) & 0x1)#define ecap_dit (e) ((e >> 41) & 0x1)
Signature · 1 line
2.17.1
PATCH v6 11/12 iommu/vt-d: Shared virtual address in scalable mode
In reply to Lu BaoluThis patch enables the current SVA (Shared Virtual Address) implementation to work in the scalable mode.
--- drivers/iommu/intel-iommu.c | 38 ------------------------ drivers/iommu/intel-pasid.c | 2 +- drivers/iommu/intel-pasid.h | 1 - drivers/iommu/intel-svm.c | 58 ++++++++++++------------------------- include/linux/intel-iommu.h | 9 +----- 5 files changed, 20 insertions(+), 88 deletions(-)
drivers/iommu/intel-iommu.c +0 -38
@@ -5257,18 +5257,6 @@ static void intel_iommu_put_resv_regions(struct device *dev, }#ifdef CONFIG_INTEL_IOMMU_SVM -static inline unsigned long intel_iommu_get_pts (struct device *dev) -{ -int pts, max_pasid; - - max_pasid =intel_pasid_get_dev_max_id (dev); - pts =find_first_bit ((unsigned long *)&max_pasid,MAX_NR_PASID_BITS ); -if (pts <5 ) -return 0 ; - -return pts -5 ; -} -int intel_iommu_enable_pasid (struct intel_iommu *iommu,struct intel_svm_dev *sdev) {struct device_domain_info *info;
@@ -5300,33 +5288,7 @@ int intel_iommu_enable_pasid(struct intel_iommu *iommu, struct intel_svm_dev *sd sdev->sid =PCI_DEVID (info->bus, info->devfn);if (!(ctx_lo &CONTEXT_PASIDE )) { -if (iommu->pasid_state_table) - context[1 ].hi = (u64 )virt_to_phys (iommu->pasid_state_table); - context[1 ].lo = (u64 )virt_to_phys (info->pasid_table->table) | -intel_iommu_get_pts (sdev->dev); - -wmb (); -/* CONTEXT_TT_MULTI_LEVEL and CONTEXT_TT_DEV_IOTLB are both -* extended to permit requests-with-PASID if the PASIDE bit -* is set. which makes sense. For CONTEXT_TT_PASS_THROUGH, -* however, the PASIDE bit is ignored and requests-with-PASID -* are unconditionally blocked. Which makes less sense. -* So convert from CONTEXT_TT_PASS_THROUGH to one of the new -* "guest mode" translation types depending on whether ATS -* is available or not. Annoyingly, we can't use the new -* modes *unless* PASIDE is set. */ -if ((ctx_lo &CONTEXT_TT_MASK ) == (CONTEXT_TT_PASS_THROUGH <<2 )) { - ctx_lo &= ~CONTEXT_TT_MASK ; -if (info->ats_supported) - ctx_lo |=CONTEXT_TT_PT_PASID_DEV_IOTLB <<2 ; -else - ctx_lo |=CONTEXT_TT_PT_PASID <<2 ; - } ctx_lo |=CONTEXT_PASIDE ; -if (iommu->pasid_state_table) - ctx_lo |=CONTEXT_DINVE ; -if (info->pri_supported) - ctx_lo |=CONTEXT_PRS ; context[0 ].lo = ctx_lo;wmb (); iommu->flush.flush_context(iommu, sdev->did, sdev->sid,
drivers/iommu/intel-pasid.c +1 -1
@@ -286,7 +286,7 @@ static inline void pasid_clear_entry(struct pasid_entry *pe)WRITE_ONCE (pe->val[7 ],0 ); } -void intel_pasid_clear_entry (struct device *dev,int pasid) +static void intel_pasid_clear_entry (struct device *dev,int pasid) {struct pasid_entry *pe;
drivers/iommu/intel-pasid.h +0 -1
@@ -58,7 +58,6 @@ void intel_pasid_free_table(struct device *dev);struct pasid_table *intel_pasid_get_table (struct device *dev);int intel_pasid_get_dev_max_id (struct device *dev);struct pasid_entry *intel_pasid_get_entry (struct device *dev,int pasid); -void intel_pasid_clear_entry (struct device *dev,int pasid);int intel_pasid_setup_first_level (struct intel_iommu *iommu,struct device *dev,pgd_t *pgd,int pasid,u16 did,int flags);
drivers/iommu/intel-svm.c +18 -40
99 lines not on this page · raw message
include/linux/intel-iommu.h +1 -8
@@ -54,14 +54,7 @@#define CONTEXT_TT_MULTI_LEVEL 0#define CONTEXT_TT_DEV_IOTLB 1#define CONTEXT_TT_PASS_THROUGH 2 -/* Extended context entry types */ -#define CONTEXT_TT_PT_PASID 4 -#define CONTEXT_TT_PT_PASID_DEV_IOTLB 5 -#define CONTEXT_TT_MASK (7ULL << 2) - -#define CONTEXT_DINVE (1ULL << 8) -#define CONTEXT_PRS (1ULL << 9) -#define CONTEXT_PASIDE (1ULL << 11) +#define CONTEXT_PASIDE BIT_ULL(3)/* * Intel IOMMU register specification per version 1.0 public spec.
Signature · 1 line
2.17.1
PATCH v6 12/12 iommu/vt-d: Remove deferred invalidation
In reply to Lu BaoluDeferred invalidation is an ECS specific feature. It will not be supported when IOMMU works in scalable mode. As we deprecated the ECS support, remove deferred invalidation and cleanup the code.
--- drivers/iommu/intel-iommu.c | 1 - drivers/iommu/intel-svm.c | 45 ------------------------------------- include/linux/intel-iommu.h | 8 ------- 3 files changed, 54 deletions(-)
drivers/iommu/intel-iommu.c +0 -1
@@ -1700,7 +1700,6 @@ static void free_dmar_iommu(struct intel_iommu *iommu)if (pasid_supported (iommu)) {if (ecap_prs (iommu->ecap))intel_svm_finish_prq (iommu); -intel_svm_exit (iommu); }#endif }
drivers/iommu/intel-svm.c +0 -45
65 lines not on this page · raw message
include/linux/intel-iommu.h +0 -8
@@ -541,15 +541,8 @@ struct intel_iommu {struct iommu_flush flush;#endif #ifdef CONFIG_INTEL_IOMMU_SVM -/* These are large and need to be contiguous, so we allocate just -* one for now. We'll maybe want to rethink that if we truly give -* devices away to userspace processes (e.g. for DPDK) and don't -* want to trust that userspace will use *only* the PASID it was -* told to. But while it's all driver-arbitrated, we're fine. */ -struct pasid_state_entry *pasid_state_table;struct page_req_dsc *prq;unsigned char prq_name[16 ];/* Name for PRQ interrupt */ -u32 pasid_max;#endif struct q_inval *qi;/* Queued invalidation info */ u32 *iommu_state;/* Store iommu states between suspend and resume.*/
@@ -663,7 +656,6 @@ void iommu_flush_write_buffer(struct intel_iommu *iommu);#ifdef CONFIG_INTEL_IOMMU_SVM int intel_svm_init (struct intel_iommu *iommu); -int intel_svm_exit (struct intel_iommu *iommu);extern int intel_svm_enable_prq (struct intel_iommu *iommu);extern int intel_svm_finish_prq (struct intel_iommu *iommu);
Signature · 1 line
2.17.1
Quoted 5 lines · Lu Baolu
Lu Baolu (12):
iommu/vt-d: Enumerate the scalable mode capability iommu/vt-d: Manage scalalble mode PASID tables iommu/vt-d: Move page table helpers into header iommu/vt-d: Add 256-bit invalidation descriptor support
Quoted 8 lines
iommu/vt-d: Reserve a domain id for FL and PT modes iommu/vt-d: Add second level page table interface iommu/vt-d: Setup pasid entry for RID2PASID support iommu/vt-d: Pass pasid table to context mapping iommu/vt-d: Setup context and enable RID2PASID support iommu/vt-d: Add first level page table interface iommu/vt-d: Shared virtual address in scalable mode iommu/vt-d: Remove deferred invalidation
Applied, thanks.