drm/i915/selftest: Make sure to init i915_ggtt_view in igt_vma_rotate_remap()
authorImre Deak <imre.deak@intel.com>
Thu, 25 Mar 2021 21:47:48 +0000 (23:47 +0200)
committerImre Deak <imre.deak@intel.com>
Mon, 29 Mar 2021 19:54:50 +0000 (22:54 +0300)
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 <imre.deak@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210325214808.2071517-6-imre.deak@intel.com
drivers/gpu/drm/i915/selftests/i915_vma.c

index 23f6a212a391dad620fa6fbe6e9875a951aa7d9f..9dd6799105e666e6e6994d6b4e53068aeb6c80c3 100644 (file)
@@ -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,