Mesa (master): intel/tools/aub_dump: move aub file initialization to maybe_init()
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Tue Feb 25 19:19:28 UTC 2020
Module: Mesa
Branch: master
Commit: 76bf38eaf0b6c839eaa4a36990e4b14b8095b7eb
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=76bf38eaf0b6c839eaa4a36990e4b14b8095b7eb
Author: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Date: Sat Jan 11 17:04:49 2020 +0200
intel/tools/aub_dump: move aub file initialization to maybe_init()
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Reviewed-by: Rafael Antognolli <rafael.antognolli at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/3705>
---
src/intel/tools/intel_dump_gpu.c | 71 +++++++++++++++++++---------------------
1 file changed, 33 insertions(+), 38 deletions(-)
diff --git a/src/intel/tools/intel_dump_gpu.c b/src/intel/tools/intel_dump_gpu.c
index 198eb9891fd..2e1941a18af 100644
--- a/src/intel/tools/intel_dump_gpu.c
+++ b/src/intel/tools/intel_dump_gpu.c
@@ -113,7 +113,7 @@ align_u32(uint32_t v, uint32_t a)
}
static struct gen_device_info devinfo = {0};
-static uint32_t device = 0;
+static int device = 0;
static struct aub_file aub_file;
static void *
@@ -338,23 +338,6 @@ remove_bo(int fd, int handle)
bo->map = NULL;
}
-static uint32_t
-dump_create_context(int fd, uint32_t *ctx_id)
-{
- if (!aub_file.file) {
- aub_file_init(&aub_file, output_file,
- verbose == 2 ? stdout : NULL,
- device, program_invocation_short_name);
- aub_write_default_setup(&aub_file);
-
- if (verbose)
- printf("[running, output file %s, chipset id 0x%04x, gen %d]\n",
- output_filename, device, devinfo.gen);
- }
-
- return aub_write_context_create(&aub_file, ctx_id);
-}
-
__attribute__ ((visibility ("default"))) int
close(int fd)
{
@@ -364,8 +347,23 @@ close(int fd)
return libc_close(fd);
}
+static int
+get_pci_id(int fd, int *pci_id)
+{
+ struct drm_i915_getparam gparam;
+
+ if (device_override) {
+ *pci_id = device;
+ return 0;
+ }
+
+ gparam.param = I915_PARAM_CHIPSET_ID;
+ gparam.value = pci_id;
+ return libc_ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gparam);
+}
+
static void
-maybe_init(void)
+maybe_init(int fd)
{
static bool initialized = false;
FILE *config;
@@ -412,6 +410,18 @@ maybe_init(void)
bos = calloc(MAX_FD_COUNT * MAX_BO_COUNT, sizeof(bos[0]));
fail_if(bos == NULL, "out of memory\n");
+
+ int ret = get_pci_id(fd, &device);
+ assert(ret == 0);
+
+ aub_file_init(&aub_file, output_file,
+ verbose == 2 ? stdout : NULL,
+ device, program_invocation_short_name);
+ aub_write_default_setup(&aub_file);
+
+ if (verbose)
+ printf("[running, output file %s, chipset id 0x%04x, gen %d]\n",
+ output_filename, device, devinfo.gen);
}
__attribute__ ((visibility ("default"))) int
@@ -435,27 +445,12 @@ ioctl(int fd, unsigned long request, ...)
}
if (fd == drm_fd) {
- maybe_init();
+ maybe_init(fd);
switch (request) {
case DRM_IOCTL_I915_GETPARAM: {
struct drm_i915_getparam *getparam = argp;
-
- if (device_override && getparam->param == I915_PARAM_CHIPSET_ID) {
- *getparam->value = device;
- return 0;
- }
-
- ret = libc_ioctl(fd, request, argp);
-
- /* If the application looks up chipset_id
- * (they typically do), we'll piggy-back on
- * their ioctl and store the id for later
- * use. */
- if (ret == 0 && getparam->param == I915_PARAM_CHIPSET_ID)
- device = *getparam->value;
-
- return ret;
+ return get_pci_id(fd, getparam->value);
}
case DRM_IOCTL_I915_GEM_EXECBUFFER: {
@@ -487,7 +482,7 @@ ioctl(int fd, unsigned long request, ...)
}
if (ret == 0)
- create->ctx_id = dump_create_context(fd, ctx_id);
+ create->ctx_id = aub_write_context_create(&aub_file, ctx_id);
return ret;
}
@@ -502,7 +497,7 @@ ioctl(int fd, unsigned long request, ...)
}
if (ret == 0)
- create->ctx_id = dump_create_context(fd, ctx_id);
+ create->ctx_id = aub_write_context_create(&aub_file, ctx_id);
return ret;
}
More information about the mesa-commit
mailing list