[igt-dev] [PATCH i-g-t] benchmarks/gem_userptr_benchmark: Dead code removal
Zbigniew Kempczyński
zbigniew.kempczynski at intel.com
Fri May 21 14:17:38 UTC 2021
It seems nobody cares of this benchmark a long time - it is still having
libdrm dependency what is not true nor initializing code is not within
igt_fixture. As it is not supposed to work lets rid off it from IGT.
Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
Cc: Petri Latvala <petri.latvala at intel.com>
---
benchmarks/gem_userptr_benchmark.c | 506 -----------------------------
benchmarks/meson.build | 1 -
2 files changed, 507 deletions(-)
delete mode 100644 benchmarks/gem_userptr_benchmark.c
diff --git a/benchmarks/gem_userptr_benchmark.c b/benchmarks/gem_userptr_benchmark.c
deleted file mode 100644
index d7a495209..000000000
--- a/benchmarks/gem_userptr_benchmark.c
+++ /dev/null
@@ -1,506 +0,0 @@
-/*
- * Copyright © 2014 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- *
- * Authors:
- * Tvrtko Ursulin <tvrtko.ursulin at intel.com>
- *
- */
-
-/** @file gem_userptr_benchmark.c
- *
- * Benchmark the userptr code and impact of having userptr surfaces
- * in process address space on some normal operations.
- *
- */
-
-#include <stdlib.h>
-#include <stdio.h>
-#include <string.h>
-#include <fcntl.h>
-#include <inttypes.h>
-#include <errno.h>
-#include <assert.h>
-#include <sys/stat.h>
-#include <sys/time.h>
-#include <sys/mman.h>
-#include <signal.h>
-
-#include "drm.h"
-#include "i915_drm.h"
-
-#include "drmtest.h"
-#include "intel_batchbuffer.h"
-#include "intel_chipset.h"
-#include "ioctl_wrappers.h"
-#include "igt_aux.h"
-
-#ifndef PAGE_SIZE
- #define PAGE_SIZE 4096
-#endif
-
-static uint32_t userptr_flags = I915_USERPTR_UNSYNCHRONIZED;
-
-#define BO_SIZE (65536)
-
-static void gem_userptr_test_unsynchronized(void)
-{
- userptr_flags = I915_USERPTR_UNSYNCHRONIZED;
-}
-
-static void gem_userptr_test_synchronized(void)
-{
- userptr_flags = 0;
-}
-
-static void **handle_ptr_map;
-static unsigned int num_handle_ptr_map;
-
-static void add_handle_ptr(uint32_t handle, void *ptr)
-{
- if (handle >= num_handle_ptr_map) {
- handle_ptr_map = realloc(handle_ptr_map,
- (handle + 1000) * sizeof(void*));
- num_handle_ptr_map = handle + 1000;
- }
-
- handle_ptr_map[handle] = ptr;
-}
-
-static void *get_handle_ptr(uint32_t handle)
-{
- return handle_ptr_map[handle];
-}
-
-static void free_handle_ptr(uint32_t handle)
-{
- igt_assert(handle < num_handle_ptr_map);
- igt_assert(handle_ptr_map[handle]);
-
- free(handle_ptr_map[handle]);
- handle_ptr_map[handle] = NULL;
-}
-
-static uint32_t create_userptr_bo(int fd, int size)
-{
- void *ptr;
- uint32_t handle;
- int ret;
-
- ret = posix_memalign(&ptr, PAGE_SIZE, size);
- igt_assert(ret == 0);
-
- gem_userptr(fd, (uint32_t *)ptr, size, 0, userptr_flags, &handle);
- add_handle_ptr(handle, ptr);
-
- return handle;
-}
-
-static void free_userptr_bo(int fd, uint32_t handle)
-{
- gem_close(fd, handle);
- free_handle_ptr(handle);
-}
-
-static int has_userptr(int fd)
-{
- uint32_t handle = 0;
- void *ptr;
- uint32_t oldflags;
- int ret;
-
- assert(posix_memalign(&ptr, PAGE_SIZE, PAGE_SIZE) == 0);
- oldflags = userptr_flags;
- gem_userptr_test_unsynchronized();
- ret = __gem_userptr(fd, ptr, PAGE_SIZE, 0, userptr_flags, &handle);
- userptr_flags = oldflags;
- if (ret != 0) {
- free(ptr);
- return 0;
- }
-
- gem_close(fd, handle);
- free(ptr);
-
- return handle != 0;
-}
-
-static const unsigned int nr_bos[] = {0, 1, 10, 100, 1000, 10000};
-static const unsigned int test_duration_sec = 3;
-
-static volatile unsigned int run_test;
-
-static void alarm_handler(int sig)
-{
- assert(run_test == 1);
- run_test = 0;
-}
-
-static void start_test(unsigned int duration)
-{
- run_test = 1;
- if (duration == 0)
- duration = test_duration_sec;
- signal(SIGALRM, alarm_handler);
- alarm(duration);
-}
-
-static void exchange_ptr(void *array, unsigned i, unsigned j)
-{
- void **arr, *tmp;
- arr = (void **)array;
-
- tmp = arr[i];
- arr[i] = arr[j];
- arr[j] = tmp;
-}
-
-static void test_malloc_free(int random)
-{
- unsigned long iter = 0;
- unsigned int i, tot = 1000;
- void *ptr[tot];
-
- start_test(test_duration_sec);
-
- while (run_test) {
- for (i = 0; i < tot; i++) {
- ptr[i] = malloc(1000);
- assert(ptr[i]);
- }
- if (random)
- igt_permute_array(ptr, tot, exchange_ptr);
- for (i = 0; i < tot; i++)
- free(ptr[i]);
- iter++;
- }
-
- printf("%8lu iter/s\n", iter / test_duration_sec);
-}
-
-static void test_malloc_realloc_free(int random)
-{
- unsigned long iter = 0;
- unsigned int i, tot = 1000;
- void *ptr[tot];
-
- start_test(test_duration_sec);
-
- while (run_test) {
- for (i = 0; i < tot; i++) {
- ptr[i] = malloc(1000);
- assert(ptr[i]);
- }
- if (random)
- igt_permute_array(ptr, tot, exchange_ptr);
- for (i = 0; i < tot; i++) {
- ptr[i] = realloc(ptr[i], 2000);
- assert(ptr[i]);
- }
- if (random)
- igt_permute_array(ptr, tot, exchange_ptr);
- for (i = 0; i < tot; i++)
- free(ptr[i]);
- iter++;
- }
-
- printf("%8lu iter/s\n", iter / test_duration_sec);
-}
-
-static void test_mmap_unmap(int random)
-{
- unsigned long iter = 0;
- unsigned int i, tot = 1000;
- void *ptr[tot];
-
- start_test(test_duration_sec);
-
- while (run_test) {
- for (i = 0; i < tot; i++) {
- ptr[i] = mmap(NULL, 1000, PROT_READ | PROT_WRITE,
- MAP_ANONYMOUS | MAP_PRIVATE, -1, 0);
- assert(ptr[i] != MAP_FAILED);
- }
- if (random)
- igt_permute_array(ptr, tot, exchange_ptr);
- for (i = 0; i < tot; i++)
- munmap(ptr[i], 1000);
- iter++;
- }
-
- printf("%8lu iter/s\n", iter / test_duration_sec);
-}
-
-static void test_ptr_read(void *ptr)
-{
- unsigned long iter = 0;
- volatile unsigned long *p;
- unsigned long i, loops;
-
- loops = BO_SIZE / sizeof(unsigned long) / 4;
-
- start_test(test_duration_sec);
-
- while (run_test) {
- p = (unsigned long *)ptr;
- for (i = 0; i < loops; i++) {
- (void)*p++;
- (void)*p++;
- (void)*p++;
- (void)*p++;
- }
- iter++;
- }
-
- printf("%8lu MB/s\n", iter / test_duration_sec * BO_SIZE / 1000000);
-}
-
-static void test_ptr_write(void *ptr)
-{
- unsigned long iter = 0;
- volatile unsigned long *p;
- register unsigned long i, loops;
-
- loops = BO_SIZE / sizeof(unsigned long) / 4;
-
- start_test(test_duration_sec);
-
- while (run_test) {
- p = (unsigned long *)ptr;
- for (i = 0; i < loops; i++) {
- *p++ = i;
- *p++ = i;
- *p++ = i;
- *p++ = i;
- }
- iter++;
- }
-
- printf("%8lu MB/s\n", iter / test_duration_sec * BO_SIZE / 1000000);
-}
-
-static void do_impact_tests(unsigned int n, const char *pfix, const char *pfix2,
- void *ptr)
-{
- printf("%s%sptr-read, %5u bos = ", pfix, pfix2, n);
- test_ptr_read(ptr);
-
- printf("%s%sptr-write %5u bos = ", pfix, pfix2, n);
- test_ptr_write(ptr);
-
- printf("%s%smalloc-free, %5u bos = ", pfix, pfix2, n);
- test_malloc_free(0);
- printf("%s%smalloc-free-random %5u bos = ", pfix, pfix2, n);
- test_malloc_free(1);
-
- printf("%s%smalloc-realloc-free, %5u bos = ", pfix, pfix2, n);
- test_malloc_realloc_free(0);
- printf("%s%smalloc-realloc-free-random, %5u bos = ", pfix, pfix2, n);
- test_malloc_realloc_free(1);
-
- printf("%s%smmap-unmap, %5u bos = ", pfix, pfix2, n);
- test_mmap_unmap(0);
- printf("%s%smmap-unmap-random, %5u bos = ", pfix, pfix2, n);
- test_mmap_unmap(1);
-}
-
-static void test_impact_overlap(int fd, const char *prefix)
-{
- unsigned int total = sizeof(nr_bos) / sizeof(nr_bos[0]);
- unsigned int subtest, i;
- uint32_t handles[nr_bos[total-1]];
- void *block = NULL;
- void *ptr;
- unsigned char *p;
- char buffer[BO_SIZE];
- int ret;
-
- for (subtest = 0; subtest < total; subtest++) {
- if (nr_bos[subtest] > 0) {
- igt_assert(PAGE_SIZE < BO_SIZE);
- ret = posix_memalign(&block, PAGE_SIZE,
- PAGE_SIZE * nr_bos[subtest] + BO_SIZE);
- igt_assert(ret == 0);
-
- for (i = 0, p = block; i < nr_bos[subtest];
- i++, p += PAGE_SIZE)
- gem_userptr(fd, (uint32_t *)p, BO_SIZE, 0, userptr_flags, &handles[i]);
- }
-
- if (nr_bos[subtest] > 0)
- ptr = block;
- else
- ptr = buffer;
-
- do_impact_tests(nr_bos[subtest], prefix, "overlap-", ptr);
-
- for (i = 0; i < nr_bos[subtest]; i++)
- gem_close(fd, handles[i]);
-
- free(block);
- block = NULL;
- }
-}
-
-static void test_impact(int fd, const char *prefix)
-{
- unsigned int total = sizeof(nr_bos) / sizeof(nr_bos[0]);
- unsigned int subtest, i;
- uint32_t handles[nr_bos[total-1]];
- void *ptr;
- char buffer[BO_SIZE];
-
- for (subtest = 0; subtest < total; subtest++) {
- for (i = 0; i < nr_bos[subtest]; i++)
- handles[i] = create_userptr_bo(fd, BO_SIZE);
-
- if (nr_bos[subtest] > 0)
- ptr = get_handle_ptr(handles[0]);
- else
- ptr = buffer;
-
- do_impact_tests(nr_bos[subtest], prefix, "no-overlap-", ptr);
-
- for (i = 0; i < nr_bos[subtest]; i++)
- free_userptr_bo(fd, handles[i]);
- }
-}
-
-static void test_single(int fd)
-{
- char *ptr, *bo_ptr;
- uint32_t handle = 0;
- unsigned long iter = 0;
- unsigned long map_size = BO_SIZE + PAGE_SIZE - 1;
-
- ptr = mmap(NULL, map_size, PROT_READ | PROT_WRITE,
- MAP_ANONYMOUS | MAP_PRIVATE, -1, 0);
- assert(ptr != MAP_FAILED);
-
- bo_ptr = (char *)ALIGN((unsigned long)ptr, PAGE_SIZE);
-
- start_test(test_duration_sec);
-
- while (run_test) {
- gem_userptr(fd, bo_ptr, BO_SIZE, 0, userptr_flags, &handle);
- gem_close(fd, handle);
- iter++;
- }
-
- munmap(ptr, map_size);
-
- printf("%8lu iter/s\n", iter / test_duration_sec);
-}
-
-static void test_multiple(int fd, unsigned int batch, int random)
-{
- char *ptr, *bo_ptr;
- uint32_t handles[10000];
- int map[10000];
- unsigned long iter = 0;
- int i;
- unsigned long map_size = batch * BO_SIZE + PAGE_SIZE - 1;
-
- assert(batch < (sizeof(handles) / sizeof(handles[0])));
- assert(batch < (sizeof(map) / sizeof(map[0])));
-
- ptr = mmap(NULL, map_size, PROT_READ | PROT_WRITE,
- MAP_ANONYMOUS | MAP_PRIVATE, -1, 0);
- assert(ptr != MAP_FAILED);
-
- bo_ptr = (char *)ALIGN((unsigned long)ptr, PAGE_SIZE);
-
- for (i = 0; i < batch; i++)
- map[i] = i;
-
- start_test(test_duration_sec);
-
- while (run_test) {
- if (random)
- igt_permute_array(map, batch, igt_exchange_int);
- for (i = 0; i < batch; i++) {
- gem_userptr(fd, bo_ptr + map[i] * BO_SIZE, BO_SIZE,
- 0, userptr_flags, &handles[i]);
- }
- if (random)
- igt_permute_array(map, batch, igt_exchange_int);
- for (i = 0; i < batch; i++)
- gem_close(fd, handles[map[i]]);
- iter++;
- }
-
- munmap(ptr, map_size);
-
- printf("%8lu iter/s\n", iter * batch / test_duration_sec);
-}
-
-static void test_userptr(int fd)
-{
- printf("create-destroy = ");
- test_single(fd);
-
- printf("multi-create-destroy = ");
- test_multiple(fd, 100, 0);
-
- printf("multi-create-destroy-random = ");
- test_multiple(fd, 100, 1);
-}
-
-int main(int argc, char **argv)
-{
- int fd = -1, ret;
-
- igt_subtest_init(argc, argv);
-
- fd = drm_open_driver(DRIVER_INTEL);
- igt_assert(fd >= 0);
-
- ret = has_userptr(fd);
- igt_skip_on_f(ret == 0, "No userptr support - %s (%d)\n",
- strerror(errno), ret);
-
-
- gem_userptr_test_unsynchronized();
-
- igt_subtest("userptr-unsync")
- test_userptr(fd);
-
- igt_subtest("userptr-impact-unsync")
- test_impact(fd, "unsync-");
-
- igt_subtest("userptr-impact-unsync-overlap")
- test_impact_overlap(fd, "unsync-");
-
- gem_userptr_test_synchronized();
-
- igt_subtest("userptr-sync")
- test_userptr(fd);
-
- igt_subtest("userptr-impact-sync")
- test_impact(fd, "sync-");
-
- igt_subtest("userptr-impact-sync-overlap")
- test_impact_overlap(fd, "sync-");
-
- igt_exit();
-
- return 0;
-}
diff --git a/benchmarks/meson.build b/benchmarks/meson.build
index bede51dce..b3d16b06d 100644
--- a/benchmarks/meson.build
+++ b/benchmarks/meson.build
@@ -23,7 +23,6 @@ if libdrm_intel.found()
'intel_upload_blit_large_gtt',
'intel_upload_blit_large_map',
'intel_upload_blit_small',
- 'gem_userptr_benchmark',
]
endif
--
2.26.0
More information about the igt-dev
mailing list