Joerg Roedel 0f412a
From: Bjorn Helgaas <bhelgaas@google.com>
Joerg Roedel 0f412a
Date: Wed, 22 Sep 2021 13:47:26 +0800
Joerg Roedel 0f412a
Subject: iommu/vt-d: Drop "0x" prefix from PCI bus & device addresses
Joerg Roedel 0f412a
Git-commit: 0b482d0c75bf321b2fd87d215c3d6df095a601d1
Joerg Roedel 0f412a
Patch-mainline: v5.15-rc4
Joerg Roedel 0f412a
References: git-fixes
Joerg Roedel 0f412a
Joerg Roedel 0f412a
719a19335692 ("iommu/vt-d: Tweak the description of a DMA fault") changed
Joerg Roedel 0f412a
the DMA fault reason from hex to decimal.  It also added "0x" prefixes to
Joerg Roedel 0f412a
the PCI bus/device, e.g.,
Joerg Roedel 0f412a
Joerg Roedel 0f412a
  - DMAR: [INTR-REMAP] Request device [00:00.5]
Joerg Roedel 0f412a
  + DMAR: [INTR-REMAP] Request device [0x00:0x00.5]
Joerg Roedel 0f412a
Joerg Roedel 0f412a
These no longer match dev_printk() and other similar messages in
Joerg Roedel 0f412a
dmar_match_pci_path() and dmar_acpi_insert_dev_scope().
Joerg Roedel 0f412a
Joerg Roedel 0f412a
Drop the "0x" prefixes from the bus and device addresses.
Joerg Roedel 0f412a
Joerg Roedel 0f412a
Fixes: 719a19335692 ("iommu/vt-d: Tweak the description of a DMA fault")
Joerg Roedel 0f412a
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Joerg Roedel 0f412a
Link: https://lore.kernel.org/r/20210903193711.483999-1-helgaas@kernel.org
Joerg Roedel 0f412a
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Joerg Roedel 0f412a
Link: https://lore.kernel.org/r/20210922054726.499110-2-baolu.lu@linux.intel.com
Joerg Roedel 0f412a
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Joerg Roedel 0f412a
---
Joerg Roedel 0f412a
 drivers/iommu/intel/dmar.c | 6 +++---
Joerg Roedel 0f412a
 1 file changed, 3 insertions(+), 3 deletions(-)
Joerg Roedel 0f412a
Joerg Roedel 0f412a
diff --git a/drivers/iommu/intel/dmar.c b/drivers/iommu/intel/dmar.c
Joerg Roedel 0f412a
index 0ec5514c9980..b7708b93f3fa 100644
Joerg Roedel 0f412a
--- a/drivers/iommu/intel/dmar.c
Joerg Roedel 0f412a
+++ b/drivers/iommu/intel/dmar.c
Joerg Roedel 0f412a
@@ -1942,18 +1942,18 @@ static int dmar_fault_do_one(struct intel_iommu *iommu, int type,
Joerg Roedel 0f412a
 	reason = dmar_get_fault_reason(fault_reason, &fault_type);
Joerg Roedel 0f412a
 
Joerg Roedel 0f412a
 	if (fault_type == INTR_REMAP)
Joerg Roedel 0f412a
-		pr_err("[INTR-REMAP] Request device [0x%02x:0x%02x.%d] fault index 0x%llx [fault reason 0x%02x] %s\n",
Joerg Roedel 0f412a
+		pr_err("[INTR-REMAP] Request device [%02x:%02x.%d] fault index 0x%llx [fault reason 0x%02x] %s\n",
Joerg Roedel 0f412a
 		       source_id >> 8, PCI_SLOT(source_id & 0xFF),
Joerg Roedel 0f412a
 		       PCI_FUNC(source_id & 0xFF), addr >> 48,
Joerg Roedel 0f412a
 		       fault_reason, reason);
Joerg Roedel 0f412a
 	else if (pasid == INVALID_IOASID)
Joerg Roedel 0f412a
-		pr_err("[%s NO_PASID] Request device [0x%02x:0x%02x.%d] fault addr 0x%llx [fault reason 0x%02x] %s\n",
Joerg Roedel 0f412a
+		pr_err("[%s NO_PASID] Request device [%02x:%02x.%d] fault addr 0x%llx [fault reason 0x%02x] %s\n",
Joerg Roedel 0f412a
 		       type ? "DMA Read" : "DMA Write",
Joerg Roedel 0f412a
 		       source_id >> 8, PCI_SLOT(source_id & 0xFF),
Joerg Roedel 0f412a
 		       PCI_FUNC(source_id & 0xFF), addr,
Joerg Roedel 0f412a
 		       fault_reason, reason);
Joerg Roedel 0f412a
 	else
Joerg Roedel 0f412a
-		pr_err("[%s PASID 0x%x] Request device [0x%02x:0x%02x.%d] fault addr 0x%llx [fault reason 0x%02x] %s\n",
Joerg Roedel 0f412a
+		pr_err("[%s PASID 0x%x] Request device [%02x:%02x.%d] fault addr 0x%llx [fault reason 0x%02x] %s\n",
Joerg Roedel 0f412a
 		       type ? "DMA Read" : "DMA Write", pasid,
Joerg Roedel 0f412a
 		       source_id >> 8, PCI_SLOT(source_id & 0xFF),
Joerg Roedel 0f412a
 		       PCI_FUNC(source_id & 0xFF), addr,
Joerg Roedel 0f412a