Commit 2c33b49a authored by Philippe Lecluse's avatar Philippe Lecluse Committed by Rodrigo Vivi

drm/xe: enforce GSMBASE for DG1 instead of BAR2

On DG1, BAR2 is not reliable for reporting Vram size, need to use GSMBASE.
Simplify xe_mmio_total_vram_size to report vram size and usable size.
Signed-off-by: default avatarPhilippe Lecluse <philippe.lecluse@intel.com>
Reviewed-by: default avatarMatthew Auld <matthew.auld@intel.com>
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent c343bacf
...@@ -150,7 +150,7 @@ static bool xe_pci_resource_valid(struct pci_dev *pdev, int bar) ...@@ -150,7 +150,7 @@ static bool xe_pci_resource_valid(struct pci_dev *pdev, int bar)
return true; return true;
} }
int xe_mmio_total_vram_size(struct xe_device *xe, u64 *vram_size, u64 *flat_ccs_base) int xe_mmio_total_vram_size(struct xe_device *xe, u64 *vram_size, u64 *usable_size)
{ {
struct xe_gt *gt = xe_device_get_gt(xe, 0); struct xe_gt *gt = xe_device_get_gt(xe, 0);
struct pci_dev *pdev = to_pci_dev(xe->drm.dev); struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
...@@ -159,8 +159,12 @@ int xe_mmio_total_vram_size(struct xe_device *xe, u64 *vram_size, u64 *flat_ccs_ ...@@ -159,8 +159,12 @@ int xe_mmio_total_vram_size(struct xe_device *xe, u64 *vram_size, u64 *flat_ccs_
if (!xe->info.has_flat_ccs) { if (!xe->info.has_flat_ccs) {
*vram_size = pci_resource_len(pdev, GEN12_LMEM_BAR); *vram_size = pci_resource_len(pdev, GEN12_LMEM_BAR);
if (flat_ccs_base) if (usable_size) {
*flat_ccs_base = *vram_size; if (xe->info.platform == XE_DG1)
*usable_size = xe_mmio_read64(gt, GEN12_GSMBASE.reg);
else
*usable_size = *vram_size;
}
return 0; return 0;
} }
...@@ -170,15 +174,13 @@ int xe_mmio_total_vram_size(struct xe_device *xe, u64 *vram_size, u64 *flat_ccs_ ...@@ -170,15 +174,13 @@ int xe_mmio_total_vram_size(struct xe_device *xe, u64 *vram_size, u64 *flat_ccs_
reg = xe_gt_mcr_unicast_read_any(gt, XEHP_TILE0_ADDR_RANGE); reg = xe_gt_mcr_unicast_read_any(gt, XEHP_TILE0_ADDR_RANGE);
*vram_size = (u64)REG_FIELD_GET(GENMASK(14, 8), reg) * SZ_1G; *vram_size = (u64)REG_FIELD_GET(GENMASK(14, 8), reg) * SZ_1G;
if (flat_ccs_base) { if (usable_size) {
reg = xe_gt_mcr_unicast_read_any(gt, XEHP_FLAT_CCS_BASE_ADDR); reg = xe_gt_mcr_unicast_read_any(gt, XEHP_FLAT_CCS_BASE_ADDR);
*flat_ccs_base = (u64)REG_FIELD_GET(GENMASK(31, 8), reg) * SZ_64K; *usable_size = (u64)REG_FIELD_GET(GENMASK(31, 8), reg) * SZ_64K;
drm_info(&xe->drm, "lmem_size: 0x%llx usable_size: 0x%llx\n",
*vram_size, *usable_size);
} }
if (flat_ccs_base)
drm_info(&xe->drm, "lmem_size: 0x%llx flat_ccs_base: 0x%llx\n",
*vram_size, *flat_ccs_base);
return xe_force_wake_put(gt_to_fw(gt), XE_FW_GT); return xe_force_wake_put(gt_to_fw(gt), XE_FW_GT);
} }
...@@ -190,7 +192,7 @@ int xe_mmio_probe_vram(struct xe_device *xe) ...@@ -190,7 +192,7 @@ int xe_mmio_probe_vram(struct xe_device *xe)
u64 lmem_size; u64 lmem_size;
u64 original_size; u64 original_size;
u64 current_size; u64 current_size;
u64 flat_ccs_base; u64 usable_size;
int resize_result, err; int resize_result, err;
if (!IS_DGFX(xe)) { if (!IS_DGFX(xe)) {
...@@ -212,11 +214,9 @@ int xe_mmio_probe_vram(struct xe_device *xe) ...@@ -212,11 +214,9 @@ int xe_mmio_probe_vram(struct xe_device *xe)
} }
gt = xe_device_get_gt(xe, 0); gt = xe_device_get_gt(xe, 0);
lmem_size = xe_mmio_read64(gt, GEN12_GSMBASE.reg);
original_size = pci_resource_len(pdev, GEN12_LMEM_BAR); original_size = pci_resource_len(pdev, GEN12_LMEM_BAR);
err = xe_mmio_total_vram_size(xe, &lmem_size, &flat_ccs_base); err = xe_mmio_total_vram_size(xe, &lmem_size, &usable_size);
if (err) if (err)
return err; return err;
...@@ -244,7 +244,7 @@ int xe_mmio_probe_vram(struct xe_device *xe) ...@@ -244,7 +244,7 @@ int xe_mmio_probe_vram(struct xe_device *xe)
xe->mem.vram.mapping = ioremap_wc(xe->mem.vram.io_start, xe->mem.vram.size); xe->mem.vram.mapping = ioremap_wc(xe->mem.vram.io_start, xe->mem.vram.size);
#endif #endif
xe->mem.vram.size = min_t(u64, xe->mem.vram.size, flat_ccs_base); xe->mem.vram.size = min_t(u64, xe->mem.vram.size, usable_size);
drm_info(&xe->drm, "TOTAL VRAM: %pa, %pa\n", &xe->mem.vram.io_start, &xe->mem.vram.size); drm_info(&xe->drm, "TOTAL VRAM: %pa, %pa\n", &xe->mem.vram.io_start, &xe->mem.vram.size);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment