Messages in this thread | | | From | Sohil Mehta <> | Subject | [PATCH v8 0/6] Add Intel IOMMU debugfs support | Date | Tue, 11 Sep 2018 17:11:35 -0700 |
| |
Hi All,
This series aims to add debugfs support for Intel IOMMU. It exposes IOMMU registers, internal context and dumps individual table entries to help debug Intel IOMMUs.
Patches 1-3 do the ground work for the following patches by reorganizing some Intel IOMMU data structures, creating a new config option - INTEL_IOMMU_DEBUGFS and adding an Intel IOMMU debugfs directory. The following patches add debugfs support for IOMMU resigter contents, Interrupt remapping and context internals in that order. The information can be accessed in sysfs at '/sys/kernel/debug/iommu/intel/'.
This series is based on the new VT-d 3.0 specification [1]. The scalable mode and PASID table debugfs patches are not part of this series. They would be submitted later once the IOMMU driver patches for the same are accepted and merged upstream.
Regards, Sohil
[1] https://software.intel.com/sites/default/files/managed/c5/15/vt-directed-io-spec.pdf
Changes since v7: - Move to the common iommu directory approach suggested by Joerg - Remove ECS debugfs support that has been deprecated with the new VT-d spec - Change filename to intel-iommu-debugfs.c - Remove the pasid table debugfs patch - Add MTRR and Virtual command register support - Use locks to protect register and table access - Improve seq_printf formatting - Simplify and split the patches differently
Changes since v6: - Split patch 1/5 and 2/5 differently - Simplify and improve code formatting - Use macro for register set definitions - Fix compiler warning for readq - Add Co-Developed-by tag to commit messages
Changes since v5: - Change the order of includes to an alphabetical order - Change seq_printf and seq_puts formatting
Changes since v4: - Change to a SPDX license tag - Fix seq_printf formatting and remove leading '\n's
Changes since v3: - Remove an unused function parameter from some of the functions - Fix checkpatch.pl warnings - Remove error reporting for debugfs_create_file functions - Fix unnecessary reprogramming of the context entries - Simplify and merge the show context and extended context patch into one - Remove redundant IOMMU null check under for_each_active_iommu - Update the commit title to be consistent
Changes since v2: - Added a macro for seq file operations based on recommendation by Andy Shevchenko. The marco can be moved to seq_file.h at a future point - Changed the debugfs file names to more relevant ones - Added information for MTRR registers in the regset file
Changes since v1: - Fixed seq_printf formatting - Handled the case when Interrupt remapping is not enabled
Gayatri Kammela (1): iommu/vt-d: Add debugfs support to show register contents
Sohil Mehta (5): iommu/vt-d: Relocate struct/function declarations to its header files iommu/vt-d: Update register definitions to VT-d 3.0 specification iommu/vt-d: Enable base Intel IOMMU debugfs support iommu/vt-d: Add debugfs support for Interrupt remapping iommu/vt-d: Add debugfs support to show context internals
arch/x86/include/asm/irq_remapping.h | 2 + drivers/iommu/Kconfig | 13 ++ drivers/iommu/Makefile | 1 + drivers/iommu/intel-iommu-debugfs.c | 314 +++++++++++++++++++++++++++ drivers/iommu/intel-iommu.c | 32 +-- drivers/iommu/intel_irq_remapping.c | 2 +- include/linux/intel-iommu.h | 72 ++++++ 7 files changed, 407 insertions(+), 29 deletions(-) create mode 100644 drivers/iommu/intel-iommu-debugfs.c
-- 2.18.0
| |