[PATCH v5 12/12] drm/xe/vram: Report VRAM errors using structured xe_log API

[Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>]

Replace few drm_err()/drm_info() with xe_log_err_fatal()/xe_log_info()
using the VRAM component tag. The "no CPU visible VRAM" error uses
tile or device as the log location depending on the caller context.

Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>

---
v2:
 - Update the ternary condition (Mallesh)
---
---
 drivers/gpu/drm/xe/xe_vram.c | 27 +++++++++++++++------------
 1 file changed, 15 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vram.c b/drivers/gpu/drm/xe/xe_vram.c
index 2441ffe77edb..d44a92e97195 100644
--- a/drivers/gpu/drm/xe/xe_vram.c
+++ b/drivers/gpu/drm/xe/xe_vram.c
@@ -17,6 +17,7 @@
 #include "xe_device.h"
 #include "xe_force_wake.h"
 #include "xe_gt_mcr.h"
+#include "xe_log.h"
 #include "xe_map.h"
 #include "xe_migrate.h"
 #include "xe_mmio.h"
@@ -46,7 +47,8 @@ static int determine_lmem_bar_size(struct xe_device *xe, struct xe_vram_region *
 	struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
 
 	if (!resource_is_valid(pdev, LMEM_BAR)) {
-		drm_err(&xe->drm, "pci resource is not valid\n");
+		xe_log_err_fatal(xe, VRAM, -ENXIO,
+				 "PCI resource is not valid\n");
 		return -ENXIO;
 	}
 
@@ -227,15 +229,14 @@ struct xe_vram_region *xe_vram_region_alloc(struct xe_device *xe, u8 id, u32 pla
 
 static void print_vram_region_info(struct xe_device *xe, struct xe_vram_region *vram)
 {
-	struct drm_device *drm = &xe->drm;
 
 	if (vram->io_size < vram->usable_size)
-		drm_info(drm, "Small BAR device\n");
+		xe_log_info(xe, VRAM, "Small BAR device\n");
 
-	drm_info(drm,
-		 "VRAM[%u]: Actual physical size %pa, usable size exclude stolen %pa, CPU accessible size %pa\n",
-		 vram->id, &vram->actual_physical_size, &vram->usable_size, &vram->io_size);
-	drm_info(drm, "VRAM[%u]: DPA range: [%pa-%llx], io range: [%pa-%llx]\n",
+	xe_log_info(xe, VRAM,
+		    "VRAM[%u]: Actual physical size %pa, usable size exclude stolen %pa, CPU accessible size %pa\n",
+		    vram->id, &vram->actual_physical_size, &vram->usable_size, &vram->io_size);
+	xe_log_info(xe, VRAM, "VRAM[%u]: DPA range: [%pa-%llx], io range: [%pa-%llx]\n",
 		 vram->id, &vram->dpa_base, vram->dpa_base + (u64)vram->actual_physical_size,
 		 &vram->io_start, vram->io_start + (u64)vram->io_size);
 }
@@ -254,8 +255,9 @@ static int vram_region_init(struct xe_device *xe, struct xe_vram_region *vram,
 	vram->io_size = min_t(u64, usable_size, remain_io_size);
 
 	if (!vram->io_size) {
-		drm_err(&xe->drm, "%s without any CPU visible VRAM. Aborting.\n",
-			tile ? "Tile" : "Device");
+		xe_log_err_fatal(xe, VRAM, -ENODEV,
+				 "%s without any CPU visible VRAM. Aborting.\n",
+				 tile ? "Tile" : "Device");
 		return -ENODEV;
 	}
 
@@ -320,7 +322,7 @@ int xe_vram_probe(struct xe_device *xe)
 	err = determine_lmem_bar_size(xe, &lmem_bar);
 	if (err)
 		return err;
-	drm_info(&xe->drm, "VISIBLE VRAM: %pa, %pa\n", &lmem_bar.io_start, &lmem_bar.io_size);
+	xe_log_info(xe, VRAM, "VISIBLE VRAM: %pa, %pa\n", &lmem_bar.io_start, &lmem_bar.io_size);
 
 	remain_io_size = lmem_bar.io_size;
 
@@ -342,8 +344,9 @@ int xe_vram_probe(struct xe_device *xe)
 			return err;
 
 		if (total_size > lmem_bar.io_size) {
-			drm_info(&xe->drm, "VRAM: %pa is larger than resource %pa\n",
-				 &total_size, &lmem_bar.io_size);
+			xe_log_info(xe, VRAM,
+				    "Total size %pa is larger than resource %pa\n",
+				    &total_size, &lmem_bar.io_size);
 		}
 
 		remain_io_size -= min_t(u64, tile->mem.vram->actual_physical_size, remain_io_size);
-- 
2.54.0