[Intel-gfx] [drm-intel:drm-intel-next-queued 273/275] drivers/gpu/drm/i915/i915_gem_gtt.c:2748:2: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t'

[Daniel Vetter]

On Wed, Jun 24, 2015 at 01:23:48PM +0100, Tvrtko Ursulin wrote:
> 
> On 06/24/2015 01:00 PM, kbuild test robot wrote:
> >tree:   git://anongit.freedesktop.org/drm-intel drm-intel-next-queued
> >head:   cfbcbcf86a2fe9e6841abf3ba798f62de5f14c68
> >commit: 3c77ff539bce5ab9fe3c5da7929d311a44919b8c [273/275] drm/i915: Remove mostly unused variable in intel_rotate_fb_obj_pages
> >config: i386-randconfig-x0-06241944 (attached as .config)
> >reproduce:
> >   git checkout 3c77ff539bce5ab9fe3c5da7929d311a44919b8c
> >   # save the attached .config to linux build tree
> >   make ARCH=i386
> >
> >All warnings (new ones prefixed by >>):
> >
> >    drivers/gpu/drm/i915/i915_gem_gtt.c: In function 'gen8_ppgtt_init':
> >    drivers/gpu/drm/i915/i915_gem_gtt.c:955:22: warning: large integer implicitly truncated to unsigned type [-Woverflow]
> >      ppgtt->base.total = 1ULL << 32;
> >                          ^
> >    drivers/gpu/drm/i915/i915_gem_gtt.c: In function 'intel_rotate_fb_obj_pages':
> >>>drivers/gpu/drm/i915/i915_gem_gtt.c:2748:2: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t' [-Wformat=]
> >      DRM_DEBUG_KMS(
> >      ^
> 
> Ah size_t again. :) Sounds like it should be %zu for obj->base.size.
> Interestingly this warning did not trigger on my machine.
> 
> Would you prefer a fixup patch, a resend of the offender, or manual
> maintainer intervention in git history? (In the last option note there are
> two instances to be fixed up, line 2748 and 2763.)

Rectified directly, but please double-check I didn't botch it ;-)
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch