summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLukasz Laguna <lukasz.laguna@intel.com>2026-07-30 11:38:06 +0200
committerLukasz Laguna <lukasz.laguna@intel.com>2026-08-04 08:13:23 +0200
commitde6fcadec516caa8942775f27d17aef9b12dc358 (patch)
treeaec6664f859e634561ef0217106f99752abfa23e
parent270bc13954ccab274f11ea80b4a857ad8b9e194b (diff)
lib/xe/xe_ggtt: Extract generic GGTT code into library
Add a shared xe_ggtt library with a common GGTT definitions, including bitfields, bitmasks and helpers. Signed-off-by: Lukasz Laguna <lukasz.laguna@intel.com> Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
-rw-r--r--lib/meson.build1
-rw-r--r--lib/xe/xe_ggtt.c22
-rw-r--r--lib/xe/xe_ggtt.h21
-rw-r--r--lib/xe/xe_mmio.h3
-rw-r--r--tests/intel/xe_sriov_flr.c31
5 files changed, 61 insertions, 17 deletions
diff --git a/lib/meson.build b/lib/meson.build
index 12d78de20..3001b473e 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -126,6 +126,7 @@ lib_sources = [
'igt_msm.c',
'igt_dsc.c',
'igt_hook.c',
+ 'xe/xe_ggtt.c',
'xe/xe_gt.c',
'xe/xe_ioctl.c',
'xe/xe_legacy.c',
diff --git a/lib/xe/xe_ggtt.c b/lib/xe/xe_ggtt.c
new file mode 100644
index 000000000..96b189b7a
--- /dev/null
+++ b/lib/xe/xe_ggtt.c
@@ -0,0 +1,22 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright(c) 2026 Intel Corporation. All rights reserved.
+ */
+
+#include "igt.h"
+#include "xe/xe_ggtt.h"
+
+xe_ggtt_pte_mask_t xe_ggtt_get_gpa_mask(int pf_fd)
+{
+ if (IS_TIGERLAKE(intel_get_drm_devid(pf_fd)))
+ return TGL_GGTT_PTE_ADDR_MASK;
+
+ return GGTT_PTE_ADDR_MASK;
+}
+
+uint64_t xe_ggtt_pte_get_gpa(int pf_fd, xe_ggtt_pte_t pte)
+{
+ xe_ggtt_pte_mask_t mask = xe_ggtt_get_gpa_mask(pf_fd);
+
+ return (uint64_t)((pte & mask) >> GGTT_PTE_ADDR_SHIFT);
+}
diff --git a/lib/xe/xe_ggtt.h b/lib/xe/xe_ggtt.h
new file mode 100644
index 000000000..5e4038591
--- /dev/null
+++ b/lib/xe/xe_ggtt.h
@@ -0,0 +1,21 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * Copyright(c) 2026 Intel Corporation. All rights reserved.
+ */
+
+#ifndef __XE_GGTT_H__
+#define __XE_GGTT_H__
+
+#include <stdint.h>
+
+#define GGTT_PTE_ADDR_MASK GENMASK_ULL(45, 12)
+#define TGL_GGTT_PTE_ADDR_MASK GENMASK_ULL(38, 12)
+#define GGTT_PTE_ADDR_SHIFT 12
+
+typedef uint64_t xe_ggtt_pte_t;
+typedef uint64_t xe_ggtt_pte_mask_t;
+
+xe_ggtt_pte_mask_t xe_ggtt_get_gpa_mask(int pf_fd);
+uint64_t xe_ggtt_pte_get_gpa(int pf_fd, xe_ggtt_pte_t pte);
+
+#endif /* __XE_GGTT_H__ */
diff --git a/lib/xe/xe_mmio.h b/lib/xe/xe_mmio.h
index 030c50a2e..65aea65ea 100644
--- a/lib/xe/xe_mmio.h
+++ b/lib/xe/xe_mmio.h
@@ -5,6 +5,7 @@
#include "lib/intel_io.h"
#include "lib/igt_sizes.h"
+#include "lib/xe/xe_ggtt.h"
#ifndef XE_MMIO_H
#define XE_MMIO_H
@@ -12,8 +13,6 @@
#define TILE_MMIO_SIZE SZ_16M
#define GGTT_OFFSET_IN_TILE SZ_8M
-typedef uint64_t xe_ggtt_pte_t;
-
struct xe_mmio {
int fd;
bool init;
diff --git a/tests/intel/xe_sriov_flr.c b/tests/intel/xe_sriov_flr.c
index bb702d354..556979d7b 100644
--- a/tests/intel/xe_sriov_flr.c
+++ b/tests/intel/xe_sriov_flr.c
@@ -16,6 +16,7 @@
#include "intel_chipset.h"
#include "intel_vram.h"
#include "linux_scaffold.h"
+#include "xe/xe_ggtt.h"
#include "xe/xe_mmio.h"
#include "xe/xe_query.h"
#include "xe/xe_sriov_provisioning.h"
@@ -646,8 +647,6 @@ static int execute_parallel_flr_twice(int pf_fd, int num_vfs,
}
#define GEN12_VF_CAP_REG 0x1901f8
-#define GGTT_PTE_TEST_FIELD_MASK GENMASK_ULL(19, 12)
-#define GGTT_PTE_ADDR_SHIFT 12
struct ggtt_ops {
void (*set_pte)(struct xe_mmio *mmio, uint8_t tile, uint32_t pte_offset, xe_ggtt_pte_t pte);
@@ -693,26 +692,21 @@ static void intel_mtl_set_pte(struct xe_mmio *mmio, uint8_t tile,
static bool set_pte_gpa(struct ggtt_ops *ggtt, struct xe_mmio *mmio, uint8_t tile,
uint32_t pte_offset, uint8_t gpa, xe_ggtt_pte_t *out)
{
+ xe_ggtt_pte_mask_t mask = xe_ggtt_get_gpa_mask(mmio->fd);
xe_ggtt_pte_t pte;
pte = ggtt->get_pte(mmio, tile, pte_offset);
- pte &= ~GGTT_PTE_TEST_FIELD_MASK;
- pte |= ((xe_ggtt_pte_t)gpa << GGTT_PTE_ADDR_SHIFT) & GGTT_PTE_TEST_FIELD_MASK;
+ pte &= ~mask;
+ pte |= ((xe_ggtt_pte_t)gpa << GGTT_PTE_ADDR_SHIFT) & mask;
ggtt->set_pte(mmio, tile, pte_offset, pte);
*out = ggtt->get_pte(mmio, tile, pte_offset);
return *out == pte;
}
-static bool check_pte_gpa(struct ggtt_ops *ggtt, struct xe_mmio *mmio, uint8_t tile,
- uint32_t pte_offset, uint8_t expected_gpa, xe_ggtt_pte_t *out)
+static bool check_pte_gpa(int pf_fd, xe_ggtt_pte_t pte, uint8_t expected)
{
- uint8_t val;
-
- *out = ggtt->get_pte(mmio, tile, pte_offset);
- val = (uint8_t)((*out & GGTT_PTE_TEST_FIELD_MASK) >> GGTT_PTE_ADDR_SHIFT);
-
- return val == expected_gpa;
+ return (xe_ggtt_pte_get_gpa(pf_fd, pte) == expected);
}
static int populate_ggtt_pte_offsets(struct ggtt_data *gdata)
@@ -817,17 +811,24 @@ static void ggtt_subcheck_prepare_vf(int vf_id, struct subcheck_data *data)
static void ggtt_subcheck_verify_vf(int vf_id, int flr_vf_id, struct subcheck_data *data)
{
struct ggtt_data *gdata = (struct ggtt_data *)data;
- uint8_t expected = (vf_id == flr_vf_id) ? 0 : vf_id;
struct xe_mmio *mmio = xe_mmio_for_vf(0);
+ int pf_fd = gdata->base.pf_fd;
xe_ggtt_pte_t pte;
uint32_t pte_offset;
+ bool failed = false;
if (data->stop_reason)
return;
for_each_pte_offset(pte_offset, &gdata->pte_offsets[vf_id]) {
- if (!check_pte_gpa(&gdata->ggtt, mmio, data->tile, pte_offset,
- expected, &pte)) {
+ pte = gdata->ggtt.get_pte(mmio, data->tile, pte_offset);
+
+ if (!check_pte_gpa(pf_fd, pte, (vf_id == flr_vf_id) ? 0 : vf_id)) {
+ igt_debug("Wrong GGTT GPA on VF%u after VF%u FLR\n", vf_id, flr_vf_id);
+ failed = true;
+ }
+
+ if (failed) {
set_fail_reason(data,
"GGTT check after VF%u FLR failed on VF%u: Read PTE: %#" PRIx64 " at offset: %#x\n",
flr_vf_id, vf_id, pte, pte_offset);