From b21b38c46cd61d9b301a38ea7d868ac1eedb6d44 Mon Sep 17 00:00:00 2001 From: Lionel Landwerlin Date: Mon, 23 Jul 2018 15:39:12 +0100 Subject: [PATCH] intel: tools: dump: only store device id on success MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit We might fail on master node drm fd because we won't have the right permissions. Signed-off-by: Lionel Landwerlin Reviewed-by: Tapani Pälli --- src/intel/tools/intel_dump_gpu.c | 4 ++-- 1 file 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; -- 2.30.2