drm/i915: Silence sparse by using gfp_t
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 1 Sep 2017 14:57:28 +0000 (15:57 +0100)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Tue, 5 Sep 2017 17:00:16 +0000 (10:00 -0700)
Sparse enforces that GFP flags are only manipulated inside gfp_t locals.

Fixes: 4d470f7359c4 ("drm/i915: Avoid undefined behaviour of "u32 >> 32"")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170901145729.21363-1-chris@chris-wilson.co.uk
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
(cherry picked from commit 0d95c883bab5c5507fac3c34bc506f735971a2ee)
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/i915/i915_gem_execbuffer.c

index 944e9e543573e230df28727ca7ece6e05255bbbf..f187ca573e625ad8017260d6f8912c8ef2a55a9d 100644 (file)
@@ -285,7 +285,7 @@ static int eb_create(struct i915_execbuffer *eb)
                 * direct lookup.
                 */
                do {
-                       unsigned int flags;
+                       gfp_t flags;
 
                        /* While we can still reduce the allocation size, don't
                         * raise a warning and allow the allocation to fail.