[igt-dev] [PATCH i-g-t v3 4/5] tests/xe_evict_ccs: Add evict ccs test

Matthew Auld matthew.william.auld at gmail.com
Tue Oct 24 14:57:25 UTC 2023


On Thu, 19 Oct 2023 at 20:33, Zbigniew Kempczyński
<zbigniew.kempczynski at intel.com> wrote:
>
> Exercise is flat-ccs eviction working fine in the kernel driver
> when buffers takes more than available vram. Differentiate with
> standalone/parallel execution, same or separate drm fd and buffer
> freeing time. Tests are divided to two groups - first which won't
> exceed vram memory size (thus don't trigger eviction, but it is
> good for the reference logic is properly compress/decompress
> buffers) and second which exceeds.
>
> v2:
> - Add command line switches to exercise kernel with different
>   sizes, number of objects and vram overcommitment
> - Add -simple test which creates single big object which enforces
>   eviction (Matt)
> v3:
> - Address review comments (Matt)
>
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> Cc: Matthew Auld <matthew.auld at intel.com>
> ---
>  tests/intel/xe_evict_ccs.c | 520 +++++++++++++++++++++++++++++++++++++
>  tests/meson.build          |   1 +
>  2 files changed, 521 insertions(+)
>  create mode 100644 tests/intel/xe_evict_ccs.c
>
> diff --git a/tests/intel/xe_evict_ccs.c b/tests/intel/xe_evict_ccs.c
> new file mode 100644
> index 0000000000..6f3a142683
> --- /dev/null
> +++ b/tests/intel/xe_evict_ccs.c
> @@ -0,0 +1,520 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * Copyright © 2023 Intel Corporation
> + */
> +
> +/**
> + * TEST: Check flat-ccs eviction
> + * Category: Software building block
> + * Sub-category: Flat-CCS
> + * Functionality: evict
> + * GPU requirements: GPU needs to have dedicated VRAM
> + */
> +
> +#include "igt.h"
> +#include "igt_list.h"
> +#include "intel_blt.h"
> +#include "intel_mocs.h"
> +#include "lib/igt_syncobj.h"
> +#include "lib/intel_reg.h"
> +#include "xe_drm.h"
> +
> +#include "xe/xe_ioctl.h"
> +#include "xe/xe_query.h"
> +#include <math.h>
> +#include <string.h>
> +
> +#define OVERCOMMIT_VRAM_PERCENT 110
> +#define MIN_OBJ_KB 64
> +#define MAX_OBJ_KB (256 * 1024)
> +#define DUMP_FILENAME "/tmp/object.data"
> +#define DUMP_EXPFILENAME "/tmp/object.expected"
> +
> +static struct param {
> +       bool print_bb;
> +       bool disable_compression;
> +       bool dump_corrupted_surface;
> +       int num_objs;
> +       int vram_percent;
> +       int min_size_kb;
> +       int max_size_kb;
> +       bool verify;
> +} params = {
> +       .num_objs = 0,
> +       .vram_percent = OVERCOMMIT_VRAM_PERCENT,
> +       .min_size_kb = MIN_OBJ_KB,
> +       .max_size_kb = MAX_OBJ_KB,
> +};
> +
> +struct object {
> +       uint64_t size;
> +       uint32_t start_value;
> +       struct blt_copy_object *blt_obj;
> +       struct igt_list_head link;
> +};
> +
> +#define TEST_PARALLEL          (1 << 0)
> +#define TEST_INSTANTFREE       (1 << 1)
> +#define TEST_REOPEN            (1 << 2)
> +#define TEST_SIMPLE            (1 << 3)
> +
> +#define MAX_NPROC 8
> +struct config {
> +       uint32_t flags;
> +       int nproc;
> +       int free_mb, total_mb;
> +       int test_mb, mb_per_proc;
> +       const struct param *param;
> +};
> +
> +static void copy_obj(struct blt_copy_data *blt,
> +                    struct blt_copy_object *src_obj,
> +                    struct blt_copy_object *dst_obj,
> +                    intel_ctx_t *ctx,
> +                    uint64_t ahnd)
> +{
> +       struct blt_block_copy_data_ext ext = {};
> +       int fd = blt->fd;
> +       uint64_t bb_size = xe_get_default_alignment(fd);
> +       uint32_t bb;
> +       uint32_t w, h;
> +
> +       w = src_obj->x2;
> +       h = src_obj->y2;
> +
> +       bb = xe_bo_create_flags(fd, 0, bb_size, visible_vram_memory(fd, 0));
> +
> +       blt->color_depth = CD_32bit;
> +       blt->print_bb = params.print_bb;
> +       blt_set_copy_object(&blt->src, src_obj);
> +       blt_set_copy_object(&blt->dst, dst_obj);
> +       blt_set_object_ext(&ext.src, 0, w, h, SURFACE_TYPE_2D);
> +       blt_set_object_ext(&ext.dst, 0, w, h, SURFACE_TYPE_2D);
> +       blt_set_batch(&blt->bb, bb, bb_size, vram_if_possible(fd, 0));
> +       blt_block_copy(fd, ctx, NULL, ahnd, blt, &ext);
> +       intel_ctx_xe_sync(ctx, true);
> +
> +       gem_close(fd, bb);
> +       put_offset(ahnd, bb);
> +       put_offset(ahnd, blt->src.handle);
> +       put_offset(ahnd, blt->dst.handle);
> +       intel_allocator_bind(ahnd, 0, 0);
> +}
> +
> +static uint32_t rand_and_update(uint32_t *left, uint32_t min, uint32_t max)
> +{
> +       int left_bit, min_bit, max_bit, rand_id, rand_kb;
> +
> +       left_bit = igt_fls(*left) - 1;
> +       min_bit = igt_fls(min) - 1;
> +       max_bit = max_t(int, min_t(int, igt_fls(max) - 1, left_bit), igt_fls(max));
> +       rand_id = rand() % (max_bit - min_bit);
> +       rand_kb = 1 << (rand_id + min_bit);
> +
> +       if (*left >= rand_kb)
> +               *left -= rand_kb;
> +       else
> +               *left = 0;
> +
> +       return rand_kb;
> +}
> +
> +static struct object *create_obj(struct blt_copy_data *blt,
> +                                intel_ctx_t *ctx, uint64_t ahnd,
> +                                uint64_t size, int start_value,
> +                                bool disable_compression)
> +{
> +       int fd = blt->fd;
> +       struct object *obj;
> +       uint32_t w, h;
> +       uint8_t uc_mocs = intel_get_uc_mocs_index(fd);
> +       int i;
> +       struct blt_copy_object *src;
> +
> +       obj = calloc(1, sizeof(*obj));
> +       igt_assert(obj);
> +       obj->size = size;
> +       obj->start_value = start_value;
> +
> +       w = max_t(int, 1024, roundup_power_of_two(sqrt(size/4)));
> +       h = size / w / 4; /* /4 - 32bpp */
> +
> +       igt_debug("[%8d] Obj size: %ldKiB (%ldMiB) <w: %d, h: %d>\n",
> +                 getpid(), size / SZ_1K, size / SZ_1M, w, h);
> +
> +       src = blt_create_object(blt,
> +                               system_memory(fd),
> +                               w, h, 32, uc_mocs,
> +                               T_LINEAR, COMPRESSION_DISABLED,
> +                               COMPRESSION_TYPE_3D, true);
> +
> +       obj->blt_obj = blt_create_object(blt, vram_memory(fd, 0),
> +                                        w, h, 32, uc_mocs,
> +                                        T_LINEAR,
> +                                        disable_compression ? COMPRESSION_DISABLED :
> +                                                              COMPRESSION_ENABLED,
> +                                        COMPRESSION_TYPE_3D, true);
> +
> +       for (i = 0; i < size / sizeof(uint32_t); i++)
> +               src->ptr[i] = start_value++;
> +
> +       copy_obj(blt, src, obj->blt_obj, ctx, ahnd);
> +
> +       blt_destroy_object_and_alloc_free(fd, ahnd, src);
> +       intel_allocator_bind(ahnd, 0, 0);
> +
> +       return obj;
> +}
> +
> +static void dump_obj(const struct blt_copy_object *obj, int start_value)
> +{
> +       FILE *out;
> +
> +       if (!params.dump_corrupted_surface)
> +               return;
> +
> +       out = fopen(DUMP_FILENAME, "wb");
> +       fwrite(obj->ptr, obj->size, 1, out);
> +       fclose(out);
> +
> +       out = fopen(DUMP_EXPFILENAME, "wb");
> +       for (int i = 0; i < obj->size / 4; i++) {
> +               int v = start_value + i;
> +
> +               fwrite(&v, sizeof(int), 1, out);
> +       }
> +       fclose(out);
> +}
> +
> +static void check_obj(const char *check_mode,
> +                     const struct blt_copy_object *obj, uint64_t size,
> +                     int start_value, int num_obj)
> +{
> +       int i, idx;
> +
> +       if (obj->ptr[0] != start_value ||
> +           (obj->ptr[size/4 - 1] != start_value + size/4 - 1)) {
> +               igt_info("[%s] Failed object w: %d, h: %d, size: %ldKiB (%ldMiB)\n",
> +                        check_mode, obj->x2, obj->y2, obj->size / SZ_1K, obj->size / SZ_1M);
> +               dump_obj(obj, start_value);
> +       }
> +
> +       igt_assert_eq(obj->ptr[0], start_value);
> +       igt_assert_eq(obj->ptr[size/4 - 1], start_value + size/4 - 1);
> +
> +       /* Couple of checks of random indices */
> +       for (i = 0; i < 128; i++) {
> +               idx = rand() % (size/4);
> +
> +               if (obj->ptr[idx] != start_value + idx) {
> +                       igt_info("[%s] Failed object w: %d, h: %d, size: %ldKiB (%ldMiB)\n",
> +                                check_mode, obj->x2, obj->y2,
> +                                obj->size / SZ_1K, obj->size / SZ_1M);
> +                       dump_obj(obj, start_value);
> +               }
> +
> +               igt_assert_f(obj->ptr[idx] == start_value + idx,
> +                            "[%s] Object number %d doesn't contain valid data",
> +                            check_mode, num_obj);
> +       }
> +}
> +
> +static void evict_single(int fd, int child, const struct config *config)
> +{
> +       struct blt_copy_data blt = {};
> +       struct blt_copy_object *orig_obj;
> +       uint32_t kb_left = config->mb_per_proc * SZ_1K;
> +       uint32_t min_alloc_kb = config->param->min_size_kb;
> +       uint32_t max_alloc_kb = config->param->max_size_kb;
> +       uint32_t vm = xe_vm_create(fd, DRM_XE_VM_CREATE_ASYNC_DEFAULT, 0);
> +       uint64_t ahnd = intel_allocator_open(fd, vm, INTEL_ALLOCATOR_RELOC);
> +       uint8_t uc_mocs = intel_get_uc_mocs_index(fd);
> +       struct object *obj, *tmp;
> +       struct igt_list_head list;
> +       struct drm_xe_engine_class_instance inst = {
> +               .engine_class = DRM_XE_ENGINE_CLASS_COPY,
> +       };
> +       intel_ctx_t *ctx;
> +       uint32_t exec_queue, big_obj;
> +       int num_obj = 0;
> +
> +       srandom(time(NULL) + getpid());
> +       IGT_INIT_LIST_HEAD(&list);
> +       igt_debug("[%2d] child : to allocate: %uMiB\n", child, kb_left/SZ_1K);
> +
> +       blt_copy_init(fd, &blt);
> +
> +       exec_queue = xe_exec_queue_create(fd, vm, &inst, 0);
> +       ctx = intel_ctx_xe(fd, vm, exec_queue, 0, 0, 0);
> +
> +       while (kb_left) {
> +               struct blt_copy_object *verify_obj;
> +               uint64_t obj_size = rand_and_update(&kb_left, min_alloc_kb, max_alloc_kb) * SZ_1K;
> +               int start_value = rand();
> +
> +               if (config->flags & TEST_SIMPLE)
> +                       obj_size = max_alloc_kb * SZ_1K;
> +
> +               obj = create_obj(&blt, ctx, ahnd, obj_size, start_value,
> +                                config->param->disable_compression);
> +               igt_list_add(&obj->link, &list);
> +
> +               if (config->param->verify) {
> +                       verify_obj = blt_create_object(&blt, system_memory(fd),
> +                                                      obj->blt_obj->x2,
> +                                                      obj->blt_obj->y2,
> +                                                      32, uc_mocs,
> +                                                      T_LINEAR, COMPRESSION_DISABLED,
> +                                                      0, true);
> +                       copy_obj(&blt, obj->blt_obj, verify_obj, ctx, ahnd);
> +                       check_obj("Verify", verify_obj, obj->blt_obj->size,
> +                                 obj->start_value, num_obj++);
> +                       blt_destroy_object_and_alloc_free(fd, ahnd, verify_obj);
> +                       intel_allocator_bind(ahnd, 0, 0);
> +               }
> +
> +               if (config->flags & TEST_SIMPLE) {
> +                       big_obj = xe_bo_create_flags(fd, vm, kb_left * SZ_1K,
> +                                                    vram_memory(fd, 0));
> +                       break;
> +               }
> +
> +               if (config->param->num_objs && ++num_obj == config->param->num_objs)
> +                       break;
> +       }
> +
> +       if (config->param->verify)
> +               igt_info("[%8d] Verify ok\n", getpid());
> +
> +       num_obj = 0;
> +       igt_list_for_each_entry_safe(obj, tmp, &list, link) {
> +               orig_obj = blt_create_object(&blt, system_memory(fd),
> +                                            obj->blt_obj->x2,
> +                                            obj->blt_obj->y2,
> +                                            32, uc_mocs,
> +                                            T_LINEAR, COMPRESSION_DISABLED,
> +                                            0, true);
> +               copy_obj(&blt, obj->blt_obj, orig_obj, ctx, ahnd);
> +               check_obj("Check", orig_obj, obj->blt_obj->size, obj->start_value, num_obj++);
> +               blt_destroy_object_and_alloc_free(fd, ahnd, orig_obj);
> +
> +               if (config->flags & TEST_INSTANTFREE) {
> +                       igt_list_del(&obj->link);
> +                       blt_destroy_object_and_alloc_free(fd, ahnd, obj->blt_obj);
> +                       free(obj);
> +               }
> +               intel_allocator_bind(ahnd, 0, 0);
> +       }
> +
> +       if (!(config->flags & TEST_INSTANTFREE))
> +               igt_list_for_each_entry_safe(obj, tmp, &list, link) {
> +                       igt_list_del(&obj->link);
> +                       blt_destroy_object_and_alloc_free(fd, ahnd, obj->blt_obj);
> +                       free(obj);
> +               }
> +
> +       if (config->flags & TEST_SIMPLE)
> +               gem_close(fd, big_obj);
> +}
> +
> +static void set_config(int fd, uint32_t flags, const struct param *param,
> +                      struct config *config)
> +{
> +       int nproc = 1;
> +
> +       config->param = param;
> +       config->flags = flags;
> +       config->free_mb = xe_visible_vram_size(fd, 0) / SZ_1M;
> +       config->total_mb = xe_vram_available(fd, 0) / SZ_1M;
> +       config->test_mb = min_t(int, config->free_mb * config->param->vram_percent / 100,
> +                               config->total_mb * config->param->vram_percent / 100);
> +
> +       igt_debug("VRAM memory size: %dMB/%dMB (use %dMB), overcommit perc: %d\n",
> +                 config->free_mb, config->total_mb,
> +                 config->test_mb, config->param->vram_percent);
> +
> +       if (flags & TEST_PARALLEL)
> +               nproc = min_t(int, sysconf(_SC_NPROCESSORS_ONLN), MAX_NPROC);
> +       config->nproc = nproc;
> +       config->mb_per_proc = config->test_mb / nproc;
> +
> +       igt_debug("nproc: %d, mem per proc: %dMB\n", nproc, config->mb_per_proc);
> +}
> +
> +static void evict_ccs(int fd, uint32_t flags, const struct param *param)
> +{
> +       struct config config;
> +       char numstr[32];
> +
> +       igt_info("Test mode <parallel: %d, instant free: %d, reopen: %d, simple: %d>\n",
> +                !!(flags & TEST_PARALLEL),
> +                !!(flags & TEST_INSTANTFREE),
> +                !!(flags & TEST_REOPEN),
> +                !!(flags & TEST_SIMPLE));
> +       if (param->num_objs)
> +               snprintf(numstr, sizeof(numstr), "%d", param->num_objs);
> +       else
> +               strncpy(numstr, "limited to vram", sizeof(numstr));
> +       igt_info("Params: compression: %s, num objects: %s, vram percent: %d, kb <min: %d, max: %d>\n",
> +                param->disable_compression ? "disabled" : "enabled",
> +                numstr, param->vram_percent,
> +                param->min_size_kb, param->max_size_kb);
> +
> +       set_config(fd, flags, param, &config);
> +
> +       if (flags & TEST_PARALLEL) {
> +               igt_fork(n, config.nproc) {
> +                       if (flags & TEST_REOPEN) {
> +                               fd = drm_reopen_driver(fd);
> +                               intel_allocator_init();
> +                       }
> +                       evict_single(fd, n, &config);
> +               }
> +               igt_waitchildren();
> +       } else {
> +               if (flags & TEST_REOPEN)
> +                       fd = drm_reopen_driver(fd);
> +               evict_single(fd, 0, &config);
> +       }
> +}
> +
> +/**
> + *
> + * SUBTEST: evict-ccs-overcommit-simple
> + * Description: FlatCCS eviction test.
> + * Feature: flatccs
> + * Test category: stress test
> + */
> +/**
> + *
> + * SUBTEST: evict-ccs-overcommit-%s-%s-%s
> + * Description: FlatCCS eviction test.
> + * Feature: flatccs
> + * Test category: stress test
> + *
> + * arg[1]:
> + *
> + * @standalone:                        single process
> + * @parallel:                  multiple processes
> + *
> + * arg[2]:
> + *
> + * @nofree:                    keep objects till the end of the test
> + * @instantfree:               free object after it was verified and it won't
> + *                             be used anymore
> + *
> + * arg[3]:
> + *
> + * @samefd:                    operate on same opened drm fd
> + * @reopen:                    use separately opened drm fds
> + *
> + */
> +static int opt_handler(int opt, int opt_index, void *data)
> +{
> +       switch (opt) {
> +       case 'b':
> +               params.print_bb = true;
> +               igt_debug("Print bb: %d\n", params.print_bb);
> +               break;
> +       case 'd':
> +               params.disable_compression = true;
> +               igt_debug("Print bb: %d\n", params.disable_compression);
> +               break;
> +       case 'D':
> +               params.dump_corrupted_surface = true;
> +               igt_debug("Print bb: %d\n", params.dump_corrupted_surface);
> +               break;
> +       case 'n':
> +               params.num_objs = atoi(optarg);
> +               igt_debug("Number objects: %d\n", params.num_objs);
> +               break;
> +       case 'p':
> +               params.vram_percent = atoi(optarg);
> +               igt_debug("Percent vram: %d\n", params.vram_percent);
> +               break;
> +       case 's':
> +               params.min_size_kb = atoi(optarg);
> +               igt_debug("Min size kb: %d\n", params.min_size_kb);
> +               break;
> +       case 'S':
> +               params.max_size_kb = atoi(optarg);
> +               igt_debug("Max size kb: %d\n", params.max_size_kb);
> +               break;
> +       case 'V':
> +               params.verify = true;
> +               igt_debug("Verify: %d\n", params.verify);
> +               break;
> +       default:
> +               return IGT_OPT_HANDLER_ERROR;
> +       }
> +
> +       return IGT_OPT_HANDLER_SUCCESS;
> +}
> +
> +const char *help_str =
> +       "  -b\tPrint bb\n"
> +       "  -d\tDisable compression (don't use flatccs area)\n"
> +       "  -D\tDump surface which doesn't match\n"
> +       "  -e\tAdd temporary object which enforce eviction\n"
> +       "  -n\tNumber of objects to create (0 - 31)\n"
> +       "  -p\tPercent of VRAM to alloc\n"
> +       "  -s\tMinimum size of object in kb\n"
> +       "  -S\tMaximum size of object in kb\n"
> +       "  -V\tVerify object after compressing\n"
> +       ;
> +
> +igt_main_args("bdDn:p:s:S:V", NULL, help_str, opt_handler, NULL)
> +{
> +       struct drm_xe_engine_class_instance *hwe;
> +
> +       const struct ccs {
> +               const char *name;
> +               uint32_t flags;
> +       } ccs[] = {
> +               { "simple",
> +                       TEST_SIMPLE },
> +               { "standalone-nofree-samefd",
> +                       0 },
> +               { "standalone-nofree-reopen",
> +                       TEST_REOPEN },
> +               { "standalone-instantfree-samefd",
> +                       TEST_INSTANTFREE },
> +               { "standalone-instantfree-reopen",
> +                       TEST_INSTANTFREE | TEST_REOPEN },
> +               { "parallel-nofree-samefd",
> +                       TEST_PARALLEL },
> +               { "parallel-nofree-reopen",
> +                       TEST_PARALLEL | TEST_REOPEN },
> +               { "parallel-instantfree-samefd",
> +                       TEST_PARALLEL | TEST_INSTANTFREE },
> +               { "parallel-instantfree-reopen",
> +                       TEST_PARALLEL | TEST_INSTANTFREE | TEST_REOPEN },
> +               { },
> +       };
> +       uint64_t vram_size;
> +       int fd;
> +
> +       igt_fixture {
> +               fd = drm_open_driver(DRIVER_XE);
> +               igt_require(xe_has_vram(fd));
> +               vram_size = xe_visible_vram_size(fd, 0);
> +               igt_assert(vram_size);
> +
> +               xe_for_each_hw_engine(fd, hwe)
> +                       if (hwe->engine_class != DRM_XE_ENGINE_CLASS_COPY)
> +                               break;

What is the purpose of this loop? Maybe I'm blind.

Otherwise,
Reviewed-by: Matthew Auld <matthew.auld at intel.com>

> +       }
> +
> +       igt_fixture
> +               intel_allocator_multiprocess_start();
> +
> +       for (const struct ccs *s = ccs; s->name; s++) {
> +               igt_subtest_f("evict-ccs-overcommit-%s", s->name)
> +                       evict_ccs(fd, s->flags, &params);
> +       }
> +
> +       igt_fixture {
> +               intel_allocator_multiprocess_stop();
> +               drm_close_driver(fd);
> +       }
> +}
> diff --git a/tests/meson.build b/tests/meson.build
> index 14b5bfda26..df64bf18b9 100644
> --- a/tests/meson.build
> +++ b/tests/meson.build
> @@ -279,6 +279,7 @@ intel_xe_progs = [
>         'xe_debugfs',
>         'xe_drm_fdinfo',
>         'xe_evict',
> +       'xe_evict_ccs',
>         'xe_exec_balancer',
>         'xe_exec_basic',
>         'xe_exec_compute_mode',
> --
> 2.34.1
>


More information about the igt-dev mailing list