Skip to content

Commit e5567f5

Browse files
Kuppuswamy Sathyanarayananjoergroedel
authored andcommitted
PCI/ATS: Add pci_prg_resp_pasid_required() interface.
Return the PRG Response PASID Required bit in the Page Request Status Register. As per PCIe spec r4.0, sec 10.5.2.3, if this bit is Set, the device expects a PASID TLP Prefix on PRG Response Messages when the corresponding Page Requests had a PASID TLP Prefix. If Clear, the device does not expect PASID TLP Prefixes on any PRG Response Message, and the device behavior is undefined if the device receives a PRG Response Message with a PASID TLP Prefix. Also the device behavior is undefined if this bit is Set and the device receives a PRG Response Message with no PASID TLP Prefix when the corresponding Page Requests had a PASID TLP Prefix. This function will be used by drivers like IOMMU, if it is required to check the status of the PRG Response PASID Required bit before enabling the PASID support of the device. Cc: Ashok Raj <ashok.raj@intel.com> Cc: Jacob Pan <jacob.jun.pan@linux.intel.com> Cc: Keith Busch <keith.busch@intel.com> Suggested-by: Ashok Raj <ashok.raj@intel.com> Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com> Acked-by: Bjorn Helgaas <bhelgaas@google.com> Signed-off-by: Joerg Roedel <jroedel@suse.de>
1 parent 3f0c625 commit e5567f5

File tree

3 files changed

+36
-0
lines changed

3 files changed

+36
-0
lines changed

drivers/pci/ats.c

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -368,6 +368,36 @@ int pci_pasid_features(struct pci_dev *pdev)
368368
}
369369
EXPORT_SYMBOL_GPL(pci_pasid_features);
370370

371+
/**
372+
* pci_prg_resp_pasid_required - Return PRG Response PASID Required bit
373+
* status.
374+
* @pdev: PCI device structure
375+
*
376+
* Returns 1 if PASID is required in PRG Response Message, 0 otherwise.
377+
*
378+
* Even though the PRG response PASID status is read from PRI Status
379+
* Register, since this API will mainly be used by PASID users, this
380+
* function is defined within #ifdef CONFIG_PCI_PASID instead of
381+
* CONFIG_PCI_PRI.
382+
*/
383+
int pci_prg_resp_pasid_required(struct pci_dev *pdev)
384+
{
385+
u16 status;
386+
int pos;
387+
388+
pos = pci_find_ext_capability(pdev, PCI_EXT_CAP_ID_PRI);
389+
if (!pos)
390+
return 0;
391+
392+
pci_read_config_word(pdev, pos + PCI_PRI_STATUS, &status);
393+
394+
if (status & PCI_PRI_STATUS_PASID)
395+
return 1;
396+
397+
return 0;
398+
}
399+
EXPORT_SYMBOL_GPL(pci_prg_resp_pasid_required);
400+
371401
#define PASID_NUMBER_SHIFT 8
372402
#define PASID_NUMBER_MASK (0x1f << PASID_NUMBER_SHIFT)
373403
/**

include/linux/pci-ats.h

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ void pci_disable_pasid(struct pci_dev *pdev);
4040
void pci_restore_pasid_state(struct pci_dev *pdev);
4141
int pci_pasid_features(struct pci_dev *pdev);
4242
int pci_max_pasids(struct pci_dev *pdev);
43+
int pci_prg_resp_pasid_required(struct pci_dev *pdev);
4344

4445
#else /* CONFIG_PCI_PASID */
4546

@@ -66,6 +67,10 @@ static inline int pci_max_pasids(struct pci_dev *pdev)
6667
return -EINVAL;
6768
}
6869

70+
static int pci_prg_resp_pasid_required(struct pci_dev *pdev)
71+
{
72+
return 0;
73+
}
6974
#endif /* CONFIG_PCI_PASID */
7075

7176

include/uapi/linux/pci_regs.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -880,6 +880,7 @@
880880
#define PCI_PRI_STATUS_RF 0x001 /* Response Failure */
881881
#define PCI_PRI_STATUS_UPRGI 0x002 /* Unexpected PRG index */
882882
#define PCI_PRI_STATUS_STOPPED 0x100 /* PRI Stopped */
883+
#define PCI_PRI_STATUS_PASID 0x8000 /* PRG Response PASID Required */
883884
#define PCI_PRI_MAX_REQ 0x08 /* PRI max reqs supported */
884885
#define PCI_PRI_ALLOC_REQ 0x0c /* PRI max reqs allowed */
885886
#define PCI_EXT_CAP_PRI_SIZEOF 16

0 commit comments

Comments
 (0)