From: Imre Deak Date: Thu, 25 Mar 2021 21:47:48 +0000 (+0200) Subject: drm/i915/selftest: Make sure to init i915_ggtt_view in igt_vma_rotate_remap() X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=54c2921f7e9eeb7bb17e7174fb83c833503277f5;p=linux.git drm/i915/selftest: Make sure to init i915_ggtt_view in igt_vma_rotate_remap() This probably doesn't cause an issue, since the code checks the view type dependent size of the views before comparing them, but let's follow the practice to bzero the whole struct when initializing it. v2: Use {} instead of { } struct intializer. (Ville) Signed-off-by: Imre Deak Reviewed-by: Ville Syrjälä Link: https://patchwork.freedesktop.org/patch/msgid/20210325214808.2071517-6-imre.deak@intel.com --- diff --git a/drivers/gpu/drm/i915/selftests/i915_vma.c b/drivers/gpu/drm/i915/selftests/i915_vma.c index 23f6a212a391d..9dd6799105e66 100644 --- a/drivers/gpu/drm/i915/selftests/i915_vma.c +++ b/drivers/gpu/drm/i915/selftests/i915_vma.c @@ -515,7 +515,7 @@ static int igt_vma_rotate_remap(void *arg) for (t = types; *t; t++) { for (a = planes; a->width; a++) { for (b = planes + ARRAY_SIZE(planes); b-- != planes; ) { - struct i915_ggtt_view view; + struct i915_ggtt_view view = {}; unsigned int n, max_offset; max_offset = max(a->stride * a->height,