[igt-dev] [PATCH v2 i-g-t] test/xe_spin_batch: Add spin-fixed-duration-with-preempter
Kumar, Janga Rahul
janga.rahul.kumar at intel.com
Mon Dec 4 10:15:08 UTC 2023
LGTM,
Reviewed-by: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> -----Original Message-----
> From: Das, Nirmoy <nirmoy.das at intel.com>
> Sent: Thursday, November 30, 2023 11:31 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Das, Nirmoy <nirmoy.das at intel.com>; Kempczynski, Zbigniew
> <zbigniew.kempczynski at intel.com>; Kumar, Janga Rahul
> <janga.rahul.kumar at intel.com>
> Subject: [PATCH v2 i-g-t] test/xe_spin_batch: Add spin-fixed-duration-with-
> preempter
>
> Add a variation of spin-fixed-duration where the spinners gets preempted with a
> short duration high priority task.
>
> This validates preemption in GPU and xe_spin.
>
> v2: rebase
>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> Cc: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> Signed-off-by: Nirmoy Das <nirmoy.das at intel.com>
> ---
> tests/intel/xe_spin_batch.c | 126 ++++++++++++++++++++++++++++++++++--
> 1 file changed, 122 insertions(+), 4 deletions(-)
>
> diff --git a/tests/intel/xe_spin_batch.c b/tests/intel/xe_spin_batch.c index
> 6ab604d9b..345776875 100644
> --- a/tests/intel/xe_spin_batch.c
> +++ b/tests/intel/xe_spin_batch.c
> @@ -136,12 +136,102 @@ static void spin_all(int fd, int gt, int class)
> xe_vm_destroy(fd, vm);
> }
>
> +struct data {
> + uint32_t batch[16];
> + uint64_t pad;
> + uint32_t data;
> + uint64_t addr;
> +};
> +
> +static void store_dword_batch(struct data *data, uint64_t addr, int
> +value) {
> + int b;
> + uint64_t batch_offset = (char *)&(data->batch) - (char *)data;
> + uint64_t batch_addr = addr + batch_offset;
> + uint64_t sdi_offset = (char *)&(data->data) - (char *)data;
> + uint64_t sdi_addr = addr + sdi_offset;
> +
> + b = 0;
> + data->batch[b++] = MI_STORE_DWORD_IMM_GEN4;
> + data->batch[b++] = sdi_addr;
> + data->batch[b++] = sdi_addr >> 32;
> + data->batch[b++] = value;
> + data->batch[b++] = MI_BATCH_BUFFER_END;
> + igt_assert(b <= ARRAY_SIZE(data->batch));
> +
> + data->addr = batch_addr;
> +}
> +
> +static void preempter(int fd, struct drm_xe_engine_class_instance *hwe)
> +{
> + struct drm_xe_sync sync = {
> + .flags = DRM_XE_SYNC_FLAG_SYNCOBJ |
> DRM_XE_SYNC_FLAG_SIGNAL
> + };
> + struct drm_xe_exec exec = {
> + .num_batch_buffer = 1,
> + .num_syncs = 1,
> + .syncs = to_user_pointer(&sync),
> + };
> + struct drm_xe_ext_set_property ext = {
> + .base.next_extension = 0,
> + .base.name =
> DRM_XE_EXEC_QUEUE_EXTENSION_SET_PROPERTY,
> + .property = DRM_XE_EXEC_QUEUE_SET_PROPERTY_PRIORITY,
> + .value = 2, /* High priority */
> + };
> + struct data *data;
> + uint32_t vm;
> + uint32_t exec_queue;
> + uint32_t syncobj;
> + size_t bo_size;
> + int value = 0x123456;
> + uint64_t addr = 0x100000;
> + uint32_t bo = 0;
> +
> + syncobj = syncobj_create(fd, 0);
> + sync.handle = syncobj;
> +
> + vm = xe_vm_create(fd, DRM_XE_VM_CREATE_FLAG_ASYNC_DEFAULT,
> 0);
> + bo_size = sizeof(*data);
> + bo_size = ALIGN(bo_size + xe_cs_prefetch_size(fd),
> + xe_get_default_alignment(fd));
> +
> + bo = xe_bo_create_flags(fd, vm, bo_size,
> + visible_vram_if_possible(fd, hwe->gt_id));
> +
> + xe_vm_bind_async(fd, vm, hwe->gt_id, bo, 0, addr, bo_size, &sync, 1);
> + data = xe_bo_map(fd, bo, bo_size);
> + store_dword_batch(data, addr, value);
> +
> + exec_queue = xe_exec_queue_create(fd, vm, hwe,
> to_user_pointer(&ext));
> + exec.exec_queue_id = exec_queue;
> + exec.address = data->addr;
> + sync.flags &= DRM_XE_SYNC_FLAG_SIGNAL;
> + xe_exec(fd, &exec);
> +
> + igt_assert(syncobj_wait(fd, &syncobj, 1, INT64_MAX, 0, NULL));
> + igt_assert_eq(data->data, value);
> +
> + syncobj_destroy(fd, syncobj);
> + munmap(data, bo_size);
> + gem_close(fd, bo);
> +
> + xe_exec_queue_destroy(fd, exec_queue);
> + xe_vm_destroy(fd, vm);
> +}
> +
> +#define SPIN_FIX_DURATION_NORMAL 0
> +#define SPIN_FIX_DURATION_PREEMPT 1
> /**
> * SUBTEST: spin-fixed-duration
> * Description: Basic test which validates the functionality of xe_spin with fixed
> duration.
> * Run type: FULL
> */
> -static void xe_spin_fixed_duration(int fd)
> +/**
> + * SUBTEST: spin-fixed-duration-with-preempter
> + * Description: Basic test which validates the functionality of xe_spin
> preemption which gets preempted with a short duration high-priority task.
> + * Run type: FULL
> + */
> +static void xe_spin_fixed_duration(int fd, int gt, int class, int
> +flags)
> {
> struct drm_xe_sync sync = {
> .handle = syncobj_create(fd, 0),
> @@ -152,12 +242,20 @@ static void xe_spin_fixed_duration(int fd)
> .num_syncs = 1,
> .syncs = to_user_pointer(&sync),
> };
> + struct drm_xe_ext_set_property ext_prio = {
> + .base.next_extension = 0,
> + .base.name =
> DRM_XE_EXEC_QUEUE_EXTENSION_SET_PROPERTY,
> + .property = DRM_XE_EXEC_QUEUE_SET_PROPERTY_PRIORITY,
> + .value = 0, /* Low priority */
> + };
> + struct drm_xe_engine_class_instance *hwe = NULL, *_hwe;
> const uint64_t duration_ns = NSEC_PER_SEC / 10; /* 100ms */
> uint64_t spin_addr;
> uint64_t ahnd;
> uint32_t exec_queue;
> uint32_t vm;
> uint32_t bo;
> + uint64_t ext = 0;
> size_t bo_size;
> struct xe_spin *spin;
> struct timespec tv;
> @@ -165,8 +263,18 @@ static void xe_spin_fixed_duration(int fd)
> igt_stats_t stats;
> int i;
>
> + if (flags & SPIN_FIX_DURATION_PREEMPT)
> + ext = to_user_pointer(&ext_prio);
> +
> + xe_for_each_hw_engine(fd, _hwe)
> + if (_hwe->engine_class == class && _hwe->gt_id == gt)
> + hwe = _hwe;
> +
> + if (!hwe)
> + return;
> +
> vm = xe_vm_create(fd, 0, 0);
> - exec_queue = xe_exec_queue_create_class(fd, vm,
> DRM_XE_ENGINE_CLASS_COPY);
> + exec_queue = xe_exec_queue_create(fd, vm, hwe, ext);
> ahnd = intel_allocator_open(fd, 0, INTEL_ALLOCATOR_RELOC);
> bo_size = ALIGN(sizeof(*spin) + xe_cs_prefetch_size(fd),
> xe_get_default_alignment(fd));
> bo = xe_bo_create(fd, 0, vm, bo_size); @@ -186,13 +294,17 @@ static
> void xe_spin_fixed_duration(int fd)
> igt_gettime(&tv);
> xe_exec(fd, &exec);
> xe_spin_wait_started(spin);
> + if (flags & SPIN_FIX_DURATION_PREEMPT)
> + preempter(fd, hwe);
> +
> igt_assert(syncobj_wait(fd, &sync.handle, 1, INT64_MAX, 0,
> NULL));
> igt_stats_push_float(&stats, igt_nsec_elapsed(&tv) * 1e-6);
> syncobj_reset(fd, &sync.handle, 1);
> igt_debug("i=%d %.2fms\n", i, stats.values_f[i]);
> }
> elapsed_ms = igt_stats_get_median(&stats);
> - igt_info("%.0fms spin took %.2fms (median)\n", duration_ns * 1e-6,
> elapsed_ms);
> + igt_info("%s: %.0fms spin took %.2fms (median)\n",
> xe_engine_class_string(hwe->engine_class),
> + duration_ns * 1e-6, elapsed_ms);
> igt_assert(elapsed_ms < duration_ns * 1.5e-6 && elapsed_ms >
> duration_ns * 0.5e-6);
>
> xe_vm_unbind_sync(fd, vm, 0, spin_addr, bo_size); @@ -230,7 +342,13
> @@ igt_main
> }
>
> igt_subtest("spin-fixed-duration")
> - xe_spin_fixed_duration(fd);
> + xe_spin_fixed_duration(fd, 0, DRM_XE_ENGINE_CLASS_COPY,
> +SPIN_FIX_DURATION_NORMAL);
> +
> +
> + igt_subtest("spin-fixed-duration-with-preempter")
> + xe_for_each_gt(fd, gt)
> + xe_for_each_hw_engine_class(class)
> + xe_spin_fixed_duration(fd, gt, class,
> SPIN_FIX_DURATION_PREEMPT);
>
> igt_fixture
> drm_close_driver(fd);
> --
> 2.42.0
More information about the igt-dev
mailing list