diff options
| author | Lionel Landwerlin <lionel.g.landwerlin@intel.com> | 2018-07-23 15:39:12 +0100 |
|---|---|---|
| committer | Lionel Landwerlin <lionel.g.landwerlin@intel.com> | 2018-07-25 16:53:06 +0100 |
| commit | b21b38c46cd61d9b301a38ea7d868ac1eedb6d44 (patch) | |
| tree | 8d033c34be575eee4b77badf43ff0a7892cf9982 /src/intel | |
| parent | 82fc6bdebfe3c4e2b224c12257b1f9317f7242cf (diff) | |
intel: tools: dump: only store device id on success
We might fail on master node drm fd because we won't have the right
permissions.
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
Diffstat (limited to 'src/intel')
| -rw-r--r-- | src/intel/tools/intel_dump_gpu.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/intel/tools/intel_dump_gpu.c b/src/intel/tools/intel_dump_gpu.c index 1abe54147cf..a71103f1889 100644 --- a/src/intel/tools/intel_dump_gpu.c +++ b/src/intel/tools/intel_dump_gpu.c @@ -111,7 +111,7 @@ align_u32(uint32_t v, uint32_t a) } static struct gen_device_info devinfo = {0}; -static uint32_t device; +static uint32_t device = 0; static struct aub_file aub_file; static void * @@ -419,7 +419,7 @@ ioctl(int fd, unsigned long request, ...) * (they typically do), we'll piggy-back on * their ioctl and store the id for later * use. */ - if (getparam->param == I915_PARAM_CHIPSET_ID) + if (ret == 0 && getparam->param == I915_PARAM_CHIPSET_ID) device = *getparam->value; return ret; |