[igt-dev] [PATCH i-g-t 4/9] i915: Start putting the mmio_base to wider use
Lionel Landwerlin
lionel.g.landwerlin at intel.com
Thu Nov 21 12:04:42 UTC 2019
On 13/11/2019 14:52, Chris Wilson wrote:
> Several tests depend upon the implicit engine->mmio_base but have no
> means of determining the physical layout. Since the kernel has started
> providing this information, start putting it to use.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
> lib/i915/gem_engine_topology.c | 84 ++++++++++++++++++++++++++++++++++
> lib/i915/gem_engine_topology.h | 5 ++
> tests/i915/gem_ctx_shared.c | 38 +++++----------
> tests/i915/gem_exec_latency.c | 17 ++++---
> 4 files changed, 111 insertions(+), 33 deletions(-)
>
> diff --git a/lib/i915/gem_engine_topology.c b/lib/i915/gem_engine_topology.c
> index 790d455ff..bd200a4b9 100644
> --- a/lib/i915/gem_engine_topology.c
> +++ b/lib/i915/gem_engine_topology.c
> @@ -21,7 +21,12 @@
> * IN THE SOFTWARE.
> */
>
> +#include <fcntl.h>
> +#include <unistd.h>
> +
> #include "drmtest.h"
> +#include "igt_sysfs.h"
> +#include "intel_chipset.h"
> #include "ioctl_wrappers.h"
>
> #include "i915/gem_engine_topology.h"
> @@ -337,3 +342,82 @@ bool gem_engine_is_equal(const struct intel_execution_engine2 *e1,
> {
> return e1->class == e2->class && e1->instance == e2->instance;
> }
> +
> +static int descend(int dir, const char *path)
> +{
> + int fd;
> +
> + fd = openat(dir, path, O_RDONLY);
> + close(dir);
> +
> + return fd;
> +}
> +
Not sure I understand what file the function below is supposed to parse.
Is that /sys/kernel/debug/dri/0/i915_engine_info?
Probably doesn't work on my system as I get a 0 mmio_base for vcs1.
-Lionel
> +int gem_engine_property_scanf(int i915, const char *engine, const char *attr,
> + const char *fmt, ...)
> +{
> + FILE *file;
> + va_list ap;
> + int ret;
> + int fd;
> +
> + fd = igt_sysfs_open(i915);
> + if (fd < 0)
> + return fd;
> +
> + fd = descend(fd, "engine");
> + if (fd < 0)
> + return fd;
> +
> + fd = descend(fd, engine);
> + if (fd < 0)
> + return fd;
> +
> + fd = descend(fd, attr);
> + if (fd < 0)
> + return fd;
> +
> + file = fdopen(fd, "r");
> + if (!file) {
> + close(fd);
> + return -1;
> + }
> +
> + va_start(ap, fmt);
> + ret = vfscanf(file, fmt, ap);
> + va_end(ap);
> +
> + fclose(file);
> + return ret;
> +}
> +
> +uint32_t gem_engine_mmio_base(int i915, const char *engine)
> +{
> + unsigned int mmio = 0;
> +
> + if (gem_engine_property_scanf(i915, engine, "mmio_base",
> + "%x", &mmio) < 0) {
> + int gen = intel_gen(intel_get_drm_devid(i915));
> +
> + /* The layout of xcs1+ is unreliable -- hence the property! */
> + if (!strcmp(engine, "rcs0")) {
> + mmio = 0x2000;
> + } else if (!strcmp(engine, "bcs0")) {
> + mmio = 0x22000;
> + } else if (!strcmp(engine, "vcs0")) {
> + if (gen < 6)
> + mmio = 0x4000;
> + else if (gen < 11)
> + mmio = 0x12000;
> + else
> + mmio = 0x1c0000;
> + } else if (!strcmp(engine, "vecs0")) {
> + if (gen < 11)
> + mmio = 0x1a000;
> + else
> + mmio = 0x1c8000;
> + }
> + }
> +
> + return mmio;
> +}
> diff --git a/lib/i915/gem_engine_topology.h b/lib/i915/gem_engine_topology.h
> index d98773e06..e728ebd93 100644
> --- a/lib/i915/gem_engine_topology.h
> +++ b/lib/i915/gem_engine_topology.h
> @@ -74,4 +74,9 @@ struct intel_execution_engine2 gem_eb_flags_to_engine(unsigned int flags);
> ((e__) = intel_get_current_physical_engine(&i__)); \
> intel_next_engine(&i__))
>
> +__attribute__((format(scanf, 4, 5)))
> +int gem_engine_property_scanf(int i915, const char *engine, const char *attr,
> + const char *fmt, ...);
> +uint32_t gem_engine_mmio_base(int i915, const char *engine);
> +
> #endif /* GEM_ENGINE_TOPOLOGY_H */
> diff --git a/tests/i915/gem_ctx_shared.c b/tests/i915/gem_ctx_shared.c
> index a6eee16dd..949e1f3d4 100644
> --- a/tests/i915/gem_ctx_shared.c
> +++ b/tests/i915/gem_ctx_shared.c
> @@ -38,6 +38,7 @@
>
> #include <drm.h>
>
> +#include "i915/gem_engine_topology.h"
> #include "igt_rand.h"
> #include "igt_vgem.h"
> #include "sync_file.h"
> @@ -556,6 +557,14 @@ static uint32_t store_timestamp(int i915,
> return obj.handle;
> }
>
> +static uint32_t ring_base(int i915, unsigned ring)
> +{
> + if (ring == I915_EXEC_DEFAULT)
> + ring = I915_EXEC_RENDER; /* XXX */
> +
> + return gem_engine_mmio_base(i915, gem_eb_flags_to_engine(ring).name);
> +}
> +
> static void independent(int i915, unsigned ring, unsigned flags)
> {
> const int TIMESTAMP = 1023;
> @@ -563,33 +572,8 @@ static void independent(int i915, unsigned ring, unsigned flags)
> igt_spin_t *spin[MAX_ELSP_QLEN];
> unsigned int mmio_base;
>
> - /* XXX i915_query()! */
> - switch (ring) {
> - case I915_EXEC_DEFAULT:
> - case I915_EXEC_RENDER:
> - mmio_base = 0x2000;
> - break;
> -#if 0
> - case I915_EXEC_BSD:
> - mmio_base = 0x12000;
> - break;
> -#endif
> - case I915_EXEC_BLT:
> - mmio_base = 0x22000;
> - break;
> -
> -#define GEN11_VECS0_BASE 0x1c8000
> -#define GEN11_VECS1_BASE 0x1d8000
> - case I915_EXEC_VEBOX:
> - if (intel_gen(intel_get_drm_devid(i915)) >= 11)
> - mmio_base = GEN11_VECS0_BASE;
> - else
> - mmio_base = 0x1a000;
> - break;
> -
> - default:
> - igt_skip("mmio base not known\n");
> - }
> + mmio_base = ring_base(i915, ring);
> + igt_require_f(mmio_base, "mmio base not known\n");
>
> for (int n = 0; n < ARRAY_SIZE(spin); n++) {
> const struct igt_spin_factory opts = {
> diff --git a/tests/i915/gem_exec_latency.c b/tests/i915/gem_exec_latency.c
> index 3d99182a0..d2159f317 100644
> --- a/tests/i915/gem_exec_latency.c
> +++ b/tests/i915/gem_exec_latency.c
> @@ -109,7 +109,7 @@ poll_ring(int fd, unsigned ring, const char *name)
> igt_spin_free(fd, spin[0]);
> }
>
> -#define RCS_TIMESTAMP (0x2000 + 0x358)
> +#define TIMESTAMP (0x358)
> static void latency_on_ring(int fd,
> unsigned ring, const char *name,
> unsigned flags)
> @@ -119,6 +119,7 @@ static void latency_on_ring(int fd,
> struct drm_i915_gem_exec_object2 obj[3];
> struct drm_i915_gem_relocation_entry reloc;
> struct drm_i915_gem_execbuffer2 execbuf;
> + const uint32_t mmio_base = gem_engine_mmio_base(fd, name);
> igt_spin_t *spin = NULL;
> IGT_CORK_HANDLE(c);
> volatile uint32_t *reg;
> @@ -128,7 +129,8 @@ static void latency_on_ring(int fd,
> double gpu_latency;
> int i, j;
>
> - reg = (volatile uint32_t *)((volatile char *)igt_global_mmio + RCS_TIMESTAMP);
> + igt_require(mmio_base);
> + reg = (volatile uint32_t *)((volatile char *)igt_global_mmio + mmio_base + TIMESTAMP);
>
> memset(&execbuf, 0, sizeof(execbuf));
> execbuf.buffers_ptr = to_user_pointer(&obj[1]);
> @@ -176,7 +178,7 @@ static void latency_on_ring(int fd,
> map[i++] = 0x24 << 23 | 1;
> if (has_64bit_reloc)
> map[i-1]++;
> - map[i++] = RCS_TIMESTAMP; /* ring local! */
> + map[i++] = mmio_base + TIMESTAMP;
> map[i++] = offset;
> if (has_64bit_reloc)
> map[i++] = offset >> 32;
> @@ -266,11 +268,14 @@ static void latency_from_ring(int fd,
> struct drm_i915_gem_exec_object2 obj[3];
> struct drm_i915_gem_relocation_entry reloc;
> struct drm_i915_gem_execbuffer2 execbuf;
> + const uint32_t mmio_base = gem_engine_mmio_base(fd, name);
> const unsigned int repeats = ring_size / 2;
> uint32_t *map, *results;
> uint32_t ctx[2] = {};
> int i, j;
>
> + igt_require(mmio_base);
> +
> if (flags & PREEMPT) {
> ctx[0] = gem_context_create(fd);
> gem_context_set_priority(fd, ctx[0], -1023);
> @@ -351,7 +356,7 @@ static void latency_from_ring(int fd,
> map[i++] = 0x24 << 23 | 1;
> if (has_64bit_reloc)
> map[i-1]++;
> - map[i++] = RCS_TIMESTAMP; /* ring local! */
> + map[i++] = mmio_base + TIMESTAMP;
> map[i++] = offset;
> if (has_64bit_reloc)
> map[i++] = offset >> 32;
> @@ -376,7 +381,7 @@ static void latency_from_ring(int fd,
> map[i++] = 0x24 << 23 | 1;
> if (has_64bit_reloc)
> map[i-1]++;
> - map[i++] = RCS_TIMESTAMP; /* ring local! */
> + map[i++] = mmio_base + TIMESTAMP;
> map[i++] = offset;
> if (has_64bit_reloc)
> map[i++] = offset >> 32;
> @@ -669,7 +674,7 @@ igt_main
> ring_size = 1024;
>
> intel_register_access_init(&mmio_data, intel_get_pci_device(), false, device);
> - rcs_clock = clockrate(device, RCS_TIMESTAMP);
> + rcs_clock = clockrate(device, 0x2000 + TIMESTAMP);
> igt_info("RCS timestamp clock: %.0fKHz, %.1fns\n",
> rcs_clock / 1e3, 1e9 / rcs_clock);
> rcs_clock = 1e9 / rcs_clock;
More information about the igt-dev
mailing list