diff options
author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2018-07-06 19:35:50 +0300 |
---|---|---|
committer | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2018-07-09 15:35:13 +0300 |
commit | 764160f214cd916ddb79408b9f28ac0ad2df40e0 (patch) | |
tree | 8bb5281046fe188a5ea38d1d4460e19fd7a694bb /lib/igt_draw.c | |
parent | 366eed37c7c71217e1cb1f3be5e26358a41f0001 (diff) |
lib/igt_draw: Zero initalize the igt_bufs
I missed these when sprinkling the memsets. Using stack garbage as the
aux surface state isn't a good idea. Causes kms_front_buffer_tracking
to fail on skl+.
Fixes: a4393c3951ec ("lib: Add aux surface state to igt_buf")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Antonio Argenziano <antonio.argenziano@intel.com>
Diffstat (limited to 'lib/igt_draw.c')
-rw-r--r-- | lib/igt_draw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/igt_draw.c b/lib/igt_draw.c index ebae9151..c7d5770d 100644 --- a/lib/igt_draw.c +++ b/lib/igt_draw.c @@ -568,7 +568,7 @@ static void draw_rect_render(int fd, struct cmd_data *cmd_data, drm_intel_bo *src, *dst; uint32_t devid = intel_get_drm_devid(fd); igt_render_copyfunc_t rendercopy = igt_get_render_copyfunc(devid); - struct igt_buf src_buf, dst_buf; + struct igt_buf src_buf = {}, dst_buf = {}; struct intel_batchbuffer *batch; uint32_t tiling, swizzle; struct buf_data tmp; |